Merge lp:~valavanisalex/ubuntu/saucy/inkscape/fix-1194855 into lp:ubuntu/saucy/inkscape

Proposed by Alex Valavanis
Status: Merged
Merge reported by: James Page
Merged at revision: not available
Proposed branch: lp:~valavanisalex/ubuntu/saucy/inkscape/fix-1194855
Merge into: lp:ubuntu/saucy/inkscape
Diff against target: 300 lines (+239/-18)
5 files modified
debian/changelog (+29/-0)
debian/control (+3/-2)
debian/patches/04-Fix_FTBFS_on_gcc-4.8.patch (+206/-0)
debian/patches/bzr-fix-for-gcc48.patch (+0/-15)
debian/patches/series (+1/-1)
To merge this branch: bzr merge lp:~valavanisalex/ubuntu/saucy/inkscape/fix-1194855
Reviewer Review Type Date Requested Status
James Page Approve
Ubuntu branches Pending
Review via email: mp+171599@code.launchpad.net

This proposal supersedes a proposal from 2013-06-26.

To post a comment you must log in.
Revision history for this message
James Page (james-page) :
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 2013-04-29 11:22:17 +0000
3+++ debian/changelog 2013-06-26 16:12:27 +0000
4@@ -1,3 +1,32 @@
5+inkscape (0.48.4-1ubuntu1) saucy; urgency=low
6+
7+ * Merge from Debian Unstable (LP: #1194855)
8+ - debian/control:
9+ + Ubuntu maintainers
10+ + Build against lcms2
11+ + promote libgnomevfs2-extra to recommends
12+ + Demote pstoedit to Suggests (universe package).
13+ + Suggest transfig to allow import of xfig files
14+ + Drop unnecessary python and skencil suggestions
15+ - Handle translation tasks for Ubuntu main:
16+ + debian/rules: Add dh_translations to binary-arch rule
17+ + debian/control: Build-Depends dh-translations
18+ - Drop debian/patches/bzr-fix-for-gcc48.patch: Superseded by
19+ 04-Fix_FTBFS_on_gcc-4.8.patch
20+ - debian/patches/01_add_unity_quicklist_support.patch: add "New Drawing"
21+ to Unity quicklist
22+
23+ -- Alexander Valavanis <valavanisalex@gmail.com> Wed, 26 Jun 2013 13:37:53 +0100
24+
25+inkscape (0.48.4-1) unstable; urgency=low
26+
27+ * Upload to unstable
28+ * debian/control: added myself to Uploaders
29+ * debian/patches/: patchset updated
30+ - 04-Fix_FTBFS_on_gcc-4.8.patch added (Closes: #701297)
31+
32+ -- Matteo F. Vescovi <mfv.debian@gmail.com> Fri, 21 Jun 2013 15:13:32 +0200
33+
34 inkscape (0.48.4-0.1ubuntu3) saucy; urgency=low
35
36 * debian/patches/bzr-fix-for-gcc48.patch: fix build with gcc 4.8.
37
38=== modified file 'debian/control'
39--- debian/control 2013-02-11 19:00:07 +0000
40+++ debian/control 2013-06-26 16:12:27 +0000
41@@ -6,6 +6,7 @@
42 Homepage: http://www.inkscape.org/
43 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
44 XSBC-Original-Maintainer: Wolfram Quester <wolfi@sigxcpu.org>
45+Uploaders: Matteo F. Vescovi <mfv.debian@gmail.com>
46 Build-Depends: autoconf,
47 automake,
48 debhelper (>= 7.0.0),
49@@ -30,8 +31,8 @@
50 libpopt-dev,
51 libsigc++-2.0-dev (>= 2.0.16-2),
52 libtool,
53- libwpd-dev,
54- libwpg-dev,
55+ libwpd-dev (>= 0.9.0),
56+ libwpg-dev (>= 0.2.0),
57 libxml-parser-perl,
58 libxml2-dev (>= 2-2.4.24),
59 libxslt1-dev,
60
61=== added file 'debian/patches/04-Fix_FTBFS_on_gcc-4.8.patch'
62--- debian/patches/04-Fix_FTBFS_on_gcc-4.8.patch 1970-01-01 00:00:00 +0000
63+++ debian/patches/04-Fix_FTBFS_on_gcc-4.8.patch 2013-06-26 16:12:27 +0000
64@@ -0,0 +1,206 @@
65+From: "Matteo F. Vescovi" <mfv.debian@gmail.com>
66+Date: Fri, 21 Jun 2013 15:05:53 +0200
67+Subject: Fix_FTBFS_on_gcc-4.8
68+
69+---
70+ src/connector-context.cpp | 22 +++++++++++-----------
71+ src/extension/effect.cpp | 2 +-
72+ src/gradient-drag.cpp | 2 +-
73+ src/libavoid/vpsc.cpp | 2 +-
74+ src/libcola/straightener.cpp | 6 +++---
75+ src/libcroco/cr-parser.c | 2 +-
76+ src/libvpsc/block.cpp | 2 +-
77+ src/widgets/desktop-widget.h | 2 +-
78+ src/widgets/toolbox.cpp | 4 ++--
79+ 9 files changed, 22 insertions(+), 22 deletions(-)
80+
81+diff --git a/src/connector-context.cpp b/src/connector-context.cpp
82+index f7d18a3..f0dea9a 100644
83+--- a/src/connector-context.cpp
84++++ b/src/connector-context.cpp
85+@@ -1287,15 +1287,15 @@ cc_connector_rerouting_finish(SPConnectorContext *const cc, Geom::Point *const p
86+ if (found) {
87+ if (cc->clickedhandle == cc->endpt_handle[0]) {
88+ sp_object_setAttribute(cc->clickeditem,
89+- "inkscape:connection-start", shape_label, false);
90++ "inkscape:connection-start", shape_label, NULL);
91+ sp_object_setAttribute(cc->clickeditem,
92+- "inkscape:connection-start-point", cpid, false);
93++ "inkscape:connection-start-point", cpid, NULL);
94+ }
95+ else {
96+ sp_object_setAttribute(cc->clickeditem,
97+- "inkscape:connection-end", shape_label, false);
98++ "inkscape:connection-end", shape_label, NULL);
99+ sp_object_setAttribute(cc->clickeditem,
100+- "inkscape:connection-end-point", cpid, false);
101++ "inkscape:connection-end-point", cpid, NULL);
102+ }
103+ g_free(shape_label);
104+ }
105+@@ -1435,26 +1435,26 @@ spcc_flush_white(SPConnectorContext *cc, SPCurve *gc)
106+
107+ bool connection = false;
108+ sp_object_setAttribute(cc->newconn, "inkscape:connector-type",
109+- cc->isOrthogonal ? "orthogonal" : "polyline", false);
110++ cc->isOrthogonal ? "orthogonal" : "polyline", NULL);
111+ sp_object_setAttribute(cc->newconn, "inkscape:connector-curvature",
112+- Glib::Ascii::dtostr(cc->curvature).c_str(), false);
113++ Glib::Ascii::dtostr(cc->curvature).c_str(), NULL);
114+ if (cc->shref)
115+ {
116+ sp_object_setAttribute(cc->newconn, "inkscape:connection-start",
117+- cc->shref, false);
118++ cc->shref, NULL);
119+ if (cc->scpid)
120+ sp_object_setAttribute(cc->newconn, "inkscape:connection-start-point",
121+- cc->scpid, false);
122++ cc->scpid, NULL);
123+ connection = true;
124+ }
125+
126+ if (cc->ehref)
127+ {
128+ sp_object_setAttribute(cc->newconn, "inkscape:connection-end",
129+- cc->ehref, false);
130++ cc->ehref, NULL);
131+ if (cc->ecpid)
132+ sp_object_setAttribute(cc->newconn, "inkscape:connection-end-point",
133+- cc->ecpid, false);
134++ cc->ecpid, NULL);
135+ connection = true;
136+ }
137+ // Process pending updates.
138+@@ -1937,7 +1937,7 @@ void cc_selection_set_avoid(bool const set_avoid)
139+
140+ if (cc_item_is_shape(item)) {
141+ sp_object_setAttribute(item, "inkscape:connector-avoid",
142+- value, false);
143++ value, NULL);
144+ item->avoidRef->handleSettingChange();
145+ changes++;
146+ }
147+diff --git a/src/extension/effect.cpp b/src/extension/effect.cpp
148+index 9a22c07..192c72e 100644
149+--- a/src/extension/effect.cpp
150++++ b/src/extension/effect.cpp
151+@@ -323,7 +323,7 @@ Effect::set_last_effect (Effect * in_effect)
152+ Inkscape::XML::Node *
153+ Effect::find_menu (Inkscape::XML::Node * menustruct, const gchar *name)
154+ {
155+- if (menustruct == NULL) return false;
156++ if (menustruct == NULL) return NULL;
157+ for (Inkscape::XML::Node * child = menustruct;
158+ child != NULL;
159+ child = child->next()) {
160+diff --git a/src/gradient-drag.cpp b/src/gradient-drag.cpp
161+index 0459cd1..d620817 100644
162+--- a/src/gradient-drag.cpp
163++++ b/src/gradient-drag.cpp
164+@@ -1969,7 +1969,7 @@ GrDrag::deleteSelected (bool just_one)
165+ {
166+ if (!selected) return;
167+
168+- SPDocument *document = false;
169++ SPDocument *document = NULL;
170+
171+ struct StructStopInfo {
172+ SPStop * spstop;
173+diff --git a/src/libavoid/vpsc.cpp b/src/libavoid/vpsc.cpp
174+index 19d3603..1646dda 100644
175+--- a/src/libavoid/vpsc.cpp
176++++ b/src/libavoid/vpsc.cpp
177+@@ -422,7 +422,7 @@ Constraint* IncSolver::mostViolated(Constraints &l) {
178+ // downwards. There is always at least 1 element in the
179+ // vector because of search.
180+ // TODO check this logic and add parens:
181+- if((deletePoint != end) && ((minSlack < ZERO_UPPERBOUND) && !v->active || v->equality)) {
182++ if((deletePoint != end) && (((minSlack < ZERO_UPPERBOUND) && !v->active) || v->equality)) {
183+ *deletePoint = l[l.size()-1];
184+ l.resize(l.size()-1);
185+ }
186+diff --git a/src/libcola/straightener.cpp b/src/libcola/straightener.cpp
187+index e237c03..7c73cb9 100644
188+--- a/src/libcola/straightener.cpp
189++++ b/src/libcola/straightener.cpp
190+@@ -108,7 +108,7 @@ namespace straightener {
191+ int compare_events(const void *a, const void *b) {
192+ Event *ea=*(Event**)a;
193+ Event *eb=*(Event**)b;
194+- if(ea->v!=NULL&&ea->v==eb->v||ea->e!=NULL&&ea->e==eb->e) {
195++ if((ea->v!=NULL&&ea->v==eb->v)||(ea->e!=NULL&&ea->e==eb->e)) {
196+ // when comparing opening and closing from object
197+ // open must come first
198+ if(ea->type==Open) return -1;
199+@@ -263,8 +263,8 @@ namespace straightener {
200+ // node is on an edge
201+ Edge *edge=(*i)->edge;
202+ if(!edge->isEnd(v->id)
203+- &&(l!=NULL&&!edge->isEnd(l->id)||l==NULL)
204+- &&(r!=NULL&&!edge->isEnd(r->id)||r==NULL)) {
205++ &&((l!=NULL&&!edge->isEnd(l->id))||l==NULL)
206++ &&((r!=NULL&&!edge->isEnd(r->id))||r==NULL)) {
207+ if(lastNode!=NULL) {
208+ //printf(" Rule A: Constraint: v%d +g <= v%d\n",lastNode->id,(*i)->id);
209+ cs.push_back(createConstraint(lastNode,*i,dim));
210+diff --git a/src/libcroco/cr-parser.c b/src/libcroco/cr-parser.c
211+index 5b0a56f..a8e2de5 100644
212+--- a/src/libcroco/cr-parser.c
213++++ b/src/libcroco/cr-parser.c
214+@@ -2408,7 +2408,7 @@ cr_parser_parse_stylesheet (CRParser * a_this)
215+ import_string,
216+ NULL, &location) ;
217+
218+- if ((PRIVATE (a_this)->sac_handler->resolve_import == TRUE)) {
219++ if (PRIVATE (a_this)->sac_handler->resolve_import == TRUE) {
220+ /*
221+ *TODO: resolve the
222+ *import rule.
223+diff --git a/src/libvpsc/block.cpp b/src/libvpsc/block.cpp
224+index 221df53..0bd662f 100644
225+--- a/src/libvpsc/block.cpp
226++++ b/src/libvpsc/block.cpp
227+@@ -72,7 +72,7 @@ void Block::setUpConstraintHeap(PairingHeap<Constraint*>* &h,bool in) {
228+ for (Cit j=cs->begin();j!=cs->end();++j) {
229+ Constraint *c=*j;
230+ c->timeStamp=blockTimeCtr;
231+- if (c->left->block != this && in || c->right->block != this && !in) {
232++ if ((c->left->block != this && in) || (c->right->block != this && !in)) {
233+ h->insert(c);
234+ }
235+ }
236+diff --git a/src/widgets/desktop-widget.h b/src/widgets/desktop-widget.h
237+index d853e09..aab73c9 100644
238+--- a/src/widgets/desktop-widget.h
239++++ b/src/widgets/desktop-widget.h
240+@@ -239,7 +239,7 @@ struct SPDesktopWidget {
241+ private:
242+ GtkWidget *tool_toolbox;
243+ GtkWidget *aux_toolbox;
244+- GtkWidget *commands_toolbox,;
245++ GtkWidget *commands_toolbox;
246+ GtkWidget *snap_toolbox;
247+
248+ static void init(SPDesktopWidget *widget);
249+diff --git a/src/widgets/toolbox.cpp b/src/widgets/toolbox.cpp
250+index dfa5e08..8c1600d 100644
251+--- a/src/widgets/toolbox.cpp
252++++ b/src/widgets/toolbox.cpp
253+@@ -7932,7 +7932,7 @@ static void sp_connector_orthogonal_toggled( GtkToggleAction* act, GObject *tbl
254+
255+ if (cc_item_is_connector(item)) {
256+ sp_object_setAttribute(item, "inkscape:connector-type",
257+- value, false);
258++ value, NULL);
259+ item->avoidRef->handleSettingChange();
260+ modmade = true;
261+ }
262+@@ -7981,7 +7981,7 @@ static void connector_curvature_changed(GtkAdjustment *adj, GObject* tbl)
263+
264+ if (cc_item_is_connector(item)) {
265+ sp_object_setAttribute(item, "inkscape:connector-curvature",
266+- value, false);
267++ value, NULL);
268+ item->avoidRef->handleSettingChange();
269+ modmade = true;
270+ }
271
272=== removed file 'debian/patches/bzr-fix-for-gcc48.patch'
273--- debian/patches/bzr-fix-for-gcc48.patch 2013-04-29 11:22:17 +0000
274+++ debian/patches/bzr-fix-for-gcc48.patch 1970-01-01 00:00:00 +0000
275@@ -1,15 +0,0 @@
276-Description: Fix FTBFS with gcc 4.8
277-Origin: upstream, part of r10359 of lp:inkscape
278-Last-Update: 2013-04-29
279-
280---- a/src/widgets/desktop-widget.h
281-+++ b/src/widgets/desktop-widget.h
282-@@ -239,7 +239,7 @@
283- private:
284- GtkWidget *tool_toolbox;
285- GtkWidget *aux_toolbox;
286-- GtkWidget *commands_toolbox,;
287-+ GtkWidget *commands_toolbox;
288- GtkWidget *snap_toolbox;
289-
290- static void init(SPDesktopWidget *widget);
291
292=== modified file 'debian/patches/series'
293--- debian/patches/series 2013-04-29 11:22:17 +0000
294+++ debian/patches/series 2013-06-26 16:12:27 +0000
295@@ -2,4 +2,4 @@
296 02-drop-pdf-ps.diff
297 01_add_unity_quicklist_support.patch
298 03-fix-LP911146.diff
299-bzr-fix-for-gcc48.patch
300+04-Fix_FTBFS_on_gcc-4.8.patch

Subscribers

People subscribed via source and target branches