Package: mdadm
Version: 2.5
Submitter: Jørn V. Christensen <[EMAIL PROTECTED]>
Tags: confirmed, patch

----- Forwarded message from "\"Jørn V. Christensen\"" <[EMAIL PROTECTED]> -----

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 <[EMAIL PROTECTED]>
: :'  :  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

Attachment: signature.asc
Description: Digital signature (GPG/PGP)

Reply via email to