[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: This was committed a while back, I guess I forgot to close this issue. -- resolution: -> fixed stage: commit review -> committed/rejected status: open -> closed ___ Python tracker

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: And for 3.1 in r81677. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: ht

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: Committed for 2.6 in r81674 -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: Ported to 3.2 in r81673. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: I've applied a new version of the patch in r81662. I'll be monitoring the buildbot farm to ensure that any issues that might crop up get fixed ASAP. -- ___ Python tracker __

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: The attached version should fix the issue found by Stefan. I'm going to do builds on OSX as well as Linux before committing though. -- Added file: http://bugs.python.org/file17316/issue7724-v3.patch ___ Python trac

[issue7724] setup.py ignores SDK root on OSX

Stefan Krah added the comment: Just to save you some time investigating, it broke the build because of missing parentheses in setup.py: if sys.platform == 'darwin' and dir.startswith('/System') or dir.startswith('/usr') -- nosy: +skrah ___ Python t

[issue7724] setup.py ignores SDK root on OSX

Antoine Pitrou added the comment: Ronald, I've reverted the patch since it broke compilation on almost every buildbot, and we're close to beta2 release. -- nosy: +benjamin.peterson stage: committed/rejected -> commit review ___ Python tracker

[issue7724] setup.py ignores SDK root on OSX

Changes by Ronald Oussoren : -- stage: patch review -> committed/rejected ___ Python tracker ___ ___ Python-bugs-list mailing list Unsu

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: Committed to the trunk in r80963. I'm not closing this yet because the patch needs to be ported to 3.2 at least, and preferably 2.6 and 3.1 as well. -- ___ Python tracker __

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: I've cleaned up the patch and made it clearer that platforms other than OSX aren't affected by rewriting code like this: f = os.path.join(d, "db.h") if sys.platform == "darwin" and is_macosx_sdk_path(d): f = os.path.join(sysroot, d[1:], "db.h") The new ve

[issue7724] setup.py ignores SDK root on OSX

Antoine Pitrou added the comment: There's a bunch of debug stuff in that patch ("if 1", commented out lines...). Also, the patch should not change behaviour on non-OS X platforms, which is not obvious right now. -- nosy: +pitrou ___ Python tracker

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: As a short explanation of OSX SDKs: those are basicly directories containing header files and stub libraries with the same directory structure as a real system. As an example, /Developer/SDKs/MacOSX10.4u.sdk contains the 10.4u SDK, that tree contains /Deve

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: The attached patch (for the trunk) fixes this issue on my machine. The patch also fixes issue 8444. I wouldn't mind some review of the patch, it does affect core bits of setup.py. -- keywords: +needs review stage: needs patch -> patch review Added fi

[issue7724] setup.py ignores SDK root on OSX

Ned Deily added the comment: I suppose all of the relevant setup.py build-time tests could be restructured as autoconf-style tests using gcc & friends with consistent arguments (with the build and with Distutils) so there wouldn't need to be special knowledge in setup.py or configure about wh

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: I agree, our usage of -isysroot seems to be correct. What's rather annoying is that the documentation seems to claim that the sysroot value gets prepended to every search location, while that is obviously not try. My guess is that it is only prepended for b

[issue7724] setup.py ignores SDK root on OSX

Ned Deily added the comment: It's not exactly the same issue but I think it is closely related since effectively both document the need for setup.py to build with a non-default system root (or SDK) and some of the hardcoded paths should be being satisfied from the SDK. So at least parts of t

[issue7724] setup.py ignores SDK root on OSX

Ronald Oussoren added the comment: Issue7713 is not the same: it asks for a way to affect the hardcoded paths in setup.py, this issue asks to honor the SDK-root by setup.py. I've done some further research and this affects distutils in general: the compiler has methods to look for files and t

[issue7724] setup.py ignores SDK root on OSX

Ned Deily added the comment: See also newly opened Issue7713 which expands the issue to other platforms . -- nosy: +ned.deily ___ Python tracker ___ _

[issue7724] setup.py ignores SDK root on OSX

New submission from Ronald Oussoren : On OSX it is possible to compile using an SDK, which is basicly a directory tree containing include files and shared library stubs. When building using an SDK (such as the 10.4u SDK) the compiler looks in the SDK subtree instead of / (that is, look for inc