[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2008-10-02 Thread bugzilla
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=209617





--- Comment #6 from Huzaifa S. Sidhpurwala [EMAIL PROTECTED]  2008-10-03 
01:01:46 EDT ---
cvs done

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

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


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2008-10-02 Thread bugzilla
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=209617


Huzaifa S. Sidhpurwala [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2008-10-01 Thread bugzilla
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=209617


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Mamoru Tasaka [EMAIL PROTECTED]  2008-10-02 01:53:01 EDT 
---
Package Change Request
==
Package Name: comix
New Branches: F-10
Owners: mtasaka

Early branching request.

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

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


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2006-10-07 Thread bugzilla
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: comix - A user-friendly, customizable image viewer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-10-07 12:10 EST ---
(In reply to comment #2)
  You should use dl.sourceforge.net instead of
 prdownloads.sourceforge.net.  

I changed URL as you recommended.


 This package owns /etc/gconf, which is owned by GConf in FC5 but which is
 completely unowned in FC6.  Ouch. 

Well, I checked FC5, however, /etc/gconf is not owned by GConf2 (I have
not checked GConf because this package does not require GConf) and..
/etc/gconf is not owned by any packages required by this package in FC5,
either. So I have to make this package own /etc/gconf in FC5, too

(I strongly think that some other package should own /etc/gconf and
/etc/gconf/schemas)

 This package also owns /etc/gconf/schemas,
 which unfortunately doesn't seem to be owned by GConf as it should.  
This is the same for FC5.



-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2006-10-06 Thread bugzilla
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: comix - A user-friendly, customizable image viewer


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





--- Additional Comments From [EMAIL PROTECTED]  2006-10-06 09:41 EST ---
Note: rpmlint says:

W: comix non-conffile-in-etc /etc/gconf/schemas/comix.schemas

However, this is not a configuration file and files under
/etc/gconf/schemas are usually not marked as %config
( as you can see these files in epiphany. yelp, planner, etc)

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2006-10-06 Thread bugzilla
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: comix - A user-friendly, customizable image viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2006-10-06 Thread bugzilla
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: comix - A user-friendly, customizable image viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-10-06 20:23 EST ---
Hmm, the source doesn't match upstream.  Ahh, because the source URL gives me
some HTML for selecting a mirror.  You should use dl.sourceforge.net instead of
prdownloads.sourceforge.net.  I also had to manually add a mirror (easynews) in
order to actually download the source, since the sourceforge mirror system is so
broken.

The non-conffile-in-etc bit is known to be bogus for gconf schemas.

This package owns /etc/gconf, which is owned by GConf in FC5 but which is
completely unowned in FC6.  Ouch.  This package also owns /etc/gconf/schemas,
which unfortunately doesn't seem to be owned by GConf as it should.  I really
don't know what this package should do; I guess owning those directories is
better than leaving them unowned, but this should be fixed in Core and if it is
this package will need to stop owning those directories.

So I'll approve this package for FC6/rawhide, but for FC5 you will need to stop
owning /etc/gconf.

* source files match upstream:
   2d6034c5c032fcb5fe257048d64364a8  comix-3.6.tar.gz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* 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 (development, x86_64).
* package installs properly
* rpmlint has only bogus complaints.
* final provides and requires are sane:
   comix = 3.6-1.fc6
  =
   /bin/sh
   /usr/bin/env
   /usr/bin/jpegtran
   GConf2
   pygtk2
   python-imaging
* %check is not present; no test suite upstream.  Package manually tested and
seems to work fine.
* owns the directories it creates.
? owns /etc/gconf; in FC5, it must not.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (gconf schema installation, icon cache update, desktop
database update, mime database update)
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* .desktop file installed properly

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 209617] Review Request: comix - A user-friendly, customizable image viewer

2006-10-06 Thread bugzilla
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: comix - A user-friendly, customizable image viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-10-07 00:07 EST ---
Thank you for a quick review and approving this package!!

* Rebuild for FE-devel succeeded.
* SyncNeeded is requested for FE-5.

Now I close this bug as CLOSED NEXTRELEASE.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review