forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork0
Commitae41bb4
committed
On OS X, link libpython normally, ignoring the "framework" framework.
As of Xcode 5.0, Apple isn't including the Python framework as part of theSDK-level files, which means that linking to it might fail depending onwhether Xcode thinks you've selected a specific SDK version. According totheir Tech Note 2328, they've basically deprecated the framework method oflinking to libpython and are telling people to link to the shared librarynormally. (I'm pretty sure this is in direct contradiction to the advicethey were giving a few years ago, but whatever.) Testing says that thisapproach works fine at least as far back as OS X 10.4.11, so let's justrip out the framework special case entirely. We do still need a specialcase to decide that OS X provides a shared library at all, unfortunately(I wonder why the distutils check doesn't work ...). But this is stillless of a special case than before, so it's fine.Back-patch to all supported branches, since we'll doubtless be hearingabout this more as more people update to recent Xcode.1 parentfd78544 commitae41bb4
1 file changed
+1
-3
lines changedLines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
| 24 | + | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
28 | | - | |
29 | 27 | | |
30 | 28 | | |
31 | 29 | | |
| |||
0 commit comments
Comments
(0)