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

Fix compilation on Linux #3454

Closed
wants to merge 2 commits into from
Closed

Conversation

Stilic
Copy link

@Stilic Stilic commented Sep 25, 2024

Some fixes for compilation on Linux.

@Stilic Stilic mentioned this pull request Sep 25, 2024
@AbnormalPoof
Copy link
Contributor

AbnormalPoof commented Sep 25, 2024

Wouldn't this inadvertently break ARM64 macOS compilation? The commit hash the PR uses is really old. (It's from the develop2 branch instead of dev-funkin-arm64!)

If something broke in one of the newer commits, I recommend pinpointing the issue and then submitting a fix on FunkinCrew's lime fork directly!

@NotHyper-474
Copy link
Contributor

NotHyper-474 commented Sep 25, 2024

I personally use Debian 12 and had no issues building to Linux with that branch (except for a missing lime.ndll error that was easily solved by lime rebuild cpp -64 -release -nocffi -clean)

Also according to openfl/lime@d5b3294 the newer HxCpp is required for Mac ARM64 compilation (this seems to be only for NDlls tho?)

@Stilic
Copy link
Author

Stilic commented Sep 26, 2024

Wouldn't this inadvertently break ARM64 macOS compilation? The commit hash the PR uses is really old. (It's from the develop2 branch instead of dev-funkin-arm64!)

If something broke in one of the newer commits, I recommend pinpointing the issue and then submitting a fix on FunkinCrew's lime fork directly!

Oops... I thought changing the version wouldn't bring other issues...

@Stilic
Copy link
Author

Stilic commented Sep 26, 2024

I personally use Debian 12 and had no issues building to Linux with that branch (except for a missing lime.ndll error that was easily solved by lime rebuild cpp -64 -release -nocffi -clean)

Also according to openfl/lime@d5b3294 the newer HxCpp is required for Mac ARM64 compilation (this seems to be only for NDlls tho?)

It was complaining about missing Freetype librairies on my laptop running Arch Linux because the structure got changed in the newest versions I think (I had all my packages up-to-date).
I got this issue fixed for pixman by symlinking to the appropriate locations but it didn't do the trick for Freetype.
I may look into fixing this issue at the level of Lime itself.

@Stilic
Copy link
Author

Stilic commented Sep 26, 2024

Checked the current version of hxcpp and turns out it's newer that the one that was used before (4.3.45 < 4.3.65).
I will re-add hxcpp in hmm.json with the commit hash of that release.

@Stilic Stilic marked this pull request as draft September 26, 2024 05:22
@NotHyper-474
Copy link
Contributor

It was complaining about missing Freetype librairies on my laptop running Arch Linux because the structure got changed in the newest versions I think (I had all my packages up-to-date).
I got this issue fixed for pixman by symlinking to the appropriate locations but it didn't do the trick for Freetype.
I may look into fixing this issue at the level of Lime itself.

Perhaps that can be solved by following
this part of the troubleshooting?

@Stilic
Copy link
Author

Stilic commented Sep 26, 2024

I've found a way to build a Lime NDLL.
Run the following commands and then you're good to go:

cd .haxelib/lime/git
git submodule init
git submodule sync
git submodule update
lime rebuild cpp -64 -release -nocffi -clean
lime rebuild cpp -64 -release -nocffi -clean -debug

@Stilic Stilic closed this Sep 26, 2024
@Stilic Stilic deleted the fix-develop-linux branch September 26, 2024 19:41
@Stilic
Copy link
Author

Stilic commented Sep 26, 2024

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.

3 participants