I need one more trivial fixup, to start the service at level 59 so it starts 
after ntpd.  I'll include that in the final posting.

Not a Review but found one thing in SPEC
SOURCE is not URL

A few comments:

As in comment #2, Source0 is not a full URL.  I suggest using:
Source0:   http://www.sixxs.net/archive/sixxs/aiccu/unix/aiccu_%{version}.tar.gz

Builds fine in mock; rpmlint says:

E: aiccu non-readable /etc/aiccu.conf 0600
Understandable since this includes password information.

W: aiccu service-default-enabled /etc/init.d/aiccu
The init script has:
# chkconfig: 345 15 85
So the service will start enabled in runlevels 3, 4, and 5.  You should s/345/-/.

The makefile strips the binary, which breaks -debuginfo generation.

Other than those, things look pretty good although I suspect that once you fix
-debuginfo generation, the fact that many of the source files are executable for
some reason will cause several other warnings.

Fixed the above, no further warnings are emitted.

Spec URL: http://domsch.com/linux/fedora/extras/aiccu/aiccu.spec
SRPM URL: http://domsch.com/linux/fedora/extras/aiccu/aiccu-2005.01.31-3.src.rpm

Now builds in mock and rpmlint has only the ignorable non-readable error.

The compiler isn't being called with the proper arguments; I see you pass in
RPM_OPT_FLAGS, but that doesn't seem to be used by the Makefiles.  I think
you'll probably have to patch.

* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* source files match upstream:
   7c3da5feab3d59fb5a99a45203e0ca56  aiccu_2005.01.31.tar.gz
* latest version is being packaged.
* BuildRequires are proper.
* package builds in mock (development, x86_64).
* rpmlint has only ignorable complaints.
* final provides and requires are sane:
   config(aiccu) = 2005.01.31-3.fc6
   aiccu = 2005.01.31-3.fc6
  =
   /bin/sh
   chkconfig
   config(aiccu) = 2005.01.31-3.fc6
   iproute
* no shared libraries are present.
* package is not relocatable.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* %clean is present.
* %check is not present; no test suite upstream.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no libtool .la droppings.
* not a GUI app.

Fixed RPM_OPT_FLAGS usage, and removed executable bits on files going into 
debuginfo so that's sane now.

Spec URL: http://domsch.com/linux/fedora/extras/aiccu/aiccu.spec
SRPM URL: http://domsch.com/linux/fedora/extras/aiccu/aiccu-2005.01.31-
4.src.rpm

Everything looks fine now; the debuginfo package has the source and build.log
shows the proper flags being passed to the compiler.

APPROVED