FS#20968 - [snort] bug in PKGBUILD

Attached to Project: Community Packages
Opened by jason (bostonvaulter) - Monday, 27 September 2010, 22:25 GMT
Last edited by Lukas Fleischer (lfleischer) - Saturday, 06 November 2010, 19:02 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Lukas Fleischer (lfleischer)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 3
Private No

Details

Description:
Line 41 of the PKGBUILD install lines is incorrect. It only has one pipe when it should have two like the other install lines.

install -D -m644 ${srcdir}/snort.conf.d ${pkgdir}/etc/conf.d/snort | return 1
install -D -m644 ${srcdir}/snort.conf.d ${pkgdir}/etc/conf.d/snort || return 1

Additional info:
* 2.8.3.2

Steps to reproduce:
Download PKGBUILD using abs.
Somehow get the original source tarball (I was unable to find it online, so I used a newer version)
Run makepkg
Returns 2 when it gets to the line with the single |

Here's some more info if necessary (my forum post)
https://bbs.archlinux.org/viewtopic.php?id=105111

Here's an updated PKGBUILD:
http://gist.github.com/59999
This task depends upon

Closed by  Lukas Fleischer (lfleischer)
Saturday, 06 November 2010, 19:02 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in 2.9.0-1.
Comment by Brad Fanella (cesura) - Tuesday, 28 September 2010, 03:59 GMT
The "|| return 1" should just be removed altogether. It is no longer needed.
Comment by Stefan Husmann (stefanhusmann) - Wednesday, 13 October 2010, 15:56 GMT
2.8.3.2 is not downloadable anymore. 2.9.0 is out.
Comment by Lukas Fleischer (lfleischer) - Sunday, 31 October 2010, 19:23 GMT
I pushed 2.9.0 to [community-testing]. Can you guys please test? :)
Comment by Lukas Fleischer (lfleischer) - Saturday, 06 November 2010, 19:02 GMT
Moved to [community].

Loading...