Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Olivier Berger
Le vendredi 10 octobre 2008 à 14:19 +0200, Stefan Hornburg a écrit : > It should be more robust by principle as the replacement string is read from > environment > and special quoting isn't needed. Hmmm... the fact that it keeps in the environment afterwards is something a bit problematic (thin

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Olivier Berger wrote: Le vendredi 10 octobre 2008 à 14:19 +0200, Stefan Hornburg a écrit : It should be more robust by principle as the replacement string is read from environment and special quoting isn't needed. Hmmm... the fact that it keeps in the environment afterwards is something a bit

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Olivier Berger wrote: On Thu, Oct 09, 2008 at 05:25:50PM +0200, Patrick Schoenfeld wrote: Hi, On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: Line 212 in the postinst breaks if sympa/key_password contains sed metacharacters, for instance 'a/b/c' as above. attached is a patch for th

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Patrick Schoenfeld wrote: Hi, On Fri, Oct 10, 2008 at 09:46:27AM +0200, Stefan Hornburg wrote: Thanks for the patch! Did you test it already? I tested it manually with a testscript containing only that code and RET="fo4b/!bar" or something. That worked fine. I didn't test it under real cond

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Olivier Berger wrote: On Thu, Oct 09, 2008 at 05:25:50PM +0200, Patrick Schoenfeld wrote: Hi, On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: Line 212 in the postinst breaks if sympa/key_password contains sed metacharacters, for instance 'a/b/c' as above. attached is a patch for th

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Olivier Berger
On Thu, Oct 09, 2008 at 05:25:50PM +0200, Patrick Schoenfeld wrote: > Hi, > > On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: > > Line 212 in the postinst breaks if sympa/key_password contains sed > > metacharacters, for instance 'a/b/c' as above. > > attached is a patch for this issue

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Patrick Schoenfeld wrote: Hi, On Fri, Oct 10, 2008 at 09:46:27AM +0200, Stefan Hornburg wrote: Thanks for the patch! Did you test it already? I tested it manually with a testscript containing only that code and RET="fo4b/!bar" or something. That worked fine. I didn't test it under real cond

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Patrick Schoenfeld
Hi, On Fri, Oct 10, 2008 at 09:46:27AM +0200, Stefan Hornburg wrote: > Thanks for the patch! Did you test it already? I tested it manually with a testscript containing only that code and RET="fo4b/!bar" or something. That worked fine. I didn't test it under real conditions though. Its probably a

Bug#501605: sympa: broken post-inst script

2008-10-10 Thread Stefan Hornburg
Patrick Schoenfeld wrote: Tags 501605 +patch thanks Hi, On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: Line 212 in the postinst breaks if sympa/key_password contains sed metacharacters, for instance 'a/b/c' as above. attached is a patch for this issue. Thanks for the patch! Did

Processed: Re: Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]: > Tags 501605 +patch Bug#501605: sympa: broken post-inst script There were no tags set. Tags added: patch > thanks Stopping processing here. Please contact me if you need assistance. Debian bug tracking system administrator (administrator, Debia

Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Patrick Schoenfeld
Tags 501605 +patch thanks Hi, On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: > Line 212 in the postinst breaks if sympa/key_password contains sed > metacharacters, for instance 'a/b/c' as above. attached is a patch for this issue. Best Regards, Patrick --- sympa-5.3.4/debian/postins

Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Patrick Schoenfeld
Hi, On Thu, Oct 09, 2008 at 03:17:43PM +0300, Niko Tyni wrote: > Line 212 in the postinst breaks if sympa/key_password contains sed > metacharacters, for instance 'a/b/c' as above. > > This looks RC to me, so upgrading back. I already had this line in suspect, but I didn't think about that. Good

Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Stefan Hornburg (Racke)
Niko Tyni wrote: severity 501605 serious tag 501605 - unreproducible thanks On Thu, Oct 09, 2008 at 01:42:39PM +0200, Patrick Schoenfeld wrote: Severity 501605 important thanks I have tested the installation of sympa as well and I can't reproduce the problem. Additional I auditted the post-i

Processed: Re: Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Debian Bug Tracking System
Processing commands for [EMAIL PROTECTED]: > severity 501605 serious Bug#501605: sympa: broken post-inst script Severity set to `serious' from `important' > tag 501605 - unreproducible Bug#501605: sympa: broken post-inst script Tags were: unreproducible Tags removed: unreprod

Bug#501605: sympa: broken post-inst script

2008-10-09 Thread Patrick Schoenfeld
Severity 501605 important thanks Hi, I have tested the installation of sympa as well and I can't reproduce the problem. Additional I auditted the post-inst script and I can't see any condition under which one of the sed commands could fail. All seem reasonable and don't fail when I test them manu

Bug#501605: sympa: broken post-inst script

2008-10-08 Thread Carl Fürstenberg
Package: sympa Version: 5.3.4-6 Severity: grave Justification: renders package unusable Installation fails with following output: Setting up sympa (5.3.4-6) ... sed: -e expression #1, char 51: unknown option to `s' dpkg: error processing sympa (--configure): subprocess post-installation script r