Bug 606127

Only a brief look at the spec:


> Release: 20102006svn%{?dist}

https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages


> Requires: qt    

https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires


> %description

The guidelines are not very specific, but you are supposed to construct _full sentences_ in the description. Would not be a blocker criterion, though.


> /usr/bin/colortool
> /usr/share/pixmaps/colortool.xpm
> /usr/share/applications/ColorTool.desktop
> /usr/share/applnk/Graphics/ColorTool.kdelnk
> /usr/share/colortool/translations/*

https://fedoraproject.org/wiki/Packaging:Guidelines#Macros
 and
https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage


> /usr/share/colortool/translations/*

https://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging:UnownedDirectories

I am sorry for the incorrect build.
Hopefully, this one is better...

SPEC: http://colortool.googlecode.com/files/colortool-0-0.1.20102006svn0042.fc13.spec

SRPM: http://colortool.googlecode.com/files/colortool-0-0.1.20102006svn0042.fc13.src.rpm

I just updated the package; so the current file urls are:

SPEC:
http://colortool.googlecode.com/files/colortool-1.0-1.fc13.spec

SRPM:
http://colortool.googlecode.com/files/colortool-1.0-1.fc13.src.rpm

Thanks!

Hi Tobias,

here are some more quick comments on your spec file:

- The summary is too generic. It should tell what the program does, not who the intended users are. :)

- Drop at least the revision number (-7) from BR: qt-devel >= 4.5.3-7
  (probably, "4.5" instead of "4.5.3" is sufficient too)

- Add BR: desktop-file-utils.

- You can drop Requires: qt (it's automatically picked up as a dependency).

- The creation of the .desktop looks a bit complicated. I recommend to replace it with a heredoc, e.g. like this:

cat <<EOT >%{name}.desktop
[Desktop Entry]
Name=ColorTool
Comment=Colorspace converter and color chooser
Exec=colortool
Icon=colortool
Terminal=false
Type=Application
Categories=GNOME;GTK;Graphics;
EOT

Also, move these lines to the %build section. 

- In the %install section install the .desktop file with
desktop-file-install --dir=%{buildroot}%{_datadir}/applications %{name}.desktop
(the additional parameters given in your spec are not necessary)

- drop export BINDIR/DATADIR from %install (not required)

- the "install" line for the manpage can be shortened as follows:
  install -m 644 -p colortool.1 %{buildroot}%{_mandir}/man1/

- Don't mix $RPM_BUILD_ROOT and %{buildroot}. Use only one of both.

- In %files, replace 
  %{_mandir}/man1/colortool.1.gz
  with
  %{_mandir}/man1/colortool.1* or %{_mandir}/man1/%{name}.1*
  because you should not rely on a concrete compression format in the spec file

- Add a newline between the %changelog revisions to increase legibility.

That's all for now. :)

It's been several months since that commentary with no response from the submitter.  I will close this soon if there is no further progress.

I am sorry for my inactivity for this bug. I learned a lot and adjusted my SPEC-file according to this advices.
Because I am currently working on a new version of my tool, I didn't post a newer SPEC-file so far, as dependencies are changing while I'm working on the new version.
I will be back soon, with the new version of this tool and an updated SPEC-file for both, this and the upcoming version.

After another 1.5 years with no progress, I'll just close this out.  You can reopen in the future if you do come up with that new package.