Your message dated Tue, 18 Nov 2014 19:24:51 +0100
with message-id <546b8ef3.9030...@thykier.net>
and subject line Re: Bug#769880: unblock: hello/2.9-2
has caused the Debian Bug report #769880,
regarding unblock: hello/2.9-2
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
769880: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=769880
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian....@packages.debian.org
Usertags: unblock
I'd like to ask the unblocking of "hello".
It fixes #767172 which is some kind of brown-paper bug (one of the
star features of "hello" is that it says "hello" according to LANG,
which was not the case).
I would have liked to include the copyright file contributed by Osamu Aoki,
but unfortunately didn't find the time to review it appropriately, so
this is deferred for "stretch".
debdiff follows, excluding the config.guess/config.sub part.
Thanks.
diff -Nru --exclude '*99-config-guess-config-sub' hello-2.9/debian/changelog
hello-2.9/debian/changelog
--- hello-2.9/debian/changelog 2014-04-12 15:23:34.000000000 +0200
+++ hello-2.9/debian/changelog 2014-11-06 12:04:53.000000000 +0100
@@ -1,3 +1,17 @@
+hello (2.9-2) unstable; urgency=low
+
+ * Apply patch from Reuben Thomas to fix i18n of default message.
+ This is upstream commit c4aed00. Closes: #767172.
+ * The previous change in src/hello.c trigger a rebuild of man/hello.1
+ that we don't need. Add a "touch man/hello.1" to avoid it.
+ * Use Breaks: hello-debhelper (<< 2.9), not Conflicts,
+ as hello-debhelper is deprecated.
+ * Restore simple watch file from old hello package that was lost
+ when the packages were renamed.
+ * Update 99-config-guess-config-sub patch.
+
+ -- Santiago Vila <sanv...@debian.org> Thu, 06 Nov 2014 12:03:40 +0100
+
hello (2.9-1) unstable; urgency=low
* New upstream release. Closes: #744195.
diff -Nru --exclude '*99-config-guess-config-sub' hello-2.9/debian/control
hello-2.9/debian/control
--- hello-2.9/debian/control 2014-04-12 15:00:00.000000000 +0200
+++ hello-2.9/debian/control 2014-11-06 12:00:00.000000000 +0100
@@ -9,8 +9,9 @@
Package: hello
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}
-Conflicts: hello-traditional, hello-debhelper (<< 2.9)
+Conflicts: hello-traditional
Replaces: hello-traditional, hello-debhelper (<< 2.9)
+Breaks: hello-debhelper (<< 2.9)
Description: example package based on GNU hello
The GNU hello program produces a familiar, friendly greeting. It
allows non-programmers to use a classic computer science tool which
diff -Nru --exclude '*99-config-guess-config-sub'
hello-2.9/debian/patches/01-fix-i18n-of-default-message
hello-2.9/debian/patches/01-fix-i18n-of-default-message
--- hello-2.9/debian/patches/01-fix-i18n-of-default-message 1970-01-01
01:00:00.000000000 +0100
+++ hello-2.9/debian/patches/01-fix-i18n-of-default-message 2014-11-06
12:01:00.000000000 +0100
@@ -0,0 +1,34 @@
+From: Reuben Thomas <r...@sc3d.org>
+Subject: Fix i18n of default message: don't initialise it before we set up
gettext
+Bug-Debian: http://bugs.debian.org/767172
+
+--- a/src/hello.c
++++ b/src/hello.c
+@@ -1,7 +1,7 @@
+ /* hello.c -- print a greeting message and exit.
+
+ Copyright 1992, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2005,
+- 2006, 2007, 2008, 2010, 2011 Free Software Foundation, Inc.
++ 2006, 2007, 2008, 2010, 2011, 2013 Free Software Foundation, Inc.
+
+ This program is free software: you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+@@ -47,7 +47,7 @@
+ {
+ int optc;
+ int lose = 0;
+- const char *greeting_msg = _("Hello, world!");
++ const char *greeting_msg;
+ wchar_t *mb_greeting;
+ size_t len;
+ greeting_type g = greet_traditional;
+@@ -63,6 +63,9 @@
+ textdomain (PACKAGE);
+ #endif
+
++ /* Having initialized gettext, get the default message. */
++ greeting_msg = _("Hello, world!");
++
+ /* Even exiting has subtleties. On exit, if any writes failed, change
+ the exit status. The /dev/full device on GNU/Linux can be used for
+ testing; for instance, hello >/dev/full should exit unsuccessfully.
diff -Nru --exclude '*99-config-guess-config-sub'
hello-2.9/debian/patches/series hello-2.9/debian/patches/series
--- hello-2.9/debian/patches/series 2014-04-12 15:00:00.000000000 +0200
+++ hello-2.9/debian/patches/series 2014-11-06 12:00:00.000000000 +0100
@@ -1 +1,2 @@
+01-fix-i18n-of-default-message
99-config-guess-config-sub
diff -Nru --exclude '*99-config-guess-config-sub' hello-2.9/debian/rules
hello-2.9/debian/rules
--- hello-2.9/debian/rules 2014-04-12 15:00:00.000000000 +0200
+++ hello-2.9/debian/rules 2014-11-06 12:00:00.000000000 +0100
@@ -2,6 +2,10 @@
%:
dh $@
+override_dh_auto_build:
+ touch man/hello.1
+ dh_auto_build
+
override_dh_auto_clean:
[ ! -f Makefile ] || $(MAKE) distclean
diff -Nru --exclude '*99-config-guess-config-sub' hello-2.9/debian/watch
hello-2.9/debian/watch
--- hello-2.9/debian/watch 1970-01-01 01:00:00.000000000 +0100
+++ hello-2.9/debian/watch 2014-11-06 12:00:00.000000000 +0100
@@ -0,0 +1,9 @@
+# debian/watch
+# Debian uscan file for hello package
+# Manpage: uscan(1)
+
+# Compulsory line, format version of this file
+version=3
+
+# Current version from GNU site
+http://ftp.gnu.org/gnu/hello/hello-(.*).tar.gz
--- End Message ---
--- Begin Message ---
On 2014-11-18 19:20, Santiago Vila wrote:
> On Tue, Nov 18, 2014 at 07:06:46PM +0100, Niels Thykier wrote:
>> [...]
>>
>> Hi,
>>
>> You *also* need a Replaces, since hello ships files that hello-debhelper
>> used to ship. Breaks without Replaces is insufficient to avoid upgrade
>> issues for this case.
>
> Of course. There *is* a Replaces.
>
> [...]
> Thanks.
>
>
Indeed, silly me. Unblocked, thanks.
~Niels
--- End Message ---