Merge lp:~vibhavp/ubuntu/raring/gramophone2/merge-from-debian into lp:ubuntu/raring/gramophone2

Proposed by Vibhav Pant
Status: Work in progress
Proposed branch: lp:~vibhavp/ubuntu/raring/gramophone2/merge-from-debian
Merge into: lp:ubuntu/raring/gramophone2
Diff against target: 134 lines (+57/-6)
11 files modified
.pc/.quilt_patches (+1/-0)
.pc/.quilt_series (+1/-0)
.pc/.version (+1/-0)
debian/changelog (+17/-0)
debian/control (+2/-2)
debian/patches/makefile.patch (+14/-0)
debian/patches/series (+2/-0)
debian/patches/typoCorrection.patch (+12/-0)
debian/rules (+5/-2)
debian/source/format (+1/-0)
debian/watch (+1/-2)
To merge this branch: bzr merge lp:~vibhavp/ubuntu/raring/gramophone2/merge-from-debian
Reviewer Review Type Date Requested Status
Micah Gersten (community) Needs Fixing
Ubuntu branches Pending
Review via email: mp+141277@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Micah Gersten (micahg) wrote :

The makefile changes should be converted to a quilt patch. I'd suggest that the Debian patch be disabled and the previous changes be used in their stead. I believe the Debian patch is incorrect.

review: Needs Fixing

Unmerged revisions

7. By Vibhav Pant

* Merge from Debian unstable. Remaining changes:
  - Makefile: Fix linking with --as-needed

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory '.pc'
2=== added file '.pc/.quilt_patches'
3--- .pc/.quilt_patches 1970-01-01 00:00:00 +0000
4+++ .pc/.quilt_patches 2012-12-26 17:40:28 +0000
5@@ -0,0 +1,1 @@
6+debian/patches
7
8=== added file '.pc/.quilt_series'
9--- .pc/.quilt_series 1970-01-01 00:00:00 +0000
10+++ .pc/.quilt_series 2012-12-26 17:40:28 +0000
11@@ -0,0 +1,1 @@
12+series
13
14=== added file '.pc/.version'
15--- .pc/.version 1970-01-01 00:00:00 +0000
16+++ .pc/.version 2012-12-26 17:40:28 +0000
17@@ -0,0 +1,1 @@
18+2
19
20=== modified file 'debian/changelog'
21--- debian/changelog 2010-12-03 17:37:23 +0000
22+++ debian/changelog 2012-12-26 17:40:28 +0000
23@@ -1,3 +1,20 @@
24+gramophone2 (0.8.13a-2ubuntu1) raring; urgency=low
25+
26+ * Merge from Debian unstable. Remaining changes:
27+ - Makefile: Fix linking with --as-needed.
28+
29+ -- Vibhav Pant <vibhav@ubuntu.com> Wed, 26 Dec 2012 23:06:41 +0530
30+
31+gramophone2 (0.8.13a-2) unstable; urgency=low
32+
33+ * debian/watch: fixed. (Closes: #673379)
34+ * Switch to dpkg-source 3.0 (quilt) format
35+ * debian/control: added dep on dpkg-dev (>= 1.16.1~)
36+ * added makefile.patch to corectly handle LDFLAGS on Makefile.
37+ * added typoCorrection.patch to fix a typo on source file.
38+
39+ -- Francesco Namuri <francesco@namuri.it> Sun, 12 Aug 2012 19:38:49 +0200
40+
41 gramophone2 (0.8.13a-1ubuntu1) natty; urgency=low
42
43 * Makefile: Fix linking with --as-needed.
44
45=== modified file 'debian/control'
46--- debian/control 2010-12-03 17:37:23 +0000
47+++ debian/control 2012-12-26 17:40:28 +0000
48@@ -3,8 +3,8 @@
49 Priority: optional
50 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
51 XSBC-Original-Maintainer: Francesco Namuri <francesco@namuri.it>
52-Build-Depends: debhelper (>= 5), cdbs, bison, flex
53-Standards-Version: 3.8.0
54+Build-Depends: debhelper (>= 5), cdbs, bison, flex, dpkg-dev (>= 1.16.1~)
55+Standards-Version: 3.9.3
56 Homepage: http://gramophone2.sourceforge.net/
57
58 Package: gramophone2
59
60=== added directory 'debian/patches'
61=== added file 'debian/patches/makefile.patch'
62--- debian/patches/makefile.patch 1970-01-01 00:00:00 +0000
63+++ debian/patches/makefile.patch 2012-12-26 17:40:28 +0000
64@@ -0,0 +1,14 @@
65+adds LDFLAGS to makefile
66+Index: gramophone2/Makefile
67+===================================================================
68+--- gramophone2.orig/Makefile 2012-08-12 21:54:14.886343552 +0200
69++++ gramophone2/Makefile 2012-08-13 19:29:23.430691399 +0200
70+@@ -6,7 +6,7 @@
71+ DESTDIR=/usr/local
72+
73+ default: GRAMophone.tab.c
74+- $(CC) $(CFLAGS) -o gramophone2 GRAMophone.c\
75++ $(CC) $(CPPFLAGS) $(CFLAGS) $(LDFLAGS) -o gramophone2 GRAMophone.c\
76+ grammyVM.c init.c midicode.c\
77+ midifile.c expcode.c debug.c errors.c\
78+ hash.c GRAMophone.tab.c
79
80=== added file 'debian/patches/series'
81--- debian/patches/series 1970-01-01 00:00:00 +0000
82+++ debian/patches/series 2012-12-26 17:40:28 +0000
83@@ -0,0 +1,2 @@
84+typoCorrection.patch
85+makefile.patch
86
87=== added file 'debian/patches/typoCorrection.patch'
88--- debian/patches/typoCorrection.patch 1970-01-01 00:00:00 +0000
89+++ debian/patches/typoCorrection.patch 2012-12-26 17:40:28 +0000
90@@ -0,0 +1,12 @@
91+Fixes typo on source files.
92+Index: gramophone2/grammyVM.c
93+===================================================================
94+--- gramophone2.orig/grammyVM.c 2012-08-12 21:40:22.302339825 +0200
95++++ gramophone2/grammyVM.c 2012-08-12 21:41:18.794340078 +0200
96+@@ -1552,5 +1552,5 @@
97+ Mf_writetrack=write_player_track;
98+ mfwrite(0, playerCount, global_resolution, midi);
99+ if(!debugOption)
100+- printf("midi file succesfully generated!\n");
101++ printf("midi file successfully generated!\n");
102+ }
103
104=== modified file 'debian/rules'
105--- debian/rules 2007-08-26 19:34:04 +0000
106+++ debian/rules 2012-12-26 17:40:28 +0000
107@@ -3,6 +3,9 @@
108 include /usr/share/cdbs/1/rules/debhelper.mk
109 include /usr/share/cdbs/1/class/makefile.mk
110
111+DPKG_EXPORT_BUILDFLAGS = 1
112+include /usr/share/dpkg/buildflags.mk
113+
114 DEB_MAKE_INSTALL_TARGET = install DESTDIR=$(CURDIR)/debian/tmp/
115-DEB_MAKE_ENVVARS := CFLAGS="-lm"
116-DEB_MAKE_INVOKE := $(DEB_MAKE_ENVVARS) make -C $(DEB_BUILDDIR)
117+DEB_MAKE_INVOKE := make -C $(DEB_BUILDDIR)
118+
119
120=== added directory 'debian/source'
121=== added file 'debian/source/format'
122--- debian/source/format 1970-01-01 00:00:00 +0000
123+++ debian/source/format 2012-12-26 17:40:28 +0000
124@@ -0,0 +1,1 @@
125+3.0 (quilt)
126
127=== modified file 'debian/watch'
128--- debian/watch 2008-07-12 21:25:34 +0000
129+++ debian/watch 2012-12-26 17:40:28 +0000
130@@ -1,3 +1,2 @@
131 version=3
132-http://sourceforge.net/projects/gramophone2/files \
133- http://downloads.sourceforge.net/.+/gramophone2?[-_](.+)\.tar\.gz.+
134+http://sf.net/gramophone2/gramophone2_(.*)\.tar\.gz

Subscribers

People subscribed via source and target branches

to all changes: