Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/show_bug.cgi?id=448236
Summary: Review Request: xfbib - Lightweight BibTeX editor for the Xfce desktop environment Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: fedora@christoph-wickert.de QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com,notting@redhat.com
Spec URL: http://cwickert.fedorapeople.org/review/xfbib.spec SRPM URL: http://cwickert.fedorapeople.org/review/xfbib-0.0.2-1.fc10.src.rpm Description: Xfbib is a lightweight BibTeX editor developed for the Xfce desktop environment. The intention of Xfbib is to provide an easy and efficient way of editing BibTeX files.
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
michel.sylvan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |michel.sylvan@gmail.com Status|NEW |ASSIGNED Flag| |fedora-review?
------- Additional Comments From michel.sylvan@gmail.com 2008-06-04 15:20 EST ------- desktop-file-install should probably be called with --vendor="fedora", not "" ?
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From fedora@christoph-wickert.de 2008-06-04 15:53 EST ------- We are not using "vendor" for the other Xfce packages as well (except for xfce4-taskmanager), but I can change it if you like. Thinking about this a little more I tend to agree with you: fedora-xfbib.desktop is better than xfbib.desktop, because xfbib has no "xfce" prefix.
Do you want me to change this now or after/during review?
BTW: I just found another trivial error in %changelog: "Initial Fedora _Extras_ package" of course is wrong because there are no Extras any longer.
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From michel.sylvan@gmail.com 2008-06-05 01:12 EST ------- In case where the vendor is not already set, then yes, adding "fedora" is probably the thing to do (or add "xfce" instead? I'll leave it to your discretion). And yes, the "Extras" thing can be done without.
As it turns out, there are a couple of other problems, all easily fixable. Need to ask a small favour -- could you review a couple of my requests in turn? Much obliged:
https://bugzilla.redhat.com/buglist.cgi?query_format=&short_desc_type=al...
MUSTFIX: • own all directories: FAIL installs files in %{_datadir}/icons/hicolor: need to require hicolor-icon-theme
SHOULDFIX: • scriplets are sane: add vendor (fedora or xfce)
MUST, passed: • rpmlint: clean • package name: ok • spec file name: ok • package guideline-compliant: ok • license complies with guidelines: ok • license field accurate: ok • license file not deleted: ok • spec in US English: ok • spec legible: ok • source matches upstream: ok • builds under >= 1 archs, others excluded: ok • build dependencies complete: ok • locales handled using %find_lang, no %{_datadir}/locale: ok • no dupes in %files: ok • permission: ok • %clean RPM_BUILD_ROOT: ok • macros used consistently: ok • Package contains code: ok • desktop file uses desktop-file-install: ok • clean buildroot before install: ok • filenames UTF-8: ok
SHOULD • desc and summary contain translations if available: ok • package build in mock on all architectures: ok http://koji.fedoraproject.org/koji/taskinfo?taskID=647081 • package functioned as described: ok • require package not files: ok
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From fedora@christoph-wickert.de 2008-06-05 05:46 EST ------- (In reply to comment #3)
could you review a couple of my requests in turn?
Of course, I will see what I can do.
MUSTFIX: • own all directories: FAIL installs files in %{_datadir}/icons/hicolor: need to require hicolor-icon-theme
Really? We already have an dependency on hicolor-icon-theme, although we don't require it explicitly: $ rpm -q --requires xfbib | grep gtk libgtk-x11-2.0.so.0 $ rpm -q --whatprovides libgtk-x11-2.0.so.0 gtk2-2.12.9-5.fc9.i386 $ rpm -q --requires gtk2 | grep hicolor hicolor-icon-theme
I used to think that this kind of dependency chain is enough.
SHOULDFIX: • scriplets are sane: add vendor (fedora or xfce)
Will do.
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
michel.sylvan@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
------- Additional Comments From michel.sylvan@gmail.com 2008-06-05 15:15 EST ------- Ah yes. Good point -- hicolor-icon-theme is in the dependency chain, so I'll approve this review and you can add the vendor when uploading.
APPROVED
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From fedora@christoph-wickert.de 2008-06-06 04:13 EST ------- New Package CVS Request ======================= Package Name: xfbib Short Description: Lightweight BibTeX editor for the Xfce desktop environment Owners: cwickert Branches: F-8 F-9 devel InitialCC: Cvsextras Commits: yes
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
fedora@christoph-wickert.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-cvs? |fedora-cvs+
------- Additional Comments From kevin@tummy.com 2008-06-06 11:43 EST ------- cvs done.
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
fedora@christoph-wickert.de changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
------- Additional Comments From fedora@christoph-wickert.de 2008-06-11 19:05 EST ------- Packages for all branches have been build successfully. Closing.
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From updates@fedoraproject.org 2008-06-11 19:34 EST ------- xfbib-0.0.2-1.fc8 has been submitted as an update for Fedora 8
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From updates@fedoraproject.org 2008-06-11 19:35 EST ------- xfbib-0.0.2-1.fc9 has been submitted as an update for Fedora 9
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: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From updates@fedoraproject.org 2008-06-12 22:18 EST ------- xfbib-0.0.2-1.fc8 has been pushed to the Fedora 8 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 report.
Summary: Review Request: xfbib - Lightweight BibTeX editor for the Xfce desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=448236
------- Additional Comments From updates@fedoraproject.org 2008-06-12 22:24 EST ------- xfbib-0.0.2-1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report.
package-review@lists.fedoraproject.org