Skip to content

Allow building on Mac OS X#139

Open
daveisfera wants to merge 1 commit into
seb-m:masterfrom
daveisfera:master
Open

Allow building on Mac OS X#139
daveisfera wants to merge 1 commit into
seb-m:masterfrom
daveisfera:master

Conversation

@daveisfera
Copy link
Copy Markdown

Fixes #138.

@seanroark
Copy link
Copy Markdown

Is this going to get merged? I'm running into same problem.

Copy link
Copy Markdown

@jacebrowning jacebrowning left a comment

Choose a reason for hiding this comment

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

Because Pipfile supports PEP508, setup.py should not include platform checks like this.

👍 On this change.

@jacebrowning
Copy link
Copy Markdown

@seb-m Can you merge this?

Comment thread setup.py

# check linux platform
if not platform.startswith('linux') and not platform.startswith('freebsd'):
if not platform.startswith('linux') and not platform.startswith('freebsd') and not platform.startswith('macosx'):
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Why this package can build on macos, is there any restriction ?

@rpasta42
Copy link
Copy Markdown

rpasta42 commented Aug 24, 2018

pyinotify successfully builds on OSX with this modification, but then gives a runtime error: OSError: Cannot initialize new instance of inotify, Errno=Invalid argument (EINVAL)

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.

5 participants