Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 101758 - dynamic iptables firewalls article links to tar with broken script
Summary: dynamic iptables firewalls article links to tar with broken script
Status: RESOLVED FIXED
Alias: None
Product: [OLD] Docs-user
Classification: Unclassified
Component: Other (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Shyam Mani (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-08 09:21 UTC by Łukasz Damentko (RETIRED)
Modified: 2005-09-08 04:43 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Łukasz Damentko (RETIRED) gentoo-dev 2005-08-08 09:21:23 UTC
Hm, so....

We have two articles:

http://www.gentoo.org/doc/en/articles/dynamic-iptables-firewalls.xml
http://www.gentoo.org/proj/en/dynfw.xml

First one links to second, where's link to that tarball:

http://tirpitz.iat.sfu.ca/~robbat2/dynfw-1.0.1.tar.gz

In it we have described in the article set of scripts and one to install them
all, "install.sh"

In it there's line:

        chown 0.0 ${PREFIX}/sbin/${x}

Which for me should be changed to:

       chown 0:0 ${PREFIX}/sbin/${x}

(colon instead of dot in a chown parameter)

My proposition of fixing it is:

Confirm a bug with some bash hackers (we have some in team i believe), then fix
it and also host a fixed tar file locally, not outside of gentoo.org. Then there
also needs to be changed link to in in the article.

That's all, waiting comments.
Comment 1 Shyam Mani (RETIRED) gentoo-dev 2005-09-08 03:51:44 UTC
Yeah, chown x.y was accepted earlier. Now it gives : 

trinity ~ # chown shyam.users test.c
chown: `shyam.users' syntax is obsolete; use `:' since this will be removed in
the future

Which means, it'll still work....but we don't know for how long. 
Comment 2 Shyam Mani (RETIRED) gentoo-dev 2005-09-08 04:43:12 UTC
Fixed in CVS, added updated tar.bz2 and updated the article and the dynfw.xml in
/proj/en.

Thanks for reporting!