Bug#866148: [general] Do not require POSIX style argument order

2017-06-27 Thread Joonas Kylmälä
Package: devscripts
Version: 2.17.6
Severity: wishlist
Tags: patch
User: devscri...@packages.debian.org

Dear Maintainer,

the scripts "bts", "chdist" and "dpkg-depcheck" currently require the user
to put command-line arguments in POSIX way, i.e., option arguments
should come first and then non-option ones. As Perl's Getopt module
supports the environment variable POSIXLY_CORRECT (if it is enabled
then the POSIX order is required) I suggest we leave out the hard
coding of this order requirement out from the scripts. I
have included a patch.

Thank you,
Joona Kylmälä

-- Package-specific info:

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

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

-- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages devscripts depends on:
ii  dpkg-dev  1.18.24
ii  libc6 2.24-12
ii  libfile-homedir-perl  1.00-1
ii  perl  5.24.1-4
ii  python3   3.5.3-1

Versions of packages devscripts recommends:
ii  apt 1.4.6
ii  at  3.1.20-3
ii  dctrl-tools 2.24-2+b1
ii  debian-keyring  2017.05.28
ii  dput0.12.1
ii  equivs  2.0.10
ii  fakeroot1.21-3.1
ii  file1:5.30-1
ii  gnupg   2.1.18-8
ii  libdistro-info-perl 0.15
ii  libdpkg-perl1.18.24
ii  libencode-locale-perl   1.05-1
ii  libgit-wrapper-perl 0.047-1
ii  liblist-compare-perl0.53-1
ii  liblwp-protocol-https-perl  6.06-2
ii  libsoap-lite-perl   1.20-1
ii  liburi-perl 1.71-1
ii  libwww-perl 6.15-1
ii  licensecheck3.0.29-1
ii  lintian 2.5.51
ii  man-db  2.7.6.1-2
ii  patch   2.7.5-1+b2
ii  patchutils  0.3.4-2
ii  python3-debian  0.1.30
ii  python3-magic   1:5.30-1
ii  sensible-utils  0.0.9
ii  strace  4.15-2
ii  unzip   6.0-21
ii  wdiff   1.2.2-2
ii  wget1.19.1-4
ii  xz-utils5.2.2-1.2+b1

Versions of packages devscripts suggests:
pn  adequate 
pn  autopkgtest  
pn  bls-standalone   
ii  build-essential  12.3
pn  check-all-the-things 
pn  cvs-buildpackage 
pn  devscripts-el
pn  diffoscope   
pn  disorderfs   
pn  dose-extra   
pn  duck 
pn  faketime 
pn  gnuplot  
ii  gpgv 2.1.18-8
pn  how-can-i-help   
ii  libauthen-sasl-perl  2.1600-1
pn  libfile-desktopentry-perl
pn  libnet-smtps-perl
pn  libterm-size-perl
ii  libtimedate-perl 2.3000-2
pn  libyaml-syck-perl
ii  mailutils [mailx]1:3.1.1-1
pn  mozilla-devscripts   
ii  mutt 1.8.3+neomutt20170609-2
ii  openssh-client [ssh-client]  1:7.5p1-5
pn  piuparts 
pn  ratt 
pn  reprotest
pn  svn-buildpackage 
pn  w3m  

-- no debconf information
>From dbc7b25c02abcb5669d924adb162cb28a3270c41 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= 
Date: Tue, 27 Jun 2017 18:40:13 +
Subject: [PATCH] Don't require POSIX style argument ordering

---
 scripts/bts.pl   | 2 +-
 scripts/chdist.pl| 2 +-
 scripts/dpkg-depcheck.pl | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/scripts/bts.pl b/scripts/bts.pl
index 2468978d..2be102ed 100755
--- a/scripts/bts.pl
+++ b/scripts/bts.pl
@@ -597,7 +597,7 @@ my $opt_ccemail = "";
 my $use_default_cc = 1;
 my $ccsecurity="";
 
-Getopt::Long::Configure(qw(gnu_compat bundling require_order));
+Getopt::Long::Configure(qw(gnu_compat bundling));
 GetOptions("help|h" => \$opt_help,
   "version" => \$opt_version,
   "o" => \$offlinemode,
diff --git a/scripts/chdist.pl b/scripts/chdist.pl
index 832036c9..3e5ed7be 100755
--- a/scripts/chdist.pl
+++ b/scripts/chdist.pl
@@ -146,7 +146,7 @@ use File::Copy qw(cp);
 use File::HomeDir;
 use File::Path qw(make_path);
 use File::Basename;
-use Getopt::Long qw(:config gnu_compat bundling require_order);
+use Getopt::Long qw(:config gnu_compat bundling);
 use Cwd qw(abs_path cwd);
 use Dpkg::Version;
 use Pod::Usage;
diff --git a/scripts/dpkg-depcheck.pl 

Processed: your mail

2017-06-27 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org:

> tags 205416 - patch
Bug #205416 [devscripts] [new] scripts for mass-mailing to bug reporters
Removed tag(s) patch.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
205416: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=205416
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems

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


Bug#205416: scripts for mass-mailing to bug reporters

2017-06-27 Thread Joonas Kylmälä
Hi,

the script doesn't work anymore. Do you have any time fixing this and do
you think the script is useful anymore as there might have come new
tools that provide this functionality? If it would be useful and you
have time to fix the script then we can keep this bug report open,
otherwise we should close this.

Joonas

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


Bug#866117: devscripts: [bts] Couldn't mkdir /home/user/.cache/devscripts

2017-06-27 Thread Joonas Kylmälä
Package: devscripts
Version: 2.17.6
Severity: normal
Tags: patch

Dear Maintainer,

If I run bts command on a system where there is no ~/.cache directory
I will get the following error:

user@host:~$ bts show 4324324
bts: couldn't mkdir /home/user/.cache/devscripts: No such file or directory

Replacing mkdir function call with make_path fixes the problem. I will
include a patch to this bug report.

Best regards,
Joonas Kylmälä

-- Package-specific info:

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

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

-- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-3-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages devscripts depends on:
ii  dpkg-dev  1.18.24
ii  libc6 2.24-12
ii  libfile-homedir-perl  1.00-1
ii  perl  5.24.1-4
ii  python3   3.5.3-1

Versions of packages devscripts recommends:
ii  apt 1.4.6
ii  at  3.1.20-3
ii  dctrl-tools 2.24-2+b1
ii  debian-keyring  2017.05.28
ii  dput0.12.1
ii  equivs  2.0.10
ii  fakeroot1.21-3.1
ii  file1:5.30-1
ii  gnupg   2.1.18-8
ii  libdistro-info-perl 0.15
ii  libdpkg-perl1.18.24
ii  libencode-locale-perl   1.05-1
ii  libgit-wrapper-perl 0.047-1
ii  liblist-compare-perl0.53-1
ii  liblwp-protocol-https-perl  6.06-2
ii  libsoap-lite-perl   1.20-1
ii  liburi-perl 1.71-1
ii  libwww-perl 6.15-1
ii  licensecheck3.0.29-1
ii  lintian 2.5.51
ii  man-db  2.7.6.1-2
ii  patch   2.7.5-1+b2
ii  patchutils  0.3.4-2
ii  python3-debian  0.1.30
ii  python3-magic   1:5.30-1
ii  sensible-utils  0.0.9
ii  strace  4.15-2
ii  unzip   6.0-21
ii  wdiff   1.2.2-2
ii  wget1.19.1-4
ii  xz-utils5.2.2-1.2+b1

Versions of packages devscripts suggests:
pn  adequate 
pn  autopkgtest  
pn  bls-standalone   
ii  build-essential  12.3
pn  check-all-the-things 
pn  cvs-buildpackage 
pn  devscripts-el
pn  diffoscope   
pn  disorderfs   
pn  dose-extra   
pn  duck 
pn  faketime 
pn  gnuplot  
ii  gpgv 2.1.18-8
pn  how-can-i-help   
ii  libauthen-sasl-perl  2.1600-1
pn  libfile-desktopentry-perl
pn  libnet-smtps-perl
pn  libterm-size-perl
ii  libtimedate-perl 2.3000-2
pn  libyaml-syck-perl
ii  mailutils [mailx]1:3.1.1-1
pn  mozilla-devscripts   
pn  mutt 
ii  openssh-client [ssh-client]  1:7.5p1-5
pn  piuparts 
pn  ratt 
pn  reprotest
pn  svn-buildpackage 
pn  w3m  

-- no debconf information
>From a8368077c3e5e25b6435b4f9b6d7c1a89d2261dd Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= 
Date: Tue, 27 Jun 2017 13:17:09 +
Subject: [PATCH] Make bts work even when there is no ~/.cache

---
 scripts/bts.pl | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/scripts/bts.pl b/scripts/bts.pl
index 2468978d..7c88a23e 100755
--- a/scripts/bts.pl
+++ b/scripts/bts.pl
@@ -3690,12 +3690,12 @@ sub browse {
 elsif ($caching && have_lwp() && $thing ne '') {
if (! $hascache) {
if (! -d dirname($cachedir)) {
-   unless (mkdir(dirname($cachedir))) {
+   unless (make_path(dirname($cachedir))) {
warn "$progname: couldn't mkdir ".dirname($cachedir).": 
$!\n";
goto LIVE;
}
}
-   unless (mkdir($cachedir)) {
+   unless (make_path($cachedir)) {
warn "$progname: couldn't mkdir $cachedir: $!\n";
goto LIVE;
}
-- 
2.13.2

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