Package: devscripts Version: 2.10.35 Severity: normal Tags: patch
When running debcommit with the --release option, the --edit and --confirm options do nothing, which I assume is undesired behaviour. This patch fixes that. If I am wrong, and the behaviour is desired, please ignore. -- Package-specific info: --- /etc/devscripts.conf --- --- ~/.devscripts --- Not present -- System Information: Debian Release: lenny/sid APT prefers testing APT policy: (500, 'testing') Architecture: i386 (i686) Kernel: Linux 2.6.26-1-686 (SMP w/1 CPU core) Locale: LANG=en_CA.UTF-8, LC_CTYPE=en_CA.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/bash Versions of packages devscripts depends on: ii dpkg-dev 1.14.20 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 conkeror [www-brow 0.9~git080629-1 keyboard focused web browser with 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 equivs 2.0.7-0.1 Circumvent Debian package dependen ii fakeroot 1.9.5 Gives a fake root environment ii git-core 1:1.5.6.5-1 fast, scalable, distributed revisi ii gnupg 1.4.9-3 GNU privacy guard - a free PGP rep 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 lintian 1.24.2.1 Debian package checker ii lsb-release 3.2-19 Linux Standard Base version report 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 unzip 5.52-12 De-archiver for .zip files 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) pn libfile-desktopentry-perl <none> (no description available) pn libnet-smtp-ssl-perl <none> (no description available) pn mutt <none> (no description available) pn svn-buildpackage <none> (no description available) -- no debconf information
--- /usr/bin/debcommit.old 2008-07-26 17:19:54.000000000 -0400 +++ /usr/bin/debcommit 2008-09-05 18:45:16.000000000 -0400 @@ -222,6 +222,7 @@ my $release=0; my $message; +my $version; my $release_use_changelog=0; my $noact=0; my $diffmode=0; @@ -332,20 +333,24 @@ } close C; - my $version=`dpkg-parsechangelog | grep '^Version:' | cut -f 2 -d ' '`; + $version=`dpkg-parsechangelog | grep '^Version:' | cut -f 2 -d ' '`; chomp $version; $message="releasing version $version" if ! defined $message; - commit($message); - tag($version); } -else { - if ($edit) { - my $modified = 0; - ($message, $modified) = edit($message); - die "$progname: Commit message not modified / saved; aborting\n" unless $modified; + +if ($edit) { + my $modified = 0; + ($message, $modified) = edit($message); + die "$progname: Commit message not modified / saved; aborting\n" unless $modified; +} + +if (not $confirm or confirm($message)) { + commit($message); + + if ($release) { + tag($version); } - commit($message) if not $confirm or confirm($message); } sub getprog {