Merge lp:~ubuntu-branches/ubuntu/quantal/avarice/quantal-201210070312 into lp:ubuntu/quantal/avarice

Proposed by Ubuntu Package Importer
Status: Rejected
Rejected by: James Westby
Proposed branch: lp:~ubuntu-branches/ubuntu/quantal/avarice/quantal-201210070312
Merge into: lp:ubuntu/quantal/avarice
Diff against target: 376 lines (+197/-81) (has conflicts)
9 files modified
configure (+0/-72)
configure.ac (+0/-2)
debian/patches/00_debian_origin.patch (+160/-0)
debian/patches/01_support_bfd_include_guard.patch (+31/-0)
debian/patches/series (+2/-0)
debian/source/format (+1/-0)
doc/avarice.1 (+1/-0)
src/Makefile.in (+1/-1)
src/jtag2usb.cc (+1/-6)
Conflict adding file debian/patches.  Moved existing file to debian/patches.moved.
Conflict adding file debian/source.  Moved existing file to debian/source.moved.
To merge this branch: bzr merge lp:~ubuntu-branches/ubuntu/quantal/avarice/quantal-201210070312
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+128374@code.launchpad.net

Description of the change

The package importer has detected a possible inconsistency between the package history in the archive and the history in bzr. As the archive is authoritative the importer has made lp:ubuntu/quantal/avarice reflect what is in the archive and the old bzr branch has been pushed to lp:~ubuntu-branches/ubuntu/quantal/avarice/quantal-201210070312. This merge proposal was created so that an Ubuntu developer can review the situations and perform a merge/upload if necessary. There are three typical cases where this can happen.
  1. Where someone pushes a change to bzr and someone else uploads the package without that change. This is the reason that this check is done by the importer. If this appears to be the case then a merge/upload should be done if the changes that were in bzr are still desirable.
  2. The importer incorrectly detected the above situation when someone made a change in bzr and then uploaded it.
  3. The importer incorrectly detected the above situation when someone just uploaded a package and didn't touch bzr.

If this case doesn't appear to be the first situation then set the status of the merge proposal to "Rejected" and help avoid the problem in future by filing a bug at https://bugs.launchpad.net/udd linking to this merge proposal.

(this is an automatically generated message)

To post a comment you must log in.

Unmerged revisions

11. By Andrew Starr-Bochicchio

* Fix FTBFS (LP: #1058667)
  - debian/patches/01_support_bfd_include_guard.patch
    include autoconf.h before bfd.h to support new binutils
* Replace SYSFS with ATTRS in avarice.udev from Debian: #603630
* Switch to dpkg-source 3.0 (quilt) format
  - 00_debian_origin.patch
    split already applied patch to upstream source by debian package

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'configure'
2--- configure 2010-02-06 12:35:01 +0000
3+++ configure 2012-10-07 03:21:21 +0000
4@@ -4834,78 +4834,6 @@
5
6 fi
7
8-## New bfd requires libz
9-
10-{ echo "$as_me:$LINENO: checking for deflate in -lz" >&5
11-echo $ECHO_N "checking for deflate in -lz... $ECHO_C" >&6; }
12-if test "${ac_cv_lib_z_deflate+set}" = set; then
13- echo $ECHO_N "(cached) $ECHO_C" >&6
14-else
15- ac_check_lib_save_LIBS=$LIBS
16-LIBS="-lz $LIBS"
17-cat >conftest.$ac_ext <<_ACEOF
18-/* confdefs.h. */
19-_ACEOF
20-cat confdefs.h >>conftest.$ac_ext
21-cat >>conftest.$ac_ext <<_ACEOF
22-/* end confdefs.h. */
23-
24-/* Override any GCC internal prototype to avoid an error.
25- Use char because int might match the return type of a GCC
26- builtin and then its argument prototype would still apply. */
27-#ifdef __cplusplus
28-extern "C"
29-#endif
30-char deflate ();
31-int
32-main ()
33-{
34-return deflate ();
35- ;
36- return 0;
37-}
38-_ACEOF
39-rm -f conftest.$ac_objext conftest$ac_exeext
40-if { (ac_try="$ac_link"
41-case "(($ac_try" in
42- *\"* | *\`* | *\\*) ac_try_echo=\$ac_try;;
43- *) ac_try_echo=$ac_try;;
44-esac
45-eval "echo \"\$as_me:$LINENO: $ac_try_echo\"") >&5
46- (eval "$ac_link") 2>conftest.er1
47- ac_status=$?
48- grep -v '^ *+' conftest.er1 >conftest.err
49- rm -f conftest.er1
50- cat conftest.err >&5
51- echo "$as_me:$LINENO: \$? = $ac_status" >&5
52- (exit $ac_status); } && {
53- test -z "$ac_c_werror_flag" ||
54- test ! -s conftest.err
55- } && test -s conftest$ac_exeext &&
56- $as_test_x conftest$ac_exeext; then
57- ac_cv_lib_z_deflate=yes
58-else
59- echo "$as_me: failed program was:" >&5
60-sed 's/^/| /' conftest.$ac_ext >&5
61-
62- ac_cv_lib_z_deflate=no
63-fi
64-
65-rm -f core conftest.err conftest.$ac_objext conftest_ipa8_conftest.oo \
66- conftest$ac_exeext conftest.$ac_ext
67-LIBS=$ac_check_lib_save_LIBS
68-fi
69-{ echo "$as_me:$LINENO: result: $ac_cv_lib_z_deflate" >&5
70-echo "${ECHO_T}$ac_cv_lib_z_deflate" >&6; }
71-if test $ac_cv_lib_z_deflate = yes; then
72- cat >>confdefs.h <<_ACEOF
73-#define HAVE_LIBZ 1
74-_ACEOF
75-
76- LIBS="-lz $LIBS"
77-
78-fi
79-
80
81 if test "x$ac_found_bfd" = "xno"; then
82 { { $as_echo "$as_me:$LINENO: error: You need to install libbfd.a from binutils." >&5
83
84=== modified file 'configure.ac'
85--- configure.ac 2010-02-06 12:35:01 +0000
86+++ configure.ac 2012-10-07 03:21:21 +0000
87@@ -66,8 +66,6 @@
88 AC_CHECK_LIB([iberty], [xmalloc])
89 AC_CHECK_LIB([bfd], [bfd_init], , [ac_found_bfd=no])
90 AC_CHECK_LIB([usb], [usb_get_string_simple])
91-## New bfd requires libz
92-AC_CHECK_LIB([z], [deflate])
93
94 if test "x$ac_found_bfd" = "xno"; then
95 AC_MSG_ERROR([You need to install libbfd.a from binutils.])
96
97=== added directory 'debian/patches'
98=== renamed directory 'debian/patches' => 'debian/patches.moved'
99=== added file 'debian/patches/00_debian_origin.patch'
100--- debian/patches/00_debian_origin.patch 1970-01-01 00:00:00 +0000
101+++ debian/patches/00_debian_origin.patch 2012-10-07 03:21:21 +0000
102@@ -0,0 +1,160 @@
103+Index: avarice/configure
104+===================================================================
105+--- avarice.orig/configure 2012-09-30 18:15:11.625148244 +0900
106++++ avarice/configure 2012-09-30 18:15:50.725149730 +0900
107+@@ -4834,6 +4834,78 @@
108+
109+ fi
110+
111++## New bfd requires libz
112++
113++{ echo "$as_me:$LINENO: checking for deflate in -lz" >&5
114++echo $ECHO_N "checking for deflate in -lz... $ECHO_C" >&6; }
115++if test "${ac_cv_lib_z_deflate+set}" = set; then
116++ echo $ECHO_N "(cached) $ECHO_C" >&6
117++else
118++ ac_check_lib_save_LIBS=$LIBS
119++LIBS="-lz $LIBS"
120++cat >conftest.$ac_ext <<_ACEOF
121++/* confdefs.h. */
122++_ACEOF
123++cat confdefs.h >>conftest.$ac_ext
124++cat >>conftest.$ac_ext <<_ACEOF
125++/* end confdefs.h. */
126++
127++/* Override any GCC internal prototype to avoid an error.
128++ Use char because int might match the return type of a GCC
129++ builtin and then its argument prototype would still apply. */
130++#ifdef __cplusplus
131++extern "C"
132++#endif
133++char deflate ();
134++int
135++main ()
136++{
137++return deflate ();
138++ ;
139++ return 0;
140++}
141++_ACEOF
142++rm -f conftest.$ac_objext conftest$ac_exeext
143++if { (ac_try="$ac_link"
144++case "(($ac_try" in
145++ *\"* | *\`* | *\\*) ac_try_echo=\$ac_try;;
146++ *) ac_try_echo=$ac_try;;
147++esac
148++eval "echo \"\$as_me:$LINENO: $ac_try_echo\"") >&5
149++ (eval "$ac_link") 2>conftest.er1
150++ ac_status=$?
151++ grep -v '^ *+' conftest.er1 >conftest.err
152++ rm -f conftest.er1
153++ cat conftest.err >&5
154++ echo "$as_me:$LINENO: \$? = $ac_status" >&5
155++ (exit $ac_status); } && {
156++ test -z "$ac_c_werror_flag" ||
157++ test ! -s conftest.err
158++ } && test -s conftest$ac_exeext &&
159++ $as_test_x conftest$ac_exeext; then
160++ ac_cv_lib_z_deflate=yes
161++else
162++ echo "$as_me: failed program was:" >&5
163++sed 's/^/| /' conftest.$ac_ext >&5
164++
165++ ac_cv_lib_z_deflate=no
166++fi
167++
168++rm -f core conftest.err conftest.$ac_objext conftest_ipa8_conftest.oo \
169++ conftest$ac_exeext conftest.$ac_ext
170++LIBS=$ac_check_lib_save_LIBS
171++fi
172++{ echo "$as_me:$LINENO: result: $ac_cv_lib_z_deflate" >&5
173++echo "${ECHO_T}$ac_cv_lib_z_deflate" >&6; }
174++if test $ac_cv_lib_z_deflate = yes; then
175++ cat >>confdefs.h <<_ACEOF
176++#define HAVE_LIBZ 1
177++_ACEOF
178++
179++ LIBS="-lz $LIBS"
180++
181++fi
182++
183+
184+ if test "x$ac_found_bfd" = "xno"; then
185+ { { $as_echo "$as_me:$LINENO: error: You need to install libbfd.a from binutils." >&5
186+Index: avarice/configure.ac
187+===================================================================
188+--- avarice.orig/configure.ac 2012-09-30 18:15:11.557148242 +0900
189++++ avarice/configure.ac 2012-09-30 18:15:50.721149730 +0900
190+@@ -66,6 +66,8 @@
191+ AC_CHECK_LIB([iberty], [xmalloc])
192+ AC_CHECK_LIB([bfd], [bfd_init], , [ac_found_bfd=no])
193+ AC_CHECK_LIB([usb], [usb_get_string_simple])
194++## New bfd requires libz
195++AC_CHECK_LIB([z], [deflate])
196+
197+ if test "x$ac_found_bfd" = "xno"; then
198+ AC_MSG_ERROR([You need to install libbfd.a from binutils.])
199+Index: avarice/doc/avarice.1
200+===================================================================
201+--- avarice.orig/doc/avarice.1 2012-09-30 18:15:11.609148243 +0900
202++++ avarice/doc/avarice.1 2012-09-30 18:15:50.729149731 +0900
203+@@ -23,7 +23,6 @@
204+
205+ ..
206+ .TH avarice 1 "September 29, 2008"
207+-.LO 1
208+ .SH NAME
209+ avarice \- Provides an interface from avr-gdb to Atmel's JTAGICE box.
210+ .SH SYNOPSIS
211+Index: avarice/src/Makefile.in
212+===================================================================
213+--- avarice.orig/src/Makefile.in 2012-09-30 18:15:11.573148242 +0900
214++++ avarice/src/Makefile.in 2012-09-30 18:15:50.725149730 +0900
215+@@ -101,7 +101,7 @@
216+ INSTALL_STRIP_PROGRAM = @INSTALL_STRIP_PROGRAM@
217+ LDFLAGS = @LDFLAGS@
218+ LIBOBJS = @LIBOBJS@
219+-LIBS = @LIBS@
220++LIBS = -lusb /usr/lib/libbfd.a -lz -liberty -ldl
221+ LTLIBOBJS = @LTLIBOBJS@
222+ MAKEINFO = @MAKEINFO@
223+ MKDIR_P = @MKDIR_P@
224+Index: avarice/src/jtag2usb.cc
225+===================================================================
226+--- avarice.orig/src/jtag2usb.cc 2012-09-30 18:15:11.593148243 +0900
227++++ avarice/src/jtag2usb.cc 2012-09-30 18:15:50.725149730 +0900
228+@@ -40,6 +40,7 @@
229+ #include <signal.h>
230+ #include <string.h>
231+ #include <errno.h>
232++#include <assert.h>
233+
234+ #include <usb.h>
235+
236+@@ -71,6 +72,7 @@
237+ struct usb_device *dev;
238+ usb_dev_handle *udev;
239+ char *serno, *cp2;
240++ char *s;
241+ u_int16_t pid;
242+ size_t x;
243+
244+@@ -95,7 +97,9 @@
245+ * right-to-left, so only the least significant nibbles need to be
246+ * specified.
247+ */
248+- if ((serno = strchr(jtagDeviceName, ':')) != NULL)
249++ s = strdup(jtagDeviceName);
250++ assert(s != NULL);
251++ if ((serno = strchr(s, ':')) != NULL)
252+ {
253+ /* first, drop all colons there if any */
254+ cp2 = ++serno;
255+@@ -109,6 +113,7 @@
256+
257+ unixCheck(strlen(serno) <= 12, "invalid serial number \"%s\"", serno);
258+ }
259++ free(s);
260+
261+ usb_init();
262+
263
264=== added file 'debian/patches/01_support_bfd_include_guard.patch'
265--- debian/patches/01_support_bfd_include_guard.patch 1970-01-01 00:00:00 +0000
266+++ debian/patches/01_support_bfd_include_guard.patch 2012-10-07 03:21:21 +0000
267@@ -0,0 +1,31 @@
268+Support new bfd.h of binutils which depends to include autoconf.h.
269+
270+For more information please see:
271+http://sourceware.org/bugzilla/show_bug.cgi?id=14072
272+http://sourceware.org/bugzilla/show_bug.cgi?id=14243
273+Index: avarice/src/jtag2prog.cc
274+===================================================================
275+--- avarice.orig/src/jtag2prog.cc 2012-09-29 23:35:33.105618000 +0900
276++++ avarice/src/jtag2prog.cc 2012-09-30 00:03:43.606657880 +0900
277+@@ -27,6 +27,8 @@
278+ */
279+
280+
281++#include "autoconf.h"
282++
283+ #include <stdarg.h>
284+ #include <stdio.h>
285+ #include <stdlib.h>
286+Index: avarice/src/jtagprog.cc
287+===================================================================
288+--- avarice.orig/src/jtagprog.cc 2012-09-29 23:35:33.105618000 +0900
289++++ avarice/src/jtagprog.cc 2012-09-30 00:02:50.994655881 +0900
290+@@ -26,6 +26,8 @@
291+ */
292+
293+
294++#include "autoconf.h"
295++
296+ #include <stdarg.h>
297+ #include <stdio.h>
298+ #include <stdlib.h>
299
300=== added file 'debian/patches/series'
301--- debian/patches/series 1970-01-01 00:00:00 +0000
302+++ debian/patches/series 2012-10-07 03:21:21 +0000
303@@ -0,0 +1,2 @@
304+00_debian_origin.patch
305+01_support_bfd_include_guard.patch
306
307=== added directory 'debian/source'
308=== renamed directory 'debian/source' => 'debian/source.moved'
309=== added file 'debian/source/format'
310--- debian/source/format 1970-01-01 00:00:00 +0000
311+++ debian/source/format 2012-10-07 03:21:21 +0000
312@@ -0,0 +1,1 @@
313+3.0 (quilt)
314
315=== modified file 'doc/avarice.1'
316--- doc/avarice.1 2010-01-24 11:21:12 +0000
317+++ doc/avarice.1 2012-10-07 03:21:21 +0000
318@@ -23,6 +23,7 @@
319
320 ..
321 .TH avarice 1 "September 29, 2008"
322+.LO 1
323 .SH NAME
324 avarice \- Provides an interface from avr-gdb to Atmel's JTAGICE box.
325 .SH SYNOPSIS
326
327=== modified file 'src/Makefile.in'
328--- src/Makefile.in 2010-02-06 12:35:01 +0000
329+++ src/Makefile.in 2012-10-07 03:21:21 +0000
330@@ -101,7 +101,7 @@
331 INSTALL_STRIP_PROGRAM = @INSTALL_STRIP_PROGRAM@
332 LDFLAGS = @LDFLAGS@
333 LIBOBJS = @LIBOBJS@
334-LIBS = -lusb /usr/lib/libbfd.a -lz -liberty -ldl
335+LIBS = @LIBS@
336 LTLIBOBJS = @LTLIBOBJS@
337 MAKEINFO = @MAKEINFO@
338 MKDIR_P = @MKDIR_P@
339
340=== modified file 'src/jtag2usb.cc'
341--- src/jtag2usb.cc 2010-02-06 12:35:01 +0000
342+++ src/jtag2usb.cc 2012-10-07 03:21:21 +0000
343@@ -40,7 +40,6 @@
344 #include <signal.h>
345 #include <string.h>
346 #include <errno.h>
347-#include <assert.h>
348
349 #include <usb.h>
350
351@@ -72,7 +71,6 @@
352 struct usb_device *dev;
353 usb_dev_handle *udev;
354 char *serno, *cp2;
355- char *s;
356 u_int16_t pid;
357 size_t x;
358
359@@ -97,9 +95,7 @@
360 * right-to-left, so only the least significant nibbles need to be
361 * specified.
362 */
363- s = strdup(jtagDeviceName);
364- assert(s != NULL);
365- if ((serno = strchr(s, ':')) != NULL)
366+ if ((serno = strchr(jtagDeviceName, ':')) != NULL)
367 {
368 /* first, drop all colons there if any */
369 cp2 = ++serno;
370@@ -113,7 +109,6 @@
371
372 unixCheck(strlen(serno) <= 12, "invalid serial number \"%s\"", serno);
373 }
374- free(s);
375
376 usb_init();
377

Subscribers

People subscribed via source and target branches

to all changes: