Bug#413330: postinst script can't deal with multiple, comma-separate MAILADDR entries

[martin f krafft]

Package: mdadm
Version: 2.5
Submitter: Jørn V. Christensen <mail at mettle.dk>
Tags: confirmed, patch

----- Forwarded message from "\"Jørn V. Christensen\"" <mail at mettle.dk> -----

Hi,

I am writing to you because you are listed as the maintainer of the 
mdadm package in Debian. I have found (and fixed?) a bug in the post 
installation script for mdadm 2.5.6-8.

Basically what happened, was that sed complained about a misstated command:
   sed: -e expression #1, char 30: unknown option to `s'

I investigated the error and found that the it happened while sed was 
trying to update the mail addresses. The problem is that I have listed 
two mail addresses (separated by a comma) and the sed command delimiter 
character was also chosen to be commas. Thus the resulting sed command 
would have 4 commas instead of 3.

The fix: Changing the sed delimiter character to / instead of comma.

73c73
<         sed -i -e "s,^MAILADDR.*,MAILADDR $MAILADDR," $CONFIG
---
>         sed -i -e "s/^MAILADDR.*/MAILADDR $MAILADDR/" $CONFIG

The bug is rather old (several months) but I did not do anything about 
it because I thought you (as the maintainer) would see the bug and make 
a fix. I did not realize until now that it was because I have two 
addresses listed instead of one.

Anyway... hope it is useful to you...

Best regards
Jørn

----- End forwarded message -----

-- 
 .''`.   martin f. krafft <madduck at debian.org>
: :'  :  proud Debian developer, author, administrator, and user
`. `'`   http://people.debian.org/~madduck - http://debiansystem.info
  `-  Debian - when you have better things to do than fixing systems
 
"the question of whether computers can think
 is like the question of whether submarines can swim."
                                                 -- edsgar w. dijkstra
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature (GPG/PGP)
Url : http://lists.alioth.debian.org/pipermail/pkg-mdadm-devel/attachments/20070304/29930fac/attachment.pgp