-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Setup cleanup #1790
Setup cleanup #1790
Conversation
11cbff6
to
954abf3
Compare
Sorry, couldn't resist. As for the issues you raised, they are unfortunately not surprising. Our setup.py has to be complex, but doesn't need to be complicated. I've been trying to avoid a complete rewrite, but maybe it's time.
Changes Unknown when pulling 233afec on wiredfool:setup_cleanup into * on python-pillow:master*. |
Changes Unknown when pulling 233afec on wiredfool:setup_cleanup into * on python-pillow:master*. |
print("library, and set the corresponding ROOT variable in the") | ||
print("setup.py script.") | ||
print("library and headers.") | ||
print("See http://pillow.readthedocs.org/en/latest/installation.html#building-from-source") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This can be https
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That reminds me, because of the way that RTD is setup with versions, I can't find a deep permalink for the installation docs. Which means that all of the googable links to the documentation 404 eventually, unless they're going off of latest.
Looks good to merge. |
Changes Unknown when pulling a2bc14c on wiredfool:setup_cleanup into * on python-pillow:master*. |
So, I'm trying to track down a compile error on one of my machines and the hood came off setup.py and I nearly threw up a little.
So.
Some continuing WTFs:
The debug logging isn't complete, and it's still not answering my compile question. Comments welcome.
Fixes #679