Bug 321721 - Review Request: shorewall-shell - Shell-based compiler for Shoreline Firewall
Summary: Review Request: shorewall-shell - Shell-based compiler for Shoreline Firewall
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 321731
TreeView+ depends on / blocked
Reported: 2007-10-07 00:54 UTC by Jonathan Underwood
Modified: 2007-11-30 22:12 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2007-10-07 23:38:57 UTC
Type: ---

Attachments (Terms of Use)

Description Jonathan Underwood 2007-10-07 00:54:10 UTC
Spec URL: http://jgu.fedorapeople.org/shorewall-shell.spec
SRPM URL: http://jgu.fedorapeople.org/shorewall-shell-4.0.4-1.fc7.src.rpm
The Shoreline Firewall, more commonly known as "Shorewall", is a
Netfilter (iptables) based firewall that can be used on a dedicated
firewall system, a multi-function gateway/ router/server or on a
standalone GNU/Linux system. 

The version 3 release series of Shorewall is already available in Fedora. With the release of version 4, upstream has added a new perl based rule compiler and completely changed the way the package is distributed. The shell-based and perl-based compilers are each distributed as individual tarballs, and files required to run shorewall with either compiler are packaged as a third tarball, shorewall-common. 

The shorewall-perl compilers is suggested for new
installed systems and shorewall-shell is provided for backwards
compatibility and smooth legacy system upgrades because shorewall perl
is not fully compatible with all legacy configurations.

This package contains the files comprising the legacy shell based compiler

Comment 1 Jonathan Underwood 2007-10-07 00:55:17 UTC
$ rpmlint -i ../RPMS/noarch/shorewall-shell-4.0.4-1.fc7.noarch.rpm 
shorewall-shell.noarch: W: empty-%post
shorewall-shell.noarch: W: empty-%preun
shorewall-shell.noarch: W: empty-%postun

--> These can be ignored.

Comment 2 Jonathan Underwood 2007-10-07 00:57:16 UTC
Added current shorewall package owner to cc. 

Robert - I'm not trying to usurp your package here, but I thought that because
upstream has changed so much, and because Id done the packaging work for other
reasons, it would be useful to put them into BZ for review. I am more than happy
if you want to continue owning this package. Am also happy to co-maintain
shorewall with you, if you like.

Comment 3 Jonathan Underwood 2007-10-07 23:38:57 UTC
Following discussion with Robert, and also on #fedora-devel, consensus seems to
be that it is better to have a single package with all tarballs. Therefore, I'm
closing this review, and discussion of the multitarball package will continue in
BZ #321731

Note You need to log in before you can comment on or make changes to this bug.