[Bug 480572] Review Request: xgridloc - An application for the calculation of Maidenhead QRA Locators

bugzilla at redhat.com bugzilla at redhat.com
Fri Jul 10 03:03:49 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=480572





--- Comment #3 from Randall Berry <randyn3lrx at gmail.com>  2009-07-09 23:03:48 EDT ---
(In reply to comment #2)

> The tarball in the src.rpm does not match the tarball I downloaded from the
> Source0: URL.  Do you know why this might be the case?  It looks like the
> upstream source differs in that it includes a COPYING file, although it's v3 of
> the copying file while the source still seems to be GPLv2+ (which is OK; the
> version of the copying file has no bearing on the version of the GPL the
> software is under).

I wrote to the author earlier and asked if he could include a GPL license looks
like he added it. I'll use the newer tarball and rebuild the src rpm. 


> Please note the following output from desktop-file-install:
> 
> /builddir/build/BUILDROOT/xgridloc-0.9-5.fc12.x86_64/usr/share/applications/xgridloc.desktop:
> warning: value "xgridloc.png" for key "Icon" in group "Desktop Entry" is an
> icon name with an extension, but there should be no extension as described in
> the Icon Theme Specification if the value is not an absolute path

Fixed: removed the extension from the icon in .desktop file. 

Spec URL: http://dp67.fedorapeople.org/pkgs/SPECS/xgridloc.spec
SRPM URL: http://dp67.fedorapeople.org/pkgs/SRPMS/xgridloc-0.9-6.fc11.src.rpm
Koji Build: https://koji.fedoraproject.org/koji/taskinfo?taskID=1464685

* Thu Jul 9 2009 Randall J. Berry 'Dp67' <dp67(at)fedoraproject.org> - 0.9-6
- Upstream source added COPYING file
- Fix .desktop file removed ext from icon
- Mock build f11/devel i386
- Test build on Koji all arches

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list