Bug#373664: bts: don't hardcode sendmail -t option

[Bastian Kleineidam]

Package: devscripts
Version: 2.9.20
Severity: normal
Tags: patch

Hi,

I want to use this config entry in ~/.devscripts:
BTS_SENDMAIL_COMMAND='/usr/sbin/qmail-inject [EMAIL PROTECTED]'

But since bts hardcodes the "-t" option, qmail-inject fails to send
mail.
I made patch that only adds "-t" to $sendmailcmd if it starts with
the default /usr/sbin/sendmail. This should be backwards compatible,
and allows me to use qmail-inject as mailer.


Regards,
  Bastian

-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.16-treasure18
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=ISO-8859-15)

Versions of packages devscripts depends on:
ii  debianutils                   2.16.1     Miscellaneous utilities specific t
ii  dpkg-dev                      1.13.21    package building tools for Debian
ii  libc6                         2.3.6-15   GNU C Library: Shared libraries
ii  perl                          5.8.8-6    Larry Wall's Practical Extraction 
ii  sed                           4.1.5-1    The GNU sed stream editor

Versions of packages devscripts recommends:
ii  fakeroot                      1.5.8      Gives a fake root environment

-- no debconf information

--- bts.pl      2006-04-16 00:35:16.000000000 +0200
+++ /home/calvin/bts.pl 2006-06-14 23:16:26.974550394 +0200
@@ -396,6 +396,9 @@
     }
 }
 $sendmailcmd = $opt_sendmail if $opt_sendmail;
+if ($sendmailcmd =~ /^\/usr\/sbin\/sendmail/) {
+    $sendmailcmd .= " -t";
+}
 
 if ($opt_cachemode) {
     if ($opt_cachemode =~ /^(min|mbox|full)$/) {
@@ -1570,7 +1573,7 @@
                exec("/bin/cat")
                    or die "bts: error running cat: $!\n";
            } else {
-               exec(split(' ', $sendmailcmd), "-t")
+               exec(split(' ', $sendmailcmd))
                    or die "bts: error running sendmail: $!\n";
            }
        }