Your message dated Thu, 31 Dec 2015 03:49:13 +0000
with message-id <e1aeuej-0004qr...@franck.debian.org>
and subject line Bug#751984: fixed in devscripts 2.15.10
has caused the Debian Bug report #751984,
regarding devscripts: mk-origtargz do not handle xpi files correctly
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.)


-- 
751984: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=751984
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: devscripts
Version: 2.14.5
Severity: normal

Dear Maintainer,

   * What led up to the situation?

I plan to package a mozilla addon; following xpi-repack(1) (from
mozilla-devscripts), here is my debian/watch file:

---------8<---------
quidame@sid:/tmp/profileswitcher$ cat debian/watch
version=3
https://freeshell.de/~kaosmos/profileswitcher-([\d\.]+)-tb\+fx\.xpi debian 
xpi-repack
--------->8---------

This works fine in wheezy; but when I run uscan in a fresh sid
environment, I get the following results:

---------8<---------
quidame@sid:/tmp/profileswitcher$ uscan --verbose
-- Scanning for watchfiles in .
-- Found watchfile in ./debian
-- In debian/watch, processing watchfile line:
   https://freeshell.de/~kaosmos/profileswitcher-([\d\.]+)-tb\+fx\.xpi debian 
xpi-repack
-- Found the following matching hrefs:
     profileswitcher-1.6.2-tb+fx.xpi (1.6.2)
Newest version on remote site is 1.6.2, local version is 1.6.1
 => Newer version available from
    https://freeshell.de/~kaosmos/profileswitcher-1.6.2-tb+fx.xpi
-- Downloading updated package profileswitcher-1.6.2-tb+fx.xpi
-- Checking for common possible upstream OpenPGP signatures
-- Successfully downloaded updated package profileswitcher-1.6.2-tb+fx.xpi
-- Successfully repacked ../profileswitcher-1.6.2-tb+fx.xpi as 
../profileswitcher_1.6.2.orig.tar.gz.
-- Executing user specified script
     xpi-repack --upstream-version 1.6.2 ../profileswitcher_1.6.2.orig.tar.gz
Unpacking ../profileswitcher_1.6.2.orig.tar.gz
[../profileswitcher_1.6.2.orig.tar.gz]
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of 
../profileswitcher_1.6.2.orig.tar.gz or
        ../profileswitcher_1.6.2.orig.tar.gz.zip, and cannot find 
../profileswitcher_1.6.2.orig.tar.gz.ZIP, period.
Unpacked xpi file.
-- Scan finished
quidame@sid:/tmp/profileswitcher$ ls -dl ../profileswitcher*
drwxr-xr-x 5 quidame quidame   220 May 25 09:47 ../profileswitcher
-rw-r--r-- 1 quidame quidame 82794 Jun 18 14:31 
../profileswitcher_1.6.2.orig.tar.gz
-rw-r--r-- 1 quidame quidame   188 Jun 18 14:31 
../profileswitcher_1.6.2.orig.tar.xz
-rw-r--r-- 1 quidame quidame 88737 Jun 18 14:22 
../profileswitcher-1.6.2-tb+fx.xpi
--------->8---------

It appears that uscan automatically calls mk-origtargz first, which
creates the *.orig.tar.gz file, then xpi-repack is callled with the new
archive name as argument instead of the upstream xpi file, so xpi-repack
fails: it creates the *.orig.tar.xz file, which contains only an empty
directory.

   * What exactly did you do (or not do) that was effective (or
     ineffective)?

Removing "debian xpi-repack" from the watch file does not help very
much, because the mk-origtargz handles the xpi file very poorly: in this
case case, it contains an unpacked .jar file and the source tree is at
the root of the archive, not in a main directory (in this case, it
should be 'profileswitcher-1.6.2').

XPI means "Cross Platform Installer"; it is an extension for zip
archives. Since mk-origtargz uses file's mime detection to detect zip
files [1], it handles xpi files instead of falling into error. But the
source archive it creates is wrong.

   * What was the outcome of this action?

I have to run xpi-repack manually, with the proper arguments.

   * What outcome did you expect instead?

I think mk-origtargz should skip xpi files without error, as it was the
case in wheezy (devscripts version 2.12.6+deb7u2); or uscan could also
have a --no-repack switch, or the repack action should not be performed
if the --repack option is not explicitly specified.

Thanks
quidame

[1] https://bugs.debian.org/748462

-- Package-specific info:

--- /etc/devscripts.conf ---

--- ~/.devscripts ---
Not present

-- System Information:
Debian Release: jessie/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)

Kernel: Linux 3.14-1-486
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages devscripts depends on:
ii  dpkg-dev     1.17.10
ii  libc6        2.19-3
ii  perl         5.18.2-4
ii  python3      3.4.1-1
pn  python3:any  <none>

Versions of packages devscripts recommends:
ii  at                          3.1.14-1
ii  dctrl-tools                 2.23
ii  debian-keyring              2014.04.25
ii  dput                        0.9.6.4
ii  equivs                      2.0.9
ii  fakeroot                    1.20-3
ii  file                        1:5.19-1
ii  gnupg                       1.4.16-1.1
ii  libdistro-info-perl         0.13
ii  libencode-locale-perl       1.03-1
ii  libjson-perl                2.61-1
ii  liblwp-protocol-https-perl  6.04-3
ii  libparse-debcontrol-perl    2.005-4
ii  libsoap-lite-perl           1.11-1
ii  liburi-perl                 1.60-1
ii  libwww-perl                 6.06-1
ii  lintian                     2.5.22.1
ii  man-db                      2.6.7.1-1
ii  patch                       2.7.1-5
ii  patchutils                  0.3.3-1
ii  python3-debian              0.1.22
ii  python3-magic               1:5.19-1
ii  sensible-utils              0.0.9
ii  strace                      4.5.20-2.3
ii  unzip                       6.0-12
ii  wdiff                       1.2.2-1
ii  wget                        1.15-1+b1
ii  xz-utils                    5.1.1alpha+20120614-2

Versions of packages devscripts suggests:
ii  bsd-mailx [mailx]            8.1.2-0.20131005cvs-1
ii  build-essential              11.6
pn  cvs-buildpackage             <none>
pn  devscripts-el                <none>
pn  gnuplot                      <none>
ii  gpgv                         1.4.16-1.1
ii  libauthen-sasl-perl          2.1600-1
pn  libfile-desktopentry-perl    <none>
ii  libnet-smtp-ssl-perl         1.01-3
pn  libterm-size-perl            <none>
ii  libtimedate-perl             2.3000-2
pn  libyaml-syck-perl            <none>
ii  mutt                         1.5.23-1
ii  openssh-client [ssh-client]  1:6.6p1-5
pn  svn-buildpackage             <none>
ii  w3m                          0.5.3-15

-- no debconf information

--- End Message ---
--- Begin Message ---
Source: devscripts
Source-Version: 2.15.10

We believe that the bug you reported is fixed in the latest version of
devscripts, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 751...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
James McCoy <james...@debian.org> (supplier of updated devscripts package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Wed, 30 Dec 2015 20:44:35 -0500
Source: devscripts
Binary: devscripts
Architecture: source
Version: 2.15.10
Distribution: unstable
Urgency: low
Maintainer: Devscripts Devel Team <devscripts-devel@lists.alioth.debian.org>
Changed-By: James McCoy <james...@debian.org>
Closes: 278797 350454 363346 375138 395439 409094 415680 451051 458777 472065 
499886 516704 517637 519080 526443 526450 531321 532182 538717 539780 544638 
549178 557768 559429 570221 573631 582958 648632 663357 674812 696492 705989 
731725 733272 734748 737843 738977 740366 751984 752364 755754 762263 763071 
773390 775526 778723 788414 790629 796293 797045 797787 798133 798175 800704 
801165 802503 803724 803948 804239 804492 805501 806262 806424 807064 807065 
808271 808394
Description: 
 devscripts - scripts to make the life of a Debian Package maintainer easier
Changes:
 devscripts (2.15.10) unstable; urgency=low
 .
   [ Benjamin Drung ]
   * test_uscan: Fix failure in testFileExclusion due to wrong tarfile name
     test on Ubuntu.
 .
   [ Nicholas Bamber ]
   * Fixed issues with current bash completion (Closes: #804492)
     + Reorganised scripts/Makefile so that bash completion scripts
       are installed matching their command names - an exception being the
       generic pkgnames.bash_completion.
     + Where necessary added links to bash completion scripts
       to debian/links to reuse bash completion code
     + Removed deprecated "have" mechanism from bash completion scripts.
     + Removed undefined "$filenames" from chdist.bash_completion
     + Split debi.bash_completion off from pkgnames.bash_completion
       and added links for both debi/debc and corrected complete commands
   * Added custom bash completion support for bts (Closes: #499886)
     + Added --soap-timeout option to bts
     + Added listcachedbugs command to bts
     + Split out bts bash completion handling into its own script
   * Added custom bash completion support for debuild, checkbashisms,
     list-unreleased, dscverify, dscextract, what-patch, plotchangelog,
     debdiff and licensecheck
   * Updated custom bash completion handling for mk-origtargz
   * Added debcheckout to those commands with package names bash completion
   * licensecheck:
     + Tweaked parselicense to capture 'and or' as well as
       'and/or' in GPL licenses (Closes: #559429)
     + Added test for licensecheck versus Software::License
     + Resolved failure to find license from Software::License generated
       files from Artistic and Mozilla licenses
   * Added test for life cycle of package:
     + debuild, dscverify, dscextract, list-unreleased, debdiff
   * Rearranged cleanup of tests to use oneTimeSetUp/oneTimeTearDown
   * Streamlined structure of test/Makefile
   * Updated .gitignore
 .
   [ Dominique Dumont ]
   * licensecheck:
     + no longer attempts to find encoding or select files to parse
       using 'file' commands. By default, files are selected using file
       extension instead of MIME types provided by "file" command.  The
       list of file extensions is more boring to maintain but also more
       predictable.  (Closes: #803724)
     + added --encoding option. By default, input files are handled as utf-8
       Information is printed on STDOUT as utf-8.
     + --text option is deprecated
     + added freetype license
     + remove 'by' from copyright owners
     + remove whitespace from end of multi-line copyright statements
     + parse multi-line copyright blocks (Closes: #519080)
     + parse .lisp file (part of #806424)
     + added LLGPL license (part of #806424)
     + licensecheck: show skipped file with --skipped (Closes: #806424)
     + also parse last 5kb of a file
     + parse source file without suffix (e.g script, README)
     + parse .css or .less file
     + parse perl6 files (.pl6 .pm6 .pod6 ...)
     + parse markdown files (.md or .markdown)
     * tweaked to parse Artistic 2.0 license file
     * improved detection of artistic 2.0 licenses
 .
   [ Osamu Aoki ]
   * uscan:
     + Start uupdate with --verbose as needed.  (Closes: #350454)
     + Add test_uscan_mangle for testing the mangling rules and pgp signature.
     + Add --download-debversion.  (Closes: #755754)
     + Add pagemangle.  (Closes: #395439, #451051, #538717, #705989, #773390)
     + Add oversionmangle as a part of multiple upstream tarball support.
       (Closes: #458777)
     + Add dirversionmangle with $download-version consideration with detailed
       and sorted reporting of all available URLs.
       (Closes: #363346, #375138, #472065, #516704, #557768, #734748)
     + Add the multiple upstream tarball support and extend the OpenPGP support
       with new watch line options: component, compression, repack, pgpmode,
       versionmode, decompress.  (Closes: #526443, #531321, #549178, #731725,
       #738977, #778723, #800704)
     + Updated manpage contents and build it from POD.  (Closes: #409094,
       #415680, #573631, #648632, #733272, #775526, #790629, #796293, #797787)
     + Improve FTP listing handling. (Closes: #696492)
     + Add the --overwrite-download, --skip-signature, and similar options
       to reorganize behavior around existing downloaded files.
       (Closes: #532182, #740366, #801165)
     + Update filenamemangle to support the version pattern matching in any
       position of the URL and remove .download hack which breaks
       mk-origtargz. (Closes: #526450, #803948, #804239)
     + Reorganize the reporting by uscan and adjust its text strings.
       (Closes: #582958)
     + Add download compression ordering support to prefer xz over gz.
       (Closes: #763071)
     + Add generalized search regex with some special strings @PACKAGE@,
       @ANY_VERSION@, @ARCHIVE_EXT@, and @SIGNATURE_EXT@.
       (Closes: #517637, #539780)
     + Add direct git repository support. (Closes: #663357)
     + Add unzipopt support.
   * uupdate:
     + Prevent failure when upstream version is 0.  (Closes: #798175)
     + Work around missing debian.tar.gz for "3.0 (quilt)".  (Closes: 798133)
     + Add multiple upstream tarball support and resolve minor bugs by using
       dpkg-source as the backend.  (Closes: #278797, #544638, #570221,
       #752364, #797045)
   * mk-origtargz:
     + Support component name for the multiple upstream tarball support.
     + Fix Files-Excluded for duplicate entries. (Closes: #762263)
     + Add --unzipopt option. (Closes: #737843)
     + Add xpi files support. (Closes: #751984)
   * debian/control, dscverify, who-permits-upload, uscan:
     + Add gnupg2/gpgv2 as an alternative option to gnupg/gpgv. (Closes: 
#788414)
 .
   [ James McCoy ]
   * debuild:
     + Use _all as the .changes arch for -A builds.  (Closes: #806262)
   * dget:
     + Remove unnecessary requirement that “apt-cache policy $pkg” shows a
       priority of 0 for the Candidate version.  (Closes: #807064)
     + Fix parsing of sources.list lines with options.  (Closes: #674812)
   * Add apt to Recommends, since various scripts use tools from apt.  (Closes:
     #807065)
   * Export LC_ALL=C in test_mk-origtargz so the test doesn't fail in 
non-English
     locales.
   * chdist:
     + Make the apt-file support compatible with 3.x apt-file.
   * debcommit:
     + When a VCS requires a mangled package version for the tag name, still
       use the actual package version in the tag message.  (Closes: #802503)
   * bts:
     + Fix creation of cache dir when $XDG_CACHE_HOME doesn't exist yet.
       (Closes: #808394)
   * checkbashisms:
     + Improve detection of bashisms when a ! is included after an
       if/while/elif conditional.  (Closes: #808271)
 .
   [ Daniel Kahn Gillmor ]
   * debuild:
     + pass through DBUS_SESSION_BUS_ADDRESS so that gpg-agent can make
       pinentry-gnome3 work (Closes: #805501)
Checksums-Sha1: 
 f67218be5629e603997140fb3d230e26ed099cbf 2350 devscripts_2.15.10.dsc
 bab581df37b7d3d02329adb5f9587aa02f9c8be7 679536 devscripts_2.15.10.tar.xz
Checksums-Sha256: 
 a2442cc6aebb522d679aa2d5659faf0de37e3746c1ce3a99413a2e0d191a6e61 2350 
devscripts_2.15.10.dsc
 8659779f03563aea6187f744b083cea2dfe637da2bb12ff52e38dd2e6f3762f9 679536 
devscripts_2.15.10.tar.xz
Files: 
 62abe67fd4ccae35ac804ce2c0e7c61c 2350 devel optional devscripts_2.15.10.dsc
 a21685d0ded4b77bab71fce1e3084078 679536 devel optional 
devscripts_2.15.10.tar.xz

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQJ8BAEBCgBmBQJWhJBSXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w
ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQ5MUJGQkY0RDY5NTZCRDVERjdCNzJEMjNE
RkU2OTFBRTMzMUJBM0RCAAoJEN/mka4zG6PbkYwQAL/uCkGIls/3sas8nuiVoTyp
DZDELaK2C9InapULu6B0wB7r2hOlgJnMbGDiXCjVkP5V7x3c0P35pLwCrhoPTIxv
46E6J54MVFJtMPAuhbf1jOy+h1pbnwhAbqZpkjlDa5priCuXJkuck+VBf6Mu4apT
hy/kFF4ydQWtcBVfKSRxVoH+MNu9nlOHRl3PkOnG4xF5PV8tgDE/FxC+o3ft/zhx
u4o8pSmk0e0iBSr5k02vm9ufQuYn2/geuJlb+8zlhdW5Vlkrpx8QnVhnE1+NWBrX
8Po4dU2qzPRvEjoY0LUq4FG6ynQBmk8nvHyVL4sIO23MkQ4gjxoIJ1Q4HEkV3r8o
j1Y7fCQ1XtZrasFe7S6SbWRTy11VAO8Qp6dZtD3gDCX44PonuUOJixDQWGhkFLHI
06Ms5xgLHCMnf6vpbARLMasB7cGr2mTGmKXV5Q1AReGfESAS0qneM8+kQfAienGE
l48QkeLR+bSEJuiDWSO+pYhuIfE/FWCpYR/6usFhswbLHjaDkMaj5DkA9/YN7p2o
ORxqt5WMpxZxLnQ8MFvilq6m/A9TJZzE6jq3k9uq4DPlq6d6u7ihA6qfdWWQpKpQ
ujUYc5GHC2mfm08aCmCEyUXMLXQ9N+TytUw+8gKRCprnZ4M0QWvQJEjg1U3QuWAh
mmXkJIUiU09u84zv5msA
=c0dl
-----END PGP SIGNATURE-----

--- End Message ---
_______________________________________________
devscripts-devel mailing list
devscripts-devel@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/devscripts-devel

Reply via email to