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 MacOS support #12

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

Add MacOS support #12

wants to merge 1 commit into from

Conversation

JIaxyga
Copy link

@JIaxyga JIaxyga commented Jun 27, 2024

I’m not entirely sure if this is something we should officially support. I added it primarily for my own use since it was helpful to me on one occasion. However, it might also be useful to others

Import elf.h file from:
https://gist.github.com/mlafeldt/3885346
And use it :D

Signed-off-by: Danila Tikhonov <[email protected]>
Comment on lines +13 to +17
#if defined(__APPLE__)
#include "include/elf.h"
#else
#include <elf.h>
#endif

Choose a reason for hiding this comment

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

Everything else aside, it seems cleaner to add the local include folder to the search path from the makefile perhaps? And then rename the folder to indicate that it's a specific fallback for when the target doesn't provide elf.h.

Copy link
Contributor

@lumag lumag left a comment

Choose a reason for hiding this comment

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

Please don't brimg GPL-ed headers into a BSD-licemsed software.

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