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

Add FreeBSD CI Github Actions #3

Closed
wants to merge 21 commits into from

Conversation

ivan-volnov
Copy link

@ivan-volnov ivan-volnov commented Apr 24, 2021

This PR adds working Github Actions for FreeBSD build.

It still needs to add ant build configurations for FreeBSD.

If you need to install additional packages for building, add it to the end of pkg install command.

Related issue is here:
LWJGL#421

Here is CI output:
https://github.com/ivan-volnov/lwjgl3/runs/2426090932?check_suite_focus=true

It shows ant error because there is no such build configuration for FreeBSD:

Buildfile: /Users/runner/work/lwjgl3/lwjgl3/build.xml

BUILD FAILED
/Users/runner/work/lwjgl3/lwjgl3/build.xml:13: Cannot find /Users/runner/work/lwjgl3/lwjgl3/config/${platform}/build.xml imported from /Users/runner/work/lwjgl3/lwjgl3/build.xml

Total time: 1 second

It seems it also needs openjfx14 because we don't use zulu here. But I have no idea how to patch build system for openjfx14

But I suppose build configurations should be PR'ed to the main repo instead.

So this PR is complete for now

@ivan-volnov
Copy link
Author

ivan-volnov commented Apr 24, 2021

There is also a try to patch build configs for FreeBSD but I have no luck yet. The code is quite clean, so anybody can complete my work
https://github.com/ivan-volnov/lwjgl3/tree/freebsd_build

@Spasi Spasi force-pushed the master branch 7 times, most recently from c8c7390 to a1d29b3 Compare September 2, 2021 19:58
@Spasi
Copy link
Member

Spasi commented Nov 21, 2023

Thank you @ivan-volnov, LWJGL now has FreeBSD support.

@Spasi Spasi closed this Nov 21, 2023
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