Merge lp:~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887 into lp:ubuntu/oneiric/notification-daemon

[Ben Tucker]

Fixed LP bug #557887 by moving the homepage URL, which was in the description, into the more appropriate Homepage tag.

[Robert Roth (evfool)]

The fix looks fine, but there's a nonsense sentence left at the end of the description, although this was not reported in the bug ("as per the Desktop Notifications spec"). I guess it would also worth removing that at the same time, and the whole change should be forwarded to Debian (they should accept it without objections), to avoid having distro patches.

review: Needs Fixing

[Ben Tucker (neonofneophyte)]

> The fix looks fine, but there's a nonsense sentence left at the end of the
> description, although this was not reported in the bug ("as per the Desktop
> Notifications spec"). I guess it would also worth removing that at the same
> time, and the whole change should be forwarded to Debian (they should accept
> it without objections), to avoid having distro patches.

I will go ahead and remove that last sentence, then. Also, how do I go about forwarding the patch to Debian?

[lp:~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887]

51. By Ben Tucker on 2011-06-24

debian/control.in: Removed the very last sentence of the description,
which was nonsense.
(https://code.launchpad.net/~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887/+merge/65720)

[Robert Roth (evfool)]

Thanks, now it's OK. Regarding forwarding to debian, see the https://wiki.ubuntu.com/Debian/Bugs page or ask for help on IRC on the #ubuntu-devel channel.

[Ben Tucker (neonofneophyte)]

I have submitted the patch to Debian, and it is now pending review (sorry
for the delay, I was having some trouble with the tools)

On Fri, Jun 24, 2011 at 2:07 PM, Robert Roth <email address hidden> wrote:

> Thanks, now it's OK. Regarding forwarding to debian, see the
> https://wiki.ubuntu.com/Debian/Bugs page or ask for help on IRC on the
> #ubuntu-devel channel.
> --
>
> https://code.launchpad.net/~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887/+merge/65720
> You are the owner of
> lp:~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887.
>

[Robert Roth (evfool)]

Thanks for submitting it to debian. The tooling setup takes indeed a long time at first, but once you have it setup, you are ready for development :) So keep up the good work.

[Chris Halse Rogers (raof)]

Since the patch has been sitting in the Debian BTS for a month now I've gone ahead and cleaned this up and uploaded.

I merged the top two changelog entries into one - 0.7.1-2ubuntu1 was never in the archive, so there's no need to make a 0.7.1-2ubuntu2 changelog entry - and fixed the Launchpad bug closing line - you need (LP: #number), not (LP #number).

Thanks for your work!

=== modified file 'debian/changelog'
--- debian/changelog	2011-06-08 09:14:07 +0000
+++ debian/changelog	2011-06-24 20:36:00 +0000
@@ -1,3 +1,18 @@
+notification-daemon (0.7.1-2ubuntu2) oneiric; urgency=low
+
+  * debian/control.in: Removed the very last sentence of the description,
+    which was nonsense. 
+    (https://code.launchpad.net/~neonofneophyte/ubuntu/oneiric/notification-daemon/fix-for-557887/+merge/65720)
+
+ -- Ben Tucker <[e-mail]>  Fri, 24 Jun 2011 13:24:37 -0700
+
+notification-daemon (0.7.1-2ubuntu1) oneiric; urgency=low
+
+  * debian/control.in: Made the URL at the bottom of the description a
+    homepage instead. (LP #557887)
+
+ -- Ben Tucker <[e-mail]>  Thu, 23 Jun 2011 12:47:47 -0700
+
 notification-daemon (0.7.1-2) experimental; urgency=low
 
   * debian/control.in: Add back libdbus-glib-1-dev build dependency. configure

=== modified file 'debian/control.in'
--- debian/control.in	2011-06-08 09:14:07 +0000
+++ debian/control.in	2011-06-24 20:36:00 +0000
@@ -21,6 +21,7 @@
          ${misc:Depends}
 Conflicts: notify-daemon
 Replaces: notify-daemon
+Homepage: http://www.galago-project.org/specs/notification/index.php
 Description: daemon to displays passive pop-up notifications
  notification-daemon displays passive pop-up notifications, as per
  the Desktop Notifications Specification.
@@ -29,7 +30,5 @@
  doing passive pop-up notifications on the Linux desktop. These are
  designed to notify the user of something without interrupting their
  work with a dialog box that they must close.  Passive popups can
- automatically disappear after a short period of time.  as per the
- Desktop Notifications spec
+ automatically disappear after a short period of time.
  .
- http://www.galago-project.org/specs/notification/index.php