[Bug 716393] New: majordomo does not work with perl 5.12
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c0 Summary: majordomo does not work with perl 5.12 Classification: openSUSE Product: openSUSE 11.4 Version: Final Platform: All OS/Version: openSUSE 11.4 Status: NEW Severity: Normal Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: A.Thomas@cs.rhul.ac.uk QAContact: qa@suse.de Found By: --- Blocker: --- User-Agent: Mozilla/5.0 (compatible; MSIE 9.0; Windows NT 6.1; WOW64; Trident/5.0) Majordomo relies on split implicitely assigning to @_ which seems to have gone away sometime between Perl 5.10 and 5.12, since we encountered this problem after an openSUSE 11.2 to openSUSE 11.4 upgrade. Mail sent to lists using Majordomo is silently thrown away with no obvious clue in the logs. Patching as follows with a temporary array variable in place of @_ fixes the problem: --- majordomo.pl.DIST-11.4 2011-02-19 03:12:04.000000000 +0000 +++ majordomo.pl 2011-09-07 15:12:57.114627473 +0100 @@ -95,14 +95,14 @@ local($_) = shift; 1 while s/\([^\(\)]*\)//g; # strip comments 1 while s/"[^"]*"\s//g; # strip comments" - split(/,/); # split into parts - foreach (@_) { + my @t = split(/,/); # split into parts + foreach (@t) { 1 while s/.*<(.*)>.*/$1/; s/^\s+//; s/\s+$//; } - @_; + @t; } # Check to see if a list is valid. If it is, return the validated list Reproducible: Always -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c zj jia <zjjia@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |zjjia@suse.com AssignedTo|bnc-team-screening@forge.pr |ma@suse.com |ovo.novell.com | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c1 Michael Andres <ma@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Severity|Normal |Critical --- Comment #1 from Michael Andres <ma@suse.com> 2011-09-09 11:13:12 CEST --- Thanks a lot for the patch. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c2 Michael Andres <ma@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO InfoProvider| |maintenance@opensuse.org --- Comment #2 from Michael Andres <ma@suse.com> 2011-09-09 11:46:48 CEST --- Fixed in Factory. @maintenance: Losing mails is a bad thing. We should think about an online update for this. Fixed package is already submitted to openSUSE:11.4:Update:Test (req#81664). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c3 --- Comment #3 from Bernhard Wiedemann <bwiedemann@suse.com> 2011-09-09 12:00:18 CEST --- This is an autogenerated message for OBS integration: This bug (716393) was mentioned in https://build.opensuse.org/request/show/81661 Factory / majordomo https://build.opensuse.org/request/show/81664 11.4 / majordomo -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c4 Marcus Meissner <meissner@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |meissner@suse.com --- Comment #4 from Marcus Meissner <meissner@suse.com> 2011-09-09 11:19:17 UTC --- is fine for me +1 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c5 Christian Dengler <cdengler@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED CC| |cdengler@suse.com InfoProvider|maintenance@opensuse.org | --- Comment #5 from Christian Dengler <cdengler@suse.com> 2011-09-13 14:44:06 UTC --- yes, we should fix it. +1 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c6 Swamp Workflow Management <swamp@suse.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard| |maint:running:43171:importa | |nt --- Comment #6 from Swamp Workflow Management <swamp@suse.de> 2011-09-13 14:44:19 UTC --- The SWAMPID for this issue is 43171. This issue was rated as important. Please submit fixed packages until 2011-09-20. Also create a patchinfo file using this link: https://swamp.suse.de/webswamp/wf/43171 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c7 Michael Andres <ma@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |RESOLVED Resolution| |FIXED --- Comment #7 from Michael Andres <ma@suse.com> 2011-09-15 13:28:49 CEST --- - Also submitted to openSUSE:11.3:Update:Test (req#82206) - Patchinfo created. BTW: can't set DISTRIBUTION to '11.3-*, 11.4-*' to indicate this is for all architectures. Please add whatever else is supported. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c8 Marcus Meissner <meissner@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #8 from Marcus Meissner <meissner@suse.com> 2011-09-15 11:50:30 UTC --- the patch you submitted seems to be incorrect, it is (fully): ++++++ majordomo-1.94.5.fix-split-no-longer-assigning-to-_-bnc-716393.patch ++++++ diff --git a/majordomo.pl b/majordomo.pl index 3c8bbc2..c872751 100644 --- a/majordomo.pl +++ b/majordomo.pl @@ -69,7 +69,7 @@ sub main'ParseMailHeader ## Public $array{$keyw} = $val; } } - ($*, $/) = ($save1, $save2); + ($*, $/) = ($save1, $save2); } and so a no-op -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c9 --- Comment #9 from Michael Andres <ma@suse.com> 2011-09-15 14:01:46 CEST --- Git ate my changes :( -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c10 Michael Andres <ma@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |FIXED --- Comment #10 from Michael Andres <ma@suse.com> 2011-09-15 14:27:28 CEST --- Resubmitted with fixed patch: $ osc submitreq server:mail majordomo openSUSE:Factory created request id 82212 $ osc submitreq server:mail majordomo openSUSE:11.3:Update:Test created request id 82213 $ osc submitreq server:mail majordomo openSUSE:11.4:Update:Test created request id 82214 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c11 --- Comment #11 from Bernhard Wiedemann <bwiedemann@suse.com> 2011-09-16 10:41:12 CEST --- This is an autogenerated message for OBS integration: This bug (716393) was mentioned in https://build.opensuse.org/request/show/82206 https://build.opensuse.org/request/show/82212 https://build.opensuse.org/request/show/82213 https://build.opensuse.org/request/show/82214 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c12 Swamp Workflow Management <swamp@suse.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard|maint:running:43171:importa |maint:running:43171:importa |nt |nt | |maint:released:11.3:43203 | |maint:released:11.4:43203 --- Comment #12 from Swamp Workflow Management <swamp@suse.de> 2011-09-20 11:19:55 UTC --- Update released for: majordomo, majordomo-debuginfo Products: openSUSE 11.3 (debug, i586, x86_64) openSUSE 11.4 (debug, i586, x86_64) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=716393 https://bugzilla.novell.com/show_bug.cgi?id=716393#c Swamp Workflow Management <swamp@suse.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status Whiteboard|maint:running:43171:importa |. |nt | |maint:released:11.3:43203 | |maint:released:11.4:43203 | -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com