[Debian-med-packaging] Bug#699939: [owner at bugs.debian.org: Bug#699939 closed by smr at debian.org (Steve M. Robbins) (Bug#699939: fixed in mriconvert 2.0.250-1)]

[Yaroslav Halchenko]

Thank you Steve for the upload and explanation on #617342! I have
tried myself as well with the same "success".

FWIW -- i.e. if you care/interested -- I have uploaded backport builds
of .250 to neurodebian for nearly all debian/ubuntu releases.  To do the
backport I obviously had to lower build-depends on dh and itk.  Here is
the patch (below) I will now carry under debian/patches for the
use with  our
http://git.debian.org/?p=pkg-exppsy/neurodebian.git;a=blob;hb=HEAD;f=tools/backport-dsc
to patch while backporting for older releases.

I wondered, since the patch and all the symlinks to it (per each distribution)
are kept neatly under debian/patches and do not anyhow interfere with official
packaging -- would you mind "adopting" it into the official SVN for mriconvert?
Since it is under the team maintenance -- I could even commit it myself if you
would not mind.  It would be perfectly fine if you do not like it being there
-- I will just keep our clone of the packaging SVN with this changes for future
releases.

Also, it might be better in the longer run, if build-depends in debian/control
were multilined -- with our patching in mind that would provide
sufficient number of content lines around while getting away from "volatile"
Standads-Version etc ruining our patching ;)

--- /dev/null
+++ b/debian/patches/nd_allow-dh7-itk3.patch
@@ -0,0 +1,28 @@
+From: Yaroslav Halchenko <debian at onerussian.com>
+Subject: allow building mriconvert on older Debian/Ubuntu releases against dh>7 and itk3
+
+Origin: NeuroDebian
+Last-Update: 2013-02-11
+
+diff --git a/debian/compat b/debian/compat
+index ec63514..7f8f011 100644
+--- a/debian/compat
++++ b/debian/compat
+@@ -1 +1 @@
+-9
++7
+diff --git a/debian/control b/debian/control
+index 8fe4d6a..d5171b5 100644
+--- a/debian/control
++++ b/debian/control
+@@ -4,8 +4,8 @@ Section: science
+ Priority: optional
+ Maintainer: Debian Med Packaging Team <debian-med-packaging at lists.alioth.debian.org>
+ Uploaders: Steve M. Robbins <smr at debian.org>
+-Build-Depends: debhelper (>= 9), help2man, cmake, cableswig, libboost-program-options-dev,
+- libinsighttoolkit4-dev, libwxgtk2.8-dev
++Build-Depends: debhelper (>= 7.0.50~), help2man, cmake, cableswig, libboost-program-options-dev,
++ libinsighttoolkit3-dev|libinsighttoolkit4-dev, libwxgtk2.8-dev
+ Standards-Version: 3.9.2
+ Vcs-Browser: http://svn.debian.org/wsvn/debian-med/trunk/packages/mriconvert/trunk/
+ Vcs-Svn: svn://svn.debian.org/debian-med/trunk/packages/mriconvert/trunk/


Cheers

On Mon, 11 Feb 2013, Steve M. Robbins wrote:

> retitle 617342 32-bit multiframe image conversion ignores frames
> thanks

> On February 10, 2013 05:44:27 PM Yaroslav Halchenko wrote:
> > Hi Steve,

> > Thank you for the upload!  I wonder if I have misread
> > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=617342
> > or it should have also been fixed with the "correct" .235?

> Well, it *should* have been fixed in rev235.  It was fixed in
> the rev235 *binary* that I tried.  Unfortunately, the source tarball on the 
> site was not updated when r235 was released, so it wasn't actually fixed in the 
> Debian package.

> Also of note: while the output is indeed 32-bit pixels, the particular file 
> Mathieu supplied is still not handled properly because only one of the frames 
> is output.    So it wasn't likely fixed even with the "correct" r235.

> Thus I am leaving the bug open, with the updated title.

> Best regards,
> -Steve


-- 
Yaroslav O. Halchenko
http://neuro.debian.net http://www.pymvpa.org http://www.fail2ban.org
Postdoctoral Fellow,   Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834                       Fax: +1 (603) 646-1419
WWW:   http://www.linkedin.com/in/yarik