rpmlint complained only
kcc.i386: W: no-url-tag
The URL tag is missing.

SHOULD:
  Preserve timestamp using install or cp commands.
  Though not mandatory can use disttag.

otherwise looks ok to me.

Thanks for reviewing. well, unfortunately this package is one of things that we
lost upstream. I don't think I can provide URL tag for this package. and I
intentionally didn't give a dist tag for this package because the application
itself is stable enough and the updates doesn't appears frequently. so I just
wanted to avoid appearing the dist tag doesn't match to the actual release.

Anyway, the below should fixes the timestamp issue:

Spec file: http://tagoh.fedorapeople.org/kcc/kcc.spec
SRPM file: http://tagoh.fedorapeople.org/kcc/kcc-2.3-27.src.rpm

change %defattr(-,root,root)
to 
%defattr(-,root,root,-)
else looks fine to me.

Thanks. improved.

Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and RPM.
+ source files match upstream.
e8f4244fd5e7ffef51bcdca1c5a54ebc  kcc.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
- dist tag is NOT present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Pacakge kcc-2.3-27 ->
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1)
libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) rtld(GNU_HASH)
+ Not a GUI app.

APPROVED.

Thanks. the updated kcc has been built on koji.