Control: tags 800466 + patch Control: tags 800466 + pending [Replace XX with correct value] Dear maintainer,
I've prepared an NMU for pytables (versioned as 3.2.1-1.1) and uploaded it to DELAYED/XX. Please feel free to tell me if I should delay it longer. Regards, _g. diff -Nru pytables-3.2.1/debian/changelog pytables-3.2.1/debian/changelog --- pytables-3.2.1/debian/changelog 2015-08-14 21:42:34.000000000 +0200 +++ pytables-3.2.1/debian/changelog 2015-10-02 20:01:14.000000000 +0200 @@ -1,3 +1,12 @@ +pytables (3.2.1-1.1) unstable; urgency=medium + + * Non-maintainer upload. + * New patch: 0006-Fix-upstream-issue-481.patch + Fix test suite errors such as 'ValueError: could not broadcast + input array from shape (2) into shape (0)' (closes: #800466). + + -- Gilles Filippini <p...@debian.org> Fri, 02 Oct 2015 20:01:01 +0200 + pytables (3.2.1-1) unstable; urgency=medium * New upstream version diff -Nru pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch --- pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch 1970-01-01 01:00:00.000000000 +0100 +++ pytables-3.2.1/debian/patches/0006-Fix-upstream-issue-481.patch 2015-09-29 21:06:16.000000000 +0200 @@ -0,0 +1,40 @@ +From 44dba04d7d72f150a91553f4eb455684dfef0913 Mon Sep 17 00:00:00 2001 +From: Andrea Bedini <and...@andreabedini.com> +Date: Wed, 12 Aug 2015 10:09:43 +1000 +Subject: [PATCH] Be careful with signs while interating backward. + +This is a quick and possibily partial fix for #481. + +It seems that C dictates that istopb - istartb - 1 is an unsigned +expression (because istartb is unsigned and istopb is signed but not big +enough to contain istartb). + +With this fix I am betting that 1 - istopb + istartb is non-negative. It seems +to work but this function should be rewritten more clearly. +--- + tables/tableextension.pyx | 5 +++-- + 1 file changed, 3 insertions(+), 2 deletions(-) + +Index: pytables-3.2.1/tables/tableextension.pyx +=================================================================== +--- pytables-3.2.1.orig/tables/tableextension.pyx ++++ pytables-3.2.1/tables/tableextension.pyx +@@ -1241,7 +1241,7 @@ cdef class Row: + istartb = (j+istep) % inrowsinbuf + inextelement = inextelement + istep + i = i + inrowsinbuf +- elif 0 > istep: ++ elif istep < 0: + inrowsinbuf = self.nrowsinbuf + #istartb = self.startb + istartb = self.nrowsinbuf - 1 +@@ -1258,7 +1258,8 @@ cdef class Row: + i = i - inrowsinbuf + continue + # Compute the end for this iteration +- stopr = startr + ((istopb - istartb - 1) / istep) ++ # (we know we are going backward so try to keep indices positive) ++ stopr = startr + (1 - istopb + istartb) / (-istep) + # Read a chunk + inrowsread = inrowsread + self.table._read_records(i - inrowsinbuf + 1, + inrowsinbuf, self.iobuf) diff -Nru pytables-3.2.1/debian/patches/series pytables-3.2.1/debian/patches/series --- pytables-3.2.1/debian/patches/series 2015-08-14 21:42:34.000000000 +0200 +++ pytables-3.2.1/debian/patches/series 2015-09-29 21:05:56.000000000 +0200 @@ -3,3 +3,4 @@ 0003-Never-use-the-msse2-flag-explicitly.patch 0004-Do-not-fetch-icons-for-external-web-sites.patch 0005-Fix-setitem-return-value.patch +0006-Fix-upstream-issue-481.patch -- debian-science-maintainers mailing list debian-science-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/debian-science-maintainers
