Bug 1775

Summary: Please rebuild shorewall for new perl-5.14
Product: Mageia Reporter: Dick Gevers <dvgevers>
Component: RPM PackagesAssignee: Mageia Bug Squad <bugsquad>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: Normal Keywords: PATCH
Version: Cauldron   
Target Milestone: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Source RPM: shorewall-4.4.20.2-1.mga2 CVE:
Status comment:
Attachments: Tom Eastep provided patch

Description Dick Gevers 2011-06-12 23:53:26 CEST
Description of problem:

sudo service shorewall startCompiling...
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Config.pm line 3408.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Config.pm line 3613.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1532.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1536.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1540.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1544.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1549.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1575.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1579.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1583.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 1587.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Chains.pm line 4715.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Rules.pm line 459.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Misc.pm line 1667.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Compiler.pm line 111.
Use of qw(...) as parentheses is deprecated at /usr/share/shorewall/Shorewall/Compiler.pm line 119.
Dick Gevers 2011-06-12 23:54:29 CEST

Priority: Normal => High

Comment 1 Dick Gevers 2011-06-13 03:10:50 CEST
Thanks Ahmad for updating shorewall to 4.4.20.2. 

But the error / warning (?) reportedd remains. Is this because Mageia is ahead of shorewall providing perl's version? Is it only a warning that shorewall should soon get it's act together and harmless? or?

Thanks beforehand!
Comment 2 Ahmad Samir 2011-06-13 06:38:14 CEST
(In reply to comment #1)
> Thanks Ahmad for updating shorewall to 4.4.20.2. 
> 
> But the error / warning (?) reportedd remains. Is this because Mageia is ahead
> of shorewall providing perl's version? Is it only a warning that shorewall
> should soon get it's act together and harmless? or?
> 

Yes, shorewall upstream will have to adapt the code. I am not sure, but I think it's harmless warning, still a bug should be filed upstream.

Priority: High => Normal
Severity: major => normal

Comment 3 Dick Gevers 2011-06-13 06:41:48 CEST
I'll do what I can.
Comment 4 Dick Gevers 2011-06-13 09:38:15 CEST
Posted a question to shorewall-devel, see:
http://blog.gmane.org/gmane.comp.security.shorewall.devel
and waiting for feedback. If no replies in a reasonable time, I'll register at shorewall.net & post a ticket on their trac.

Keywords: (none) => NEEDINFO
Assignee: bugsquad => dvgevers

Comment 5 Dick Gevers 2011-06-13 17:34:03 CEST
Created attachment 559 [details]
Tom Eastep provided patch

Tom's comment:
...
Yes -- it is a harmless(but annoying) warning from the Perl compiler.
...

Ahmad: would you please rebuild and close present? Thanks!
Comment 6 Dick Gevers 2011-06-13 17:40:47 CEST
BTW: the patch is approved

Keywords: NEEDINFO => (none)
Assignee: dvgevers => bugsquad

Comment 7 Dick Gevers 2011-06-13 18:21:17 CEST
For the sake of good order, quoting Tom Eastep:

cd /usr/share/shorewall/Shorewall
patch < .../QW.patch

Keywords: (none) => PATCH
Source RPM: shorewall-4.4.19.1-3.mga1 => shorewall-4.4.20.2-1.mga2

Comment 8 Ahmad Samir 2011-06-13 18:48:33 CEST
The patch has been applied.

Thanks!

Status: NEW => RESOLVED
Resolution: (none) => FIXED