[Bug 580447] Review Request: perl-Module-Build - Build and install Perl modules
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=580447 --- Comment #10 from Marcela Mašláňová mmasl...@redhat.com 2010-06-08 02:45:01 EDT --- (In reply to comment #9) (In reply to comment #6) I wasn't able to add TEST_SIGNATURE, but I fix all other parts. I suppose it's test more for developers of this module anyway. http://mmaslano.fedorapeople.org/review/perl-Module-Build-0.3607-1.fc13.src.rpm http://mmaslano.fedorapeople.org/review/perl-Module-Build.spec I got TEST_SIGNATURE to work by creating my own keypair, sticking it in a tarball and using that: http://trac.city-fan.org/cfo-trac/browser/perl-Module-Build/trunk/perl-Module-Build.spec SRPM containing keypair: http://mirror.city-fan.org/ftp/contrib/perl-modules/SRPMS/perl-Module-Build-0.3607-2.fc14.src.rpm Thanks, I'll use 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 580447] Review Request: perl-Module-Build - Build and install Perl modules
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=580447 --- Comment #11 from Marcela Mašláňová mmasl...@redhat.com 2010-06-08 02:48:59 EDT --- http://mmaslano.fedorapeople.org/review/perl-Module-Build-0.3607-2.fc13.src.rpm http://mmaslano.fedorapeople.org/review/perl-Module-Build.spec Macro was moved to the correct position. -- 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 599316] Review Request: input-pad - Input Pad
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=599316 --- Comment #5 from fujiwara tfuji...@redhat.com 2010-06-08 02:50:45 EDT --- Added .so version info. http://github.com/fujiwarat/input-pad/commit/791ba5f3d22323e525a4153c7b0da272e9bba951 Spec URL: http://fujiwara.fedorapeople.org/ibus/input-pad/input-pad.spec SRPM URL: http://fujiwara.fedorapeople.org/ibus/input-pad/srpms/input-pad-0.1.0.20100608-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2237331 -- 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 585518] Review Request: gtranslator - Gettext po file editor for GNOME
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=585518 --- Comment #13 from Gianluca Sforna gia...@gmail.com 2010-06-08 03:25:02 EDT --- According to https://fedoraproject.org/wiki/Extras/OrphanedPackages yes, we need to file a ticket to rel-eng to unblock the devel branch If a package was last updated more than three months ago (running cvs log -r HEAD *.spec can show you this information), you will need to submit a review request and have the package approved by a reviewer as if it were new to Fedora. See the package review process for more information. There are a couple of small changes though, be sure to submit a 'update' request to 'cvs', and before you will be able to run the final 'make build' commands you will need to file a ticket w/ release engineering to unblock your package https://fedorahosted.org/rel-eng/newticket -- 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 598553] Review Request: perl-PPIx-Regexp - Represent a regular expression of some sort
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=598553 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-06-08 03:32:43 EDT --- perl-PPIx-Regexp-0.007-1.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/perl-PPIx-Regexp-0.007-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 598553] Review Request: perl-PPIx-Regexp - Represent a regular expression of some sort
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=598553 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2010-06-08 03:33:29 EDT --- perl-PPIx-Regexp-0.007-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-PPIx-Regexp-0.007-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 598553] Review Request: perl-PPIx-Regexp - Represent a regular expression of some sort
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=598553 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #13 from Petr Pisar ppi...@redhat.com 2010-06-08 03:34:54 EDT --- Package added and built for F14--F12. -- 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 600238] Review Request: maven-dependency-plugin - Plugin to manipulate, copy and unpack local and remote artifacts
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=600238 Stanislav Ochotnicky socho...@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 585518] Review Request: gtranslator - Gettext po file editor for GNOME
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=585518 --- Comment #14 from Till Maas opensou...@till.name 2010-06-08 03:57:57 EDT --- (In reply to comment #13) According to https://fedoraproject.org/wiki/Extras/OrphanedPackages yes, we need to file a ticket to rel-eng to unblock the devel branch Afaics, unblocking only refers to koji and not the PackageDB. Also the quoted text only refers to make build, which also targets at koji, but PackageDB handles commit access. So I agree that a rel-eng ticket is required, but it is unclear whether this is meant to resolve the devel info in PackageDB. -- 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 592208] Review Request: apache-commons-daemon - rename of jakarta-commons-daemon
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=592208 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #13 from Stanislav Ochotnicky socho...@redhat.com 2010-06-08 04:06:15 EDT --- Package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2237470 Thanks all. -- 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 599316] Review Request: input-pad - Input Pad
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=599316 --- Comment #6 from Parag AN(पराग) panem...@gmail.com 2010-06-08 04:07:02 EDT --- Thanks for the update. Can you fix following rpmlint messages? input-pad.x86_64: E: library-without-ldconfig-postin /usr/lib64/libinput-pad.so.0.0.1 input-pad.x86_64: E: library-without-ldconfig-postun /usr/lib64/libinput-pad.so.0.0.1 input-pad-python.x86_64: W: summary-not-capitalized C input-pad for python input-pad-xtest.x86_64: W: summary-not-capitalized C input-pad with XTEST extension other messages looks ok input-pad.x86_64: E: explicit-lib-dependency libX11 input-pad.x86_64: E: explicit-lib-dependency libxkbfile input-pad.x86_64: W: invalid-url URL: http://code.google.com/p/input-pad/ HTTP Error 404: Not Found input-pad.x86_64: W: shared-lib-calls-exit /usr/lib64/libinput-pad.so.0.0.1 e...@glibc_2.2.5 input-pad.x86_64: W: no-manual-page-for-binary input-pad input-pad-devel.x86_64: W: invalid-url URL: http://code.google.com/p/input-pad/ HTTP Error 404: Not Found input-pad-devel.x86_64: W: no-documentation input-pad-python.x86_64: W: invalid-url URL: http://code.google.com/p/input-pad/ HTTP Error 404: Not Found input-pad-python.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/input-pad-0.1/_input_pad.so _input_pad.so()(64bit) input-pad-python.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/input-pad-0.1/_input_pad_window_gtk.so _input_pad_window_gtk.so()(64bit) input-pad-python.x86_64: W: private-shared-object-provides /usr/lib64/python2.6/site-packages/input-pad-0.1/_input_pad_group.so _input_pad_group.so()(64bit) input-pad-python.x86_64: W: no-documentation input-pad-xtest.x86_64: E: explicit-lib-dependency libXtst input-pad-xtest.x86_64: W: invalid-url URL: http://code.google.com/p/input-pad/ HTTP Error 404: Not Found input-pad-xtest.x86_64: W: no-documentation -- 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 600154] Review Request: plexus-sec-dispatcher - Plexus Security Dispatcher Component
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=600154 huwang huw...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #7 from huwang huw...@redhat.com 2010-06-08 04:10:26 EDT --- Built in koji: http://koji.fedoraproject.org/koji/buildinfo?buildID=177242 -- 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 598058] Review Request: maven-docck-plugin - Maven Documentation Checker Plugin
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=598058 --- Comment #13 from huwang huw...@redhat.com 2010-06-08 04:16:00 EDT --- Built in koji: http://koji.fedoraproject.org/koji/buildinfo?buildID=177244 -- 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 598058] Review Request: maven-docck-plugin - Maven Documentation Checker Plugin
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=598058 huwang huw...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 599291] Review Request: maven-ear-plugin - Maven EAR Plugin
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=599291 huwang huw...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #7 from huwang huw...@redhat.com 2010-06-08 04:21:58 EDT --- Built in koji: http://koji.fedoraproject.org/koji/buildinfo?buildID=177248 -- 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 599316] Review Request: input-pad - Input Pad
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=599316 --- Comment #7 from fujiwara tfuji...@redhat.com 2010-06-08 04:34:52 EDT --- Revised spec file: http://github.com/fujiwarat/input-pad/commit/b5f1e4e738c0bb248d9fb59648ec06d87d58e250 Spec URL: http://fujiwara.fedorapeople.org/ibus/input-pad/input-pad.spec SRPM URL: http://fujiwara.fedorapeople.org/ibus/input-pad/srpms/input-pad-0.1.0.20100608-1.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 598902] Review Request: pxz - Parallel LZMA compressor compatible with XZ
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=598902 Jindrich Novy jn...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #8 from Jindrich Novy jn...@redhat.com 2010-06-08 04:32:34 EDT --- Jussi, Kevin, thanks for cooperation :) -- 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 578480] Review Request: spectrum - XMPP transport/gateway
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=578480
--- Comment #24 from Michal Schmidt mschm...@redhat.com 2010-06-08 04:32:30
EDT ---
(In reply to comment #22)
(In reply to comment #20)
Both approaches are acceptable in general.
I think so, I don't see any obvious packaging error in my approach. Agree?
Agreed.
Do you intend to support EL-4 too? If not, then it's pretty much equivalent
to:
%if 0%{?el5}
Except that didn't work for me on RHEL-5.
Hm, it works for me. Did you use a plain rpmbuild or mock? Both the rhel and
el5 macros are defined in the package buildsys-macros which is installed in
the buildroot by mock (and thus in Koji too).
Fair enough. But I wonder if this part of the Guidelines applies then:
http://fedoraproject.org/wiki/Packaging/Guidelines#Architecture_Build_Failures
Yes, except I have hard time to find the good failed build. It doesn't build
for me on F-12 (https://koji.fedoraproject.org/koji/taskinfo?taskID=2236823),
and there are too many missing packages on RHEL-5/PPC.
I don't think you need to look for any builds. In my understanding of the
guideline it is sufficient to file a bug describing the reason for using
ExcludeArch in your package and making it block a special tracking BZ.
--
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 601115] Review Request: lockfile-progs - safely lock and unlock 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=601115 Matthias Runge mru...@matthias-runge.de changed: What|Removed |Added Depends on||601577 -- 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 601577] New: Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV systems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV systems https://bugzilla.redhat.com/show_bug.cgi?id=601577 Summary: Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV systems 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, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.matthias-runge.de/fedora/liblockfile.spec SRPM URL: http://www.matthias-runge.de/fedora/liblockfile-1.08-1.fc13.src.rpm Description: This library implements a number of functions found in -lmail on SysV systems. These functions are designed to lock the standard mailboxes in /var/mail (or wherever the system puts them). In additions, this library adds a number of functions to create, manage and remove generic lockfiles. -- 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 601577] Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV 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=601577 --- Comment #1 from Matthias Runge mru...@matthias-runge.de 2010-06-08 04:54:14 EDT --- Building liblockfile requires 3 additional perl packages, I'm going to add as review tonight. (Everything found under http://www.matthias-runge.de/fedora/ ), but those perl packages need some work to be accepted, I know. -- 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 589867] Review Request: logcheck - analyzes logfiles and sends email
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=589867 --- Comment #8 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 2010-06-08 05:29:10 EDT --- Mohammed, no, I do not want take it form you! If you plan do it - please. Now I see it have long tail of dependencies instead I simple help you with 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 601585] Review Request: perl-mime-construct - Construct/send MIME messages from the command line
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=601585 Matthias Runge mru...@matthias-runge.de changed: What|Removed |Added Blocks||601577 -- 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 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 --- Comment #8 from Shreyank Gupta shreya...@gmail.com 2010-06-08 05:28:21 EDT --- Updated: Spec URL: http://shreyankg.fedorapeople.org/packaging/lockfile/rubygem-lockfile.spec SRPM URL: http://shreyankg.fedorapeople.org/packaging/lockfile/rubygem-lockfile-1.4.3-3.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 601585] New: Review Request: perl-mime-construct - Construct/send MIME messages from the command line
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-mime-construct - Construct/send MIME messages from the command line https://bugzilla.redhat.com/show_bug.cgi?id=601585 Summary: Review Request: perl-mime-construct - Construct/send MIME messages from the command line 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, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.matthias-runge.de/fedora/perl-mime-construct.spec SRPM URL: http://www.matthias-runge.de/fedora/perl-mime-construct-1.10-1.fc13.src.rpm Description: mime-construct constructs and (by default) mails MIME messages. It is entirely driven from the command line, it is designed to be used by other programs, or people who act like programs. -- 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 601577] Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV 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=601577 Matthias Runge mru...@matthias-runge.de changed: What|Removed |Added Depends on||601585 -- 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 601115] Review Request: lockfile-progs - safely lock and unlock 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=601115 --- Comment #2 from Matthias Runge mru...@matthias-runge.de 2010-06-08 04:50:13 EDT --- I've sent liblockfile as review too, just a few seconds ago. cf. https://bugzilla.redhat.com/show_bug.cgi?id=601577 -- 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 601577] Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV 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=601577 Matthias Runge mru...@matthias-runge.de changed: What|Removed |Added Blocks||601115 -- 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 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 --- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-06-08 05:51:25 EDT --- Well, I have not checked your new spec in detail, however - install command usually sets the permission of the file to be installed to 0755 instead of 0644. Please check if license.txt has 0644 permission. -- 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 601602] New: Review Request: google-collections - Google Collections Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: google-collections - Google Collections Library https://bugzilla.redhat.com/show_bug.cgi?id=601602 Summary: Review Request: google-collections - Google Collections Library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: huw...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://huwang.fedorapeople.org/packages/google-collections/google-collections.spec SRPM URL: http://huwang.fedorapeople.org/packages/google-collections/google-collections-1.0-1.src.rpm Description: The Google Collections Library 1.0 is a set of new collection types, implementations and related goodness for Java 5 and higher, brought to you by Google. It is a natural extension of the Java Collections Framework. Scratch built in koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2237681 -- 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 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 --- Comment #10 from Shreyank Gupta shreya...@gmail.com 2010-06-08 06:03:45 EDT --- Sorry! Thanks for pointing that out: Updated: Spec URL: http://shreyankg.fedorapeople.org/packaging/lockfile/rubygem-lockfile.spec SRPM URL: http://shreyankg.fedorapeople.org/packaging/lockfile/rubygem-lockfile-1.4.3-4.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 --- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-06-08 06:09:19 EDT --- Seems okay. Please import. -- 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 530617] Review Request: libixp - Stand-alone client/server 9P library
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=530617 Simon Wesp cassmod...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX Flag|fedora-review? | --- Comment #11 from Simon Wesp cassmod...@fedoraproject.org 2010-06-08 06:07:09 EDT --- closed, because wmii 3.9 will ship an own copy of the ixp lib. https://fedoraproject.org/wiki/Packaging:Guidelines#Duplication_of_system_libraries It makes more sense to use the shipped version instead. Thank you for your comments! -- 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 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2010-06-08 06:30:46 EDT --- rubygem-lockfile-1.4.3-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/rubygem-lockfile-1.4.3-4.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 527306] Review Request: jericho-html - Jericho HTML Parser
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=527306 --- Comment #2 from Stanislav Ochotnicky socho...@redhat.com 2010-06-08 06:53:25 EDT --- Your spec file and spec file inside src.rpm differ. You will have to fix that before I start official 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 527306] Review Request: jericho-html - Jericho HTML Parser
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=527306 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||socho...@redhat.com AssignedTo|nob...@fedoraproject.org|socho...@redhat.com Flag||fedora-review? --- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2010-06-08 06:52:23 EDT --- I will do the 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 601408] Review Request: techtalk-pse - Presentation software designed for technical people
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=601408
Thomas Spura toms...@fedoraproject.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||toms...@fedoraproject.org
AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org
Flag||fedora-review+
--- Comment #1 from Thomas Spura toms...@fedoraproject.org 2010-06-08
06:57:18 EDT ---
Nice program...
Review:
Good:
- name ok
- license ok
- group ok
- no libs
- no *.la
- arch ok
- BR ok
- %prep ok
- %build ok
- $ rpmlint ./noarch/techtalk-pse-1.0.0-1.fc13.noarch.rpm
./techtalk-pse-1.0.0-1.fc13.src.rpm
techtalk-pse.noarch: W: spelling-error %description -l en_US preloaded -
reloaded, p reloaded, freeloaded
techtalk-pse.src: W: spelling-error %description -l en_US preloaded -
reloaded, p reloaded, freeloaded
2 packages and 0 specfiles checked; 0 errors, 2 warnings.
- builds in koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2237781
Needswork:
- better use %{version} in URL
- could you please use INSTALL=install -p when installing?
- please include the example in %doc, I didn't found a good starting point
online, except the man page an this one example
#
The issues are all SHOULD.
#
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 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes
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=600914 Thomas Spura toms...@fedoraproject.org changed: What|Removed |Added CC||toms...@fedoraproject.org --- Comment #1 from Thomas Spura toms...@fedoraproject.org 2010-06-08 07:23:06 EDT --- This would surely help with Upstream Release Monitoring... Nice program. Review: Good: - sources match upstream: 83bc1f95b36fe51ade8d130fab390103 - name ok - BR ok - %build could be simplified, with a for loop, but ok - locales handled properly - no libs - no *.la - group ok - %install ok - macros everywhere - $ rpmlint ./noarch/txt2regex-0.8-1.fc13.noarch.rpm txt2regex.noarch: W: spelling-error Summary(en_US) regexes - regexps, reg exes, reg-exes txt2regex.noarch: W: spelling-error %description -l en_US Txt - Tet, Ext, Tut txt2regex.noarch: W: spelling-error %description -l en_US regex - regexp, reg ex, reg-ex txt2regex.noarch: W: spelling-error %description -l en_US regexes - regexps, reg exes, reg-exes txt2regex.noarch: W: spelling-error %description -l en_US softwares - software, software's, soft wares txt2regex.noarch: W: spelling-error %description -l en_US egrep - grep, egret, e grep txt2regex.noarch: W: spelling-error %description -l en_US lex - ex, flex, Alex txt2regex.noarch: W: spelling-error %description -l en_US mawk - maw, maws, mask 1 packages and 0 specfiles checked; 0 errors, 8 warnings. - builds in koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2237821 Needswork; - license is unclear: The webside says GPL, the COPYING contains GPLv2. Is it GPLv2 only, GPLv2+ ? Please ask upstream to clearify this and make proper headers in a next release. - %desctiption: When searching for python or emacs or this program will be found, which is not so good... What's your opinion towards this? ## Main blocker: unclear license. -- 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 601636] New: Review Request: sup - A console-based email client written in ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sup - A console-based email client written in ruby https://bugzilla.redhat.com/show_bug.cgi?id=601636 Summary: Review Request: sup - A console-based email client written in ruby Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: shreya...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://shreyankg.fedorapeople.org/packaging/sup/sup.spec SRPM URL: http://shreyankg.fedorapeople.org/packaging/sup/sup-0.10.2-1.fc13.src.rpm Description: Sup is a console-based email client for people with a lot of email. It supports tagging, very fast full-text search, automatic contact-list management, and more. -- 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 591926] Review Request: erlang-mochiweb - An Erlang library for building lightweight HTTP servers
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=591926 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2010-06-08 07:24:51 EDT --- erlang-mochiweb-1.3-0.3.20100507svn159.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/erlang-mochiweb-1.3-0.3.20100507svn159.el5 -- 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 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes
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=600914 Thomas Spura toms...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org 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 598376] Review Request: maven-one-plugin - Maven One Plugin
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=598376 Yang Yong yy...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #11 from Yang Yong yy...@redhat.com 2010-06-08 07:15:26 EDT --- Kevin, cvs is not done after 12 hours passed, may caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-one-plugin Short Description: Plugin provides some integration tasks with Maven 1.x Owners: yyang Branches: devel 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 593959] Review Request: maven-assembly-plugin - Maven Assembly Plugin
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=593959 Yang Yong yy...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Yang Yong yy...@redhat.com 2010-06-08 07:14:15 EDT --- Kevin, cvs is not done after 12 hours passed, may caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-assembly-plugin Short Description: Plugin to create archives from project's descriptors Owners: yyang Branches: devel 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 600236] Review Request: maven-doap-plugin - Plugins which generate a DOAP file from information in a POM
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=600236 --- Comment #7 from Yang Yong yy...@redhat.com 2010-06-08 07:35:22 EDT --- Hi Kevin, cvs is not done yet after 12 hours passed, may be caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-doap-plugin Short Description: Plugins which generate a DOAP file from information in a POM Owners: yyang Branches: devel 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 598891] Review Request: maven-help-plugin - Plugin to to get relative information about a project or the system
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=598891 --- Comment #7 from Yang Yong yy...@redhat.com 2010-06-08 07:31:11 EDT --- Hi Kevin, cvs is not done yet after 12 hours passed, may be caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-help-plugin Short Description: Plugin to get relative information about a project or the system Owners: yyang Branches: devel 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 598916] Review Request: maven-stage-plugin - Plugin to copy artifacts from one repository to another
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=598916 --- Comment #9 from Yang Yong yy...@redhat.com 2010-06-08 07:34:09 EDT --- Hi Kevin, cvs is not done yet after 12 hours passed, may be caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-stage-plugin Short Description: Plugin to copy artifacts from one repository to another system Owners: yyang Branches: devel 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 600236] Review Request: maven-doap-plugin - Plugins which generate a DOAP file from information in a POM
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=600236 Yang Yong yy...@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 600227] Review Request: maven-repository-plugin - Plugin to create bundles of artifacts that should be manually uploaded to the central Maven repository
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=600227 Yang Yong yy...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Yang Yong yy...@redhat.com 2010-06-08 07:36:16 EDT --- Hi Kevin, cvs is not done yet after 12 hours passed, may be caused by wrong CC, please create cvs again and leave CC blank, thanks. New Package CVS Request === Package Name: maven-repository-plugin Short Description: Plugin to create bundles of artifacts for manual uploaded to repository Owners: yyang Branches: devel 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 601641] Review Request: sup - A console-based email client written in 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=601641 Shreyank Gupta shreya...@gmail.com changed: What|Removed |Added Depends on||588461, 598980 -- 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 588461] Review Request: rubygem-net-ssh - A Ruby ssh client library
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=588461 Shreyank Gupta shreya...@gmail.com changed: What|Removed |Added Blocks||601641 -- 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 601641] New: Review Request: sup - A console-based email client written in ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sup - A console-based email client written in ruby https://bugzilla.redhat.com/show_bug.cgi?id=601641 Summary: Review Request: sup - A console-based email client written in ruby Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: shreya...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://shreyankg.fedorapeople.org/packaging/sup/sup.spec SRPM URL: http://shreyankg.fedorapeople.org/packaging/sup/sup-0.10.2-1.fc13.src.rpm Description: Sup is a console-based email client for people with a lot of email. It supports tagging, very fast full-text search, automatic contact-list management, and more. -- 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 581619] Review Request: mlterm - Multi Lingual TERMinal emulator on X
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=581619
--- Comment #13 from Hicham HAOUARI hicham.haou...@gmail.com 2010-06-08
07:49:40 EDT ---
1)Please provide a direct link to the spec everytime you update the package.
2)The %{_libexecdir}/* should be replaced IMHO with the corresponding files
separately.
3)You forgot to update the changelog.
4)BuildRequires : gtk+-devel, better put gtk2-devel
5)Patch should be upstreamed ( put a link to the corresponding upstream
bugzilla bug report )
6)As mentioned above, change sed -i -re 's/#ifdef HAVE_ISASTREAM/#ifdef
HAVE_STROPTS_H/' kiklib/src/kik_pty_streams.c to a patch and upstream it. Same
for fribidi-config
7)The desktop file comment should be changed IMHO :
Use the command line with Arabic support to Use the command line with
multilingual support, and the arabic comment also.
8)It is up to you, but the pixmapdir macro is really unneeded
--
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 601633] Review Request: sup - A console-based email client written in 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=601633 --- Comment #2 from Parag AN(पराग) panem...@gmail.com 2010-06-08 07:52:20 EDT --- *** Bug 601636 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 555990] Review Request: rubygem-acts-as-list - Sorting and reordering capabilities for ActiveRecord 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=555990 Karel Klíč kk...@redhat.com changed: What|Removed |Added Flag|needinfo?(kk...@redhat.com) | --- Comment #8 from Karel Klíč kk...@redhat.com 2010-06-08 07:54:02 EDT --- I imported the package under wrong name (rubygems-acts-as-list). It should be named rubygem-acts-as-list. I'm sorry. Can that be fixed before importing the package into Fedora? I hope we will be able to avoid following https://fedoraproject.org/wiki/Package_Renaming_Process for this new, not-yet-imported package. Maybe file a ticket to rel-eng? -- 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 601641] Review Request: sup - A console-based email client written in 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=601641 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED CC||panem...@gmail.com Resolution||DUPLICATE --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-06-08 07:52:04 EDT --- *** This bug has been marked as a duplicate of bug 601633 *** -- 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 601633] New: Review Request: sup - A console-based email client written in ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sup - A console-based email client written in ruby https://bugzilla.redhat.com/show_bug.cgi?id=601633 Summary: Review Request: sup - A console-based email client written in ruby Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: shreya...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://shreyankg.fedorapeople.org/packaging/sup/sup.spec SRPM URL: http://shreyankg.fedorapeople.org/packaging/sup/sup-0.10.2-1.fc13.src.rpm Description: Sup is a console-based email client for people with a lot of email. It supports tagging, very fast full-text search, automatic contact-list management, and more. --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-06-08 07:52:04 EDT --- *** Bug 601641 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 601636] Review Request: sup - A console-based email client written in 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=601636 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED CC||panem...@gmail.com Resolution||DUPLICATE --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-06-08 07:52:20 EDT --- *** This bug has been marked as a duplicate of bug 601633 *** -- 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 594806] Review Request: jbcrypt - Strong password hashing for Java
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=594806 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added CC||socho...@redhat.com Flag||fedora-review? --- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2010-06-08 08:05:49 EDT --- I will do the review. Before starting official review though, I have one question: Was it really your intention to package java file inside jar? I assume you meant to package .class file. Please fix this before we go any further (quick rpmlint was clean though) -- 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 594806] Review Request: jbcrypt - Strong password hashing for Java
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=594806 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|socho...@redhat.com -- 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 588461] Review Request: rubygem-net-ssh - A Ruby ssh client library
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=588461 Shreyank Gupta shreya...@gmail.com changed: What|Removed |Added Blocks||601633 -- 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 601633] Review Request: sup - A console-based email client written in 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=601633 Shreyank Gupta shreya...@gmail.com changed: What|Removed |Added Depends on||588461, 598980 -- 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 598980] Review Request: rubygem-lockfile - Ruby library for creating NFS safe lockfiles
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=598980 Shreyank Gupta shreya...@gmail.com changed: What|Removed |Added Blocks||601633 -- 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 517488] Review Request: vhostmd - Virtualization host metrics daemon
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=517488 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #23 from Richard W.M. Jones rjo...@redhat.com 2010-06-08 08:54:18 EDT --- I noticed this bug should be closed, since vhostmd has been added to Fedora. -- 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 521319] Review Request: virt-v2v - Convert a virtual machine to run on KVM
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=521319 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #17 from Richard W.M. Jones rjo...@redhat.com 2010-06-08 09:06:49 EDT --- I'm pretty sure this bug should be closed now since virt-v2v has been in Fedora for quite a long time. -- 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 525346] Review Request: papi - Performance Application Programming Interface
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=525346 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #26 from Richard W.M. Jones rjo...@redhat.com 2010-06-08 09:08:40 EDT --- This package is in Fedora now, and so I think this bug should be closed. https://admin.fedoraproject.org/pkgdb/acls/name/papi -- 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 594575] Review Request: mingw32-wxWidgets - MinGW Windows wxWidgets library
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=594575 Bug 594575 depends on bug 594581, which changed state. Bug 594581 Summary: Add mingw32(rpcrt4.dll) in mingw32-filesystem provides for wxWidgets https://bugzilla.redhat.com/show_bug.cgi?id=594581 What|Old Value |New Value Status|NEW |CLOSED Resolution||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 578480] Review Request: spectrum - XMPP transport/gateway
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=578480 --- Comment #25 from Matej Cepl mc...@redhat.com 2010-06-08 09:18:15 EDT --- New packages will not have bugzilla entries during the review process, so they should put this description in the comment until the package is approved, then file the bugzilla entry, and replace the long explanation with the bug number. There are two reasons why this package should have ExcludeArch: PPC for RHEL-5. The really important one is that there is no RHEL-5/Desktop for PPC so there are many libraries (especially and including libpurple) which are not available for PPC. The second is that I just cannot imagine somebody running spectrum on high-end IBM PowerPC servers, so there is not much motivation to even investigate possibility of building it on PPC. -- 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 601160] Review Request: rubygem-rdiscount - Fast Implementation of Gruber's Markdown in C
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=601160 Gerd Pokorra g...@zimt.uni-siegen.de changed: What|Removed |Added Flag||fedora-review? --- Comment #2 from Gerd Pokorra g...@zimt.uni-siegen.de 2010-06-08 09:30:40 EDT --- Thank you for the first look. Would you please make a second look? I did not increment the revision (release) number. I only overwrite the files at Spec URL: ftp://ftp.uni-siegen.de/pub/review/rubygem-rdiscount.spec and SRPM URL: ftp://ftp.uni-siegen.de/pub/review/rubygem-rdiscount-1.6.3.2-1.fc12.src.rpm The URL from the scratch build of the current files is: http://koji.fedoraproject.org/koji/taskinfo?taskID=2238352 I do the best I can for Enabling tests. May be you know a better solution for that. -- 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 555990] Review Request: rubygem-acts-as-list - Sorting and reordering capabilities for ActiveRecord 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=555990 --- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-06-08 09:47:03 EDT --- Ah, I guess rel-eng team will help this case if you file a ticket for rel-eng. -- 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 601160] Review Request: rubygem-rdiscount - Fast Implementation of Gruber's Markdown in C
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=601160 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Flag|fedora-review? | --- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-06-08 09:49:35 EDT --- fedora-review flag must be set by reviewer (assignee), once resetting. Also would you bump release number to avoid confusion? (and bumping release number is helpful when using diff) -- 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 601633] Review Request: sup - A console-based email client written in 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=601633 --- Comment #4 from Shreyank Gupta shreya...@gmail.com 2010-06-08 10:14:21 EDT --- (In reply to comment #3) Would you update rubygem-net-ssh review request? I was talking to the guy who did the review request for rubygem-net-ssh. Could I add a new specfile/srpm on behalf of the requestor? Or do I have to wait until he doesn't respond? Also where is rubygem(xapian-full)? I will package it as soon as rubygems.org comes back up. I was hoping I could use xapian-bindings-ruby inplace of rubygem(xapian-full), but as it turns out I can't. -- 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 581619] Review Request: mlterm - Multi Lingual TERMinal emulator on X
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=581619 --- Comment #15 from Muayyad Alsadi als...@ojuba.org 2010-06-08 10:19:49 EDT --- 7)The desktop file comment should be changed IMHO : Use the command line with Arabic support to Use the command line with multilingual support, and the arabic comment also. actually (from my fedora IRC talks with users) it's intended for Arabic users, since Chinese and Japanese prefer another terminal emulator. -- 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 581619] Review Request: mlterm - Multi Lingual TERMinal emulator on X
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=581619
--- Comment #14 from Muayyad Alsadi als...@ojuba.org 2010-06-08 10:17:54 EDT
---
1) Please provide a direct link to the spec everytime you update the package.
no problem, I'll do that
2)The %{_libexecdir}/* should be replaced IMHO with the corresponding files
separately.
is there a reason for that ? they are files that might change from version to
another
3)You forgot to update the changelog.
ok
4)BuildRequires : gtk+-devel, better put gtk2-devel
we can't do that, the config window of mlterm is a gtk1 application
5)Patch should be upstreamed ( put a link to the corresponding upstream
bugzilla bug report )
if you are referring to your patch in #11 then it just -lx11 to build option,
it could be missed because of many reasons, and BTW it used to build in F11 and
F12 without this patch, so if the upstream is using fedora13 he won't be
convinced that there is any problem
6)As mentioned above, change sed -i -re 's/#ifdef HAVE_ISASTREAM/#ifdef
HAVE_STROPTS_H/' kiklib/src/kik_pty_streams.c to a patch and upstream it. Same
for fribidi-config
the fribidi-config patch is no longer needed that's why it's commented (it was
needed because the upstream used to use fribidi-config instead of pkg-config
which is in fedora)
and the sed change is needed to build it, try building without it.
the function it uses isastream is from stropts.h I guess both are not in
fedora so HAVE_STROPTS_H is equivalent to HAVE_ISASTREAM
http://www.opengroup.org/onlinepubs/009695399/functions/isastream.html
--
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 555990] Review Request: rubygem-acts-as-list - Sorting and reordering capabilities for ActiveRecord 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=555990 --- Comment #10 from Karel Klíč kk...@redhat.com 2010-06-08 10:28:11 EDT --- https://fedorahosted.org/rel-eng/ticket/3782 -- 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 532524] Review request: jcifs - Common Internet File System Client in 100% Java
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=532524 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||socho...@redhat.com AssignedTo|nob...@fedoraproject.org|socho...@redhat.com Flag||fedora-review? --- Comment #1 from Stanislav Ochotnicky socho...@redhat.com 2010-06-08 10:30:39 EDT --- I'll do the review, but would you mind updating to latest version first? As far as I can see, upstream keeps only latest version of sources available for each main version. It might be a good idea to contact them about it. It makes verifying sources quite hard. -- 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 601408] Review Request: techtalk-pse - Presentation software designed for technical people
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=601408 Ian Weller i...@ianweller.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ian Weller i...@ianweller.org 2010-06-08 11:02:33 EDT --- Thanks, Thomas! I fixed the needswork stuff. New Package CVS Request === Package Name: techtalk-pse Short Description: Presentation software designed for technical people Owners: ianweller Branches: F-12 F-13 EL-6 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 601771] New: Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series Adapters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series Adapters https://bugzilla.redhat.com/show_bug.cgi?id=601771 Summary: Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series Adapters Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: linvi...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://linville.fedorapeople.org/iwl6050-firmware.spec SRPM URL: http://linville.fedorapeople.org/iwl6050-firmware-9.201.4.1-1.fc14.src.rpm Description: This package contains the firmware required by the iwlagn driver for Linux to support the iwl6050 hardware. Usage of the firmware is subject to the terms and conditions contained inside the provided LICENSE file. Please read it carefully. -- 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 587438] Review Request: rubygem-snmp - A Ruby implementation of SNMP (the Simple Network Management 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=587438 Michael Stahnke mastah...@gmail.com changed: What|Removed |Added Flag|needinfo?(mastah...@gmail.c | |om) | --- Comment #12 from Michael Stahnke mastah...@gmail.com 2010-06-08 11:09:44 EDT --- I was working on the testing with upstream a bit. Tests seem to work on Mac OS so I don't think upstream is too interested in fixing. I will apply patch provided and run tests that way (rather than with Rake). -- 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 601771] Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series 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=601771 Bill Nottingham nott...@redhat.com changed: What|Removed |Added 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 601771] Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series 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=601771
Bill Nottingham nott...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #1 from Bill Nottingham nott...@redhat.com 2010-06-08 13:06:33
EDT ---
Well, it appears to just be s/6000/6050/ on the iwl6000-firmware package. But
still, for completeness sake...
MUST items:
- Package meets naming and packaging guidelines - ok
- Spec file matches base package name. - ok
- Spec has consistant macro usage. - ok
- Meets Packaging Guidelines. - ok
- License - standard firmware license
- License field in spec matches - Redistributable, no modification permitted
- License file included in package - ok
- Spec in American English - ok
- Spec is legible. - ok
- Sources match upstream md5sum:
1cbca537d7c9af2c8d1f40b00cb430055b6f06de4b6d29e2a318bb0eae3352ff
iwlwifi-6050-ucode-9.201.4.1.tgz
OK
- Package needs ExcludeArch - N/A
- BuildRequires correct - OK
- Spec handles locales/find_lang - N/A
- Package is relocatable and has a reason to be. - N/A
- Package has %defattr and permissions on files is good. - OK
- Package has a correct %clean section. - OK
- Package has correct buildroot - OK
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
- Package is code or permissible content. - oK
- Doc subpackage needed/used. - N/A
- Packages %doc files don't affect runtime. - N/A
- Package compiles and builds on at least one arch. - OK (tested x86_64)
- Package has no duplicate files in %files. - OK
- Package doesn't own any directories other packages own. - OK
- Package owns all the directories it creates. -OK
- No rpmlint output. - ***
iwl6050-firmware.noarch: W: spurious-executable-perm
/usr/share/doc/iwl6050-firmware-9.201.4.1/README
iwl6050-firmware.noarch: W: spurious-executable-perm
/usr/share/doc/iwl6050-firmware-9.201.4.1/LICENSE
That can be fixed, but it's not critical.
- final provides and requires are sane: - OK
SHOULD Items:
- Should build in mock. - didn't try
- Should build on all supported archs - didn't try
- Should function as described. - OK (tested with 6250 part)
- Should have dist tag - OK
- Should package latest version - OK
APPROVED. Feel free to fix the docs perms.
--
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 601160] Review Request: rubygem-rdiscount - Fast Implementation of Gruber's Markdown in C
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=601160 --- Comment #4 from Gerd Pokorra g...@zimt.uni-siegen.de 2010-06-08 13:11:12 EDT --- I am sorry for the mistake. I was in a hurry because I had an appointment and my last review request is some time ago. I increment the release number in the spec file, it is still stored under the URL: ftp://ftp.uni-siegen.de/pub/review/rubygem-rdiscount.spec The new SRPM is at: ftp://ftp.uni-siegen.de/pub/review/rubygem-rdiscount-1.6.3.2-2.fc12.src.rpm Scratch build URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=2238903 -- 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 601771] Review Request: iwl6050-firmware - Firmware for Intel(R) Wireless WiFi Link 6050 Series 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=601771 John W. Linville linvi...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from John W. Linville linvi...@redhat.com 2010-06-08 13:25:10 EDT --- New Package CVS Request === Package Name: iwl6050-firmware Short Description: Firmware for Intel(R) Wireless WiFi Link 6050 Series Adapters Owners: linville Branches: F-12 F-13 InitialCC: linville -- 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Alias||FE-NEEDSPONSER -- 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) |505154(FE-SCITECH) -- 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added CC||steve.tray...@cern.ch --- Comment #4 from Steve Traylen steve.tray...@cern.ch 2010-06-08 14:18:24 EDT --- This is an informal review, as I understand from a mail you are seeking sponsership and should block this bug on FE-NEEDSPONSER accordingly. Just looking at the .spec file for now 1) The .la should be removed somehow. See: http://fedoraproject.org/wiki/PackagingGuidelines#Packaging_Static_Libraries 2) Your release tag is 2.1 which is odd, it typically starts at 1. :-) See: http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Package_Version In a nutshell it's the number of .spec file modifications or builds in fact since the Version was last update. 3) Not important I would say but in principal packages take the name of the tar ball so JAGS but I would never not approve based on that. http://fedoraproject.org/wiki/Packaging:NamingGuidelines 4) BuildRequires: gcc-c++ certainly is not needed, see http://fedoraproject.org/wiki/PackagingGuidelines#Exceptions_2 5) I see you have a .tex file in your docs have you considered converting it to something more readable such as ps or pdf. Steve. -- 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Blocks|505154(FE-SCITECH) |177841(FE-NEEDSPONSOR) -- 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Alias|FE-NEEDSPONSER | -- 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 505154] Tracker: Review Requests for Science and Technology related packages
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=505154 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Depends on|600467 | -- 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 505154] Tracker: Review Requests for Science and Technology related packages
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=505154 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Depends on||600467 Bug 505154 depends on bug 575541, which changed state. Bug 575541 Summary: Review Request: xcalc - Scientific Calculator X11 Client https://bugzilla.redhat.com/show_bug.cgi?id=575541 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |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 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes
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=600914 --- Comment #2 from Richard Fearn richardfe...@gmail.com 2010-06-08 14:19:40 EDT --- Thanks for the review Thomas! - %build could be simplified, with a for loop, but ok I don't know why I didn't do that before! That will be in the next release. - license is unclear: The webside says GPL, the COPYING contains GPLv2. Is it GPLv2 only, GPLv2+ ? Please ask upstream to clearify this and make proper headers in a next release. I have asked. - %desctiption: When searching for python or emacs or this program will be found, which is not so good... What's your opinion towards this? I see your point. I've removed the list of programs now. This means that if support for any new programs is added, the description won't need to be modified. I also fixed a few typos. The current description is: txt2regex is a regular expression wizard that converts human sentences to regexes. In a simple interactive interface, you just answer questions and build your own regex for a large variety of software and programming languages. I'll upload a new spec/SRPM when I've heard from upstream. Thanks for your 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 600467] Review Request: jags - Just Another Gibbs Sampler
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=600467 chris desjardins cddesjard...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 593851] Review Request: clide - Color and style highlighting program for text
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=593851
--- Comment #4 from Martin Gieseking martin.giesek...@uos.de 2010-06-08
14:28:43 EDT ---
Richard, the package looks almost fine to me. I suggest to explicitly assign
values to BINDIR and MANDIR in make rpminstall, e.g.
make BINDIR=%{buildroot}%{_bindir} \
MANDIR=%{buildroot}%{_mandir}/man1 \
rpminstall
to ensure the files will be installed in the proper (rpm-defined) directories.
--
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 593851] Review Request: clide - Color and style highlighting program for text
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=593851
--- Comment #5 from Richard Fearn richardfe...@gmail.com 2010-06-08 14:44:21
EDT ---
Richard, the package looks almost fine to me. I suggest to explicitly assign
values to BINDIR and MANDIR in make rpminstall, e.g.
make BINDIR=%{buildroot}%{_bindir} \
MANDIR=%{buildroot}%{_mandir}/man1 \
rpminstall
to ensure the files will be installed in the proper (rpm-defined)
directories.
Good suggestion - given that the rpminstall target only uses BINDIR/MANDIR*,
it certainly makes things clearer.
* I remove the doc file install (which uses DOCDIR) in clide-Makefile.patch.
New spec/SRPM:
http://richardfearn.fedorapeople.org/clide/0.9-3/clide.spec
http://richardfearn.fedorapeople.org/clide/0.9-3/clide-0.9-3.fc13.src.rpm
Thanks!
--
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 601037] Review Request: perl-MooseX-SetOnce - Write-once, read-many attributes for Moose
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=601037 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |ON_QA --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-06-08 15:25:27 EDT --- perl-MooseX-SetOnce-0.100471-1.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-MooseX-SetOnce'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-MooseX-SetOnce-0.100471-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 601036] Review Request: perl-MooseX-LazyRequire - Required attributes which fail only when trying to use them
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=601036 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |ON_QA --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-06-08 15:29:46 EDT --- perl-MooseX-LazyRequire-0.05-1.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-MooseX-LazyRequire'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-MooseX-LazyRequire-0.05-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 601038] Review Request: perl-MooseX-Types-Perl - Moose types that check against Perl syntax
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=601038 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |ON_QA --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-06-08 15:31:11 EDT --- perl-MooseX-Types-Perl-0.101340-1.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-MooseX-Types-Perl'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-MooseX-Types-Perl-0.101340-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 578759] Review request: django-mako - Mako Templates Plugin 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=578759 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||django-mako-0.1.4-0.1.pre.f ||c13 Resolution||ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 593125] Review Request: gridsite - Grid Security for the Web, Web platforms for Grids
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=593125 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||gridsite-1.5.18-4.fc13 Resolution||ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 578759] Review request: django-mako - Mako Templates Plugin 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=578759 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-06-08 15:33:05 EDT --- django-mako-0.1.4-0.1.pre.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 593125] Review Request: gridsite - Grid Security for the Web, Web platforms for Grids
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=593125 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2010-06-08 15:36:01 EDT --- gridsite-1.5.18-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
