Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 279926 - net-analyzer/snort-2.8.4.2: typo in ebuild doesn't remove trailing '/'
Summary: net-analyzer/snort-2.8.4.2: typo in ebuild doesn't remove trailing '/'
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Patrick Lauer
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-01 17:10 UTC by Todd Partridge
Modified: 2009-09-09 15:35 UTC (History)
2 users (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 Todd Partridge 2009-08-01 17:10:50 UTC
snort-2.8.4.2.ebuild type doesn't remove trailing '/'

Line 295:

  sed -i -e 's:snort_dynamicrule/$:snort_dynamicrules:g' \

should be:

  sed -i -e 's:snort_dynamicrules/$:snort_dynamicrules:g' \

Reproducible: Always
Comment 1 Wormo (RETIRED) gentoo-dev 2009-08-01 19:00:59 UTC
Thanks for reporting this typo bug. Assigning to maintainers.
Comment 2 Jason Wallace 2009-08-08 22:29:22 UTC
(In reply to comment #0)
> snort-2.8.4.2.ebuild type doesn't remove trailing '/'
> 
> Line 295:
> 
>   sed -i -e 's:snort_dynamicrule/$:snort_dynamicrules:g' \
> 
> should be:
> 
>   sed -i -e 's:snort_dynamicrules/$:snort_dynamicrules:g' \
> 
> Reproducible: Always
> 

Thx for reporting this. I expect snort-2.8.5 will come out of beta very soon. I'll add this to my to do list when I releases the new ebuild for the new version.
Comment 3 Patrick Lauer gentoo-dev 2009-08-17 16:18:34 UTC
+  17 Aug 2009; Patrick Lauer <patrick@gentoo.org> snort-2.8.4.1.ebuild:
+  Small typo fix, closes #279926

That should fix this.
Comment 4 Jason Wallace 2009-09-09 15:35:51 UTC
(In reply to comment #3)
> +  17 Aug 2009; Patrick Lauer <patrick@gentoo.org> snort-2.8.4.1.ebuild:
> +  Small typo fix, closes #279926
> 
> That should fix this.
> 

This status needs to change. The fix proposed above was not correct.

In the snort.conf shipped in the tarball (which gets moved to snort.conf.distrib) this is the line that needs to change...

# dynamicdetection directory /usr/local/lib/snort_dynamicrule/

it needs to change to...

dynamicdetection directory /usr/<either lib or lib64>/snort_dynamicrule

so the correct line in the ebuild should be...

sed -i -e 's:snort_dynamicrule/$:snort_dynamicrule:g' \