[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2012-04-14 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=604501

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|gtk-chtheme-0.3.1-6.fc13|gtk-chtheme-0.3.1-11.el6

--- Comment #21 from Fedora Update System  
2012-04-14 13:58:36 EDT ---
gtk-chtheme-0.3.1-11.el6 has been pushed to the Fedora EPEL 6 stable
repository.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2012-03-30 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=604501

--- Comment #20 from Fedora Update System  
2012-03-30 04:30:16 EDT ---
gtk-chtheme-0.3.1-11.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/gtk-chtheme-0.3.1-11.el6

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2012-02-29 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=604501

--- Comment #19 from Jon Ciesla  2012-02-29 08:22:51 EST 
---
Git done (by process-git-requests).

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2012-02-29 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=604501

Germán Racca  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #18 from Germán Racca  2012-02-29 08:05:45 EST ---
Package Change Request
==
Package Name: gtk-chtheme
New Branches: el6
Owners: splinux skytux

I'm authorizing the user splinux to co-maintain this epel branch for
gtk-chtheme.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-08-05 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=604501

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|gtk-chtheme-0.3.1-6.fc12|gtk-chtheme-0.3.1-6.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-08-05 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=604501

--- Comment #17 from Fedora Update System  
2010-08-05 19:53:24 EDT ---
gtk-chtheme-0.3.1-6.fc13 has been pushed to the Fedora 13 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-08-05 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=604501

--- Comment #16 from Fedora Update System  
2010-08-05 19:34:30 EDT ---
gtk-chtheme-0.3.1-6.fc12 has been pushed to the Fedora 12 stable repository. 
If problems still persist, please make note of it in this bug report.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-08-05 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=604501

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version||gtk-chtheme-0.3.1-6.fc12
 Resolution|NEXTRELEASE |ERRATA

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-16 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=604501

Germán Racca  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-16 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=604501

--- Comment #15 from Fedora Update System  
2010-07-16 16:46:47 EDT ---
gtk-chtheme-0.3.1-6.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/gtk-chtheme-0.3.1-6.fc13

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-16 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=604501

--- Comment #14 from Fedora Update System  
2010-07-16 16:40:58 EDT ---
gtk-chtheme-0.3.1-6.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/gtk-chtheme-0.3.1-6.fc12

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-16 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=604501

--- Comment #13 from Kevin Fenzi  2010-07-16 13:30:02 EDT ---
CVS done (by process-cvs-requests.py).

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-15 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=604501

Germán Racca  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #12 from Germán Racca  2010-07-15 19:18:45 EDT ---
New Package CVS Request
===
Package Name: gtk-chtheme
Short Description: Gtk+ 2.0 theme preview and selection made slick
Owners: skytux
Branches: F-12 F-13
InitialCC:

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-15 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=604501

--- Comment #11 from Germán Racca  2010-07-15 19:15:10 EDT ---
(In reply to comment #10)
> Thanks, all is good.
> 
>  The package gtk-chtheme is APPROVED.

Thank you very much for the review Terje!

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-15 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=604501

Terje Røsten  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #10 from Terje Røsten  2010-07-15 18:49:42 
EDT ---
Thanks, all is good.

 The package gtk-chtheme is APPROVED.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-15 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=604501

--- Comment #9 from Germán Racca  2010-07-15 14:35:12 EDT ---
(In reply to comment #7)
> Seems like a change in GTK is the problem, I added a patch and it built fine:
> 
>  http://koji.fedoraproject.org/koji/taskinfo?taskID=2318320
> 
> spec: http://terjeros.fedorapeople.org/gtk-chtheme/gtk-chtheme.spec
> srpm:
> http://terjeros.fedorapeople.org/gtk-chtheme/gtk-chtheme-0.3.1-5.fc12.src.rpm
> 
> Note: only built, not tested.

Hello Terje:

Many thanks for your help with the patch! I also have corrected the permission
of the man page in the right way. I've tested it under FC13 and works fine.

Koji builds from scratch:

dist-f12: http://koji.fedoraproject.org/koji/taskinfo?taskID=2322311
dist-f13: http://koji.fedoraproject.org/koji/taskinfo?taskID=2322367
dist-rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=2322382

$ rpmlint rpmbuild/{SPECS,SRPMS,RPMS/i686}/gtk-chtheme*
gtk-chtheme.i686: W: unstripped-binary-or-object /usr/bin/gtk-chtheme
2 packages and 1 specfiles checked; 0 errors, 1 warnings.

Please find the updated files here:

Spec: http://skytux.fedorapeople.org/packages/gtk-chtheme.spec
SRPM: http://skytux.fedorapeople.org/packages/gtk-chtheme-0.3.1-6.fc13.src.rpm

Regards,
German.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-14 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=604501

--- Comment #8 from Terje Røsten  2010-07-14 05:00:03 
EDT ---
Forgot: patch might work on all active Fedora releases, no special cace for
F-14 needed.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-14 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=604501

--- Comment #7 from Terje Røsten  2010-07-14 04:54:48 
EDT ---
Seems like a change in GTK is the problem, I added a patch and it built fine:

 http://koji.fedoraproject.org/koji/taskinfo?taskID=2318320

spec: http://terjeros.fedorapeople.org/gtk-chtheme/gtk-chtheme.spec
srpm:
http://terjeros.fedorapeople.org/gtk-chtheme/gtk-chtheme-0.3.1-5.fc12.src.rpm

Note: only built, not tested.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-13 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=604501

--- Comment #6 from Germán Racca  2010-07-13 22:06:27 EDT ---
Hello Terje:

Thanks for the full review! But I can't find what is the problem in the build
for F-14. For F-12 and F-13 it builds fine. With respect to the man page, no
problem, I'll correct it, but please can you give me some hints about the build
error?

Regards,
German.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-07-11 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=604501

Terje Røsten  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|terje...@phys.ntnu.no
   Flag||fedora-review?

--- Comment #5 from Terje Røsten  2010-07-11 05:48:57 
EDT ---

ok rpmlint
ok named and versioned according to the Package Naming Guidelines
ok spec file name matches base package name
ok license : GPLv2+ tag correct and all files with license info
ok COPYING packaged as %doc
ok source matches upstream:
 dbea31f4092877e786fe040fae1374238fada94a  gtk-chtheme-0.3.1.tar.bz2
 dbea31f4092877e786fe040fae1374238fada94a  gtk-chtheme-0.3.1.tar.bz2.1
!  builds in koji: something is wrong here, might be missing #include, 
   please have a look. Ref:
 http://koji.fedoraproject.org/koji/taskinfo?taskID=2310790
 http://koji.fedoraproject.org/koji/getfile?taskID=2310792&name=build.log
ok no missing BuildRequires (builds in mock)
ok no translations, so translation/locale guidelines don't apply
ok no shared libraries, so no ldconfig calls neeed
ok no duplicated system libraries
ok package not relocatable
ok directory ownership correct (doesn't own directories owned by another
package, owns all package-specific directories)
ok no duplicate files in %files
!  permissions correct, defattr used correctly
 please drop %attr on man page, change to correct mode in %install. 
 %attr is for special cases.
ok macros
ok no non-code content
ok no large documentation files, so no -doc package needed
ok no %doc files required at runtime
ok no header files which would need to be in a -devel subpackage
ok no static libraries, so no -static package needed
ok no devel symlinks which would need to be in a -devel subpackage
ok no -devel package,
ok no .la files
ok .desktop file
ok desktop-file-validate is used in %install and the .desktop file passes
validation
ok all filenames are valid UTF-8
ok complies with the FHS
  ok proper changelog, tags, BuildRoot, BuildRequires, Summary, Description
  ok no macros in Summary and Description
  ok no non-UTF-8 characters
  ok all relevant documentation included as %doc
  ok RPM_OPT_FLAGS are used (%cmake macro)
  ok debuginfo package is valid
  ok no rpaths
  ok no configuration files, so %config guideline doesn't apply
  ok no init scripts, so init script guideline doesn't apply
  ok no timestamp-clobbering file commands
  ok _smp_mflags used
  ok not a web application, so web application guideline doesn't apply
  ok no conflicts


Please have a look on items marked with ! .

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-06-28 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=604501

--- Comment #4 from Germán Racca  2010-06-28 18:15:09 EDT ---
Files moved to fedorapeople.org:

Spec URL: http://skytux.fedorapeople.org/packages/gtk-chtheme.spec

SRPM URL:
http://skytux.fedorapeople.org/packages/gtk-chtheme-0.3.1-4.fc13.src.rpm

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-06-17 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=604501

Fabian Affolter  changed:

   What|Removed |Added

 CC||iamde...@gmail.com

--- Comment #3 from Fabian Affolter  2010-06-17 
14:43:02 EDT ---
*** Bug 541724 has been marked as a duplicate of this bug. ***

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-06-17 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=604501

--- Comment #2 from Germán Racca  2010-06-17 03:18:53 EDT ---
OK, both issues fixed. Please find updated files here:

Spec URL: http://sites.google.com/site/gracca/gtk-chtheme.spec

SRPM URL: http://sites.google.com/site/gracca/gtk-chtheme-0.3.1-4.fc13.src.rpm

Cheers,
German.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 604501] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

2010-06-16 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=604501

Terje Røsten  changed:

   What|Removed |Added

 CC||terje...@phys.ntnu.no

--- Comment #1 from Terje Røsten  2010-06-16 06:31:50 
EDT ---
Please fix exec bits on man page, no reason not to fix that.

desktop file has  Icon=style , however no style image is shipped.

-- 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review