Merge lp:~scarneiro/ubuntu/oneiric/googlizer/fix-for-770806 into lp:ubuntu/oneiric/googlizer

[Sebastian Carneiro]

Changed linker parameters in Makefile to work with GCC-4.6. Make some changes based on lintian analysis.

Thanks and Best Regards.

[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)]

Similar to the other merge request you should send your diff to Debian but limit the Ubuntu merge request to fixes, changing the source format or standards version for example is not really a delta we need over Debian

[lp:~scarneiro/ubuntu/oneiric/googlizer/fix-for-770806]

5. By Sebastian Carneiro on 2011-06-22

Removed changes not required to fix the associated bug, as adviced.

[Sebastian Carneiro (scarneiro)]

Branch updated with the following:

- Removed file debian/compat.
- Restored Standards-Version and Build-Depends to previous values.
- Removed changes from debian/rules.
- Removed entries in changelog corresponding to these reverted changes.

Please, if you see anything else that needs changing, let me know. I'm grateful for your advice, so I can learn to do correct packaging.

Thanks for the patience!

Best Regards.

[Michael Terry (mterry)]

Thanks, Sebastian. I uploaded with only one change: I ran 'update-maintainer' because it is Ubuntu policy to do so whenever we introduce a delta with Debian.

I also forwarded the misc:Depends and debian/watch changes on to Debian.

review: Approve

[Sebastian Carneiro (scarneiro)]

Thanks to you, Michael.

I will take your comments into account, so I don't miss those changes from next merge proposals.

Best regards,

Sebastian.

=== modified file 'Makefile'
--- Makefile	2003-04-02 00:51:16 +0000
+++ Makefile	2011-06-22 00:31:25 +0000
@@ -4,7 +4,7 @@
 mandir=$(datadir)/man
 
 googlizer: googlizer.o
-	gcc `pkg-config --libs libgnomeui-2.0 libgnome-2.0 gtk+-2.0 glib-2.0` $< -o $@
+	gcc $< -o $@ `pkg-config --libs libgnomeui-2.0 libgnome-2.0 gtk+-2.0 glib-2.0` 
 
 googlizer.o: googlizer.c
 	gcc `pkg-config --cflags libgnomeui-2.0 libgnome-2.0 gtk+-2.0 glib-2.0` -c $<

=== modified file 'debian/changelog'
--- debian/changelog	2005-04-24 17:48:11 +0000
+++ debian/changelog	2011-06-22 00:31:25 +0000
@@ -1,3 +1,11 @@
+googlizer (0.3-3ubuntu1) oneiric; urgency=low
+
+  * Makefile: reordered linker parameters to work with GCC-4.6 (LP: #770806)
+  * debian/control: Added dependency misc dependency to binary package section. 
+  * Added debian/watch to allow uscan to check for new versions.
+
+ -- Sebastian Carneiro <[e-mail]>  Tue, 21 Jun 2011 21:20:59 -0300
+
 googlizer (0.3-3) unstable; urgency=medium
 
   * Rebuild because liblinc1 has gone.                  (closes: #306152)

=== modified file 'debian/control'
--- debian/control	2003-04-02 00:51:16 +0000
+++ debian/control	2011-06-22 00:31:25 +0000
@@ -7,7 +7,7 @@
 
 Package: googlizer
 Architecture: any
-Depends: ${shlibs:Depends}
+Depends: ${shlibs:Depends}, ${misc:Depends}
 Recommends: gnome-panel
 Description: utility to search Google via your GNOME menu/panel
  This is a very simple and very handy utility that just spawns the

=== added file 'debian/watch'
--- debian/watch	1970-01-01 00:00:00 +0000
+++ debian/watch	2011-06-22 00:31:25 +0000
@@ -0,0 +1,2 @@
+version=3
+http://ftp.acc.umu.se/pub/GNOME/sources/googlizer/(\d\.\d)/googlizer-([\d\.]*)\.tar\.gz