Bug 475032

I: Testing

Ignore this comment.

MUST Items: 

OK - rpmlint is clean

xx - does not follow Naming Guidelines
    + Looking at the upstream release announcement, it looks like 1.0.20080808
      should be the value of the version tag, although they say that the 1.0
      release is "a stable snapshot of the beta Gnaural2". Ubuntu has also
      chosen to include the date in the version tag and not in the release tag.

OK - spec file is named as %{name}.spec

xx - package meets Packaging Guidelines
    + The description is too verbose. Apart from the first sentence, the
      content mainly describes the surrounding research and not Gnaural itself.
    + To preserve timestamps you could consider using:
      make install INSTALL="%{__install} -p" DESTDIR=$RPM_BUILD_ROOT
    + According to
      https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps
      'install -p' should be used to preserve timestamps.
    + Instead of putting gnaural-icon.png in /usr/share/pixmaps you could
      consider putting it in /usr/share/icons/hicolor/48x48/apps and add
      'Requires: hicolor-icon-theme'. If you do so, then you should use the
      necessary post and postun scriptlets from
      https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#GTK.2B_icon_cache
      gnaural-icon.xpm is a better candidate for /usr/share/pixmaps.

OK - Fedora approved license and meets Licensing Guidelines
OK - License field meets actual license
OK - upstream license file included in %doc
OK - spec file uses American English
OK - spec file is legible
OK - sources match upstream sources
OK - package builds successfully
OK - ExcludeArch not needed

xx - redundant and extra build dependencies listed
    + pkgconfig is brought in by all the -devel packages providing *.pc files

OK - no locales
OK - no shared libraries
OK - package is not relocatable
OK - file and directory ownership
OK - no duplicates in %file
OK - file permissions set properly
OK - %clean present
OK - macros used consistently
OK - contains code and permissable content
OK - -doc is not needed
OK - contents of %doc does not affect the runtime
OK - no header files
OK - no static libraries
OK - no pkgconfig files
OK - no library files
OK - -devel is not needed
OK - no libtool archives

xx - %{name}.desktop file is not properly installed
    + If the package installs a .desktop file, then desktop-file-install must
      be run. In this case mv can not be used. See
      https://fedoraproject.org/wiki/Packaging/Guidelines#desktop-file-install_usage
    + According to
      https://fedoraproject.org/wiki/Packaging/Guidelines#Icon_tag_in_Desktop_Files it is better to use only 'gnaural-icon' as the value for the Icon tag in
      order to support theming.

OK - does not own files or directories owned by other packages
OK - buildroot correctly prepped
OK - all file names valid UTF-8

SHOULD Items:

OK - upstream provides license text
xx - no translations for description and summary
OK - package builds in mock successfully
OK - package builds on all supported architectures
OK - package functions as expected
OK - scriptlets are sane
OK - subpackages are not needed
OK - no pkgconfig files
OK - no file dependencies

Fixed all issues.

SPEC: http://rakesh.fedorapeople.org/spec/gnaural.spec
SRPM: http://rakesh.fedorapeople.org/srpm/gnaural-1.0.20080808-2.fc10.src.rpm

> MUST Items: 

> xx - package meets Packaging Guidelines
>
> [...]
>
>     + According to
>       https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps
>       'install -p' should be used to preserve timestamps.

You missed this one. :-)

Recently the Fedora Packaging Committee decided to not use --vendor=fedora anymore, but that decision was waiting to be approved by FESCo. See:
+ http://fedoraproject.org/wiki/Packaging/Minutes/20081021
+ https://fedoraproject.org/wiki/TomCallaway/DesktopFileVendor
I have just dropped a mail on fedora-devel-list to find out what the current is.

Removed vendor

SPEC: http://rakesh.fedorapeople.org/spec/gnaural.spec
SRPM: http://rakesh.fedorapeople.org/srpm/gnaural-1.0.20080808-3.fc10.src.rpm

Fixed missing install -p

SPEC: http://rakesh.fedorapeople.org/spec/gnaural.spec
SRPM: http://rakesh.fedorapeople.org/srpm/gnaural-1.0.20080808-4.fc10.src.rpm

Sorry for not pointing this out earlier, but "Application" should not be a part of the "Categories" key in the .desktop file:
[rishi@ginger SPECS]$ desktop-file-validate /usr/share/applications/gnaural.desktop 
/usr/share/applications/gnaural.desktop: warning: value "Application;AudioVideo;AudioVideoEditing;" for key "Categories" in group "Desktop Entry" contains a deprecated value "Application"
[rishi@ginger SPECS]$ 

Also, please try to discuss and submit your modifications to the .desktop file upstream.

Otherwise it looks fine.

+---------------------------------+
| This package is APPROVED by me. |
+---------------------------------+

Will fix it before importing.


New Package CVS Request
=======================
Package Name: gnaural
Short Description: A multi-platform programmable binaural-beat generator.
Owners: rakesh
Branches: F-8 F-9 F-10
InitialCC:
Cvsextras Commits: yes

F-8 branches are no longer accepted. 

cvs done for the rest.

gnaural-1.0.20080808-5.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/gnaural-1.0.20080808-5.fc9

gnaural-1.0.20080808-5.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/gnaural-1.0.20080808-5.fc10

Fixed and imported.