Bug 1775 - Please rebuild shorewall for new perl-5.14
Summary: Please rebuild shorewall for new perl-5.14
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Mageia Bug Squad
QA Contact:
URL:
Whiteboard:
Keywords: PATCH
Depends on:
Blocks:
 
Reported: 2011-06-12 23:53 CEST by Dick Gevers
Modified: 2011-06-13 18:48 CEST (History)
0 users

See Also:
Source RPM: shorewall-4.4.20.2-1.mga2
CVE:
Status comment:

Attachments
Tom Eastep provided patch (5.20 KB, patch)
2011-06-13 17:34 CEST, Dick Gevers 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

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
Note You need to log in before you can comment on or make changes to this bug.