[Bug 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 Nathan Owe ndowen...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution||WONTFIX Last Closed||2012-04-08 20:56:59 --- Comment #19 from Nathan Owe ndowen...@gmail.com 2012-04-08 20:56:59 EDT --- Deciding to withdraw from Review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
--- Comment #18 from Nathan Owe ndowen...@gmail.com 2011-09-01 14:09:02 EDT
---
I created my service file before you attached the file and it looks exactly
like that, except it didn't include Type, since if left out, the type would be
simple by default and it doesn't want to work. To install it, I simply did:
Source1: %{name}.service
%install
...
install -Dm644 %{SOURCE1} %{_unitdir}/%{name}.service
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #16 from Nathan Owe ndowen...@gmail.com 2011-08-30 15:32:53 EDT --- I don't see any attached files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #17 from Haïkel Guémar karlthe...@gmail.com 2011-08-30 16:23:48 EDT --- Created attachment 520684 -- https://bugzilla.redhat.com/attachment.cgi?id=520684 ufw sample systemd service Sorry, i forgot to add it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 Haïkel Guémar karlthe...@gmail.com changed: What|Removed |Added CC||karlthe...@gmail.com --- Comment #15 from Haïkel Guémar karlthe...@gmail.com 2011-08-28 08:33:04 EDT --- according to ufw documentation, you need to provide your own initscripts, these are only helpers. Forget about SysV initscripts, and provide a systemd service instead (i attached a working albeit basic one) * drop the BR: iptables-devel, you don't need it * using %find_lang is a must, they should be installed in location accordingly to GNU standards. You may have to check this with upstream http://www.gnu.org/prep/standards/html_node/Directory-Variables.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 Nathan Owe ndowen...@gmail.com changed: What|Removed |Added Flag|fedora-review? | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #14 from Nathan Owe ndowen...@gmail.com 2011-08-11 13:22:50 EDT --- I am not sure what to do about the following init scripts: /lib/ufw/ufw-init /lib/ufw/ufw-init-functions I know ufw enable will pretty much do /lib/ufw/ufw-init start. So really do I need to install it to /etc/rc.d/init.d even though ufw looks for them in /lib and I also tried doing something like it shows in the wiki: /sbin/chkconfig --add /lib/ufw/ufw-init and it gives a error like error reading information on service ufw-init: No such file or directory -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #12 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-03 02:34:47 EDT --- (In reply to comment #10) On the scrambled email address. I hope it is allowed to help combat spambots from getting it so my email won't get cluttered with spam. Seems like scrambling is OK. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #13 from Nathan Owe ndowen...@gmail.com 2011-08-03 21:35:35 EDT --- (In reply to comment #11) Also can somebody install ufw and add/delete a firewall rule in it and see if it does have an effect. When I do a port scan on my PC it doesn't seem to make a difference. Well just remembered that I am also behind a router so that might be why I see no difference. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
--- Comment #5 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-02 03:51:34
EDT ---
(In reply to comment #4)
To start off, yes I am a packager :)
Okay. I was just wondering why you didn't make %files simpler:
%files
%doc ChangeLog COPYING README* TODO AUTHORS
%{__libdir}/ufw/
%{_datadir}/ufw/
%{_sbindir}/ufw
%{python_sitelib}/ufw-%{version}-py*.egg-info
%{python_sitelib}/ufw/
%config(noreplace) %{_sysconfdir}/default/
%config(noreplace) %{_sysconfdir}/ufw/
%{_mandir}/man8/ufw-framework.8*
%{_mandir}/man8/ufw.8*
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #6 from Nathan Owe ndowen...@gmail.com 2011-08-02 10:09:13 EDT --- Mainly because some people say that the file list should be more detailed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
--- Comment #7 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-02 10:31:23
EDT ---
(In reply to comment #6)
Mainly because some people say that the file list should be more detailed.
I'm one of them: for instance, I abhor too freeminded use of wildcards such as
%{python_sitelib}/*
which may up owning stuff that it isn't supposed to. Or, for the current
example, one might not detect if the egg-info isn't built (which may happen
e.g. on EPEL-5, where one has to do a trick to get it).
However, there's no sense in listing every single file that goes in the RPM, if
they are placed in a package-specific directory, or obey some sane naming
scheme, e.g.
%{_bindir}/foo
%{_bindir}/foo-*
Being too specific can also bite you, as you noticed.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #8 from Nathan Owe ndowen...@gmail.com 2011-08-02 12:11:00 EDT --- SPEC: http://ndowens.fedorapeople.org/SPECS/ufw.spec SRPM: http://ndowens.fedorapeople.org/SRPM/ufw-0.30.1-3.fc15.src.rpm Those are the updated version -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi
Flag||fedora-review?
--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-02 13:24:13
EDT ---
Some more comments:
Please use the %{version} macro for
%{python_sitelib}/ufw-0.30.1*
And, since it is an egg-info file, it's better to be a bit more explicit:
%{python_sitelib}/ufw-%{version}-py*.egg-info.
**
You are still missing ownership of
%{_datadir}/ufw/iptables/
Please combine
%{_datadir}/ufw/iptables/*.rules
%dir %{_datadir}/ufw/
to simply
%{_datadir}/ufw/
**
You are mixing styles:
http://fedoraproject.org/wiki/Packaging/Guidelines#Using_.25.7Bbuildroot.7D_and_.25.7Boptflags.7D_vs_.24RPM_BUILD_ROOT_and_.24RPM_OPT_FLAGS
Since this is a noarch package, you should drop CFLAGS=$RPM_OPT_FLAGS from
the build statement.
**
I'm also not sure why you want to break lines to exceedingly small length in
%{__python} setup.py install \
--skip-build \
--root %{buildroot}
but the very next line is very long.
**
By the way, there is a reason why firewall rules are not world readable. They
should be owned by root and installed as 600.
**
What does Patch0 do? Please add a comment about it in the spec file.
**
Note that you do not need to use the %{__python} macro, plain python will do
just fine.
**
I am wondering whether the use of a scrambled email address is allowed, but the
guideline is a bit unclear. I'll get this cleared up.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Status|ASSIGNED|NEW AssignedTo|jussi.leht...@iki.fi|nob...@fedoraproject.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #10 from Nathan Owe ndowen...@gmail.com 2011-08-02 19:13:47 EDT --- SPEC: http://ndowens.fedorapeople.org/SPECS/ufw.spec SRPM: http://ndowens.fedorapeople.org/SRPM/ufw-0.30.1-4.fc15.src.rpm On the scrambled email address. I hope it is allowed to help combat spambots from getting it so my email won't get cluttered with spam. I am not sure what to do about the following init scripts: /lib/ufw/ufw-init /lib/ufw/ufw-init-functions I know ufw enable will pretty much do /lib/ufw/ufw-init start. So really do I need to install it to /etc/rc.d/init.d even though ufw looks for them in /lib and I also tried doing something like it shows in the wiki: /sbin/chkconfig --add /lib/ufw/ufw-init and it gives a error like error reading information on service ufw-init: No such file or directory -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #11 from Nathan Owe ndowen...@gmail.com 2011-08-02 19:21:24 EDT --- Also can somebody install ufw and add/delete a firewall rule in it and see if it does have an effect. When I do a port scan on my PC it doesn't seem to make a difference. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
Veeti Paananen veeti.paana...@rojekti.fi changed:
What|Removed |Added
CC||veeti.paana...@rojekti.fi
--- Comment #2 from Veeti Paananen veeti.paana...@rojekti.fi 2011-08-01
11:31:34 EDT ---
Correct me if I'm wrong, but couldn't you use %{_lib}?
(http://fedoraproject.org/wiki/Packaging:RPMMacros#Other_macros_and_variables_for_paths)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=727030
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
CC||jussi.leht...@iki.fi
--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi 2011-08-01 12:24:19
EDT ---
Why on earth does it install something into /lib anyway?
%{__libdir}/ufw/ufw*
%{__libdir}/ufw/user*.rules
looks a lot like configuration files.
Besides, you're missing ownership of the following directories:
%{__libdir}/ufw/
%{python_sitelib}/ufw/
%{_datadir}/ufw/
Are you a packager, yet..?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #4 from Nathan Owe ndowen...@gmail.com 2011-08-01 23:13:38 EDT --- To start off, yes I am a packager :) @Veeti: _lib doesn't point to /lib that is why I had to create a variable/macro Well apparently the coding they used is creating a problem even in trying to run ufw status. So I have updated my SPEC file repairing two problems: 1. Ownership of dirs 2. The coding in the file used variables like #CONFIG_DIR#/ when parsing the file the variable wouldn't be understood. So I had to create a patch pointing the variables to the correct directories. SPEC: http://ndowens.fedorapeople.org/SPECS/ufw.spec SRPM: http://ndowens.fedorapeople.org/SRPM/ufw-0.30.1-2.fc15.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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 727030] Review Request: ufw - uncomplicated firewall
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=727030 --- Comment #1 from Nathan Owe ndowen...@gmail.com 2011-08-01 01:03:37 EDT --- The only errors I get is a spelling error for netfilter I also get hardcoded-library-path /lib even though I tried to work around it by creating a macro __libdir /lib to get around that. [ndowens@revan rpmbuild]$ rpmlint SRPMS/ufw-0.30.1-1.fc15.src.rpm ufw.src: W: spelling-error %description -l en_US netfilter - net filter, net-filter, filterer ufw.src:1: E: hardcoded-library-path in /lib 1 packages and 0 specfiles checked; 1 errors, 1 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- 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
