Merge lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756 into lp:ubuntu/oneiric/pppstatus

[Sebastian Carneiro]

Changed linker parameters to work with GCC-4.6. Make other changes mainly from lintian recommendations.

[Sebastien Bacher (seb128)]

Thank you for the work you did there, the package is coming from Debian (https://wiki.ubuntu.com/Debian) though so ideally we would be sent to their bug system (http://www.debian.org/Bugs/Reporting) so they can benefit from your fixes and it Ubuntu has less changes to carry and maintain which are duplicated efforts over what Debian is doing

We also try to limit the changes we do over Debian to required ones, while you standards version update and watch files are nice cleanups they are not really required in an Ubuntu specific way also

[Sebastian Carneiro (scarneiro)]

Sebastien, thanks for your review.

I need clarification about what the right course of action would be in a case like this:

 - Do I have to revert changes leaving only the required ones, or
 - Do I have to forward to Debian only the required changes, leaving the rest out of the patch ?

Again,

Thanks for your time.

Best regards.

[Sebastien Bacher (seb128)]

What I would recommend to do is to forward your diff it is to Debian and update the merge request to drop unrequired changes, fixes are fine to keep (i.e the maintainer script tokens or watch fixes) but changing the source format in an Ubuntu specific way doesn't seem a good idea

You also don't need to list the maintainer update in the changelog that's something we do for each source we modify in Ubuntu

[lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756]

5. By Sebastian Carneiro on 2011-06-21

Removed unrequired changes, following advice from Sebastien Bacher.

6. By Sebastian Carneiro on 2011-06-21

Removed unrequired changes, following advice from Sebastien Bacher.

[Sebastian Carneiro (scarneiro)]

Sebastien, did the following update to this branch:

- Removed debian/source/format;
- Restored previous values in debian/control for Standards-Version and Build-Depends.
- Removed references to previous change from changelog.

Please let me know if it's ok now, or further changes are needed.

Thanks for the patience!

Best regasrds.-

[Michael Terry (mterry)]

Sebastian, would you mind just making this branch hold the Makefile fix? The other changes are great and would ideally be sent on to Debian. But if Ubuntu accepts the extra changes, it would make it harder for us to go back to being in sync (we'd generally wait until Debian took them too).

[Sebastian Carneiro (scarneiro)]

Sure, no problem.

I will to that and let you know.

Thanks for your help !

Sebastian.

[lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756]

7. By Sebastian Carneiro on 2011-07-06

Reverted previous changes and holding only the ones on Makefile.

[Sebastian Carneiro (scarneiro)]

Changes reverted. Only the change on Makefile remains.

Best regards.

[lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756]

8. By Sebastian Carneiro on 2011-07-06

Removed extra line in changelog entry.

[Stefano Rivera (stefanor)]

Looks good, thanks.

Only issue was overly-long changelog lines. (lintian complains about it) Fixed on upload.

review: Approve

=== modified file 'Makefile'
--- Makefile	2003-02-01 13:00:00 +0000
+++ Makefile	2011-07-06 00:19:27 +0000
@@ -17,7 +17,7 @@
 all : pppstatus printdone
 
 pppstatus : pppstatus.o
-	gcc $(CFLAGS) $(LDFLAGS) -o ${BIN} ${SRC}
+	gcc $(CFLAGS) -o ${BIN} ${SRC} $(LDFLAGS)
 
 pppstatus.o : pppstatus.h
 

=== modified file 'debian/changelog'
--- debian/changelog	2007-12-14 21:25:03 +0000
+++ debian/changelog	2011-07-06 00:19:27 +0000
@@ -1,3 +1,9 @@
+pppstatus (0.4.2-10ubuntu1) oneiric; urgency=low
+
+  * Makefile: Changed order of linker parameters to work with GCC-4.6 (LP: #770756)
+
+ -- Sebastian Carneiro <[e-mail]>  Tue, 05 Jul 2011 21:13:54 -0300
+
 pppstatus (0.4.2-10) unstable; urgency=low
 
   * Added code to postinst to remove obsolete pppstatus-clean.

=== modified file 'debian/control'
--- debian/control	2007-05-04 17:06:43 +0000
+++ debian/control	2011-07-06 00:19:27 +0000
@@ -1,7 +1,8 @@
 Source: pppstatus
 Section: net
 Priority: optional
-Maintainer: John Hasler <[e-mail]>
+Maintainer: Ubuntu Developers <[e-mail]>
+XSBC-Original-Maintainer: John Hasler <[e-mail]>
 Build-Depends: debhelper (>> 2.0.0), libncurses5-dev
 Standards-Version: 3.7.2.2