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

Change rfd features to enable manual builds on Linux. #127

Merged
merged 1 commit into from
Jan 3, 2025

Conversation

benjamin051000
Copy link
Contributor

rfd wasn't building on Linux, since its default features appear to be for Windows. This commit changes them to work on Linux.

  • TODO test on Windows to make sure it doesn't break that build (CI should tell us).

@benjamin051000 benjamin051000 marked this pull request as ready for review January 3, 2025 15:02
@benjamin051000
Copy link
Contributor Author

Looks like CI jobs have to be approved by the maintainer, makes sense. In the mean time, I'm (slowly) setting up a Windows VM on my laptop to test this change. If someone with Windows could verify it works, that'd be helpful!

@benjamin051000 benjamin051000 changed the title Change rfd features to allow builds on Linux. Change rfd features to enable builds on Linux. Jan 3, 2025
@benjamin051000 benjamin051000 changed the title Change rfd features to enable builds on Linux. Change rfd features to enable manual builds on Linux. Jan 3, 2025
@benjamin051000 benjamin051000 force-pushed the main branch 2 times, most recently from 046b297 to 96b652c Compare January 3, 2025 15:20
@benjamin051000
Copy link
Contributor Author

Looks like it built in CI on my fork: https://github.com/benjamin051000/arnis/actions/runs/12599851589

Testing in a VM to make sure it actually runs now.

@louis-e
Copy link
Owner

louis-e commented Jan 3, 2025

Huge thanks for your effort! Will merge it asap, just checking on a few things regarding the Linux compilation issues

rfd wasn't building on Linux, since its default features appear to be
for Windows. This commit changes them to work on Linux. TODO test on
Windows.
@benjamin051000
Copy link
Contributor Author

No worries. I look forward to getting this running on Linux 🐧

@louis-e louis-e merged commit ffd5a27 into louis-e:main Jan 3, 2025
2 checks passed
This was referenced Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants