Merge lp:~vorlon/python/lp.738213 into lp:~doko/python/pkg2.7-debian

Proposed by Steve Langasek
Status: Needs review
Proposed branch: lp:~vorlon/python/lp.738213
Merge into: lp:~doko/python/pkg2.7-debian
Diff against target: 63 lines (+14/-22)
2 files modified
changelog (+9/-0)
patches/deb-setup.diff (+5/-22)
To merge this branch: bzr merge lp:~vorlon/python/lp.738213
Reviewer Review Type Date Requested Status
Matthias Klose Pending
Review via email: mp+54473@code.launchpad.net

Description of the change

Matthias, please review this proposed fix to the python2.7 packaging's
detection of the multiarch path at build time, and if you find it suitable,
please upload.

To post a comment you must log in.
lp:~vorlon/python/lp.738213 updated
55. By Steve Langasek

releasing version 2.7.1-5ubuntu1

Unmerged revisions

55. By Steve Langasek

releasing version 2.7.1-5ubuntu1

54. By Steve Langasek

debian/patches/deb-setup.diff: DEB_GNU_TYPE was never being set, so
the expected multiarch path support wasn't working. Consolidate to
a single call to dpkg-architecture -qDEB_HOST_MULTIARCH in setup.py.
LP: #738213.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'changelog'
2--- changelog 2011-02-24 06:44:54 +0000
3+++ changelog 2011-03-23 22:05:59 +0000
4@@ -1,3 +1,12 @@
5+python2.7 (2.7.1-5ubuntu1) natty; urgency=low
6+
7+ * debian/patches/deb-setup.diff: DEB_GNU_TYPE was never being set, so
8+ the expected multiarch path support wasn't working. Consolidate to
9+ a single call to dpkg-architecture -qDEB_HOST_MULTIARCH in setup.py.
10+ LP: #738213.
11+
12+ -- Steve Langasek <steve.langasek@ubuntu.com> Wed, 23 Mar 2011 15:02:47 -0700
13+
14 python2.7 (2.7.1-5) experimental; urgency=low
15
16 * Update to 20110224, taken from the 2.7 branch.
17
18=== modified file 'patches/deb-setup.diff'
19--- patches/deb-setup.diff 2010-11-14 02:51:54 +0000
20+++ patches/deb-setup.diff 2011-03-23 22:05:59 +0000
21@@ -1,18 +1,10 @@
22 # DP: Don't include /usr/local/include and /usr/local/lib as gcc search paths
23 # DP: Include /usr/include/<triplet> and /usr/lib/<triplet> as gcc search paths
24
25---- a/Makefile.pre.in
26-+++ b/Makefile.pre.in
27-@@ -207,6 +207,7 @@
28-
29-
30- LIBFFI_INCLUDEDIR= @LIBFFI_INCLUDEDIR@
31-+DEB_GNU_TYPE=@DEB_GNU_TYPE@
32-
33- ##########################################################################
34- # Parser
35---- a/setup.py
36-+++ b/setup.py
37+Index: python2.7-2.7.1/setup.py
38+===================================================================
39+--- python2.7-2.7.1.orig/setup.py
40++++ python2.7-2.7.1/setup.py
41 @@ -347,8 +347,12 @@
42
43 def detect_modules(self):
44@@ -22,18 +14,9 @@
45 + # On Debian /usr/local is always used, so we don't include it twice
46 + #add_dir_to_list(self.compiler.library_dirs, '/usr/local/lib')
47 + #add_dir_to_list(self.compiler.include_dirs, '/usr/local/include')
48-+ triplet = sysconfig.get_config_var("DEB_GNU_TYPE")
49++ triplet = os.popen('dpkg-architecture -qDEB_HOST_MULTIARCH').readline()[:-1]
50 + add_dir_to_list(self.compiler.library_dirs, '/usr/lib/%s' % triplet)
51 + add_dir_to_list(self.compiler.include_dirs, '/usr/include/%s' % triplet)
52
53 # Add paths specified in the environment variables LDFLAGS and
54 # CPPFLAGS for header and library files.
55-@@ -408,6 +412,8 @@
56- '/lib', '/usr/lib',
57- ]
58- inc_dirs = self.compiler.include_dirs + ['/usr/include']
59-+ gnu_triplet = os.popen('dpkg-architecture -qDEB_HOST_GNU_TYPE').readline()[:-1]
60-+ inc_dirs.append(os.path.join('/usr/include', gnu_triplet))
61- exts = []
62- missing = []
63-

Subscribers

People subscribed via source and target branches

to all changes: