Spec Name or Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.specSRPM Name
or Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-2.src.rpm

Fixed dependency problems with files included as %doc, also misc. spec file
improvements.

Anyone care to have a look ? :)

Spec Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.spec
SRPM Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-3.src.rpm

aparently, I tickled some sort of bug in rpmbuild's find-requires. Commented out
the %doc contrib from argus-clients, so package could build on vanilla FC4 box

A review:

MUST items:

OK - Package name.
OK - Spec file name matches.
OK - Package guidelines.
OK - Licsense.
OK - License field matches in spec.
OK - License included in files
OK - Spec in american english
OK - Spec legible
OK- Md5sum of source from upstream
b90b8ea951fd0e2193b9a6bab565a139  argus-2.0.6.fixes.1.tar.gz
b90b8ea951fd0e2193b9a6bab565a139  ../argus-2.0.6.fixes.1.tar.gz
59f65159070f849211685513384fa90f  argus-clients-2.0.6.fixes.1.tar.gz
59f65159070f849211685513384fa90f  ../argus-clients-2.0.6.fixes.1.tar.gz

OK - No Forbidden buildrequires included
OK - All required buildrequires included?
OK - Owns all directories it creates.
OK - No duplicate files in %files listing.
OK - Permissions on files correct.
OK - Clean section correct.
OK - Macros consistant.
OK - Code not content.
OK - Docs must not affect runtime.
OK - Doesn't own any files/dirs that are already owned by others.

Blockers:

1. Some rpmlint output:

This might be due to the name having 'lib*' in it:

E: argus explicit-lib-dependency libpcap

Should be installed with non executable permissions:
(currently 755, should be 644?):

E: argus script-without-shellbang
/usr/share/doc/argus-2.0.6.fixes.1/support/Config/argus.conf
E: argus script-without-shellbang
/usr/share/doc/argus-2.0.6.fixes.1/support/System/crontab
E: argus script-without-shellbang
/usr/share/doc/argus-2.0.6.fixes.1/support/Deployment/sample
E: argus script-without-shellbang
/usr/share/doc/argus-2.0.6.fixes.1/support/System/magic

Should we use /var/argus? or should that be /var/lib/argus?

W: argus non-standard-dir-in-var argus

Can be ignored ('install' in comment)

W: argus dangerous-command-in-%post install

By default the service should be disabled on install...

W: argus service-default-enabled /etc/rc.d/init.d/argus

2. There are some build issues. If you have the
# prevent perl modules in contrib from being checked for dependencies
%define __perl_requires /bin/true
%define __perl_provides /bin/true
lines in the spec, rpm fails to build with:

getOutputFrom(): Broken pipe

This is to work around a .pm file in contrib triggering lots of perl dependencies.
Possible options:

- Don't package contrib script.
- Re-name to something besides .pm file?
- Some more clever alternative that some perl/rpm guru comes up with?

Spec Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.spec
SRPM Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-4.src.rpm

Currently, the contrib stuff is not being packaged, and no __perl_requires
workaround is necessary.

Fixed all other issues listed in the previous comment.

Spec Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.spec
SRPM Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-5.src.rpm

Added patch to fix several print-formatting bugs from argus mailing list

Thanks for the updates. 

1. rpmlint shows no output now. :) 

2. You might add a README.fedora explaining that the contrib packages are not
included and where to get them, since they are no longer included. 
You could also mention what patches you have applied and where to get them (the
mailing list looks like it. ;) 

In the default /etc/argus.conf, both the output file and the host/port are
disabled, meaning when you start argus it starts fine, but you can't communicate
with it. :( A default of localhost binding might be less confusing, and allow it
to work out of the box. 

With those changes, this package can be APPROVED. 

I will go ahead and sponsor you. 
You can continue the process from the "Get a Fedora Account" section of 
http://fedoraproject.org/wiki/Extras/Contributors

If you have any questions at all, please feel free to drop me an email. 

Spec Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.spec
SRPM Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-6.src.rpm

Thanks again for the comments and advice !

- default argus.conf now enables listening on localhost:561

- clients subpackage now contains %doc README.fedora, explaining the
  contrib situation and origin of the printout patch

Spec Url: http://www.contrib.andrew.cmu.edu/~somlo/argus.spec
SRPM Url: http://www.contrib.andrew.cmu.edu/~somlo/argus-2.0.6.fixes.1-7.src.rpm

minor fix to argus-2.0.6.fixes.1-makefile.patch

With those changes everything looks good... should be approved and good to go. 

Changed summary for tracking purposes.

Package Change Request
======================
Package Name: argus
New Branches: epel7
Owners: fab
InitialCC:

Git done (by process-git-requests).