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

Build support for macOS/Darwin #2

Closed
wants to merge 4 commits into from
Closed

Conversation

ghost
Copy link

@ghost ghost commented Jan 5, 2024

When trying to compile the current master revision (7929f41) for macOS, I run into some small issues. These issues can be easily solved by including the correct dependencies and supplying valid search paths to the compiler suite.

I've extended the Makefile so it can use clang and the correct search paths. When brew is installed on the macOS system this project is being built on, brew's prefix path will be used for building. Some things work different for Apple/macOS, so these changes had been implemented in a non-invasive way via define switches.

Lastly, there was a minor issue in dumpet/applepart.c, where a printf call did not take into consideration whether it's being built for 64- or 32-bit systems. This one was solved by implementing two versions of the call.

@ghost ghost closed this by deleting the head repository Nov 5, 2024
This pull request was closed.
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.

0 participants