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

set correct paths for ndll for linxuarm and linuxarm64 (raspberrypi) #1834

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

gepatto
Copy link
Contributor

@gepatto gepatto commented Aug 18, 2024

set correct paths for ndll for linxuarm and linuxarm64 (raspberrypi)
and remove conflicting legacy includepaths in build.xml

These are the changes I made which allow compilation on a raspberry pi 4 or 5 running 32 or 64bit Bullseye or Bookworm (raspberrypios). A lot of the isRaspberryPi statements have been replaced by checking for Arm64 or Armv7 so that in the future other linux arm boards could be used as well. Ndlls are built in ndll/LinuxArm and ndll/LinuxArm64 instead of the previous ndll/RPi path

Old -I flags have been removed from Build.xml because they point to libraries from older OS versions and do no longer exist on bullseye and bookworm. They prevented sdl from compiling on these os versions.
Performance on older versions of the raspberrypi was poor. So I think support should be for pi4 and up only.

SDL still has pi specific code.
That's why a -Drpi flag can still be found when calling hxcpp. Otherwise a build would fail.

One important note is that for 64bit the latest (git)build of hxcpp is needed ! There is a bug in haxelibversion that caused threads to throw a bus error. This is solved in master.

and remove conflicting legacy includepaths in build.xml
@joshtynjala joshtynjala changed the base branch from 8.2.0-Dev to develop August 19, 2024 15:25
@joshtynjala
Copy link
Member

joshtynjala commented Aug 19, 2024

8.2.0-Dev has already been merged into develop, for the upcoming release, so I changed the base branch of this PR to match.

@skial skial mentioned this pull request Aug 21, 2024
1 task
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