Merge lp:~ari-tczew/gnome-python-extras/ubuntu into lp:~ubuntu-desktop/gnome-python-extras/ubuntu

Proposed by Artur Rona
Status: Merged
Merged at revision: 24
Proposed branch: lp:~ari-tczew/gnome-python-extras/ubuntu
Merge into: lp:~ubuntu-desktop/gnome-python-extras/ubuntu
Diff against target: 219 lines (+115/-56)
5 files modified
debian/changelog (+34/-0)
debian/patches/01_xulrunner_ubuntu.patch (+63/-56)
debian/patches/04_use_PYTHON-config_check.patch (+4/-0)
debian/patches/06_gda_libpython.patch (+13/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~ari-tczew/gnome-python-extras/ubuntu
Reviewer Review Type Date Requested Status
Chris Coulson Approve
Review via email: mp+47601@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Chris Coulson (chrisccoulson) wrote :

Looks good. One minor comment though:

--- debian/patches/04_use_PYTHON-config_check.patch 2009-05-12 08:42:44 +0000
+++ debian/patches/04_use_PYTHON-config_check.patch 2011-01-26 22:42:32 +0000
@@ -1,3 +1,7 @@
+From: Chris Coulson <email address hidden>
+Description: Use python-config.
+Author: Daniel Holbach <email address hidden>
+

This change originally came from Matthias Klose (https://launchpad.net/ubuntu/+source/gnome-python-extras/2.14.2-1ubuntu2), so I've fixed that.

Also, as with g-p-m, it would be good to try using dh-autoreconf rather than the old CDBS rules for updating the build system.

I'll merge it anyway though.

Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-01-07 14:59:45 +0000
3+++ debian/changelog 2011-01-26 22:51:58 +0000
4@@ -1,3 +1,37 @@
5+gnome-python-extras (2.25.3-7ubuntu1) natty; urgency=low
6+
7+ * Merge from debian unstable. Remaining changes: (LP: #708357)
8+ - debian/control:
9+ + Build-Depends on gnome-common, needed for autotools in rules.
10+ + Bump build-depends on python-gnome2-dev.
11+ + Remove stray comma from python-gtk2 depends.
12+ - debian/patches/01_xul19.patch:
13+ + Replaced with 01_xulrunner_ubuntu.patch, support xulrunner 2.0.
14+ - debian/patches/04_use_PYTHON-config_check.patch:
15+ + Use python-config.
16+ - debian/patches/70_autotools.patch:
17+ + Dropped, autotools now runs on build.
18+ - debian/rules:
19+ + Run autotools on build.
20+ + Support xulrunner 2.0.
21+ * 01_xulrunner_ubuntu.patch: Updated to get it more professionally.
22+
23+ -- Artur Rona <ari-tczew@ubuntu.com> Wed, 26 Jan 2011 21:40:16 +0100
24+
25+gnome-python-extras (2.25.3-7) unstable; urgency=low
26+
27+ * Patch from Micah Gersten to add missing dependency on xulrunner.
28+ Closes: #609189.
29+
30+ -- Josselin Mouette <joss@debian.org> Fri, 14 Jan 2011 19:38:45 +0100
31+
32+gnome-python-extras (2.25.3-6) unstable; urgency=low
33+
34+ * 06_gda_libpython.patch: don’t link the gda module with libpython.
35+ * 70_autotools.patch: updated accordingly.
36+
37+ -- Josselin Mouette <joss@debian.org> Mon, 27 Sep 2010 09:24:41 +0200
38+
39 gnome-python-extras (2.25.3-5ubuntu4) natty; urgency=low
40
41 * Have dh_xulrunner install a dependency on xulrunner for python-gtkmozembed
42
43=== modified file 'debian/patches/01_xulrunner_ubuntu.patch'
44--- debian/patches/01_xulrunner_ubuntu.patch 2010-11-25 15:08:54 +0000
45+++ debian/patches/01_xulrunner_ubuntu.patch 2011-01-26 22:51:58 +0000
46@@ -1,5 +1,9 @@
47---- a/configure.ac
48-+++ b/configure.ac
49+Description: Update for xulrunner 2.0. Change the GREVersion boundaries and tidy up the configure.ac changes a bit too.
50+Author: Chris Coulson <chris.coulson@canonical.com>
51+
52+diff -pruN -x '*~' gnome-python-extras-2.25.3.orig/configure.ac gnome-python-extras-2.25.3/configure.ac
53+--- gnome-python-extras-2.25.3.orig/configure.ac 2009-02-15 23:20:23.000000000 +0100
54++++ gnome-python-extras-2.25.3/configure.ac 2011-01-26 21:56:35.000000000 +0100
55 @@ -19,6 +19,7 @@ m4_define(gtkhtml2_required_version,
56 m4_define(mozilla_required_version, 1.2b)
57 m4_define(firefox_required_version, 1.0)
58@@ -123,28 +127,42 @@
59 if test -n "$export_dynamic"; then
60 GTKMOZEMBED_LIBS=`echo $GTKMOZEMBED_LIBS | sed -e "s/$export_dynamic//"`
61 fi
62---- a/gtkmozembed/Makefile.am
63-+++ b/gtkmozembed/Makefile.am
64-@@ -13,9 +13,15 @@ pkgpyexecdir = $(pyexecdir)/gtk-2.0
65- pkgpyexec_LTLIBRARIES = gtkmozembed.la
66- gtkmozembed_la_LDFLAGS = -module -avoid-version \
67- -export-symbols-regex initgtkmozembed \
68-- -R$(MOZILLA_HOME) -L$(MOZILLA_HOME)
69-+ $(NULL)
70-+
71-+# only use rpath for non-glue builds
72-+if !HAVE_XPCOM_GLUE
73-+ gtkmozembed_la_LDFLAGS + = -R$(MOZILLA_HOME) -L$(MOZILLA_HOME)
74-+endif
75-+
76- gtkmozembed_la_LIBADD = $(GTKMOZEMBED_LIBS)
77--gtkmozembed_la_SOURCES = gtkmozembedmodule.c
78-+gtkmozembed_la_SOURCES = gtkmozembedmodule.cpp
79- nodist_gtkmozembed_la_SOURCES = gtkmozembed.c
80- gtkmozembed.c: gtkmozembed.defs gtkmozembed.override
81- CLEANFILES = gtkmozembed.c
82---- /dev/null
83-+++ b/gtkmozembed/gtkmozembedmodule.cpp
84+diff -pruN -x '*~' gnome-python-extras-2.25.3.orig/gtkmozembed/gtkmozembedmodule.c gnome-python-extras-2.25.3/gtkmozembed/gtkmozembedmodule.c
85+--- gnome-python-extras-2.25.3.orig/gtkmozembed/gtkmozembedmodule.c 2008-12-12 02:06:29.000000000 +0100
86++++ gnome-python-extras-2.25.3/gtkmozembed/gtkmozembedmodule.c 1970-01-01 01:00:00.000000000 +0100
87+@@ -1,29 +0,0 @@
88+-#ifdef HAVE_CONFIG_H
89+-#include "config.h"
90+-#endif
91+-
92+-/* include this first, before NO_IMPORT_PYGOBJECT is defined */
93+-#include <pygobject.h>
94+-
95+-void pygtkmozembed_register_classes (PyObject *d);
96+-void pygtkmozembed_add_constants(PyObject *module, const gchar *strip_prefix);
97+-
98+-extern PyMethodDef pygtkmozembed_functions[];
99+-
100+-DL_EXPORT(void)
101+-initgtkmozembed(void)
102+-{
103+- PyObject *m, *d;
104+-
105+- init_pygobject ();
106+-
107+- m = Py_InitModule ("gtkmozembed", pygtkmozembed_functions);
108+- d = PyModule_GetDict (m);
109+-
110+- pygtkmozembed_register_classes (d);
111+- pygtkmozembed_add_constants(m, "GTK_MOZ_EMBED_");
112+-
113+- if (PyErr_Occurred ()) {
114+- Py_FatalError ("can't initialise module gtkmozembed");
115+- }
116+-}
117+diff -pruN -x '*~' gnome-python-extras-2.25.3.orig/gtkmozembed/gtkmozembedmodule.cpp gnome-python-extras-2.25.3/gtkmozembed/gtkmozembedmodule.cpp
118+--- gnome-python-extras-2.25.3.orig/gtkmozembed/gtkmozembedmodule.cpp 1970-01-01 01:00:00.000000000 +0100
119++++ gnome-python-extras-2.25.3/gtkmozembed/gtkmozembedmodule.cpp 2011-01-26 21:56:35.000000000 +0100
120 @@ -0,0 +1,78 @@
121 +#ifdef HAVE_CONFIG_H
122 +#include "config.h"
123@@ -224,35 +242,24 @@
124 + Py_FatalError ("can't initialise module gtkmozembed");
125 + }
126 +}
127---- a/gtkmozembed/gtkmozembedmodule.c
128-+++ /dev/null
129-@@ -1,29 +0,0 @@
130--#ifdef HAVE_CONFIG_H
131--#include "config.h"
132--#endif
133--
134--/* include this first, before NO_IMPORT_PYGOBJECT is defined */
135--#include <pygobject.h>
136--
137--void pygtkmozembed_register_classes (PyObject *d);
138--void pygtkmozembed_add_constants(PyObject *module, const gchar *strip_prefix);
139--
140--extern PyMethodDef pygtkmozembed_functions[];
141--
142--DL_EXPORT(void)
143--initgtkmozembed(void)
144--{
145-- PyObject *m, *d;
146--
147-- init_pygobject ();
148--
149-- m = Py_InitModule ("gtkmozembed", pygtkmozembed_functions);
150-- d = PyModule_GetDict (m);
151--
152-- pygtkmozembed_register_classes (d);
153-- pygtkmozembed_add_constants(m, "GTK_MOZ_EMBED_");
154--
155-- if (PyErr_Occurred ()) {
156-- Py_FatalError ("can't initialise module gtkmozembed");
157-- }
158--}
159+diff -pruN -x '*~' gnome-python-extras-2.25.3.orig/gtkmozembed/Makefile.am gnome-python-extras-2.25.3/gtkmozembed/Makefile.am
160+--- gnome-python-extras-2.25.3.orig/gtkmozembed/Makefile.am 2008-12-12 02:06:29.000000000 +0100
161++++ gnome-python-extras-2.25.3/gtkmozembed/Makefile.am 2011-01-26 21:56:35.000000000 +0100
162+@@ -13,9 +13,15 @@ pkgpyexecdir = $(pyexecdir)/gtk-2.0
163+ pkgpyexec_LTLIBRARIES = gtkmozembed.la
164+ gtkmozembed_la_LDFLAGS = -module -avoid-version \
165+ -export-symbols-regex initgtkmozembed \
166+- -R$(MOZILLA_HOME) -L$(MOZILLA_HOME)
167++ $(NULL)
168++
169++# only use rpath for non-glue builds
170++if !HAVE_XPCOM_GLUE
171++ gtkmozembed_la_LDFLAGS + = -R$(MOZILLA_HOME) -L$(MOZILLA_HOME)
172++endif
173++
174+ gtkmozembed_la_LIBADD = $(GTKMOZEMBED_LIBS)
175+-gtkmozembed_la_SOURCES = gtkmozembedmodule.c
176++gtkmozembed_la_SOURCES = gtkmozembedmodule.cpp
177+ nodist_gtkmozembed_la_SOURCES = gtkmozembed.c
178+ gtkmozembed.c: gtkmozembed.defs gtkmozembed.override
179+ CLEANFILES = gtkmozembed.c
180
181=== modified file 'debian/patches/04_use_PYTHON-config_check.patch'
182--- debian/patches/04_use_PYTHON-config_check.patch 2009-05-12 08:42:44 +0000
183+++ debian/patches/04_use_PYTHON-config_check.patch 2011-01-26 22:51:58 +0000
184@@ -1,3 +1,7 @@
185+From: Chris Coulson <chris.coulson@canonical.com>
186+Description: Use python-config.
187+Author: Daniel Holbach <daniel.holbach@ubuntu.com>
188+
189 Index: gnome-python-extras-2.25.3-2ubuntu1/acinclude.m4
190 ===================================================================
191 --- gnome-python-extras-2.25.3-2ubuntu1.orig/acinclude.m4 2009-05-10 13:44:58.000000000 +0200
192
193=== added file 'debian/patches/06_gda_libpython.patch'
194--- debian/patches/06_gda_libpython.patch 1970-01-01 00:00:00 +0000
195+++ debian/patches/06_gda_libpython.patch 2011-01-26 22:51:58 +0000
196@@ -0,0 +1,13 @@
197+Index: gnome-python-extras-2.25.3/gda/Makefile.am
198+===================================================================
199+--- gnome-python-extras-2.25.3.orig/gda/Makefile.am 2010-09-27 09:21:09.079973023 +0200
200++++ gnome-python-extras-2.25.3/gda/Makefile.am 2010-09-27 09:22:06.295974778 +0200
201+@@ -34,7 +34,7 @@ gda_la_CFLAGS = $(GDA_CFLAGS)
202+ gda_la_LDFLAGS = $(common_ldflags)
203+ # gda_la_LDFLAGS = $(common_ldflags) -export-symbols-regex initgda
204+
205+-gda_la_LIBADD = $(GDA_LIBS) $(PYTHON_LDFLAGS)
206++gda_la_LIBADD = $(GDA_LIBS)
207+ gda_la_SOURCES = gdamodule.c pygdavalue_conversions.c pygdavalue_conversions.h gdaglue.h
208+
209+ argtypesdir = $(datadir)/pygtk/2.0/argtypes
210
211=== modified file 'debian/patches/series'
212--- debian/patches/series 2010-11-25 15:08:54 +0000
213+++ debian/patches/series 2011-01-26 22:51:58 +0000
214@@ -4,4 +4,5 @@
215 04_use_PYTHON-config_check.patch
216 04_gdl_2.28.patch
217 05_gdl_2.28_more.patch
218+06_gda_libpython.patch
219 10_python_m4.patch

Subscribers

People subscribed via source and target branches