[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #18 from Susi Lehtola --- Ping Christopher. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=0iy0AOZHDk&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #19 from Christopher Meng --- Hi Susi I'm busy these days. Actually I just need to write a init file now, but it takes time. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=H8UJeF6A2G&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #20 from Susi Lehtola --- So maintaining hundreds of packages is not a walk in the park eh? There should be an init file already. And you can always adapt the one from firehol. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=htdF9NL28O&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #21 from Christopher Meng --- (In reply to Susi Lehtola from comment #20) > So maintaining hundreds of packages is not a walk in the park eh? No, you're thinking too overhead. > > There should be an init file already. And you can always adapt the one from > firehol. The reason why I can't go ahead is that this init file included is designed for Debian systems, in order to follow the guideline I have to rewrite a Fedora one. BTW there also have bugs in upstream's init file. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=PYsSKOFqKh&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Björn "besser82" Esser changed: What|Removed |Added Status|NEW |ASSIGNED CC||bjoern.es...@gmail.com, ||cicku...@gmail.com Assignee|nob...@fedoraproject.org|bjoern.es...@gmail.com Flags||fedora-review? ||needinfo?(cicku...@gmail.co ||m) --- Comment #22 from Björn "besser82" Esser --- taken ;) Any news on this, yet? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Björn "besser82" Esser changed: What|Removed |Added Flags||needinfo?(cicku...@gmail.co ||m) --- Comment #23 from Björn "besser82" Esser --- Both links give me 404... -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Christopher Meng changed: What|Removed |Added Whiteboard||NotReady Flags|needinfo?(cicku...@gmail.co | |m) | |needinfo?(cicku...@gmail.co | |m) | --- Comment #24 from Christopher Meng --- (In reply to Björn "besser82" Esser from comment #23) > Both links give me 404... DO NOT DISTURB ME NOW. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Björn "besser82" Esser changed: What|Removed |Added Flags||needinfo?(cicku...@gmail.co ||m) --- Comment #25 from Björn "besser82" Esser --- (In reply to Christopher Meng from comment #24) > (In reply to Björn "besser82" Esser from comment #23) > > Both links give me 404... > > DO NOT DISTURB ME NOW. WTF??? I just wanted to review this and the links to psec/srpm gave me a 404. What's your f*g problem? What did I do wrong? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #26 from Susi Lehtola --- Ping. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Christopher Meng changed: What|Removed |Added Flags|needinfo?(cicku...@gmail.co | |m) | --- Comment #27 from Christopher Meng --- I will finish this soon. BTW, I found that firehol is not dead: http://firehol.org/download/latest/ Maybe you need to update it and I remove the provides line in sanewall spec? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #28 from Susi Lehtola --- (In reply to Christopher Meng from comment #27) > BTW, I found that firehol is not dead: > http://firehol.org/download/latest/ Well they sure took a long hiatus. And also changed the location of the tarball, so it was not picked up by release monitoring. > Maybe you need to update it and I remove the provides line in sanewall spec? OK. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Douglas Schilling Landgraf changed: What|Removed |Added CC||dougsl...@redhat.com Assignee|nob...@fedoraproject.org|dougsl...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=pQUP5wTHJg&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Susi Lehtola changed: What|Removed |Added CC||susi.leht...@iki.fi --- Comment #3 from Susi Lehtola --- IMO this package could replace firehol in Fedora; the last release of Firehol was 5 years ago and it needs systemd support and so on. If this package gains EPEL support, I'll be happy to review it and mark firehol dead. ** The Conflicts: firehol is incorrect - the packages can peacefully coexist on a system. ** What's the source for SOURCE1? Has the file been sent upstream? ** The list in the %description should IMHO be written in the form Sanewall can be used for almost any firewall need, including: * control of any number of internal/external/virtual interfaces * control of any combination of routed traffic * setting up DMZ routers and servers * all kinds of NAT * providing strong protection (flooding, spoofing, etc.) * transparent caches * source MAC verification * blacklists, whitelists -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=iPKbstykXW&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #4 from Christopher Meng --- Thanks Susi, you can reset the assignee to you, I've contacted Douglas. I think he is willing to someone can help do a review. It's too late today, I'll update the package to the latest version and check the issues you've mentioned tomorrow. Just a thought before sleep, you mean I can add obsolete tag to replace the original firehol? If so I think it's great. (like mariadb and mysql?) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=RP1pdtg0gm&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #5 from Susi Lehtola --- (In reply to Christopher Meng from comment #4) > Just a thought before sleep, you mean I can add obsolete tag to replace the > original firehol? If so I think it's great. (like mariadb and mysql?) Yes. But you'll also need to add a Provides, because sanewall really is a replacement for FireHOL. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=BdOclzTxmr&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Douglas Schilling Landgraf changed: What|Removed |Added Assignee|dougsl...@redhat.com|nob...@fedoraproject.org -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=5AtI80Mwmc&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #6 from Susi Lehtola --- Actually, the Obsoletes is out of the question, since sanewall isn't a drop-in replacement; e.g. the config file is in a different place. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=Rugri80PHk&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #7 from Christopher Meng --- Source1 is written by myself, sent to upstream, but upstream hasn't decided to support this new toy. I've fixed all your suggestions. Spec URL: http://cicku.me/sanewall.spec SRPM URL: http://cicku.me/sanewall-1.1.4-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=xUVlRIIe3k&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #8 from Susi Lehtola --- EPEL support is still missing. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=LOhK7gJmFk&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #9 from Susi Lehtola --- And the package doesn't compile. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=k77F6TInoh&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #10 from Susi Lehtola --- Any progress? -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=I6neftBfo8&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #11 from Susi Lehtola --- Second ping Christopher. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AfolmKSQhp&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #12 from Christopher Meng --- In that days 31/07~02/08, SSH connection was disturbed. I'm not sure if you found the SRPM was corrupt or whatever other reasons, all packages submitted can be built on my Rawhide machine without problems. I've reuploaded the same revision to the URL mentioned in comment 7. I don't maintain one spec for various OS, I always create a new one for the el branch. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=o2lPLzLhuI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #13 from Susi Lehtola --- (In reply to Christopher Meng from comment #12) > I'm not sure if you found the SRPM was corrupt or whatever other reasons, > all packages submitted can be built on my Rawhide machine without problems. Still does not build in mock. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=C4ScayBJ5Q&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #14 from Christopher Meng --- (In reply to Susi Lehtola from comment #13) > (In reply to Christopher Meng from comment #12) > > I'm not sure if you found the SRPM was corrupt or whatever other reasons, > > all packages submitted can be built on my Rawhide machine without problems. > > Still does not build in mock. Hi Susi, any logs available? This will help us. I'm outside now, the next time I can use computer is 15 hrs later... -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=4rrwIT3mXN&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #15 from Susi Lehtola --- (In reply to Christopher Meng from comment #14) > > Still does not build in mock. > > Hi Susi, any logs available? > > This will help us. I'm pretty sure there is more than a single problem. Now it fails due to a missing BR: hostname. Before submitting a review *do* check that the package builds in koji, or at least for one architecture and distribution in mock. Not doing so just incurs that the reviewer needs to be spending time to waste. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=QjK4vUdpqI&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #16 from Christopher Meng --- (In reply to Susi Lehtola from comment #15) > (In reply to Christopher Meng from comment #14) > > > Still does not build in mock. > > > > Hi Susi, any logs available? > > > > This will help us. > > I'm pretty sure there is more than a single problem. Now it fails due to a > missing BR: hostname. > > Before submitting a review *do* check that the package builds in koji, or at > least for one architecture and distribution in mock. > > Not doing so just incurs that the reviewer needs to be spending time to > waste. Sorry, added missing hostname/kmod/iptables/iproute/procps-ng BRs. New SPEC URL:http://cicku.me/sanewall.spec New SRPM URL:http://cicku.me/sanewall-1.1.4-2.fc20.src.rpm Koji success: http://koji.fedoraproject.org/koji/taskinfo?taskID=5818302 -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=oX42BJPX4m&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #17 from Susi Lehtola --- Fails to build in F19. error: Installed (but unpackaged) file(s) found: /usr/share/doc/sanewall/examples/adblock.sh /usr/share/doc/sanewall/examples/client-all.conf /usr/share/doc/sanewall/examples/lan-gateway.conf /usr/share/doc/sanewall/examples/office.conf /usr/share/doc/sanewall/examples/server-dmz.conf /usr/share/doc/sanewall/html/sanewall-manual.css /usr/share/doc/sanewall/html/sanewall-manual.html /usr/share/doc/sanewall/html/sanewall-services.html /usr/share/doc/sanewall/sanewall-manual.pdf Installed (but unpackaged) file(s) found: /usr/share/doc/sanewall/examples/adblock.sh /usr/share/doc/sanewall/examples/client-all.conf /usr/share/doc/sanewall/examples/lan-gateway.conf /usr/share/doc/sanewall/examples/office.conf /usr/share/doc/sanewall/examples/server-dmz.conf /usr/share/doc/sanewall/html/sanewall-manual.css /usr/share/doc/sanewall/html/sanewall-manual.html /usr/share/doc/sanewall/html/sanewall-services.html /usr/share/doc/sanewall/sanewall-manual.pdf Child return code was: 1 ** I still would like to see the EL spec files as well, since SysV init scripts are handled in a different way. I have no idea why you wouldn't want to use a unified spec file, since the differences can be handled with a few lines of %if's. -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=XL1yIoZoBi&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Maarten Bremer changed: What|Removed |Added CC||maar...@xolphin.nl --- Comment #29 from Maarten Bremer --- Firehol just released 2.0.0-RC1, largely based on the Sanewall fork. I would suggest to upgrade Firehol to version 2 and abandon Sanewall. I added a seperate bugreport for this, see https://bugzilla.redhat.com/show_bug.cgi?id=1128387 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
https://bugzilla.redhat.com/show_bug.cgi?id=960645 Christopher Meng changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NOTABUG Flags|fedora-review? | Last Closed||2014-08-10 23:04:44 --- Comment #30 from Christopher Meng --- Upstream will stop the development and merge the changes back to the firehol. Glad to see the resurrection of the firehol. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #1 from Christopher Meng --- New Upstream Version: New SPEC URL:http://cicku.me/sanewall.spec New SRPM URL:http://cicku.me/sanewall-1.1.2-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=9CkOnHHZUU&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 960645] Review Request: sanewall - A powerful firewall builder
Product: Fedora https://bugzilla.redhat.com/show_bug.cgi?id=960645 --- Comment #2 from Christopher Meng --- Note this RPM currently is not for EPEL(thus RHEL). -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=8416iWUYhx&a=cc_unsubscribe ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
