[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#768893: marked as done (unblock: libeatmydata/82-3)

Your message dated Thu, 13 Nov 2014 20:41:57 +0000
with message-id <20141113204157.GL21455@lupin.home.powdarrmonkey.net>
and subject line Re: Bug#768893: unblock: libeatmydata/82-3
has caused the Debian Bug report #768893,
regarding unblock: libeatmydata/82-3
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 owner@bugs.debian.org
immediately.)


-- 
768893: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=768893
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package libeatmydata.

This upload fix an imbortant bug,
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=765738
that renders the program useless in some cases. Note that that patch already got
included upstream, in the release 105 (see the debian vcs).
It also tries to fix an ftbfs on a couple of non-release arch, but it seems that
a patch it's also needed.
To kid the test suite and fix the ftbfs a simple

--- a/test_run.sh
+++ b/test_run.sh
@@ -9,6 +9,8 @@
 # COPYING file in the root project directory for full text.
 #
 
+`which strace` || exit 0
+
 # Get filename we want to run without path
 name=`echo $1 | sed 's/.*\/\(libeatmydata\/.*[^\/]*\)$/\1/'`

is enough.
Do you feel I should upload such a change, for including it in jessie?
I would be for it.


Current debdiff follows:

diff -Nru libeatmydata-82/debian/changelog libeatmydata-82/debian/changelog
--- libeatmydata-82/debian/changelog    2014-10-15 20:25:18.000000000 +0200
+++ libeatmydata-82/debian/changelog    2014-11-09 22:16:55.000000000 +0100
@@ -1,3 +1,13 @@
+libeatmydata (82-3) unstable; urgency=low
+
+  * debian/patches/bug-702711.patch:
+    + [c95e942] fix the bug #702711 also for 32 bit arches (Closes: #765738).
+    + [0167220] update headers.
+  * [3fabf6f] debian/control: build-depend on strace only on non hurd and non
+    kfreebsd systems, where it is unavailable.
+
+ -- Mattia Rizzolo <mattia@mapreri.org>  Sun, 09 Nov 2014 22:16:51 +0100
+
 libeatmydata (82-2) unstable; urgency=medium
 
   * Upload to unstable.
diff -Nru libeatmydata-82/debian/control libeatmydata-82/debian/control
--- libeatmydata-82/debian/control  2014-10-03 20:45:30.000000000 +0200
+++ libeatmydata-82/debian/control  2014-11-09 12:06:58.000000000 +0100
@@ -6,7 +6,7 @@
 Standards-Version: 3.9.6
 Build-Depends: autotools-dev,
                debhelper (>=9),
-               strace
+               strace [!hurd-any !kfreebsd-any]
 Homepage: https://launchpad.net/libeatmydata
 Vcs-Git: git://anonscm.debian.org/collab-maint/libeatmydata.git
 Vcs-Browser: https://anonscm.debian.org/cgit/collab-maint/libeatmydata.git
diff -Nru libeatmydata-82/debian/patches/bug-702711.patch libeatmydata-82/debian/patches/bug-702711.patch
--- libeatmydata-82/debian/patches/bug-702711.patch 2014-10-03 21:00:09.000000000 +0200
+++ libeatmydata-82/debian/patches/bug-702711.patch 2014-11-09 22:12:41.000000000 +0100
@@ -3,10 +3,12 @@
  called by ctors, do not return an unexpected error.
 Author: Roland Stigge <stigge@antcom.de>
 Reviewed-By: Petter Reinholdtsen <pere@hungry.com>
+Reviewed-By: Mattia Rizzolo <mattia@mapreri.org>
 Bug: https://launchpad.net/bugs/1374862
 Bug-Debian: https://bugs.debian.org/702711
+Bug-Debian: https://bugs.debian.org/765738
 Forwarded: https://launchpad.net/bugs/1374862
-Last-Update: 2014-10-03
+Last-Update: 2014-10-29
 
 --- a/libeatmydata/libeatmydata.c
 +++ b/libeatmydata/libeatmydata.c
@@ -40,4 +42,13 @@
 +  if (initing) {
        errno = EFAULT;
        return -1;
+   }
+@@ -171,7 +174,7 @@
+ 
+   /* In pthread environments the dlsym() may call our open(). */
+   /* We simply ignore it because libc is already loaded       */
+-  if (!libc_open64) {
++  if (initing) {
+       errno = EFAULT;
+       return -1;
    }

unblock libeatmydata/82-3

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
arm64
armhf
ppc64el
s390x
armel

Kernel: Linux 3.16-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-- 
regards,
						Mattia Rizzolo

GPG Key: 4096R/B9444540	http://goo.gl/I8TMB
more about me:		http://mapreri.org
Launchpad User:		https://launchpad.net/~mapreri
Ubuntu Wiki page:	https://wiki.ubuntu.com/MattiaRizzolo

Attachment: signature.asc
Description: Digital signature


--- End Message ---
--- Begin Message --- 
On Thu, Nov 13, 2014 at 12:54:02PM +0100, Holger Levsen wrote:
> Hi,
> 
> On Mittwoch, 12. November 2014, Mattia Rizzolo wrote:
> > On Wed Nov 12 2014 at 7:17:09 PM Jonathan Wiltshire <jmw@debian.org> wrote:
> > > Ok, so what are you asking for? an unblock? pre-approval?
> > 
> > I'm asking for an unblock for libeatmydata/82-3 AND your opinion (call it
> > pre-approval if you like (actually I'd be happier with a full
> > pre-approval)) for libeatmydata/82-4 with the only change I posted at my
> > last email.
> 
> Mattia, it's seldom a good idea to mix topics and especially in this case with 
> the release team getting ~50 bugs a day it doesn't help them at all.
> 
> About the libeatmydata/82-3 unblock request:
> 
>  libeatmydata (82-3) unstable; urgency=low
>  .
>    * debian/patches/bug-702711.patch:
>      + [c95e942] fix the bug #702711 also for 32 bit arches (Closes: #765738).
>      + [0167220] update headers.
>    * [3fabf6f] debian/control: build-depend on strace only on non hurd and non
>      kfreebsd systems, where it is unavailable.
> 
> #702711 breaks eatmydata completly on i386 (which is noticable in Debian Edu), 
> so I disagree with the severity of #702711 which is currently important, like 
> #765738 was at well. I think both bugs are serious and must be fixed for 
> jessie.
> 
> So, dear release team, could you please unblock libeatmydata? The debdiff 
> looks good to me...!

With pleasure :)

One comment: you don't need to credit your sponsor in the changelog. That's
not a packaging change, and the fact gets recorded elsewhere anyway.

> > I think a pre-approval is needed for that change because there is no bugs
> > at all regarding building libeatmydata on hurd and kfreebsd, and anyway
> > that bugs would not be important.
> 
> I'd suggest coming back to 82-4 once 82-3 is in jessie _and_ filing bugs about 
> the issues you want to fix in 82-4 _first_.

Yes, please do this.


-- 
Jonathan Wiltshire                                      jmw@debian.org
Debian Developer                         http://people.debian.org/~jmw

4096R: 0xD3524C51 / 0A55 B7C5 1223 3942 86EC  74C3 5394 479D D352 4C51

Attachment: signature.asc
Description: Digital signature


--- End Message ---
Reply to: