[Bug 230316] Review Request: jbrout - Photo manager, written in python/pygtk under the GPL licence

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

Summary: Review Request: jbrout - Photo manager, written in python/pygtk under the GPL licence


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


tibbs@xxxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |tibbs@xxxxxxxxxxx
             Status|NEW                         |ASSIGNED
               Flag|                            |fedora-review?




------- Additional Comments From tibbs@xxxxxxxxxxx  2008-06-25 15:40 EST -------
Seems there's no longer any reason for the svn checkout instructions since you
can directly fetch a tarball.

There's no real point in mentioning the License in the Summary:, is there?  Or
what language the package is written in?  These things really don't matter to
someone who is interested in what the package does.

Similarly for the %description; does any Fedora user particularly care whether
the software works on Windows 2000?  And who is "me" in the description?  Yum or
some package manager will display this to most users, and surely yum isn't going
to be conversing about itself.

rpmlint does indeed complain about the .mo files not being mentioned in %lang. 
You could list each of them separately in the %files list with %lang(foo) but at
most that would allow someone to save a little space by excluding some specific
lang files.  Nice to have, but not absolutely necessary in my opinion, although
it shouldn't be too terribly difficult if you wanted to do that.

You should probably report this issue with the desktop file upstream:
  key "Categories" is a list and does not have a semicolon as trailing 
  character, fixing
Also, there's no need to use "--vendor=fedora" when installing your desktop file.

Not sure if you noticed it, but there's no point to the %find_lang call, since
this package insatlls nothing into /usr/share/locale.  The %{name}.lang file is
empty.  You might as well just remove the %find_lang call and the -f bit from
%files.

* source files match upstream:
   b62b1bbd72400fd352deb8523a61e2b2da4f81c47f2118dd51488bee626fe77c  
   jbrout-0.2.201.sources.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
X summary could use some work.
X description could use some work.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* rpmlint has acceptable complaints.
* final provides and requires are sane:
   jbrout = 0.2.201-1.fc10
  =
   /bin/sh
   /usr/bin/env
   fbida
   jhead
   pygtk2 >= 2.6
   python >= 2.4
   python-imaging
   python-lxml

* %check is not present; no test suite upstream.  I installed and ran this 
   package; it seems to work OK.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
X desktop installed with --vendor=fedora.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]