Your message dated Wed, 05 Nov 2008 19:32:03 +0000
with message-id <[EMAIL PROTECTED]>
and subject line Bug#495743: fixed in devscripts 2.10.40
has caused the Debian Bug report #495743,
regarding [debchange] --closes encourages bad practice
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 [EMAIL PROTECTED]
immediately.)


-- 
495743: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=495743
Debian Bug Tracking System
Contact [EMAIL PROTECTED] with problems
--- Begin Message ---
Package: devscripts
Version: 2.10.35
Severity: normal

I see more and more people with changelog entries like:
* /etc/acpi/battery.d is ignored on newer kernels (Closes: #491396)

The changelog entry describes the problem instead of the fix (or the
change done to avoid the problem). After discussion with the maintainer
he told me "I was lazy enough to keep the output of
dch --closes <n>".

For reference, the proper changelog entry should have been something like
this:
* adapt foo to use the new /sys/class/power_supply interface
  instead of the /proc/acpi/ one which got removed in kernel 2.6.26.
  Closes: #491396

So I really believe that as-is this option is encouraging bad practices
concerning changelog entries. I suggest to put a warning in the manpage
and to change what's added so that it becomes a template like this:
* Fix "<bug subject>": <explain the change and why it's needed> (Closes: #<bug>)

The developer should then replace the <explain...> part.

Thanks for your consideration.

-- Package-specific info:

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

--- ~/.devscripts ---
DEBRELEASE_UPLOADER=dput
DEBCHANGE_RELEASE_HEURISTIC=changelog
DEBCHANGE_MULTIMAINT_MERGE=yes
DEBCHANGE_PRESERVE=yes
DEBUILD_LINTIAN_OPTS="--color always -I"

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 
'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.26-1-686 (SMP w/1 CPU core)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages devscripts depends on:
ii  dpkg-dev                      1.15.0     Debian package development tools
ii  libc6                         2.7-13     GNU C Library: Shared libraries
ii  perl                          5.10.0-13  Larry Wall's Practical Extraction 

Versions of packages devscripts recommends:
ii  at                 3.1.10.1              Delayed job execution and batch pr
ii  bsd-mailx [mailx]  8.1.2-0.20071201cvs-3 A simple mail user agent
ii  bzr                1.5-1                 easy to use distributed version co
ii  curl               7.18.2-7              Get a file from an HTTP, HTTPS or 
ii  cvs                1:1.12.13-11          Concurrent Versions System
ii  dctrl-tools        2.13.0                Command-line tools to process Debi
ii  debian-keyring     2008.07.22            GnuPG (and obsolete PGP) keys of D
ii  debian-maintainers 1.42                  GPG keys of Debian maintainers
ii  dput               0.9.2.32              Debian package upload tool
ii  dupload            2.6.4                 utility to upload Debian packages
ii  elinks [www-browse 0.11.4-2              advanced text-mode WWW browser
ii  epiphany-gecko [ww 2.22.3-1              Intuitive GNOME web browser - Geck
ii  equivs             2.0.7-0.1             Circumvent Debian package dependen
ii  fakeroot           1.9.6                 Gives a fake root environment
ii  galeon [www-browse 2.0.6-2               GNOME web browser for advanced use
ii  git-core           1:1.5.6.3-1.1         fast, scalable, distributed revisi
ii  gnupg              1.4.9-3               GNU privacy guard - a free PGP rep
ii  iceape-browser [ww 1.1.11-1              Iceape Navigator (Internet browser
ii  iceweasel [www-bro 3.0.1-1               lightweight web browser based on M
ii  libauthen-sasl-per 2.12-1                Authen::SASL - SASL Authentication
ii  libcrypt-ssleay-pe 0.57-1+b1             Support for https protocol in LWP
ii  libparse-debcontro 2.005-2               Easy OO parsing of Debian control-
ii  libsoap-lite-perl  0.710.08-1            Client and server side SOAP implem
ii  libterm-size-perl  0.2-4+b1              Perl extension for retrieving term
ii  libtimedate-perl   1.1600-9              Time and date functions for Perl
ii  liburi-perl        1.35.dfsg.1-1         Manipulates and accesses URI strin
ii  libwww-perl        5.813-1               WWW client/server library for Perl
ii  libyaml-syck-perl  1.05-1                Fast, lightweight YAML loader and 
ii  links [www-browser 2.1pre37-1.1          Web browser running in text mode
ii  lintian            1.24.3                Debian package checker
ii  lsb-release        3.2-19                Linux Standard Base version report
ii  lynx-cur [www-brow 2.8.7dev9-1.2         Text-mode WWW Browser with NLS sup
ii  mailx              1:20071201-3          Transitional package for mailx ren
ii  man-db             2.5.2-2               on-line manual pager
ii  openssh-client [ss 1:5.1p1-2             secure shell client, an rlogin/rsh
ii  patch              2.5.9-5               Apply a diff file to an original
ii  patchutils         0.2.31-4              Utilities to work with patches
ii  strace             4.5.17+cvs080723-2    A system call tracer
ii  subversion         1.5.1dfsg1-1          Advanced version control system
ii  tla                1.3.5+dfsg-14         GNU Arch revision control system
ii  unzip              5.52-12               De-archiver for .zip files
ii  w3m [www-browser]  0.5.2-2+b1            WWW browsable pager with excellent
ii  wdiff              0.5-18                Compares two files word by word
ii  wget               1.11.4-1              retrieves files from the web

Versions of packages devscripts suggests:
ii  build-essential               11.4       Informational list of build-essent
pn  cvs-buildpackage              <none>     (no description available)
pn  devscripts-el                 <none>     (no description available)
pn  gnuplot                       <none>     (no description available)
ii  libfile-desktopentry-perl     0.04-1     Perl module to handle freedesktop 
pn  libnet-smtp-ssl-perl          <none>     (no description available)
ii  mutt                          1.5.18-4   text-based mailreader supporting M
ii  svn-buildpackage              0.6.23     helper programs to maintain Debian

-- no debconf information



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

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:

devscripts_2.10.40.dsc
  to pool/main/d/devscripts/devscripts_2.10.40.dsc
devscripts_2.10.40.tar.gz
  to pool/main/d/devscripts/devscripts_2.10.40.tar.gz
devscripts_2.10.40_amd64.deb
  to pool/main/d/devscripts/devscripts_2.10.40_amd64.deb



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 [EMAIL PROTECTED],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Adam D. Barratt <[EMAIL PROTECTED]> (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 [EMAIL PROTECTED])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Wed, 05 Nov 2008 19:18:27 +0000
Source: devscripts
Binary: devscripts
Architecture: source amd64
Version: 2.10.40
Distribution: unstable
Urgency: low
Maintainer: Devscripts Devel Team <[email protected]>
Changed-By: Adam D. Barratt <[EMAIL PROTECTED]>
Description: 
 devscripts - scripts to make the life of a Debian Package maintainer easier
Closes: 483657 488717 495743 496065 497988 497995 499050 499264 499397 501252 
501568 502516 502517 503378 503950 504017
Changes: 
 devscripts (2.10.40) unstable; urgency=low
 .
   [ Adam D. Barratt ]
   * bts:
     + Update the manpage to mention local caching in the description
       section and correct a typo. (Closes: #499397)
     + Accept a command consisting purely of a comment. (Closes: #501252)
   * cowpoke:
     + Update to new upstream version
       - Multiple architecture and distribution support.
       - Options are now specified using --option rather than being listed
         after the .dsc file.
       - There is now a --create option to allow new build roots to be
         easily created.
       - There are two new configuration file options BUILDD_DIST (the
         default distribution for which packages will be built) and
         DEBOOTSTRAP, allowing one to specify which of {c,}debootstrap
         should be used.
       - The RESULT_DIR configuration variable has been replaced by
         PBUILDER_BASE which points to pbuilder's base folder under which
         architecture and distribution directories will be created.
       - cowpoke.conf now supports per-arch and per-distribution options.
   * debchange:
     + Don't attempt to automatically determine if the current changelog is
       an NMU if an alternative changelog location was specified using the
       CHANGELOG environment variable.
     + Update the Ubuntu distribution list. Add jaunty and remove the
       no-longer supported warty, hoary, edgy and fiesty. Also add
       "-backports" as a supported "extension".
     + Modify the message produced by --closes to make it clearer that it
       is intended to be edited to add details of the bug and fix as
       appropriate. (Closes: #495743)
   * debcommit:
     + Allow --release to be combined with --edit or --confirm. Based on
       a patch by Eric Gerlach. (Closes: #497988)
     + Allow a newly initialised (i.e. containing no HEAD) git tree to be
       committed to. Based on a patch by Eric Gerlach. (Closes: #497995)
   * debdiff: Correctly handle hyphenated distribution names when generating
     a diff against the previous version of the package. (Closes: #503950)
   * getbuildlog:
     + Apply some patches from Jörg Sommer (Closes: #496065)
       - Unescape %7Es in log URLs to allow explicitly searching for
         package versions containing tildes.
       - Simplify a couple of variable assignments and use shell string
         operations to extract the version and architecture from the logs.
     + Allow the logs for the build with the largest version number to be
       easily retrieved. Based on a patch by Jörg Sommer.
   * licensecheck: Match a couple more variants on the wording of a
     3-clause BSD license. (Closes: #503378)
   * nmudiff: Require an explicit confirmation as to whether the generated
     mail should be sent, rather than defauling to not sending it.
     (Closes: #483657)
   * tagpending: Don't output a spurious empty line near the end of the
     changelog comments under some circumstances.
   * uscan:
     + Document the USCAN_TIMEOUT configuration file variable.
       (Closes: #502517)
     + Add a USCAN_REPACK configuration file variable indicating that bzip
       tar or zip archives should always be repacked to gzip tar archives.
       (Closes: #502516)
 .
   [ Christoph Berg ]
   * debian/control: Remove the Dm-Upload-Allowed flag.
 .
   [ Stefano Zacchiroli ]
   * debcheckout:
     + try to guess repository type when given raw URLs
     + update user info about Vcs-* fields (correct developers' reference
       section, add an URL to it)
     + suggest using apt-get source as a fallback (Closes: #488717)
     + add support to define custom rules for authenticated mode, this way
       personal repos can benefit from '-a' (Closes: #501568)
     + add -d/--details option to print extended information about
       reopsitories, in machine parseable format. Implement dumping of
       TopGit-specific information for TopGit-enabled GIT repositories
     + initialize TopGit's automatically, for GIT repositories which are
       recognized as being managed with TopGit (Closes: #499264)
     + add --git-track option to require tracking of remote branches when
       checking out GIT repositories (Closes: #499050)
 .
   [ Patrick Schoenfeld ]
   * debsign: Unset GREP_OPTIONS to avoid problems when this variable is set
     in the users environment. (Closes: #504017)
Checksums-Sha1: 
 7a36779c5a779a96a7a1ce95290852c2484822d6 1363 devscripts_2.10.40.dsc
 dda43f3c41a437138ac49a2d3656569e35751fad 625530 devscripts_2.10.40.tar.gz
 acf75ada4f982092c7378ae88d7293458cbd0be8 539680 devscripts_2.10.40_amd64.deb
Checksums-Sha256: 
 cb73374f93a1ae0ea4f751fa64e8224d47e8c01b73457df6d920d725e225dc2c 1363 
devscripts_2.10.40.dsc
 73060bc625c184cb8e8efd3e5bee55f04b0ce6da499cefb49462e59333466d32 625530 
devscripts_2.10.40.tar.gz
 c1167629b3bafb39f4559d5ec14b32c9d4734110af068b0ad4f20debccf0cbfc 539680 
devscripts_2.10.40_amd64.deb
Files: 
 d801daffe9cf05b670c6db5ceb44c170 1363 devel optional devscripts_2.10.40.dsc
 9ec665f875c32c0bee14477f7f899a56 625530 devel optional 
devscripts_2.10.40.tar.gz
 b28c565e5fa83504ae0f7ee314261970 539680 devel optional 
devscripts_2.10.40_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkkR8vcACgkQokcE1TReOoXxuACfUIqhPiklWMO5vVg5Ao3ulrUY
u4QAn0DEKQUMzatG/FtNqkEbzxFv3+WP
=VA4t
-----END PGP SIGNATURE-----



--- End Message ---

Reply via email to