[Bug 771837] Review Request: gubbi-fonts - Free Kannada Opentype serif font
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=771837 Pravin Satpute psatp...@redhat.com changed: What|Removed |Added Keywords||i18n CC||i18n-bugs@lists.fedoraproje ||ct.org, panem...@gmail.com, ||svenk...@redhat.com Flag||fedora-review? -- 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 771842] Review Request: navilu-fonts - Free Kannada Opentype sans-serif font
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=771842 Pravin Satpute psatp...@redhat.com changed: What|Removed |Added Keywords||i18n CC||i18n-bugs@lists.fedoraproje ||ct.org, panem...@gmail.com, ||svenk...@redhat.com Flag||fedora-review? -- 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 771842] New: Review Request: navilu-fonts - Free Kannada Opentype sans-serif font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: navilu-fonts - Free Kannada Opentype sans-serif font https://bugzilla.redhat.com/show_bug.cgi?id=771842 Summary: Review Request: navilu-fonts - Free Kannada Opentype sans-serif font Product: Fedora Version: 16 Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: psatp...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- SPEC Url: http://pravins.fedorapeople.org/navilu-fonts.spec SRPM Url: http://pravins.fedorapeople.org/navilu-fonts-1.1-1.fc16.src.rpm Scratch build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3621205 -- 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 770152] Review Request: gnome-boxes - A simple GNOME 3 application to access remote or virtual systems
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=770152
--- Comment #9 from Christophe Fergeau cferg...@redhat.com 2012-01-05
03:54:59 EST ---
(In reply to comment #8)
Some comments:
- in the changelog, you need to escape the macro, so %%{buildroot}
Fixed (locally)
- did you intend to put something in the %pre section?
Nope, it's some copy and paste from totem, I removed it
- the address in the COPYING file is wrong - please use the most recent
version
I sent a patch upstream to fix this
- it's not required for the review, but please consider including a manpage
I opened a bug upstream about it:
https://bugzilla.gnome.org/show_bug.cgi?id=667340
--
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 770152] Review Request: gnome-boxes - A simple GNOME 3 application to access remote or virtual systems
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=770152 --- Comment #10 from Christophe Fergeau cferg...@redhat.com 2012-01-05 04:02:36 EST --- Updated SRPM at http://teuf.fedorapeople.org/reviews/gnome-boxes/gnome-boxes-3.3.3-3.fc16.src.rpm The .spec is still http://teuf.fedorapeople.org/reviews/gnome-boxes/gnome-boxes.spec -- 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 576591] Review Request: iptraf-ng
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=576591 --- Comment #27 from Nikola Pajkovsky npajk...@redhat.com 2012-01-05 04:04:00 EST --- W: non-ghost-in-var-lock /var/lock/iptraf-ng I don't have any idea, how did you get above message. All, except incoherent-version-in-changelog, warnings and error are fixed. I'm auto generating spec version of iptraf-ng from current git version, and will be off almost every time, except the time of tagging of new version in git. http://npajkovs.fedorapeople.org/iptraf-ng-1.1.0.rc0.1.gfe0c-2.el6.src.rpm please pick spec file from srpm -- 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 771851] New: Review Request: python-ladon - Multiprotocol approach to creating a webservice
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-ladon - Multiprotocol approach to creating a webservice https://bugzilla.redhat.com/show_bug.cgi?id=771851 Summary: Review Request: python-ladon - Multiprotocol approach to creating a webservice Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mru...@matthias-runge.de QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://www.matthias-runge.de/fedora/python-ladon.spec SRPM URL: http://www.matthias-runge.de/fedora/python-ladon-0.6.5-1.fc16.src.rpm Description: Ladon is a framework for exposing methods to several internet service protocols. Once a method is ladonized it is automatically served through all the interfaces that your ladon installation contains. Ladon's interface implemetations are added in a modular fashion making it very easy extend Ladon's protocol support. [mrunge@mrungexp SPECS]$ rpmlint -i /home/mrunge/rpmbuild/SRPMS/python-ladon-0.6.5-1.fc16.src.rpm /home/mrunge/rpmbuild/RPMS/noarch/python-ladon-0.6.5-1.fc16.noarch.rpm ./python-ladon.spec python-ladon.src: W: spelling-error Summary(en_US) Multiprotocol - Multiprocessor The value of this tag appears to be misspelled. Please double-check. python-ladon.src: W: spelling-error Summary(en_US) webservice - web service, web-service, service The value of this tag appears to be misspelled. Please double-check. python-ladon.src: W: spelling-error %description -l en_US ladonized - lionized The value of this tag appears to be misspelled. Please double-check. python-ladon.src: W: spelling-error %description -l en_US implemetations - implementations, implementation, supplementation The value of this tag appears to be misspelled. Please double-check. python-ladon.noarch: W: spelling-error Summary(en_US) Multiprotocol - Multiprocessor The value of this tag appears to be misspelled. Please double-check. python-ladon.noarch: W: spelling-error Summary(en_US) webservice - web service, web-service, service The value of this tag appears to be misspelled. Please double-check. python-ladon.noarch: W: spelling-error %description -l en_US ladonized - lionized The value of this tag appears to be misspelled. Please double-check. python-ladon.noarch: W: spelling-error %description -l en_US implemetations - implementations, implementation, supplementation The value of this tag appears to be misspelled. Please double-check. python-ladon.noarch: W: no-manual-page-for-binary ladon2.7ctl Each executable in standard binary directories should have a man page. python-ladon.noarch: W: no-manual-page-for-binary ladon2.7ctl.py Each executable in standard binary directories should have a man page. 2 packages and 1 specfiles checked; 0 errors, 10 warnings. -- 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 771171] Review Request: django-tastypie - A webservice API framework for Django
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=771171 Matthias Runge mru...@matthias-runge.de changed: What|Removed |Added Status|NEW |ASSIGNED CC||mru...@matthias-runge.de AssignedTo|nob...@fedoraproject.org|mru...@matthias-runge.de -- 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 771171] Review Request: django-tastypie - A webservice API framework for Django
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=771171 --- Comment #1 from Matthias Runge mru...@matthias-runge.de 2012-01-05 04:21:52 EST --- Two small notes: During build: + pushd tests ~/rpmbuild/BUILD/django-tastypie-0.9.11/tests ~/rpmbuild/BUILD/django-tastypie-0.9.11 + export PYTHONPATH=/usr/lib/python2.7/site-packages + PYTHONPATH=/usr/lib/python2.7/site-packages + ./run_all_tests.sh ** Core ** Error: No module named oauth_provider ** Basic ** 2. rpmlint -i reports ./django-tastypie.spec:18: W: mixed-use-of-spaces-and-tabs (spaces: line 5, tab: line 18) The specfile mixes use of spaces and tabs for indentation, which is a cosmetic annoyance. Use either spaces or tabs for indentation, not both. django-tastypie-doc.noarch: W: hidden-file-or-dir /usr/share/doc/django-tastypie-0.9.11/html/.buildinfo The file or directory is hidden. You should see if this is normal, and delete it from the package if not. Those are small issues, which can be easily fixed. -- 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 700818] Review Request: libircclient - Library implementing client-server IRC protocol
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=700818 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Alias||libircclient -- 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 700818] Review Request: libircclient - Library implementing client-server IRC protocol
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=700818 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Blocks||771885 -- 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 768174] Review Request: srm-ifce - Storage Resources Manager client implementation
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=768174
--- Comment #14 from adev ade...@gmail.com 2012-01-05 05:21:25 EST ---
Happy new year and thank you for your reviews,
I take care of your comments :
- README is now in the main package
- RELEASE-NOTES is now in devel, we will use it for the devs's messages.
- one empty line is added between each changelog entry
- DOC_INSTALL_DIR variable is added to cmake for the documentation directory
- every doc file is in %{_docdir}/%{name}-%{version}
- %doc macros are now removed.
Spec URL : https://firwen.org/redmine/data/docs/specs/srm-ifce.spec
SRPM URL : https://firwen.org/redmine/data/docs/specs/srm-ifce-1.12-3.src.rpm
rpmlint :
srm-ifce.src: W: invalid-url Source0: srm-ifce-1.12.tar.gz
4 packages and 0 specfiles checked; 0 errors, 1 warnings.
--
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 768183] Review Request: is-interface - library for the information system in wlcg
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=768183 --- Comment #2 from adev ade...@gmail.com 2012-01-05 05:39:41 EST --- Update of the files considering comments from 768174 about docdir : Spec URL : https://firwen.org/redmine/data/docs/specs/is-interface.spec SRPM URL: https://firwen.org/redmine/data/docs/specs/is-interface-1.12.1-3_epel.src.rpm rpmlint : is-interface.src: W: invalid-url Source0: is-interface-1.12.1.src.tar.gz 4 packages and 0 specfiles checked; 0 errors, 1 warnings. -- 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 681826] Review Request: xf86-input-multitouch - Multitouch X input driver
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=681826 Harald Hoyer har...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution||WONTFIX Last Closed||2012-01-05 05:57:37 --- Comment #4 from Harald Hoyer har...@redhat.com 2012-01-05 05:57:37 EST --- (In reply to comment #3) Is this driver still the target, or would this one be more appropriate, given the activity on the code: https://github.com/BlueDragonX/xf86-input-mtrack yeah, I think I should close this. -- 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 754583] Review Request: dnssec-trigger - Update/reconfigure DNSSEC resolving
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=754583 Adam Tkac at...@redhat.com changed: What|Removed |Added Flag||fedora-review? --- Comment #9 from Adam Tkac at...@redhat.com 2012-01-05 06:01:10 EST --- (In reply to comment #8) mock -r fedora-rawhide-i386 ~/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm State Changed: creating cache State Changed: unlock buildroot INFO: Installed packages: State Changed: setup State Changed: build INFO: Done(/home/paul/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm) Config(fedora-rawhide-i386) 13 minutes 33 seconds INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-i386/result State Changed: end [paul@bofh mock]$ cd /var/lib/mock/fedora-rawhide-i386/result [paul@bofh result]$ ls build.log dnssec-trigger-0.9-1.fc17.i686.rpm dnssec-trigger-0.9-1.fc17.src.rpm dnssec-trigger-debuginfo-0.9-1.fc17.i686.rpm root.log state.log Works for me. Please retest ? Yes, it is now fine, thanks! There is only one missing thing - package should include LICENSE file in the %doc, can you please fix this? -- 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 759757] Review Request: perl-Encode-EUCJPASCII - EucJP-ascii - An eucJP-open mapping
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=759757 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||ppi...@redhat.com AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com Flag||fedora-review? -- 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 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit
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=754754 --- Comment #6 from Daniel Berrange berra...@redhat.com 2012-01-05 07:09:09 EST --- The extra 'perl(Glib::Object::Introspection)' without version, is an automagically added dep. The versions deps are manually added, since I need to force a minimal version to ensure the package works correctly. The unversioned automatic dep is harmless so I don't think it is worth trying to filter it out. Updated SRPM with all other FIX TODO items addressed: http://berrange.fedorapeople.org/review/perl-Gtk3/perl-Gtk3-0.002-2.fc17.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 689488] Review Request: vcftools - VCF file manipulation tools
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=689488 --- Comment #9 from Adam Huffman bl...@verdurin.com 2012-01-05 07:40:40 EST --- Updated to 0.1.7: http://verdurin.fedorapeople.org/reviews/vcftools/vcftools.spec http://verdurin.fedorapeople.org/reviews/vcftools/vcftools-0.1.7-1.fc17.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 728879] Review Request: perl-WWW-GoodData - Client library for GoodData REST-ful API
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=728879 Petr Šabata psab...@redhat.com changed: What|Removed |Added CC||psab...@redhat.com Flag||needinfo?(lkund...@v3.sk) --- Comment #6 from Petr Šabata psab...@redhat.com 2012-01-05 07:49:38 EST --- I wonder why this review is still in the NEW status and why its git repo is still empty?... -- 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 758211] Review Request: perl-ExtUtils-MakeMaker - Create a module Makefile
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=758211 Petr Šabata psab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com AssignedTo|nob...@fedoraproject.org|psab...@redhat.com Flag||fedora-review? -- 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 766622] Review Request: perl-Authen-Credential - Abstraction of a credential
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=766622 Petr Šabata psab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com AssignedTo|nob...@fedoraproject.org|psab...@redhat.com Flag||fedora-review? -- 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 759757] Review Request: perl-Encode-EUCJPASCII - EucJP-ascii - An eucJP-open mapping
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=759757 --- Comment #1 from Petr Pisar ppi...@redhat.com 2012-01-05 07:58:41 EST --- Source tar ball is original. Ok. Summary verified from EUCJPASCII.pm. Ok. Description verified from EUCJPASCII.pm. Ok. Note: I'm not sure `eucJP-ascii' and `eucJP-open' are properly capitalized. Different texts use different capitalization and hyphenation. Let's hope イケダソジ, author of this module, knows better. License verified from EUCJPASCII.pm. Ok. URL and Source0 values are useful. Ok. TODO: Remove useless BuildRoot definition, it's cleaning in %install section, and whole %clean section. They are not needed anymore. TODO: Build-require perl(Encode) for tests (EUCJPASCII.pm:6) TODO: Build-require perl(XSLoader) for tests (EUCJPASCII.pm:7) TODO: Build-require perl(base) for tests (EUCJPASCII.pm:18) TODO: Build-require perl(Encode::CJKConstants) for tests (EUCJPASCII.pm:23) TODO: Build-require perl(Encode::JP::JIS7) for tests (EUCJPASCII.pm:23) FIX: Build-require perl(File::Spec) (Makefile.PL:18) All tests pass. Ok. TODO: Remove useless %defattr from %files section. $ rpmlint perl-Encode-EUCJPASCII.spec ../SRPMS/perl-Encode-EUCJPASCII-0.03-1.fc17.src.rpm ../RPMS/x86_64/perl-Encode-EUCJPASCII-* perl-Encode-EUCJPASCII.src: W: spelling-error Summary(en_US) eucJP - eggcup, equip, uncap perl-Encode-EUCJPASCII.src: W: spelling-error %description -l en_US eucJP - eggcup, equip, uncap perl-Encode-EUCJPASCII.src: W: spelling-error %description -l en_US ascii - ASCII, ASCIIs, As perl-Encode-EUCJPASCII.x86_64: W: spelling-error Summary(en_US) eucJP - eggcup, equip, uncap perl-Encode-EUCJPASCII.x86_64: W: spelling-error %description -l en_US eucJP - eggcup, equip, uncap perl-Encode-EUCJPASCII.x86_64: W: spelling-error %description -l en_US ascii - ASCII, ASCIIs, As perl-Encode-EUCJPASCII.x86_64: E: zero-length /usr/lib64/perl5/vendor_perl/auto/Encode/EUCJPASCII/EUCJPASCII.bs 3 packages and 1 specfiles checked; 1 errors, 6 warnings. FIX: Remove empty *.bs files. (These empty bootstrap files for DynaLoader are useless.) $ rpm -q -lv -p ../RPMS/x86_64/perl-Encode-EUCJPASCII-0.03-1.fc17.x86_64.rpm drwxr-xr-x2 rootroot0 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/Encode -rw-r--r--1 rootroot 5662 Oct 19 2009 /usr/lib64/perl5/vendor_perl/Encode/EUCJPASCII.pm drwxr-xr-x2 rootroot0 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/auto drwxr-xr-x2 rootroot0 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/auto/Encode drwxr-xr-x2 rootroot0 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/auto/Encode/EUCJPASCII -rw-r--r--1 rootroot0 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/auto/Encode/EUCJPASCII/EUCJPASCII.bs -rwxr-xr-x1 rootroot 759816 Jan 5 13:40 /usr/lib64/perl5/vendor_perl/auto/Encode/EUCJPASCII/EUCJPASCII.so drwxr-xr-x2 rootroot0 Jan 5 13:40 /usr/share/doc/perl-Encode-EUCJPASCII-0.03 -rw-r--r--1 rootroot 569 Oct 19 2009 /usr/share/doc/perl-Encode-EUCJPASCII-0.03/Changes -rw-r--r--1 rootroot 496 Oct 19 2009 /usr/share/doc/perl-Encode-EUCJPASCII-0.03/README Files permissions and layout are Ok. $ rpm -q --requires -p ../RPMS/x86_64/perl-Encode-EUCJPASCII-0.03-1.fc17.x86_64.rpm |sort |uniq -c 1 libc.so.6()(64bit) 1 libc.so.6(GLIBC_2.2.5)(64bit) 1 perl(base) 1 perl(bytes) 1 perl(Encode) 1 perl(Encode::CJKConstants) 1 perl(Encode::Encoding) 1 perl(Encode::JP::JIS7) 1 perl(:MODULE_COMPAT_5.14.2) 1 perl(strict) 1 perl(warnings) 1 perl(XSLoader) 1 rpmlib(CompressedFileNames) = 3.0.4-1 1 rpmlib(FileDigests) = 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) = 4.0-1 1 rpmlib(PayloadIsXz) = 5.2-1 1 rtld(GNU_HASH) Binary requires are Ok. $ rpm -q --provides -p ../RPMS/x86_64/perl-Encode-EUCJPASCII-0.03-1.fc17.x86_64.rpm |sort |uniq -c 1 perl(Encode::EUCJPASCII) = 0.03 1 perl-Encode-EUCJPASCII = 0.03-1.fc17 1 perl-Encode-EUCJPASCII(x86-64) = 0.03-1.fc17 Binary provides are Ok. $ resolvedeps rawhide ../RPMS/x86_64/perl-Encode-EUCJPASCII-0.03-1.fc17.x86_64.rpm Binary dependencies resolvable. Ok. Package builds in F17 (http://koji.fedoraproject.org/koji/taskinfo?taskID=3621985). Ok. Otherwise package is in line with Fedora and Perl packaging guidelines. Please correct all `FIX' prefixed issues, consider fixing `TODO' items, and provide new spec file. Resolution: Package NOT 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.
[Bug 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 --- Comment #13 from Jon Ciesla limburg...@gmail.com 2012-01-05 08:04:14 EST --- Summary package name and SCM request name do not match, please correct. -- 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 769173] Review Request: english-typing booster - typing booster for english language
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=769173 --- Comment #7 from Jon Ciesla limburg...@gmail.com 2012-01-05 08:03:40 EST --- Pacakge name cannot contain a space, please correct in summary and SCM request. -- 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 761474] Review Request: ibus-european-table - Predictive text for european languages
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=761474 anish apa...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- 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 752311] Review Request: gitso - A front-end to reverse VNC connections
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=752311 --- Comment #10 from Jon Ciesla limburg...@gmail.com 2012-01-05 08:02:46 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 761474] Review Request: ibus-european-table - Predictive text for european languages
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=761474 --- Comment #11 from anish apa...@redhat.com 2012-01-05 08:02:12 EST --- Package Change Request == Package Name: ibus-european-table New Branches: f16 Owners: anishpatil 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 769173] Review Request: english-typing booster - typing booster for english language
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=769173 anish apa...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- 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 769173] Review Request: english-typing booster - typing booster for english language
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=769173 --- Comment #8 from anish apa...@redhat.com 2012-01-05 08:12:06 EST --- Please find corrected version:- New Package SCM Request === Package Name: english-typing-booster Short Description: dictionary for English language Owners: anishpatil,pravins Branches: f16 InitialCC: i18n-team -- 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 769173] Review Request: english-typing-booster typing booster for english language
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=769173 anish apa...@redhat.com changed: What|Removed |Added Summary|Review Request: |Review Request: |english-typing booster -|english-typing-booster |typing booster for english |typing booster for english |language|language -- 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 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit
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=754754 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Summary|Review Request: perl-Gtk3 - |Review Request: perl-Gtk3 - |Perl interface to the 3.x |Perl interface to the 3.x |series of the gtk+ toolkit |series of the GTK+ toolkit -- 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 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the gtk+ toolkit
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=754754
Petr Pisar ppi...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #7 from Petr Pisar ppi...@redhat.com 2012-01-05 08:23:23 EST ---
Spec file changes:
--- perl-Gtk3.spec.old 2011-12-15 16:09:39.0 +0100
+++ perl-Gtk3.spec 2012-01-05 14:02:07.934998296 +0100
@@ -1,7 +1,9 @@
+%global use_x11_tests 1
+
Name: perl-Gtk3
Version:0.002
-Release:1%{?dist}
-Summary:Perl interface to the 3.x series of the gtk+ toolkit
+Release:2%{?dist}
+Summary:Perl interface to the 3.x series of the GTK+ toolkit
License:LGPLv2+
Group: Development/Libraries
URL:http://search.cpan.org/dist/Gtk3/
@@ -12,14 +14,24 @@
BuildRequires: perl(Glib::Object::Introspection) = 0.002
BuildRequires: perl(Test::More)
BuildRequires: gtk3
+BuildRequires: perl(Carp)
+BuildRequires: perl(Exporter)
+BuildRequires: perl(Test::More)
+BuildRequires: perl(Scalar::Util)
Requires: perl(Glib) = 1.240
Requires: perl(Glib::Object::Introspection) = 0.002
Requires: perl(:MODULE_COMPAT_%(eval `%{__perl} -V:version`; echo
$version))
Requires: gtk3
+%if %{use_x11_tests}
+# X11 tests:
+BuildRequires: xorg-x11-server-Xvfb
+BuildRequires: xorg-x11-xinit
+BuildRequires: font(:lang=en)
+%endif
%description
-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/
%prep
%setup -q -n Gtk3-%{version}
@@ -37,17 +49,27 @@
%{_fixperms} $RPM_BUILD_ROOT/*
%check
-# NB, we expect most of these checks to be skipped
-# because of lack of a $DISPLAY inside koji
-make test
+%if %{use_x11_tests}
+xinit /bin/sh -c 'rm -f ok; make test touch ok' -- /usr/bin/Xvfb :666
+test -e ok
+%else
+make test
+%endif
%files
-%defattr(-,root,root,-)
-%doc dist.ini LICENSE NEWS perl-Gtk3.doap README
+%doc LICENSE NEWS README
%{perl_vendorlib}/*
%{_mandir}/man3/*
%changelog
+* Thu Jan 5 2012 Daniel P. Berrange berra...@redhat.com - 0.002-2
+- Use xvfb to run test suite
+- Fix capitalization of GTK+
+- Remove dist.ini perl-Gtk3.doap
+- Remove defattr from files section
+- Add missing BuildRequires for test suite
+- Add trailing / into URIs
+
* Thu Dec 15 2011 Daniel P. Berrange berra...@redhat.com - 0.002-1
- Update to 0.002 release
TODO: Capitalize the `GTK+' to comply with official name
(http://www.gtk.org/).
-Summary:Perl interface to the 3.x series of the gtk+ toolkit
+Release:2%{?dist}
+Summary:Perl interface to the 3.x series of the GTK+ toolkit
-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/
Ok.
TODO: Append slash to the URL as conforming HTTP URL must contain path.
-The Gtk3 module allows a Perl developer to use the gtk+ graphical user
-interface library. Find out more about gtk+ at http://www.gtk.org.
+The Gtk3 module allows a Perl developer to use the GTK+ graphical user
+interface library. Find out more about GTK+ at http://www.gtk.org/
TODO: You are missing full stop at the and of the description. If you hesitate
because of URL, close URL into angle brackets (http://www.gtk.org/.).
FIX: Build-require perl(Carp) for tests (lib/Gtk3.pm:8)
TODO: Build-require perl(Exporter) for tests (lib/Gtk3.pm:10)
FIX: Build-require perl(Test::More) for tests (t/00-init.t:6)
FIX: Build-require perl(Scalar::Util) for tests (t/floating-refs.t:7)
+BuildRequires: perl(Carp)
+BuildRequires: perl(Exporter)
+BuildRequires: perl(Test::More)
+BuildRequires: perl(Scalar::Util)
Ok.
TODO: Run tests against Xvfb to perform X11 tests
(https://fedoraproject.org/wiki/Perl/Tips#Tests_require_X11_server; actually
there is easier way using xvfb-run tool).
Ok.
TODO: Remove useless %defattr from %files section.
TODO: Do not package dist.ini and perl-Gtk3.doap. They do not provide any
useful data not mentioned anywhere else.
%files
-%defattr(-,root,root,-)
-%doc dist.ini LICENSE NEWS perl-Gtk3.doap README
+%doc LICENSE NEWS README
Ok.
All tests pass. Ok.
$ rpmlint perl-Gtk3.spec ../SRPMS/perl-Gtk3-0.002-2.fc17.src.rpm
../RPMS/noarch/perl-Gtk3-0.002-2.fc17.noarch.rpm
perl-Gtk3.spec:29: W: comparison-operator-in-deptoken font(:lang=en)
perl-Gtk3.src:29: W: comparison-operator-in-deptoken font(:lang=en)
2 packages and 1 specfiles
[Bug 766622] Review Request: perl-Authen-Credential - Abstraction of a credential
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=766622
--- Comment #2 from Petr Šabata psab...@redhat.com 2012-01-05 08:30:50 EST ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[!]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
Note: Buildroot is not needed unless packager plans to package for EPEL5
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[!]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean is needed only if supporting EPEL
[x]: MUST Sources contain only permissible code or content.
[!]: MUST Each %files section contains %defattr if rpm 4.4
Note: defattr() present in %files section. This is OK if packaging
for EPEL5. Otherwise not needed
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf is only needed if supporting EPEL5
[-]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[!]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[!]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
/home/contyk/src/review/766622/Authen-Credential-0.5.tar.gz :
MD5SUM this package : 32fd2003715f1b4bb010ac4a5e477ca7
MD5SUM upstream package : 32fd2003715f1b4bb010ac4a5e477ca7
[ ]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[!]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[?]: SHOULD Package functions as described.
[x]: SHOULD Package does not include license text files separate from
upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
[x]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[-]: SHOULD Spec use %global instead of %define.
Issues:
TODO: Please, remove the Buildroot tag, the %clean section, the buildroot
removal in %build section and %defattr from files unless you plan to use this
package in EPEL5. Those are now obsolete in Fedora.
TODO: Add BuildRequires: perl(Exporter)
TODO: Since it seems you'd like to run optional POD tests, specify the needed
POD modules versions in BR, i.e.: perl(Pod::Coverage) = 0.18 and
perl(Test::Pod::Coverage) = 1.08
TODO: Remove unneeded runtime dependencies polluting the resulting RPM. rpm
adds those automatically. Those are perl(MIME::Base64), perl(Params::Validate),
[Bug 760943] Review Request: perl-Messaging-Message - This perl module provides an abstraction of a message, as used in messaging
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=760943 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flag||needinfo?(Laurence.Field@ce ||rn.ch) --- Comment #2 from Petr Pisar ppi...@redhat.com 2012-01-05 08:33:55 EST --- The SRPM URL points to compiled `binary' package. Wee need source RPM package for review (rpmbuild -bs perl-Messaging-Message.spec). Please provide it. -- 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 760943] Review Request: perl-Messaging-Message - This perl module provides an abstraction of a message, as used in messaging
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=760943 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||ppi...@redhat.com AssignedTo|nob...@fedoraproject.org|ppi...@redhat.com Flag||fedora-review? -- 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 771941] New: Review Request: bacula-docs - Bacula documentation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: bacula-docs - Bacula documentation https://bugzilla.redhat.com/show_bug.cgi?id=771941 Summary: Review Request: bacula-docs - Bacula documentation Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: negativ...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://slaanesh.fedorapeople.org/bacula.spec SRPM URL: http://slaanesh.fedorapeople.org/bacula-docs-5.2.3-6.fc16.src.rpm Description: Bacula is a set of programs that allow you to manage the backup, recovery, and verification of computer data across a network of different computers. It is based on a client/server architecture. This package contains the documentation for most of the bacula-packages. --- Notes: This is a spinoff of the bacula-docs subpackage that is present inside bacula that I'm currently co-mantaining: http://koji.fedoraproject.org/koji/buildinfo?buildID=281037 Motivations for the spin-off: - To avoid rebuilding 40 mb of docs each release that never change and to avoid uploading 40 mb for each koji scratch build. - It is pointless to have the user update all the docs each time we generate a new bacula package because of a security fix or bug. - It is also built for RHEL 4/5/6 (most of the userbase goes there), and in RHEL 4/5 there's no way to specify a different BuildArch in a subpackage, so i.e. on RHEL 5 you got x86_64 pdf files. - The package bacula-gui (currently not available in Fedora) will follow the same approach and be a separate Review Request. Package notes: - It has the release number immediately after the one which is in rawhide so it will update the one generated from the bacula package. If it's accepted I will remove the docs in the bacula package. - It passes all rpmlint checks. - It doesn't have an install section, all documents are included as %docs from the source folder where they are generated. -- 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 771941] Review Request: bacula-docs - Bacula documentation
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=771941 Simone Caronni negativ...@gmail.com changed: What|Removed |Added Priority|medium |low Severity|medium |low -- 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 771944] New: Review Request: pykka - Python library that provides concurrency using actor model
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pykka - Python library that provides concurrency using actor model https://bugzilla.redhat.com/show_bug.cgi?id=771944 Summary: Review Request: pykka - Python library that provides concurrency using actor model Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jdie...@lesbg.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://www.lesbg.com/jdieter/pykka.spec SRPM URL: http://www.lesbg.com/jdieter/pykka-0.13.0-1.fc16.src.rpm Description: The goal of Pykka is to provide easy to use concurrency abstractions for Python by using the actor model. Pykka provides an actor API with two different implementations: * ThreadingActor is built on the Python Standard Library's threading and Queue modules, and has no dependencies outside Python itself. It plays well together with non-actor threads. * GeventActor is built on the gevent library. gevent is a coroutine-based Python networking library that uses greenlet to provide a high-level synchronous API on top of libevent event loop. It is generally faster, but doesn't like playing with other threads. Much of the naming in Pykka is inspired by the Akka project which implements actors on the JVM. Though, Pykka does not aim to be a Python port of Akka. -- 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 771944] Review Request: pykka - Python library that provides concurrency using actor model
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=771944 --- Comment #1 from Jonathan Dieter jdie...@lesbg.com 2012-01-05 08:46:46 EST --- $ rpmlint pykka.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint ../RPMS/noarch/pykka-0.13.0-1.fc16.noarch.rpm pykka.noarch: W: spelling-error %description -l en_US gevent - event, g event, gent pykka.noarch: W: spelling-error %description -l en_US coroutine - co routine, co-routine, routine pykka.noarch: W: spelling-error %description -l en_US greenlet - green let, green-let, greenbelt pykka.noarch: W: spelling-error %description -l en_US libevent - lib event, lib-event, enlivenment pykka.noarch: W: spelling-error %description -l en_US doesn - does, does n 1 packages and 0 specfiles checked; 0 errors, 5 warnings. $ rpmlint ../SRPMS/pykka-0.13.0-1.fc16.src.rpm pykka.src: W: spelling-error %description -l en_US gevent - event, g event, gent pykka.src: W: spelling-error %description -l en_US coroutine - co routine, co-routine, routine pykka.src: W: spelling-error %description -l en_US greenlet - green let, green-let, greenbelt pykka.src: W: spelling-error %description -l en_US libevent - lib event, lib-event, enlivenment pykka.src: W: spelling-error %description -l en_US doesn - does, does n 1 packages and 0 specfiles checked; 0 errors, 5 warnings. -- 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 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 --- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2012-01-05 08:58:45 EST --- Alejandro: yes, the summary should read A specialized editor focused on creating structured documents Also, in the %description, please change the beginning to An editor instead of Editor. -- 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 598315] Review Request: UrJTAG - A tool to flash/program/debug hardware via JTAG adapters
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=598315 --- Comment #13 from Chitlesh GOORAH chitl...@gmail.com 2012-01-05 09:17:58 EST --- #1: libtool requires both automake and autoconf.automake requires autoconf. Remove autoconf and automake from the BR list. #2: libftdi-devel requires libusb-devel. Remove libusb-devel from the BR list. #3: Missing Group: in the main package #4: I'm ok with the naming. What is the cable you are using to test this urjtag package? -- 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 769173] Review Request: english-typing-booster typing booster for english language
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=769173 --- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-01-05 09:22:03 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 761474] Review Request: ibus-european-table - Predictive text for european languages
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=761474 --- Comment #12 from Jon Ciesla limburg...@gmail.com 2012-01-05 09:21:30 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 720857] Review Request: datalog - A Lightweight Deductive Database using Datalog
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=720857 --- Comment #22 from John D. Ramsdell ramsd...@mitre.org 2012-01-05 09:29:54 EST --- New Package SCM Request === Package Name: datalog Short Description: A Lightweight Deductive Database using Datalog Owners: ramsdell Branches: f15 f16 el6 InitialCC: tim -- 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 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 --- Comment #15 from Alejandro_Perez alejandro.perez.tor...@gmail.com 2012-01-05 09:39:20 EST --- Changes on summary and description done. Thanks Jussi -- 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 720857] Review Request: datalog - A Lightweight Deductive Database using Datalog
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=720857 --- Comment #23 from Tim Niemueller t...@niemueller.de 2012-01-05 10:08:55 EST --- This can only be done after approval. I'll look at it today. Do not put me as initial CC, that would make me a co-maintainer which we do not intend. -- 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 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 Alejandro_Perez alejandro.perez.tor...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? -- 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 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 --- Comment #16 from Alejandro_Perez alejandro.perez.tor...@gmail.com 2012-01-05 10:10:08 EST --- New Package SCM Request === Package Name: nested Short Description: A specialized editor focused on creating structured documents Owners: aeperezt Branches: f15 f16 el6 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 720857] Review Request: datalog - A Lightweight Deductive Database using Datalog
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=720857 --- Comment #24 from John D. Ramsdell ramsd...@mitre.org 2012-01-05 10:19:55 EST --- (In reply to comment #23) This can only be done after approval. I'll look at it today. Do not put me as initial CC, that would make me a co-maintainer which we do not intend. Sorry to jump the gun. I thought when you said the spec file looks good, you were approving it, but I realize now there is some flag that determines that state. I'll take you off the initial CC list next time I make the request. -- 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 771987] New: Review Request: staxmapper - StAX Mapper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: staxmapper - StAX Mapper https://bugzilla.redhat.com/show_bug.cgi?id=771987 Summary: Review Request: staxmapper - StAX Mapper Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mgold...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://goldmann.fedorapeople.org/package_review/staxmapper/1/staxmapper.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/staxmapper/1/staxmapper-1.0.0-1.fc17.src.rpm Description: StAX Mapper Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=3622377 -- 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 771987] Review Request: staxmapper - StAX Mapper
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=771987 Marek Goldmann mgold...@redhat.com changed: What|Removed |Added Blocks||652183(FE-JAVASIG) -- 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 758211] Review Request: perl-ExtUtils-MakeMaker - Create a module Makefile
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=758211
--- Comment #1 from Petr Šabata psab...@redhat.com 2012-01-05 10:32:52 EST ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[!]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Each %files section contains %defattr if rpm 4.4
Note: Note: defattr macros not found. They would be needed for EPEL5
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[!]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[?]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
/home/contyk/src/review/758211/ExtUtils-MakeMaker-6.62.tar.gz :
MD5SUM this package : 2ae291030c52999b5672b2a502eab195
MD5SUM upstream package : 2ae291030c52999b5672b2a502eab195
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[!]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[-]: SHOULD Package does not include license text files separate from
upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
[!]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[x]: SHOULD Spec use %global instead of %define.
Issues:
TODO: The following BRs are not essential at the moment, however it might
change some time in the future. Please, add them: perl(ExtUtils::Installed),
perl(File::Find), perl(File::Spec), perl(GetOpt::Long), and perl(File::Temp).
TODO: The following BRs are needed for the package to build in f17 mock/koji.
Those were originally bundled: perl(CPAN::Meta) and perl(Parse::CPAN::Meta)
Generated by fedora-review 0.1.1
External plugins:
--
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 738589] Review Request: perl-Socket-Netlink - Interface to Linux's PF_NETLINK socket family
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=738589 Petr Šabata psab...@redhat.com changed: What|Removed |Added CC||psab...@redhat.com Flag||needinfo?(bochecha@fedorapr ||oject.org) --- Comment #8 from Petr Šabata psab...@redhat.com 2012-01-05 10:37:39 EST --- Anything happening in here? Have you built the package? Can this bug be closed? -- 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 759697] Review Request: perl-Package-New - Simple base package from which to inherit
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=759697 Petr Šabata psab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com AssignedTo|nob...@fedoraproject.org|psab...@redhat.com Flag||fedora-review? -- 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 738931] Review Request: perl-Socket-Netlink-Route - Interface to Linux's NETLINK_ROUTE netlink socket protocol
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=738931 Petr Šabata psab...@redhat.com changed: What|Removed |Added CC||psab...@redhat.com Flag||needinfo?(bochecha@fedorapr ||oject.org) --- Comment #8 from Petr Šabata psab...@redhat.com 2012-01-05 10:38:00 EST --- Anything happening in here? Have you built the package? Can this bug be closed? -- 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 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit
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=754754 Daniel Berrange berra...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Daniel Berrange berra...@redhat.com 2012-01-05 10:39:15 EST --- New Package SCM Request === Package Name: perl-Gtk3 Short Description: Perl interface to the 3.x series of the GTK+ toolkit Owners: berrange Branches: f16 InitialCC: perl-sig -- 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 759697] Review Request: perl-Package-New - Simple base package from which to inherit
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=759697
--- Comment #3 from Petr Šabata psab...@redhat.com 2012-01-05 11:09:48 EST ---
Package Review
==
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
least one supported primary architecture.
[!]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
Note: Buildroot is not needed unless packager plans to package for EPEL5
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[!]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: Clean is needed only if supporting EPEL
[x]: MUST Sources contain only permissible code or content.
[!]: MUST Each %files section contains %defattr if rpm 4.4
Note: defattr() present in %files section. This is OK if packaging
for EPEL5. Otherwise not needed
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf is only needed if supporting EPEL5
[x]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[!]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package match the upstream source, as
provided in the spec URL.
/home/contyk/src/review/759697/Package-New-0.07.tar.gz :
MD5SUM this package : 00805fd07f1c330b71b2e3d120a6ad55
MD5SUM upstream package : 00805fd07f1c330b71b2e3d120a6ad55
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[?]: SHOULD Package functions as described.
[x]: SHOULD Package does not include license text files separate from
upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
[x]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[-]: SHOULD Spec use %global instead of %define.
Issues:
TODO: Add BuildRequires: perl(base)
TODO: Add BuildRequires: perl(Test::More)
TODO: Remove the BuildRoot tag, the %clean section, the buildroot removal in
the %build section and %defattr from %files unless you want to use this package
on EPEL5 as well. Those are obsolete in Fedora.
Generated by fedora-review 0.1.1
External plugins:
--
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
[Bug 754754] Review Request: perl-Gtk3 - Perl interface to the 3.x series of the GTK+ toolkit
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=754754 --- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-01-05 11:54:35 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 755484] Review Request: maven-toolchains-plugin - Maven plugin for sharing configuration across projects
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=755484
Marek Goldmann mgold...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #6 from Marek Goldmann mgold...@redhat.com 2012-01-05 11:52:25
EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[X] Rpmlint output:
rpmlint ./maven-toolchains-plugin.spec
./maven-toolchains-plugin.spec: W: invalid-url Source0:
http://repo1.org/maven2/org/apache/plugins/maven-toolchains-plugin/1.0/maven-toolchains-plugin-1.0-source-release.zip
urlopen error [Errno -2] Name or service not known
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
rpmlint ./maven-toolchains-plugin-1.0-2.fc15.src.rpm
maven-toolchains-plugin.src: I: enchant-dictionary-not-found en_US
maven-toolchains-plugin.src: W: invalid-url Source0:
http://repo1.org/maven2/org/apache/plugins/maven-toolchains-plugin/1.0/maven-toolchains-plugin-1.0-source-release.zip
urlopen error [Errno -2] Name or service not known
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
See issue #1.
[X] Package is named according to the Package Naming Guidelines[1].
[X] Spec file name must match the base package name, in the format
%{name}.spec.
[X] Package meets the Packaging Guidelines[2].
[X] Package successfully compiles and builds into binary rpms.
[X] Buildroot definition is not present
[X] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[X] License field in the package spec file matches the actual license.
License type: ASL 2.0
[-] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[-] All independent sub-packages have license of their own
[X] Spec file is legible and written in American English.
[X] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package: 05cd615acfd2acfa8c8bd29447ab7ec6
MD5SUM upstream package: 05cd615acfd2acfa8c8bd29447ab7ec6
[X] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[X] Package must own all directories that it creates or must require other
packages for directories it uses.
[X] Package does not contain duplicates in %files.
[X] File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[X] Permissions on files are set properly.
[X] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[X] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[X] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[X] Package does not own files or directories owned by other packages.
[X] Javadoc documentation files are generated and included in -javadoc
subpackage
[X] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[X] Packages have proper BuildRequires/Requires on jpackage-utils
[X] Javadoc subpackages have Require: jpackage-utils
[X] Package uses %global not %define
[-] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[X] If source tarball includes bundled jar/class files these need to be
removed prior to building
[X] All filenames in rpm packages must be valid UTF-8.
[X] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[-] If package contains pom.xml files install it (including depmaps) even when
building with ant
[X] pom files has correct add_maven_depmap
=== Maven ===
[X] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses -Dmaven.test.skip=true explain why it was needed in a
comment
[-] If package uses custom depmap -Dmaven.local.depmap.file=* explain why
it's needed in a comment
[X] Package DOES NOT use %update_maven_depmap in %post/%postun
[X] Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
=== Other suggestions ===
[X] If possible use upstream build method (maven/ant/javac)
[X] Avoid having BuildRequires on exact NVR unless necessary
[X] Package has BuildArch: noarch (if possible)
[X] Latest version is packaged.
[X] Reviewer should test that the package builds in mock.
Tested on:
[Bug 769697] Review Request: nested - A specialized editor focused on creating structured documents
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=769697 --- Comment #17 from Jon Ciesla limburg...@gmail.com 2012-01-05 11:57:15 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 770174] Review Request: kde-partitionmanager - GUI for managing disk partitions
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=770174 --- Comment #16 from Mattia mattia.ve...@tiscali.it 2012-01-05 12:12:43 EST --- Hans, do I need to do something more before being sponsored or you missed my reply in comment #14 to your request of my FAS login? Thank you. -- 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 771941] Review Request: bacula-docs - Bacula documentation
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=771941 --- Comment #1 from Simone Caronni negativ...@gmail.com 2012-01-05 12:14:45 EST --- Updated; as suggested I created a bacula-devel subpackage that's required by bacula-docs to build. Thanks Tom Callaway: Spec URL: http://slaanesh.fedorapeople.org/bacula.spec SRPM URL: http://slaanesh.fedorapeople.org/bacula-docs-5.2.3-7.fc16.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 753855] Review Request: pslib - C-library to create PostScript files
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=753855 Tom spot Callaway tcall...@redhat.com changed: What|Removed |Added CC||tcall...@redhat.com Blocks|182235(FE-Legal)| --- Comment #6 from Tom spot Callaway tcall...@redhat.com 2012-01-05 12:36:12 EST --- No real concern here. Lifting FE-Legal. -- 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 770796] Review Request: rubygem-gstreamer - Ruby binding of GStreamer
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=770796 Guillermo Gómez guillermo.go...@gmail.com changed: What|Removed |Added CC||guillermo.go...@gmail.com AssignedTo|nob...@fedoraproject.org|guillermo.go...@gmail.com Flag||fedora-review? --- Comment #2 from Guillermo Gómez guillermo.go...@gmail.com 2012-01-05 12:38:57 EST --- I'll post my review today. -- 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 770796] Review Request: rubygem-gstreamer - Ruby binding of GStreamer
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=770796
Guillermo Gómez guillermo.go...@gmail.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #3 from Guillermo Gómez guillermo.go...@gmail.com 2012-01-05
12:41:33 EST ---
- = N/A
x = Check
! = Problem
? = Not evaluated
[x] : MUST - Package is named according to the Package Naming Guidelines.
[x] : MUST - Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x] : MUST - Not have unused macro definitions (i.e. %global )
[x] : MUST - Fully versioned dependency in subpackages, if present.
[x] : MUST - Package does not contain any libtool archives (.la)
[-] : MUST - Package use %makeinstall only when make install DESTDIR=...
doesn't work.
[x] : MUST - Package successfully compiles and builds into binary rpms on at
least one supported architecture.
[x] : MUST - Sources used to build the package matches the upstream source, as
provided in the spec URL.
[-] : MUST - %config files are marked noreplace or the reason is justified.
[-] : MUST - Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application.
[-] : MUST - Header files in -devel subpackage, if present.
[-] : MUST - Development .so files in -devel subpackage, if present.
[-] : MUST - ldconfig called in %post and %postun if required.
[-] : MUST - The spec file handles locales properly.
[-] : MUST - Static libraries in -static subpackage, if present.
[x] : MUST - Rpmlint output is silent.
[-] : MUST - No %config files under /usr.
[x] : MUST - Package is licensed with an open-source compatible license and
meets other legal requirements as defined in the legal section of Packaging
Guidelines.
[x] : MUST - All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x] : MUST - Package contains no bundled libraries.
[x] : MUST - Changelog in prescribed format.
[x] : MUST - Sources contain only permissible code or content.
[x] : MUST - Macros in Summary, %description expandable at SRPM build time.
[-] : MUST - Package requires other packages for directories it uses.
[x] : MUST - Package uses nothing in %doc for runtime.
[x] : MUST - Package is not known to require ExcludeArch.
[x] : MUST - Permissions on files are set properly.
[x] : MUST - Package does not contain duplicates in %files.
[x] : MUST - Large documentation files are in a -doc subpackage, if required.
[-] : MUST - If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x] : MUST - License field in the package spec file matches the actual license.
[-] : MUST - License file installed when any subpackage combination is
installed.
[x] : MUST - Package consistently uses macros. instead of hard-coded directory
names.
[x] : MUST - Package meets the Packaging Guidelines.
[x] : MUST - Package does not generates any conflict.
[-] : MUST - Package does not contains kernel modules.
[-] : MUST - Package contains no static executables.
[x] : MUST - Package must own all directories that it creates.
[x] : MUST - Package does not own files or directories owned by other packages.
[x] : MUST - Package installs properly.
[x] : MUST - Rpath absent or only used for internal libs.
[x] : MUST - Package is not relocatable.
[x] : MUST - Requires correct, justified where necessary.
[x] : MUST - Spec file is legible and written in American English.
[x] : MUST - File names are valid UTF-8.
[x] : MUST - Useful -debuginfo package or justification otherwise.
[x] : SHOULD - Reviewer should test that the package builds in mock.
[x] : SHOULD - Dist tag is present.
[x] : SHOULD - SourceX / PatchY prefixed with %{name}.
[x] : SHOULD - SourceX is a working URL.
[x] : SHOULD - Spec use %global instead of %define.
[x] : SHOULD - The placement of pkgconfig(.pc) files are correct.
[!] : SHOULD - If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[x] : SHOULD - Final provides and requires are sane (rpm -q --provides and rpm
-q --requires).
[x] : SHOULD - Latest version is packaged.
[x] : SHOULD - Package does not include license text files separate from
upstream.
[x] : SHOULD - Patches link to upstream bugs/comments/lists or are otherwise
justified.
[x] : SHOULD - Scriptlets must be sane, if used.
[x] : SHOULD - Description and summary sections in the package spec file
contains translations for supported Non-English languages, if available.
[x] : SHOULD - Package should compile and build into binary rpms on all
supported architectures.
[x] : SHOULD
[Bug 576591] Review Request: iptraf-ng
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=576591 Terje Røsten terje...@phys.ntnu.no changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #28 from Terje Røsten terje...@phys.ntnu.no 2012-01-05 14:56:30 EST --- Thanks, package iptrag-ng 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 749299] Review Request: lcgdm-dav - HTTP/DAV frontend to the DPM/LFC services
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=749299 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|lcgdm-dav-0.5.4-1.fc16 |lcgdm-dav-0.5.4-1.el6 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:31:11 EST --- lcgdm-dav-0.5.4-1.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 726080] Review Request: Xnee - X11 environment recorder
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=726080 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|Xnee-3.10.91-1.fc15 |Xnee-3.11-1.el6 --- Comment #49 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:34:44 EST --- Xnee-3.11-1.el6 has been pushed to the Fedora EPEL 6 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 749299] Review Request: lcgdm-dav - HTTP/DAV frontend to the DPM/LFC services
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=749299 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|lcgdm-dav-0.5.4-1.el6 |lcgdm-dav-0.5.4-1.el5 --- Comment #24 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:35:15 EST --- lcgdm-dav-0.5.4-1.el5 has been pushed to the Fedora EPEL 5 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 755890] Review Request: snap A modular cross-platform system backup/restore utility
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=755890 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|snap-0.6-1.fc15 |snap-0.6-1.el6 --- Comment #38 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:33:09 EST --- snap-0.6-1.el6 has been pushed to the Fedora EPEL 6 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 752311] Review Request: gitso - A front-end to reverse VNC connections
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=752311 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:59:36 EST --- gitso-0.6-6.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/gitso-0.6-6.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 752311] Review Request: gitso - A front-end to reverse VNC connections
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=752311 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:57:00 EST --- pcl-1.3.1-2.fc15 has been pushed to the Fedora 15 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:57:00 EST --- pcl-1.3.1-2.fc15 has been pushed to the Fedora 15 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 702143] Review Request: wallaby - configuration service for Condor pools
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=702143 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:56:54 EST --- wallaby-0.12.4-1.fc15 has been pushed to the Fedora 15 testing 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 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby
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=711229 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:00:12 EST --- ruby-spqr-0.3.5-1.fc15 has been pushed to the Fedora 15 testing 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 752311] Review Request: gitso - A front-end to reverse VNC connections
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=752311 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:03:48 EST --- gitso-0.6-6.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/gitso-0.6-6.fc16 -- 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 752311] Review Request: gitso - A front-end to reverse VNC connections
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=752311 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:01:10 EST --- gitso-0.6-6.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/gitso-0.6-6.fc15 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pcl-1.3.1-2.fc16 Resolution||ERRATA Last Closed||2012-01-05 16:00:40 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:00:40 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pcl-1.3.1-2.fc16 Resolution||ERRATA Last Closed||2012-01-05 16:00:40 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:00:40 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 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 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby
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=711230 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:01:22 EST --- Package ruby-rhubarb-0.4.1-1.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing ruby-rhubarb-0.4.1-1.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-0121/ruby-rhubarb-0.4.1-1.fc16 then log in and leave karma (feedback). -- 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 689488] Review Request: vcftools - VCF file manipulation tools
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=689488 --- Comment #10 from Alex Lancaster al...@users.sourceforge.net 2012-01-05 16:09:27 EST --- koji scratch build worked fine: http://koji.fedoraproject.org/koji/taskinfo?taskID=3623169 I'll try to have a look at this in more detail soon. -- 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 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks
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=771233 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:06:49 EST --- rubygem-rack-protection-1.2.0-2.fc16 has been pushed to the Fedora 16 testing 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 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods
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=771070 --- Comment #10 from Michal Ambroz re...@seznam.cz 2012-01-05 19:08:29 EST --- SPEC URL: http://rebus.fedorapeople.org/SPECS/nwipe.spec SRPM URL: http://rebus.fedorapeople.org/SRPMS/nwipe-0.06-1.fc16.src.rpm Hello, Group field added, Any incorporated FSF address patch and manpage to the upstream package. I have tried building and run on EL5 and it works fine. There is just problem with compatibility if you use srpm from F16 the MD5 signature wont work. Try rpmbuild --nomd5 --nosignature --rebuild http://rebus.fedorapeople.org/SRPMS/nwipe-0.06-1.fc16.src.rpm See here http://koji.fedoraproject.org/koji/taskinfo?taskID=3623485 srpm for this build was created on EL5 so it works fine for the scratchbuild Best regards Michal Ambroz -- 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 769096] Review Request: gnome-shell-extension-windowoverlay-icons - Easily discover which application to select by viewing the app icons in the windows overview.
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=769096 Yader Velásquez yajo...@gmail.com changed: What|Removed |Added URL||rrttr464 -- 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 576591] Review Request: iptraf-ng
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=576591 --- Comment #29 from Nikola Pajkovsky npajk...@redhat.com 2012-01-05 21:29:26 EST --- juuuhuuu! many thanks for review please create someone git repo for f15 till the rawhide -- 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 753855] Review Request: pslib - C-library to create PostScript files
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=753855 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #7 from Parag AN(पराग) panem...@gmail.com 2012-01-05 21:51:56 EST --- Package 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 753855] Review Request: pslib - C-library to create PostScript files
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=753855 Orion Poplawski or...@cora.nwra.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Orion Poplawski or...@cora.nwra.com 2012-01-05 22:03:19 EST --- New Package SCM Request === Package Name: pslib Short Description: C-library to create PostScript files Owners: orion Branches: f16 f15 el6 el5 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 753855] Review Request: pslib - C-library to create PostScript files
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=753855 --- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-01-05 22:06:10 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 769029] Review Request: oxygen-gtk3 - Oxygen GTK+3 theme
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=769029 --- Comment #5 from Kevin Kofler ke...@tigcc.ticalc.org 2012-01-05 22:08:58 EST --- Ping? Any news? -- 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 753855] Review Request: pslib - C-library to create PostScript files
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=753855 Orion Poplawski or...@cora.nwra.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2012-01-05 22:40:20 --- Comment #10 from Orion Poplawski or...@cora.nwra.com 2012-01-05 22:40:20 EST --- Checked in and built. Thanks everyone. -- 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 738589] Review Request: perl-Socket-Netlink - Interface to Linux's PF_NETLINK socket family
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=738589 Mathieu Bridon boche...@fedoraproject.org changed: What|Removed |Added Flag|needinfo?(bochecha@fedorapr | |oject.org) | --- Comment #9 from Mathieu Bridon boche...@fedoraproject.org 2012-01-05 22:55:12 EST --- I've built it for all branches, but I'm having troubles with el6. The package fails to build on ppc64 architecture, I reported the bug upstream: https://rt.cpan.org/Public/Bug/Display.html?id=71112 I tried to disable building in ppc64 in the mean time, but then that caused issues with perl-Socket-Netlink-Route which depends on it: the latter is noarch so I can't do an ExcludeArch on it. I'd like to keep the bug report open as a reminder that I still haven't finished my work on this package, unless that's causing some issues for anyone. Of course, any help in fixing the endianness issue would be appreciated. :) -- 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 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods
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=771070 --- Comment #11 from Ivan Romanov dr...@land.ru 2012-01-05 22:56:00 EST --- Did you increase the version? Add this to changelog too. -- 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 738931] Review Request: perl-Socket-Netlink-Route - Interface to Linux's NETLINK_ROUTE netlink socket protocol
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=738931 Mathieu Bridon boche...@fedoraproject.org changed: What|Removed |Added Flag|needinfo?(bochecha@fedorapr | |oject.org) | --- Comment #9 from Mathieu Bridon boche...@fedoraproject.org 2012-01-05 22:55:41 EST --- See https://bugzilla.redhat.com/show_bug.cgi?id=738589#c9 for why I'm keeping this bug open. -- 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 770986] Review Request: rubygem-kgio - Kinder, gentler I/O for Ruby
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=770986
Mamoru Tasaka mtas...@fedoraproject.org changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #5 from Mamoru Tasaka mtas...@fedoraproject.org 2012-01-05
23:25:33 EST ---
Sorry for being late.
For -3:
* %install vs %check
- build.log shows:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3623729
http://koji.fedoraproject.org/koji/getfile?taskID=3623730name=build.log
Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.SHK0hf
+ umask 022
...
...
+ RUBYOPT='-Ilib
-I/builddir/build/BUILDROOT/rubygem-kgio-2.7.0-3.fc17.x86_64/usr/lib64/ruby/site_ruby/1.8/x86_64-linux
-Itest'
+ testrb test/test_accept_class.rb test/test_accept_flags.rb ...
...
...
255 tests, 132826 assertions, 0 failures, 0 errors
+ rm -f
/builddir/build/BUILDROOT/rubygem-kgio-2.7.0-3.fc17.x86_64/usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/.document
+ rm -f
/builddir/build/BUILDROOT/rubygem-kgio-2.7.0-3.fc17.x86_64/usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/.gitignore
---
So now the lines from rm -rf %{buildroot}%{geminstdir}/.document to
mv %{buildroot}%{geminstdir}/lib/kgio_ext.so
%{buildroot}%{ruby_sitearch}
are now executed on %check (because on the spec file%check is written
above).
Please move the lines
%check
# Patch test_autopush.rb
cd %{buildroot}%{geminstdir}
RUBYOPT=-Ilib -I%{buildroot}%{ruby_sitearch} -Itest testrb test/test_*
to below mv %{buildroot}%{geminstdir}/lib/kgio_ext.so ... line (and above
%clean section).
Please fix the above issue before importing this package
into Fedora git.
This package (rubygem-kgio) is APPROVED by mtasaka
--
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 770755] Review Request: gnome-shell-extension-calc - A simple calculator in the search overview
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=770755 --- Comment #2 from Yader Velásquez yajo...@gmail.com 2012-01-05 23:26:04 EST --- Sorry for the delay, I was on vacation :) I've updated the spec and srpm following your notes. Spec URL: http://yaderv.fedorapeople.org/rpm/gnome-shell-extension-calc/gnome-shell-extension-calc.spec SRPM URL: http://yaderv.fedorapeople.org/rpm/gnome-shell-extension-calc/gnome-shell-extension-calc-0-0.2.git2fca097.fc16.src.rpm Regards -- 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 770796] Review Request: rubygem-gstreamer - Ruby binding of GStreamer
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=770796 Mamoru Tasaka mtas...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-cvs? --- Comment #4 from Mamoru Tasaka mtas...@fedoraproject.org 2012-01-05 23:29:37 EST --- Thank you for reviewing! New Package SCM Request === Package Name: rubygem-gstreamer Short Description: Ruby binding of GStreamer Owners: mtasaka Branches: f16 f15 -- 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 761474] Review Request: ibus-european-table - Predictive text for european languages
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=761474 --- Comment #13 from anish apa...@redhat.com 2012-01-06 01:40:16 EST --- Package Change Request == Package Name: ibus-european-table New Branches: f15 Owners: anishpatil InitialCC:i18n-team -- 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 761474] Review Request: ibus-european-table - Predictive text for european languages
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=761474 anish apa...@redhat.com 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 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 769173] Review Request: english-typing-booster typing booster for english language
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=769173 --- Comment #10 from anish apa...@redhat.com 2012-01-06 01:42:58 EST --- Package Change Request == Package Name: english-typing-booster New Branches: f15 Owners: anishpatil,pravins InitialCC:i18n-team -- 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
