Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace BeIDE with Paladin if compiling on Haiku. #90

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

OscarL
Copy link
Contributor

@OscarL OscarL commented Mar 5, 2024

Pretty low-effort (3 AM, currently here :-P) attempt to address #71.

Opening as draft, as "Add file", "Remove file", and "Project->Group" menu items do not work as intended with current Paladin (see notes on the linked issue above).

@korli
Copy link
Contributor

korli commented Mar 16, 2024

Would be nicer to define application/x-mw-BeIDE in one place like #define PE_DEFAULT_IDE "..."

@korli
Copy link
Contributor

korli commented Mar 16, 2024

LGTM

@OscarL OscarL marked this pull request as ready for review March 16, 2024 17:10
@OscarL OscarL marked this pull request as draft March 16, 2024 17:11
@OscarL
Copy link
Contributor Author

OscarL commented Mar 16, 2024

Back to draft, as I remembered that "Preferences->Connection" still shows "Metrowerks BeIDE".

And I still haven't figured out how to do conditional compilation of .r files with jam, or if rez has any "#ifdef" support I could use.

@OscarL OscarL marked this pull request as ready for review October 27, 2024 23:54
@OscarL
Copy link
Contributor Author

OscarL commented Oct 27, 2024

Turns out I just needed to try using #ifdef and friends on .r files (even if Pe's syntax coloring didn't liked it). Sigh.

Anyway... up for review again.

@korli korli merged commit d624970 into HaikuArchives:master Oct 28, 2024
0 of 2 checks passed
@OscarL OscarL deleted the paladin-as-ide branch October 28, 2024 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants