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

Bug#706144: m-tx_0.60d-5.1_amd64.changes ACCEPTED into unstable



Roland Stigge wrote:
-----BEGIN PGP SIGNED MESSAGE-----
Hash: RIPEMD160

Hi you two,

On 05/05/13 04:48, Debian FTP Masters wrote:
Accepted:

Format: 1.8 Date: Sun, 05 May 2013 01:54:02 +0000 Source: m-tx Binary: m-tx Architecture: source amd64 Version: 0.60d-5.1 Distribution: unstable Urgency: low Maintainer: Roland Stigge
<stigge@antcom.de> Changed-By: Peter Michael Green
<plugwash@debian.org> Description: m-tx       - Simple
music-from-text language for use with PMX/MusiXTeX Changes: m-tx
(0.60d-5.1) unstable; urgency=low . * Non-maintainer upload. * Set
architecture list to any so the package builds on all
architectures where the build-depends are available.

Regarding this issue, the following was done in m-tx 0.60c-1:

    - Switched from p2c+gcc to fp-compiler due to upstream
      incompatibility,
      unfortunately reduces architectures to i386 powerpc sparc amd64
      arm
      and doesn't support shared library linking
Since that changelog entry was written freepascal has become available on two more architectures in unstable with a third architecture (though that third architecture is a debian-ports.org architecture rather than an official architecture) now supported by the experimental freepascal packages (should be hitting unstable soon) and another two architectures coming up upstream which are likely to be added before the jessie release.

Having a manually set architecture list on a package that is not architecture specific in itself but depends on a complier that is architecture specific has no real advantage and two significant disadvantages over leaving the architecture set to any. 1: it has to be updated manually when the compiler is ported to more architectures. This clearly was not done leaving the package unessacerally unavailable on those architectures. 2: it means it is far less obvious to porters why the package is not available on thier architecture.
For an NMU, I would expect the following:

* Filing a bug describing the problem first.
I filed a bug http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=706144 . The bug received no feedback from either you or norbert.
* Contacting the maintainers of the package - the current state may
have a reason.
I expressed my intent to NMU to the bug report about 10 days before I uploaded (the NMU guidelines say 5 days for fixing an important bug but I held it longer because of the wheezy release). My understanding is that mails sent to bug reports are forwarded to the maintainer.
* Closing the respective bug in the NMU changelog
* Attaching the NMU diff to the BTS
I apologise for forgetting these steps. I can't retroactively add a closes to the changelog entry but I have generated a debdiff and attached it to this email (which I am also sending to the bugreport).
Since you did nothing from this list, I guess sth. went wrong here.

Maybe I missed something since Norbert took over maintainership for
m-tx (including some musixtex related packages). So did you coordinate
(Peter and Norbert)? Otherwise I would be forced to revert the changes
of m-tx 0.60d-5.1 and ask you to not do like this again. (Actually,
the version from experimental should just be moved to unstable now
that wheezy is released. - Norbert?)

Thanks in advance,
Apologies for my oversights in the process but I don't belive a revert is a sensible course of action.

Peter Michael Green
Freepascal comaintainer
Arm porter


diff -Nru m-tx-0.60d/debian/changelog m-tx-0.60d/debian/changelog
--- m-tx-0.60d/debian/changelog	2012-04-01 09:59:22.000000000 +0100
+++ m-tx-0.60d/debian/changelog	2013-05-05 03:01:23.000000000 +0100
@@ -1,3 +1,11 @@
+m-tx (0.60d-5.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Set architecture list to any so the package builds on all architectures
+    where the build-depends are available. 
+
+ -- Peter Michael Green <plugwash@debian.org>  Sun, 05 May 2013 01:54:02 +0000
+
 m-tx (0.60d-5) unstable; urgency=low
 
   * Fix postinst script to adjust to new texmf path (Closes: #666678)
diff -Nru m-tx-0.60d/debian/control m-tx-0.60d/debian/control
--- m-tx-0.60d/debian/control	2012-03-30 14:48:41.000000000 +0100
+++ m-tx-0.60d/debian/control	2013-05-05 02:53:33.000000000 +0100
@@ -7,7 +7,7 @@
 Build-Depends: debhelper (>= 9), musixtex, pmx, musixtex (>= 1:0.114-2) | musixlyr, texlive-latex-recommended, texlive-fonts-recommended, fp-compiler, texlive-latex-extra
 
 Package: m-tx
-Architecture: i386 powerpc sparc amd64 arm
+Architecture: any
 Depends: ${misc:Depends}, texlive-base
 Recommends: musixtex (>= 1:0.98-1), musixlyr (>= 1.10-4), pmx (>= 2.1.0-1)
 Description: Simple music-from-text language for use with PMX/MusiXTeX

Reply to: