So, a few things going on here in the spec that should be looked at:

- Source url should not use macros
- PreReq seems a bit odd, why the Pre and the regular Req?
- Use of defines at the top seems a bit much.  Why define it to use it in only
one place? (one place if you don't hard specify Requires)
- Hard specified Requires, rpm should be able to figure this out on its own
- Graphical app but no .desktop file
- Files section is referencing directories rather than files within directories.

Other than that its a good start (;

Whoops, macro in Source url is fine.

I fixed the obvious stuff, thank you for the review.

Created attachment 127729 [details]
SPEC file

The desktop file needs to have some bits changed specific for Fedora.  You need
to add this to the %install section:

desktop-file-install --vendor fedora --delete-original	\
  --dir $RPM_BUILD_ROOT%{_datadir}/applications   	\
  --add-category X-Fedora			        \
  $RPM_BUILD_ROOT%{_datadir}/applications/%{name}.desktop

And in the %files section the desktop file needs to be:
%{_datadir}/applications/fedora-%{name}.desktop

David, is the link to your spec file valid?

Whoops, forgot to mention in comment #5, that you can drop the BuildRequires for
glib2-devel and gtk2-devel,since the libglade2-devel package will pull these in.

Thank you for the review Brian.. yes the link is valid.

I updated the spec file to include the changes you suggested and uploaded new
versions.

Now you need desktop-file-utils as a BuildRequires since you'll be using it in
the %install section.  I added that in a local copy of your spec and used that
to build packages.  Rpmlint output coming up.

rpmlint is silent, no problems.  So fix the BuildRequires and I think we're in
business.  I just have to lookup how to sponsor you.

(In reply to comment #10)
> rpmlint is silent, no problems.  So fix the BuildRequires and I think we're in
> business.  I just have to lookup how to sponsor you.

You don't have sponsor permissions.  The list of sponsors is here:

https://admin.fedora.redhat.com/accounts/dump-group.cgi?group=cvsextras&role_type=sponsor&format=html

I'm willing to sponsor this though.

Updated.. hopefully this is the final go at this untill Chris releases 0.2 :)

I have the fu to sponsor now, so David you'll need to go through the steps at
http://fedoraproject.org/wiki/Extras/Contributors to create your account so I
can sponsor it.

Also, the srpm you list isn't reachable, I did tests by using the upstream
tarball w/ your spec.  For prosperity you might want to resolve that.

My webhost seems to have been down for a brief period. sorry about that.

I'm having an issue with the user system, it has gotten in a state where on one
hand it doesn't know a user registered to david, yet claims that
one is already registered to that addy when I try to register again. 

Could someone please look at that?

David isn't as interested in maintaining this anymore, so I've gone ahead and
pushed it through.  It should wind up in extras-development soon.

(In reply to comment #15)
> David isn't as interested in maintaining this anymore, so I've gone ahead and
> pushed it through.  It should wind up in extras-development soon.

I see from owners.list that you've taken over ownership of this package yourself
(which is good). Reading the above, it could be interpreted as pushing through
an already-orphaned package (which wouldn't have been good).