Skip to content

Enable compilation on MacOS PowerPC#197

Open
bjia56 wants to merge 2 commits into
jart:masterfrom
bjia56:macppc
Open

Enable compilation on MacOS PowerPC#197
bjia56 wants to merge 2 commits into
jart:masterfrom
bjia56:macppc

Conversation

@bjia56

@bjia56 bjia56 commented Jun 14, 2025

Copy link
Copy Markdown
Contributor

These changes allow for Blink and Blinkenlights to be compiled for MacOS PowerPC hosts, tested on MacOS Tiger 10.4. Due to its legacy nature, the Tiger build was done with MacPorts GCC 14 and legacy-support-devel.

Major changes include working around a missing F_DUPD_CLOEXEC macro with two fcntls; including sys/types.h before netinet/udp.h; adding detection for the __ppc__ macro for cpuid purposes; and ensuring feature flags are properly matched to ensure mkfifo does not turn on mkfifoat.

These changes allow for Blink and Blinkenlights to be compiled for MacOS
PowerPC hosts, tested on MacOS Tiger 10.4. Due to its legacy nature, the
Tiger build was done with MacPorts GCC 14 and legacy-support-devel.

Major changes include working around a missing F_DUPD_CLOEXEC macro with
two fcntls; including sys/types.h before netinet/udp.h; adding detection
for the __ppc__ macro for cpuid purposes; and ensuring feature flags are
properly matched to ensure mkfifo does not turn on mkfifoat.
@aeiouaeiouaeiouaeiouaeiouaeiou

Copy link
Copy Markdown

In fact, this patch fixes the build on any macOS below 10.7.
I backported this to the MacPorts repository: macports/macports-ports#29177

@jart jart left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@jart

jart commented Aug 24, 2025

Copy link
Copy Markdown
Owner

@bjia56 Do you have a Cygwin environment handy to fix the build / headers issue?

@bjia56

bjia56 commented Aug 24, 2025

Copy link
Copy Markdown
Contributor Author

Not locally, but I can spin one up in a virtual environment.

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