Re: Fix for mldonkey postinst script

2009-06-30 Thread Luk Claes
Mehdi Dogguy wrote:
 Hello,
 
 I've prepared a patch (attached) which fixes a minor issue (#534516) for
 postinst script of mldonkey but several users have already reported the
 same issue.
 
 Would it be accepted for the next point-release of lenny?

No, unless there would be a fix for a more important bug accepted.

Cheers

Luk


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Fix for mldonkey postinst script

2009-06-25 Thread Julien Cristau
On Thu, Jun 25, 2009 at 15:48:04 +0200, Mehdi Dogguy wrote:

 diff --git a/debian/mldonkey-server.postinst b/debian/mldonkey-server.postinst
 index 90e9a2c..dfe2a53 100644
 --- a/debian/mldonkey-server.postinst
 +++ b/debian/mldonkey-server.postinst
 @@ -224,7 +224,7 @@ LAUNCH_AT_STARTUP=$launch_at_startup
  MLDONKEY_NICENESS=$mldonkey_niceness
  EOF
  
 -ucf $CONF_NEW $CONF
 +ucf --debconf-ok $CONF_NEW $CONF
  
  $DEBUG  echo done.
  ;;

If DEBUG is ever expected to be set, and there's some debconf
interaction going on here, this is broken (not related to your fix,
but still).  stdout is reserved for debconf.

Cheers,
Julien


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Re: Fix for mldonkey postinst script

2009-06-25 Thread Mehdi Dogguy
Julien Cristau wrote:
 
 If DEBUG is ever expected to be set, and there's some debconf
 interaction going on here, this is broken (not related to your fix,
 but still).  stdout is reserved for debconf.
 

I wanted to make the minimal change is the patch I've attached.
Maintainer scripts will be updated in next unstable versions.

Cheers,

-- 
Mehdi Dogguy مهدي الدڤي
http://www.pps.jussieu.fr/~dogguy
Tel.: (+33).1.44.27.28.38


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org