Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.x
https://bugzilla.redhat.com/show_bug.cgi?id=608973
Summary: Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.x Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: gracca@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine.spec
SRPM URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine-1.20-4.fc13.src.r...
Description: A new engine derived from Aurora 1.4. It features smooth gradients or glassy effects, subtle shadows, rounded widgets. ________________________________________________________________________________
$ rpmlint SRPMS/gtk-equinox-engine-1.20-4.fc13.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint RPMS/i686/gtk-equinox-engine-1.20-4.fc13.i686.rpm gtk-equinox-engine.i686: W: unstripped-binary-or-object /usr/lib/gtk-2.0/2.10.0/engines/libequinox.so 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ koji build --scratch dist-f13 SRPMS/gtk-equinox-engine-1.20-4.fc13.src.rpm
http://koji.fedoraproject.org/koji/taskinfo?taskID=2279586 ________________________________________________________________________________
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=608973
Julian Aloofi julian.fedora@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.fedora@googlemail.co | |m AssignedTo|nobody@fedoraproject.org |julian.fedora@googlemail.co | |m Flag| |fedora-review?
--- Comment #1 from Julian Aloofi julian.fedora@googlemail.com 2010-06-29 10:09:43 EDT --- Builds fine in mock, spec file looks good, engine seems to work. rpmlint is complaining a bit about permissions in the debuginfo package.
I'll do a full review soon.
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=608973
--- Comment #2 from Germán Racca gracca@gmail.com 2010-06-29 12:43:36 EDT --- Thank you Julian!
Regards, Germán.
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=608973
--- Comment #3 from Julian Aloofi julian.fedora@googlemail.com 2010-06-30 16:06:15 EDT --- Now when installing it under GNOME I notice that the choosable themes in the Appearance menu complain about not finding the "Faenza" and "Faenza-Dark" icon themes. (Equinox, -Dark and -Light)
The engine itself works, but this is a bit annoying.
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=608973
--- Comment #4 from Germán Racca gracca@gmail.com 2010-06-30 16:44:47 EDT --- Well...what do you suggest? Contact upstream to change it?
Regards, German.
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=608973
--- Comment #5 from Julian Aloofi julian.fedora@googlemail.com 2010-07-01 05:34:28 EDT --- I don't know how active upstream is. Looking around on gnome-look.org etc. I didn't even find a Faenza icon theme.
I'd just use sed and replace IconTheme=Faenza-Whatever with IconTheme=gnome in the index.theme files for the themes.
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=608973
--- Comment #6 from Julian Aloofi julian.fedora@googlemail.com 2010-07-01 07:11:10 EDT --- sed -i 's/Faenza-Dark/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox/index.theme sed -i 's/Faenza Dark/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox\ Glass/index.theme sed -i 's/Faenza/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox\ Light/index.theme
in %install should do the trick. replace_theme could be any icon theme, gnome or Tango or whatever, just be sure to add the Requires for it.
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=608973
--- Comment #7 from Germán Racca gracca@gmail.com 2010-07-01 17:01:49 EDT --- (In reply to comment #5)
I don't know how active upstream is. Looking around on gnome-look.org etc. I didn't even find a Faenza icon theme.
I'd just use sed and replace IconTheme=Faenza-Whatever with IconTheme=gnome in the index.theme files for the themes.
Faenza is an icon theme under development by the same creator of the engine. In gnome-look.org he said that Faenza is not ready yet.
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=608973
--- Comment #8 from Germán Racca gracca@gmail.com 2010-07-01 17:03:41 EDT --- (In reply to comment #6)
sed -i 's/Faenza-Dark/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox/index.theme sed -i 's/Faenza Dark/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox\ Glass/index.theme sed -i 's/Faenza/%{replace_theme}/g' %{buildroot}%{_datadir}/themes/Equinox\ Light/index.theme
in %install should do the trick. replace_theme could be any icon theme, gnome or Tango or whatever, just be sure to add the Requires for it.
Hello Julian:
I'm going to change the icon theme in the spec file as you suggest.
Thanks, German.
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=608973
--- Comment #9 from Julian Aloofi julian.fedora@googlemail.com 2010-07-04 09:33:59 EDT --- Okay, by the way the BuildRoot tag is not needed anymore so you can just remove it.
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=608973
--- Comment #10 from Germán Racca gracca@gmail.com 2010-07-05 23:57:11 EDT --- Hello Julian:
I've informed upstream about the changes we'll make in the Fedora package with respect to the icon theme. We'll use Gnome icon theme while Faenza is not ready. Because of this, I've added the corresponding Requires in the spec file.
I've also corrected the release number because I've made a mistake before, and I've removed the BuildRoot tag as you suggested.
Please find the updated files here:
Spec URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine.spec
SRPM URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine-1.20-2.fc13.i686.... ________________________________________________________________________________
$ rpmlint SRPMS/gtk-equinox-engine-1.20-2.fc13.src.rpm gtk-equinox-engine.src: W: no-buildroot-tag 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint RPMS/i686/gtk-equinox-engine-1.20-2.fc13.i686.rpm gtk-equinox-engine.i686: W: unstripped-binary-or-object /usr/lib/gtk-2.0/2.10.0/engines/libequinox.so
$ koji build --scratch dist-f13 rpmbuild/SRPMS/gtk-equinox-engine-1.20-2.fc13.src.rpm Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=2297296
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=608973
--- Comment #11 from Germán Racca gracca@gmail.com 2010-07-06 00:03:21 EDT --- (In reply to comment #10)
SRPM URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine-1.20-2.fc13.i686....
Sorry, that url is for the rpm package (that I removed). The source rpm is here:
SRPM URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine-1.20-2.fc13.src.r...
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=608973
Julian Aloofi julian.fedora@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #12 from Julian Aloofi julian.fedora@googlemail.com 2010-07-06 10:49:22 EDT --- MUST: The package does not yet exist in Fedora. The Review Request is not a duplicate. OK MUST: The spec file for the package is legible and macros are used consistently. OK MUST: The package must be named according to the Package Naming Guidelines. OK MUST: The spec file name must match the base package %{name}. OK MUST: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines. OK MUST: The License field in the package spec file must match the actual license. OK MUST: The sources used to build the package must match the upstream source, as provided in the spec URL. OK MUST: The package MUST successfully compile and build into binary rpms. OK MUST: The spec file MUST handle locales properly. N/A MUST: Optflags are used and time stamps preserved. OK MUST: Packages containing shared library files must call ldconfig. N/A MUST: A package must own all directories that it creates or require the package that owns the directory. OK MUST: Files only listed once in %files listings. OK MUST: Debuginfo package is complete. OK MUST: Permissions on files must be set properly. OK MUST: Clean section exists. OK MUST: Large documentation files must go in a -doc subpackage. N/A MUST: All relevant items are included in %doc. Items in %doc do not affect runtime of application. OK MUST: Header files must be in a -devel package. N/A MUST: Static libraries must be in a -static package. N/A MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A MUST: If a package contains library files with a suffix then library files ending in .so must go in a -devel package. N/A MUST: In the vast majority of cases, devel packages must require the base package using a fully versioned dependency. N/A MUST: Packages does not contain any .la libtool archives. OK MUST: Desktop files are installed properly. OK MUST: No file conflicts with other packages and no general names. OK MUST: Buildroot cleaned before install. OK SHOULD: %{?dist} tag is used in release. OK SHOULD: If the package does not include license text(s) as separate files from upstream, the packager should query upstream to include it. N/A SHOULD: The package builds in mock. - builds fine for fedora-13-i386, fedora-rawhide-i386 and fedora-12-i386
NOTE: All files in equinox-1.20/src/ are marked executable. rpmlint doesn't like that when checking the debuginfo package. A chmod 644 on these files in %prep should fix it though.
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=608973
--- Comment #13 from Germán Racca gracca@gmail.com 2010-07-07 03:31:01 EDT --- Hi Julian:
Many thanks for the full review. Executable bits are fixed now, you told me abut that the first time, but I forgot to correct it...sorry :-(
Please find updated files here:
Spec URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine.spec
SRPM URL: http://skytux.fedorapeople.org/packages/gtk-equinox-engine-1.20-3.fc13.src.r...
Cheers, Germán.
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=608973
Julian Aloofi julian.fedora@googlemail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #14 from Julian Aloofi julian.fedora@googlemail.com 2010-07-07 11:30:01 EDT --- (In reply to comment #13)
Many thanks for the full review. Executable bits are fixed now, you told me abut that the first time, but I forgot to correct it...sorry :-(
No problem at all :)
APPROVED
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=608973
--- Comment #15 from Germán Racca gracca@gmail.com 2010-07-07 20:26:05 EDT --- Thanks for the review Julian!
Germán.
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=608973
Germán Racca gracca@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #16 from Germán Racca gracca@gmail.com 2010-07-07 21:27:52 EDT --- New Package CVS Request ======================= Package Name: gtk-equinox-engine Short Description: Equinox theme engine for GTK+ 2.x Owners: skytux Branches: F-12 F-13 InitialCC:
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=608973
--- Comment #17 from Kevin Fenzi kevin@tummy.com 2010-07-09 14:22:13 EDT --- CVS done (by process-cvs-requests.py).
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=608973
--- Comment #18 from Fedora Update System updates@fedoraproject.org 2010-07-12 12:43:46 EDT --- gtk-equinox-engine-1.20-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/gtk-equinox-engine-1.20-3.fc12
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=608973
--- Comment #19 from Fedora Update System updates@fedoraproject.org 2010-07-12 12:55:23 EDT --- gtk-equinox-engine-1.20-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/gtk-equinox-engine-1.20-3.fc13
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=608973
Germán Racca gracca@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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=608973
--- Comment #20 from Fedora Update System updates@fedoraproject.org 2010-08-02 20:31:52 EDT --- gtk-equinox-engine-1.20-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
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=608973
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version| |gtk-equinox-engine-1.20-3.f | |c13 Resolution|NEXTRELEASE |ERRATA
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=608973
--- Comment #21 from Fedora Update System updates@fedoraproject.org 2010-08-02 21:01:01 EDT --- gtk-equinox-engine-1.20-3.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
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=608973
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|gtk-equinox-engine-1.20-3.f |gtk-equinox-engine-1.20-3.f |c13 |c12
package-review@lists.fedoraproject.org