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

Support building on macOS #48

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

Support building on macOS #48

wants to merge 1 commit into from

Conversation

mpctx
Copy link

@mpctx mpctx commented Oct 14, 2018

  • Wrap stat and xattr in ifdef directives.
  • Remove -Bstatic and -Bdynamic arguments in Makefile because the
    arguments aren't needed and break linking.

Tested on Linux Mint and macOS High Sierra.

- Wrap stat and xattr in ifdef directives.
- Remove -Bstatic and -Bdynamic arguments in Makefile because the
  arguments aren't needed and break linking.
@mohammad-akhlaghi
Copy link

Thanks a lot "mpctx", I just also tested this commit on several Linux systems and a Mac system and it worked nicely. I just merged it into my own fork of metastore (https://gitlab.com/makhlaghi/metastore-fork), combining it with some minor issues I had to fix. I will be using this fork until the new pull requests implemented in the main project.

@przemoc przemoc mentioned this pull request Apr 7, 2021
@przemoc
Copy link
Owner

przemoc commented Apr 8, 2021

Thank you, @mpctx, for your metastore contribution.

As it is right now, it's rather a no-go, as adding more #ifdef spaghetti code is undesirable and not maintainable.
We have to introduce OS-agnostic API internally and provide OS-specific implementation if common denominator cannot be easily found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants