[Bug 580480] Review Request: ghc-type-level - A type-level library for Haskell
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=580480 --- Comment #3 from Jens Petersen 2010-07-05 02:52:50 EDT --- For reference: type-level is needed by ForSyDe. -- 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 602279] Review Request: LibRaw - Library for reading RAW files obtained from digital photo cameras
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=602279
--- Comment #18 from Siddhesh Poyarekar 2010-07-05
02:12:53 EDT ---
(In reply to comment #17)
> OK, if you intend to package for Fedora, you can remove BuildRoot:
> %{_tmppath}/%{name}-%{version}-%{release}-root and rm -rf %{buildroot} as
> well.
Ok thanks, I'll do this in the next revision once the license details are
sorted out..
> LGPLv2.1 is not enough for LibRaw.
> See
> http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Dual_Licensing_Scenarios
>
Thanks for clarifying. I've sent an email to legal list asking for direction on
this:
http://lists.fedoraproject.org/pipermail/legal/2010-July/001313.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 603633] Review Request: scholarsfonts-cardo-fonts - This is designed for classicists, Biblical scholars, medievalists, linguists
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=603633 Parag changed: What|Removed |Added Flag||needinfo? --- Comment #4 from Parag 2010-07-05 01:19:43 EDT --- Thanks for the review! New Package CVS Request === Package Name: scholarsfonts-cardo-fonts Short Description: This is designed for classicists, Biblical scholars, medievalists, linguists Owners: pnemade Branches: InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603632] Review Request: typemade-josefinsansstd-light-fonts - This typeface was to made for geometric, elegant titling and kind of vintage
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=603632 Parag changed: What|Removed |Added Flag||needinfo? --- Comment #12 from Parag 2010-07-05 01:19:08 EDT --- Thanks for the review! New Package CVS Request === Package Name: typemade-josefinsansstd-light-fonts Short Description: This typeface was to made for geometric, elegant titling and kind of vintage Owners: pnemade Branches: InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603629] Review Request: cf-sorts-mill-goudy-fonts - Goudy Oldstyle and Italic fonts
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=603629 Parag changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Parag 2010-07-05 01:18:34 EDT --- Thanks for the review! New Package CVS Request === Package Name: cf-sorts-mill-goudy-fonts Short Description: Goudy Oldstyle and Italic fonts Owners: pnemade Branches: InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611175] Review Request: yash - Yet Another SHell
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=611175 Mamoru Tasaka changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #9 from Mamoru Tasaka 2010-07-05 00:45:45 EDT --- Rebuilt done on all branches, requested push on bodhi. Thank you for reviewing this package and CVS procedure, closing. -- 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 581220] Review Request: qtsingleapplication - Qt library to start applications only once per user
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=581220 Orcan 'oget' Ogetbil changed: What|Removed |Added Flag||fedora-cvs? --- Comment #19 from Orcan 'oget' Ogetbil 2010-07-04 23:18:40 EDT --- Thanks a bunch Kalev! New Package CVS Request === Package Name: qtsingleapplication Short Description: Qt library to start applications only once per user Owners: oget Branches: F-12 F-13 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 --- Comment #5 from Fedora Update System 2010-07-04 23:17:17 EDT --- perl-Devel-PartialDump-0.13-1.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/perl-Devel-PartialDump-0.13-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 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 --- Comment #6 from Fedora Update System 2010-07-04 23:17:22 EDT --- perl-Devel-PartialDump-0.13-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-Devel-PartialDump-0.13-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 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 Iain Arnell changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 602279] Review Request: LibRaw - Library for reading RAW files obtained from digital photo cameras
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=602279
--- Comment #17 from Chen Lei 2010-07-04 22:53:13 EDT
---
(In reply to comment #15)
> Updated:
>
> http://siddhesh.fedorapeople.org/LibRaw/0.9.1-6/LibRaw.spec
> http://siddhesh.fedorapeople.org/LibRaw/0.9.1-6/LibRaw-0.9.1-6.fc14.src.rpm
>
> (In reply to comment #13)
>
> > 2. rpmlint LibRaw-0.9.1-5.fc14.src.rpm
> > LibRaw.src: W: spelling-error %description -l en_US dcraw -> draw, craw, d
> > craw
> > LibRaw.src: W: no-cleaning-of-buildroot %clean
> > LibRaw.src: W: no-%clean-section
> > 1 packages and 0 specfiles checked; 0 errors, 3 warnings.
> >
> > %clean-section is still needed for F12 and below.
>
> I intend to package for F-13 or later, so I followed the suggestion in comment
> 2
OK, if you intend to package for Fedora, you can remove BuildRoot:
%{_tmppath}/%{name}-%{version}-%{release}-root and rm -rf %{buildroot} as well.
> > 6. License: LGPLv2+ with exceptions
> > Please explain why you use this license for LibRaw.
> > From website, I see the license is LGPLv2+ or CDDL or LibRaw
> >
> > Please try to contact fedora-legal to add LibRaw license for fedora
> >
> > See http://fedoraproject.org/wiki/Licensing#Software_License_List
>
> Fixed this to LGPLv2. I had misread the license table in
> http://fedoraproject.org/wiki/Licensing
>
> The library is distributed under all those licenses, but the website also
> mentions that "You may choose license you like more from these three.". I
> chose
> to go with LGPLv2.1 as a result.
LGPLv2.1 is not enough for LibRaw.
See
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Dual_Licensing_Scenarios
--
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 602279] Review Request: LibRaw - Library for reading RAW files obtained from digital photo cameras
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=602279 --- Comment #16 from Chen Lei 2010-07-04 22:43:30 EDT --- (In reply to comment #14) > fwiw, 1. is mostly harmless, the last flag wins, so -O4 -O2 ends up -O2 Thanks for clarification :), since shotwell 0.6.1 is released, I'll review this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 602279] Review Request: LibRaw - Library for reading RAW files obtained from digital photo cameras
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=602279 Chen Lei changed: What|Removed |Added Status|NEW |ASSIGNED 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 611372] New: Review Request: perl-Net-Twitter - Perl interface to Twitter
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-Net-Twitter - Perl interface to Twitter https://bugzilla.redhat.com/show_bug.cgi?id=611372 Summary: Review Request: perl-Net-Twitter - Perl interface to Twitter Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jd...@aquezada.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.aquezada.com/staff/julian/devel/fedora/perl-Net-Twitter.spec SRPM URL: http://www.aquezada.com/staff/julian/devel/fedora/perl-Net-Twitter-3.13006-1.fc13.src.rpm Description: This is my first package so I'm seeking a sponsor for it. Net::Twitter is a Perl API to Twitter. In order to get this to build, I had to rebuild Net::OAuth (perl-Net-OAuth) for version 1.27. I've put the SPEC and SRPM for that here: http://www.aquezada.com/staff/julian/devel/fedora/perl-Net-OAuth.spec http://www.aquezada.com/staff/julian/devel/fedora/perl-Net-OAuth-0.27-3.fc13.src.rpm Not sure if I need to open a separate bugzilla issue 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 611372] Review Request: perl-Net-Twitter - Perl interface to Twitter
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=611372 Julian C. Dunn 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 611175] Review Request: yash - Yet Another SHell
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=611175 --- Comment #8 from Kevin Fenzi 2010-07-04 21:47:17 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 --- Comment #4 from Kevin Fenzi 2010-07-04 21:46:12 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension
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=561981 --- Comment #9 from Kevin Fenzi 2010-07-04 21:43:16 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 602587] Review Request: perl-IPC-Signal - Utility functions dealing with signals
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=602587 --- Comment #19 from Kevin Fenzi 2010-07-04 21:43:52 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561985] Review Request: php-phpunit-phpdcd - Dead Code Detector (DCD) for PHP code
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=561985 --- Comment #9 from Kevin Fenzi 2010-07-04 21:43:36 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 555843] Review Request: pam_radius - PAM Module for RADIUS Authentication
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=555843 --- Comment #30 from Kevin Fenzi 2010-07-04 21:42:57 EDT --- There is already a EL-6 branch. Make sure you 'cvs update -d' to get the new directory. -- 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 607778] Review Request: xnoise-plugins-core - Core plugins for xnoise
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=607778 --- Comment #7 from Kevin Fenzi 2010-07-04 21:44:31 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 517643] Review Request: php-doctrine-Doctrine - PHP Object Relational Mapper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=517643 --- Comment #21 from Kevin Fenzi 2010-07-04 21:41:01 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 546020] Review Request: php-swift-Swift - Free Feature-rich PHP Mailer
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=546020 --- Comment #9 from Kevin Fenzi 2010-07-04 21:41:16 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
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=551721 --- Comment #13 from Kevin Fenzi 2010-07-04 21:41:27 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 473184] Review Request: clamz - Amazon Downloader
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=473184 --- Comment #30 from Kevin Fenzi 2010-07-04 21:40:09 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551723] Review Request: php-pdepend-PHP-Depend - PHP_Depend design quality metrics for PHP package
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=551723 --- Comment #14 from Kevin Fenzi 2010-07-04 21:41:56 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
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=551722 --- Comment #12 from Kevin Fenzi 2010-07-04 21:41:44 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 231772] Review Request: Siege - Siege is an http regression testing and benchmarking utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=231772 --- Comment #6 from Kevin Fenzi 2010-07-04 21:39:02 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 517641] Review Request: php-channel-doctrine - Adds doctrine project channel to PEAR
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=517641 --- Comment #18 from Kevin Fenzi 2010-07-04 21:40:35 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551724] Review Request: php-phpmd-PHP-PMD - PHPMD - PHP Mess Detector
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=551724 --- Comment #16 from Kevin Fenzi 2010-07-04 21:42:10 EDT --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 191239] Review Request: qjackctl - Qt based JACK control application
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=191239 --- Comment #25 from Kevin Fenzi 2010-07-04 21:38:22 EDT --- Have you asked if the current Fedora maintainer would like to maintain in EPEL? Please do so, if they don't want to, or don't reply in a week we can make the EPEL branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 549296] Review Request: lshell - Python-based limited Shell
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=549296 --- Comment #11 from Fedora Update System 2010-07-04 18:45:23 EDT --- lshell-0.9.12-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/lshell-0.9.12-2.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 549296] Review Request: lshell - Python-based limited Shell
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=549296 --- Comment #10 from Fedora Update System 2010-07-04 18:45:18 EDT --- lshell-0.9.12-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/lshell-0.9.12-2.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 581220] Review Request: qtsingleapplication - Qt library to start applications only once per user
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=581220
Kalev Lember changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #18 from Kalev Lember 2010-07-04 18:29:43 EDT
---
Fedora review qtsingleapplication-2.6.1-1.fc13.src.rpm 2010-07-05
+ OK
! needs attention
rpmlint output:
qtsingleapplication.src:60: W: configure-without-libdir-spec
qtsingleapplication.i686: W: unused-direct-shlib-dependency
/usr/lib/libQtSolutions_SingleApplication-2.6.so.1.0.0 /lib/libm.so.6
4 packages and 0 specfiles checked; 0 errors, 2 warnings.
+ The package is named according to the Package Naming Guidelines.
+ Spec file name matches the base package name
+ The package follows the Packaging Guidelines
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The license field in the spec file matches the actual license
+ The package contains license files (LGPL_EXCEPTION.txt, LICENSE.GPL3, and
LICENSE.LGPL)
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm. md5sum:
902795eb13ecedbdc112f00d7ec22949 qtsingleapplication-2.6_1-opensource.tar.gz
902795eb13ecedbdc112f00d7ec22949
Download/qtsingleapplication-2.6_1-opensource.tar.gz
+ The package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
n/a The spec file MUST handle locales properly
+ ldconfig is properly called in %post and %postun
+ Package does not bundle copies of system libraries
n/a Does not use Prefix: /usr
+ Package owns all directories it creates
+ No duplicate files in %files
+ Permissions are properly set and %files has %defattr
+ Consistent use of macros
+ The package must contain code, or permissable content.
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc don't affect the package
+ Header files are in -devel
n/a Static libraries should be in -static
+ Library files that end in .so are in -devel package
+ -devel requires the fully versioned base
+ Package doesn't contain any libtool .la files
n/a Packages containing GUI apps must include %{name}.desktop file
+ Directory ownership sane
+ Filenames are valid UTF-8
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 607778] Review Request: xnoise-plugins-core - Core plugins for xnoise
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=607778 Michel Alexandre Salim changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Michel Alexandre Salim 2010-07-04 17:51:30 EDT --- New Package CVS Request === Package Name: xnoise-plugins-core Short Description: Core plugins for xnoise Owners: salimma Branches: F-12 F-13 InitialCC: (In reply to comment #5) > JFTR, the review guidelines say: > > | SHOULD: The reviewer should test that the package functions as > | described. A package should not segfault instead of running, for example. > Apologies there. The plugins were in working order at some point before 0.1.6 came out, but I still should have not assumed that's still the case with the released versions. I guess xnoise is still in flux and the plugins lag a bit behind xnoise itself. > We could go on with even more disabled plugins. Eventually, no plugin may be > left. ;) Or old/new segfaults might return with one of the next updates after > package approval. > > Whether 0.1.6 is ready for Rawhide, I leave this decision to you. It's ready for Rawhide, I guess, but I'd probably keep it there and not push it to F-12 and F-13 until they are more stable. > > [...] > > About the .pc file: Consider merging this dubious -devel subpackage into the > xnoise-plugins-core package. Possibly with a virtual -devel pkg or only when > you learn about something that would need such a BuildRequires. Rationale: Good idea; will delete it for now -- 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 611328] Review Request: hydra - Fast login cracker
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=611328 Marcus Haebler 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 611328] New: Review Request: hydra - Fast login cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: hydra - Fast login cracker https://bugzilla.redhat.com/show_bug.cgi?id=611328 Summary: Review Request: hydra - Fast login cracker Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: haeb...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sites.google.com/site/haebler/home/fedora/hydra.spec?attredirects=0&d=1 SRPM URL: http://sites.google.com/site/haebler/home/fedora/hydra-5.7-0.fc12.src.rpm?attredirects=0&d=1 Description: Number one of the biggest security holes are passwords, as every password security study shows. Hydra is a parallelized login cracker which supports numerous protocols to attack. New modules are easy to add, beside that, it is flexible and very fast. Currently this tool supports: TELNET, FTP, HTTP, HTTPS, HTTP-PROXY, SMB, SMBNT, MS-SQL, MYSQL, REXEC, RSH, RLOGIN, CVS, SNMP, SMTP-AUTH, SOCKS5, VNC, POP3, IMAP, NNTP, PCNFS, ICQ, SAP/R3, LDAP2, LDAP3, Postgres, Teamspeak, Cisco auth, Cisco enable, LDAP2, Cisco AAA (incorporated in telnet module). This tool is a proof of concept code, to give researchers and security consultants the possibility to show how easy it would be to gain unauthorized access from remote to a system. Hydra modules not available in this RPM: SVN: Module uses deprecated library calls; SVN also needs APR during build, Hydra's configure script is broken for APR 1.x NCP: Required library is not available in the current Fedora repository SAP/R3: Required library is not available in the current Fedora repository. IMPORTANT: - This is my first package. I am looking for a sponsor. - TIA - I currently don't have hosting space on fedorapeople.org. - The license of Hydra has been changed to GPLv3 in the latest version 5.7!!! This should make Hydra compatible with Fedora licensing requirements. -- 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 581220] Review Request: qtsingleapplication - Qt library to start applications only once per user
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=581220
--- Comment #17 from Orcan 'oget' Ogetbil 2010-07-04
17:38:53 EDT ---
SPEC: http://oget.fedorapeople.org/review/qtsingleapplication.spec
SRPM:
http://oget.fedorapeople.org/review/qtsingleapplication-2.6.1-1.fc13.src.rpm
Changelog: 2.6.1-1
- Change version to 2.6.1. Upstream uses weird version convention 2.6_1
- Own the directory %%{_qt4_headerdir}/QtSolutions/
--
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 607778] Review Request: xnoise-plugins-core - Core plugins for xnoise
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=607778 Michael Schwendt changed: What|Removed |Added Flag||fedora-review+ --- Comment #5 from Michael Schwendt 2010-07-04 16:59:10 EDT --- Okay. The segfaults in the plugins will be left to you anyway. There are more, e.g. when starting from scratch and enabling "Leoslyrics". ABRT doesn't catch them: $ rm -rf ~/.config/xnoise $ xnoise LastfmCovers plugin failed to activate! Mediawatcher plugin failed to activate! *** glibc detected *** xnoise: malloc(): smallbin double linked list corrupted: 0x015f5f50 *** $ rm -rf ~/.config/xnoise $ xnoise LastfmCovers plugin failed to activate! Mediawatcher plugin failed to activate! closing... Segmentation fault (core dumped) JFTR, the review guidelines say: | SHOULD: The reviewer should test that the package functions as | described. A package should not segfault instead of running, for example. We could go on with even more disabled plugins. Eventually, no plugin may be left. ;) Or old/new segfaults might return with one of the next updates after package approval. Whether 0.1.6 is ready for Rawhide, I leave this decision to you. [...] About the .pc file: Consider merging this dubious -devel subpackage into the xnoise-plugins-core package. Possibly with a virtual -devel pkg or only when you learn about something that would need such a BuildRequires. Rationale: xnoise-plugins-core builds the plugins with the .pc file in xnoise-devel, so external plugins will likely use that .pc file, too. [...] Other issues have been dealt with. As the xnoise-plugins-core-devel pkg may return after the review process or be killed again in future updates, it isn't necessary to insist on its removal during this review. 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 537640] Review Request: pdfmod - A simple application for modifying PDF documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=537640 Christian Krause changed: What|Removed |Added CC||c...@plauener.de Depends on||530910 -- 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 530910] Review Request: hyena - Hyena is a library of GUI and non-GUI C# code
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=530910 Christian Krause changed: What|Removed |Added Blocks||537640 -- 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 579171] Review Request: lxpolkit - Simple PolicyKit authentication agent
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=579171 Christoph Wickert 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 579171] Review Request: lxpolkit - Simple PolicyKit authentication agent
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=579171 --- Comment #12 from Fedora Update System 2010-07-04 16:43:33 EDT --- lxpolkit-0.1.0-0.1.20100402git5087383.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/lxpolkit-0.1.0-0.1.20100402git5087383.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 579171] Review Request: lxpolkit - Simple PolicyKit authentication agent
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=579171 --- Comment #11 from Fedora Update System 2010-07-04 16:41:06 EDT --- lxpolkit-0.1.0-0.1.20100402git5087383.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/lxpolkit-0.1.0-0.1.20100402git5087383.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 191239] Review Request: qjackctl - Qt based JACK control application
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=191239 David Timms changed: What|Removed |Added CC||dti...@iinet.net.au Flag||fedora-cvs? --- Comment #24 from David Timms 2010-07-04 16:38:43 EDT --- Package Change Request == Package Name: qjackctl New Branches: EL-5 EL-6 Owners: dtimms [requirement to support rakarrack on el5,6] -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 579171] Review Request: lxpolkit - Simple PolicyKit authentication agent
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=579171 --- Comment #10 from Christoph Wickert 2010-07-04 16:37:39 EDT --- I will just go ahead and build this package. I don't really care if I break something, the GNOME folks had 3 months to object but nobody replied to my latest mail. I just sent another mail to the desktop list, lets see what happens. -- 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 540617] Review Request: django-lint - lint for (python) django web-framework
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=540617 --- Comment #47 from Till Maas 2010-07-04 16:15:54 EDT --- (In reply to comment #46) > I've checked and did not get the EL-6 path using purely cvs up. Yes, you need to use "cvs up -dP" or at least one of both arguments, but both are recommended afaik. To have them always used, you can add them to the ~/.cvsrc, these are the values that have been recommended to me: $ cat .cvsrc log -N diff -u update -dP co -P -- 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 602587] Review Request: perl-IPC-Signal - Utility functions dealing with signals
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=602587 Matthias Runge changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #18 from Matthias Runge 2010-07-04 16:06:33 EDT --- Package Change Request == Package Name: perl-IPC-Signal New Branches: EL-6 Owners: mrunge I've been asked to include perl-Proc-WaitStat into EL-6. perl-IPC-Signal is a dependency, so it's needed, too. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 540617] Review Request: django-lint - lint for (python) django web-framework
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=540617 --- Comment #46 from Matthias Runge 2010-07-04 16:00:58 EDT --- Thank you, both of you. I've checked and did not get the EL-6 path using purely cvs up. fedora-cvs ... got it for me. -- 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 579171] Review Request: lxpolkit - Simple PolicyKit authentication agent
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=579171 --- Comment #9 from Kevin Fenzi 2010-07-04 15:34:34 EDT --- Any news here? Or still in limbo? -- 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 609738] Review Request: shiboken - CPython bindings generator for C++ libraries
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=609738 Thomas Janssen changed: What|Removed |Added CC||thom...@fedoraproject.org AssignedTo|nob...@fedoraproject.org|thom...@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 607802] Review Request: Agender - A small, simple and cross-platform schedule tool
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=607802 --- Comment #2 from Mamoru Tasaka 2010-07-04 13:51:49 EDT --- Created an attachment (id=429394) --> (https://bugzilla.redhat.com/attachment.cgi?id=429394) gdb log of above gdb log when the above issue occurs is attached here. -- 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 607802] Review Request: Agender - A small, simple and cross-platform schedule tool
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=607802
--- Comment #1 from Mamoru Tasaka 2010-07-04
13:50:50 EDT ---
Created an attachment (id=429393)
--> (https://bugzilla.redhat.com/attachment.cgi?id=429393)
screenshot when Agender crashes with segv
Some notes:
* SourceURL:
- For tarball hosted on sourceforge.net, please use:
https://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net
* Explicit version dependency
- wxGTK on supported branches of Fedora have version higher than 2.8.0
and explicit version dependency ">= 2.8.0" is not needed:
https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires
* %setup
- By default %setup must be quiet (i.e. please use %setup -q)
* indentation
- Please remove unneeded spaces before each line.
* %configure
--prefix=%{_prefix} is not needed
( You can check what %configure actually does by
$ rpm --eval %configure )
* cleaning $RPM_BUILD_ROOT
- Please remove unneeded "test -n "$RPM_BUILD_ROOT" -a "$RPM_BUILD_ROOT" !=
"/" &&"
hack
! Note that on F-12 and above, $RPM_BUILD_ROOT is always cleaned up before
%install section begins, so even "%{__rm} -rf $RPM_BUILD_ROOT" is not
needed
on %install
! Also on F-13 and above writing %clean section explicitly is not needed.
* Timestamps
- Please consider to use
---
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
---
to keep timestamps on installed files as much as possible.
This method usually works for Makefiles generated by recent autotools.
* Consistent macros usage
- Please unify %{__make} and %__make .
* Desktop file
- If desktop file is installed, desktop-file-{install,validate} must be used
to verify the installed file.
https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files
* %changelog
- Please write proper %changelog entry:
https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs
By the way, when I
- launch two terminals (like uxterm)
- In one terminal, type $ Agender
- After that in another terminal type $ Agender
then Agender launched on the first terminal causes segv.
--
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 611285] New: Review Request: pyvcs - A lightweight abstraction layer over multiple VCSs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pyvcs - A lightweight abstraction layer over multiple VCSs https://bugzilla.redhat.com/show_bug.cgi?id=611285 Summary: Review Request: pyvcs - A lightweight abstraction layer over multiple VCSs Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/pyvcs.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/pyvcs-0.2.0-1.fc13.src.rpm Project URL: http://pypi.python.org/pypi/pyvcs Description: pyvcs is a minimal VCS abstraction layer for Python. It's goals are to provide as much functionality as is necessary, and no further. It doesn't try to abstract every layer or feature of a VCS, just what's necessary to build a code browsing UI. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2294135 rpmlint output: [...@laptop011 SRPMS]$ rpmlint pyvcs-0.2.0-1.fc13.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [...@laptop011 noarch]$ rpmlint pyvcs-0.2.0-1.fc13.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611285] Review Request: pyvcs - A lightweight abstraction layer over multiple VCSs
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=611285 Fabian Affolter changed: What|Removed |Added Depends on||605423(python-dulwich) Alias||pyvcs -- 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 605423] Review Request: python-dulwich - A python implementation of the Git file formats and protocols
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=605423 Fabian Affolter changed: What|Removed |Added Blocks||611285(pyvcs) -- 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 226504] Merge Review: tsclient
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=226504 --- Comment #6 from Christopher Brown 2010-07-04 13:26:25 EDT --- Errr, okay. So are you going to complete the review? Not sure I can see the value in keeping it open however as all issues raised were addressed. Happy for this to be re-opened if someone is going to do the paperwork to get it completed but this appears to be rather bureaucratic. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611054] Review Request: python-carrot - AMQP Messaging Framework for Python
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=611054 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611040] Review Request: python-importlib - Backport of importlib.import_module() from Python 2.7
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=611040 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611038] Review Request: python-anyjson - Wraps the best available JSON implementation available
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=611038 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611043] Review Request: python-billiard - Multiprocessing Pool Extensions
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=611043 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611048] Review Request: python-amqplib - Client library for AMQP
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=611048 Fabian Affolter changed: What|Removed |Added Blocks||611054 -- 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 611277] Review Request: python-celery - Task queue/job queue based on distributed message passing
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=611277 Fabian Affolter changed: What|Removed |Added Depends on||611040, 611068, 611038, ||611048, 611054, 611043 -- 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 611068] Review Request: django-picklefield - Implementation of a pickled object field
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=611068 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611054] Review Request: python-carrot - AMQP Messaging Framework for Python
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=611054 Fabian Affolter changed: What|Removed |Added Depends on||611048 -- 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 611048] Review Request: python-amqplib - Client library for AMQP
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=611048 Fabian Affolter changed: What|Removed |Added Blocks||611277 -- 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 611277] New: Review Request: python-celery - Task queue/job queue based on distributed message passing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-celery - Task queue/job queue based on distributed message passing https://bugzilla.redhat.com/show_bug.cgi?id=611277 Summary: Review Request: python-celery - Task queue/job queue based on distributed message passing Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-celery.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-celery-2.0.0-1.fc13.src.rpm Project URL: http://pypi.python.org/pypi/celery Description: Celery is a task queue/job queue based on distributed message passing. It is focused on real-time operation, but supports scheduling as well. The execution units, called tasks, are executed concurrently on a single or more worker servers. Tasks can execute asynchronously (in the background) or synchronously (wait until ready). Celery is already used in production to process millions of tasks a day. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2294047 rpmlint output: [...@laptop011 SRPMS]$ rpmlint python-celery-2.0.0-1.fc13.src.rpm python-celery.src:68: W: macro-in-comment %check 1 packages and 0 specfiles checked; 0 errors, 1 warnings. The test section is commented-out because there are plenty of issues at the moment. [...@laptop011 noarch]$ rpmlint python-celery-2.0.0-1.fc13.noarch.rpm python-celery.noarch: W: hidden-file-or-dir /usr/share/doc/python-celery-2.0.0/docs/.static/.keep python-celery.noarch: E: zero-length /usr/share/doc/python-celery-2.0.0/docs/.static/.keep python-celery.noarch: W: hidden-file-or-dir /usr/share/doc/python-celery-2.0.0/docs/.static python-celery.noarch: W: hidden-file-or-dir /usr/share/doc/python-celery-2.0.0/docs/.static python-celery.noarch: W: no-manual-page-for-binary celeryd python-celery.noarch: W: no-manual-page-for-binary camqadm python-celery.noarch: W: no-manual-page-for-binary celeryev python-celery.noarch: W: no-manual-page-for-binary celeryinit python-celery.noarch: W: no-manual-page-for-binary celeryd-multi python-celery.noarch: W: no-manual-page-for-binary celerybeat 1 packages and 0 specfiles checked; 1 errors, 9 warnings. The rpmlint issues about the hidden stuff are all related to sphinx-build. -- 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 226504] Merge Review: tsclient
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=226504 Dan Horák changed: What|Removed |Added Keywords||Reopened Status|CLOSED |ASSIGNED CC||d...@danny.cz Resolution|ERRATA | --- Comment #5 from Dan Horák 2010-07-04 12:40:11 EDT --- (In reply to comment #4) > Old bug - long since merged so closing. Yes, it's old, but I can't see that the merge review was finished. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226504] Merge Review: tsclient
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=226504 Christopher Brown changed: What|Removed |Added Status|ASSIGNED|CLOSED CC||snecklif...@gmail.com Resolution||ERRATA --- Comment #4 from Christopher Brown 2010-07-04 12:23:53 EDT --- Old bug - long since merged so closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566410] Review Request: snmpcheck - An utility to get information via SNMP protocols
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=566410 Fabian Affolter changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | --- Comment #5 from Fabian Affolter 2010-07-04 12:12:36 EDT --- My bad, sorry. -- 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 566410] Review Request: snmpcheck - An utility to get information via SNMP protocols
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=566410 --- Comment #4 from Nikolay Ulyanitsky 2010-07-04 11:39:37 EDT --- > Added the "FE-NEEDSPONSOR" blocker because it's your first package. I am already sponsored :) https://admin.fedoraproject.org/accounts/user/view/lystor -- 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 605423] Review Request: python-dulwich - A python implementation of the Git file formats and protocols
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=605423 --- Comment #6 from Christian Krause 2010-07-04 11:22:48 EDT --- Thanks for the new package. Everything is fixed now besides the regexp to filter out the provides/requires: The used syntax doesn't work: $ echo "_objects.so"| grep -v -e '_objects.so|_pack.so' _objects.so $ It is necessary to use "-E" for "extended regexp": $ echo "_objects.so"| grep -v -E '_objects.so|_pack.so' $ If you change the package for this, please just remove also the commented line with "##grep..." in the %build section. ;-) -- 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 577975] Review Request: kde-plasma-daisy - A versatile application launcher
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=577975 --- Comment #8 from Julian Aloofi 2010-07-04 11:17:50 EDT --- (In reply to comment #7) > (In reply to comment #6) > > I think it should be OK though, as daisy doesn't support the zoom effect on > > mouseover which was the problem IIRC. > > > > Well, let's wait and see. > > Is simple mouseover effect not similar to zoom effect? > http://img227.imageshack.us/img227/5585/snapshot1pc.png I think the OS X dock zooms in many icons at once (like http://blog.buerstinghaus.net/images/nexus_dock_zoom.png ), but FE-Legal probably knows better. -- 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 577975] Review Request: kde-plasma-daisy - A versatile application launcher
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=577975 --- Comment #7 from nucleo 2010-07-04 11:03:05 EDT --- (In reply to comment #6) > I think it should be OK though, as daisy doesn't support the zoom effect on > mouseover which was the problem IIRC. > > Well, let's wait and see. Is simple mouseover effect not similar to zoom effect? http://img227.imageshack.us/img227/5585/snapshot1pc.png -- 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 610934] Review Request: go - The Google Go Programming Language
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=610934 Michel Alexandre Salim changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|michael.silva...@gmail.com Flag||fedora-review? --- Comment #3 from Michel Alexandre Salim 2010-07-04 10:57:51 EDT --- (In reply to comment #1) > I forgot to mention this is my first package and I need a sponsor. no problem on that, I can sponsor. I'll take a look and start providing review 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 577975] Review Request: kde-plasma-daisy - A versatile application launcher
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=577975 --- Comment #6 from Julian Aloofi 2010-07-04 10:50:55 EDT --- I think it should be OK though, as daisy doesn't support the zoom effect on mouseover which was the problem IIRC. Well, let's wait and see. -- 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 524119] Review Request: nmon - Nigel's performance MONitor for Linux
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=524119 Mamoru Tasaka changed: What|Removed |Added Flag||needinfo?(dylan.sw...@gmail ||.com) --- Comment #15 from Mamoru Tasaka 2010-07-04 10:45:59 EDT --- What is the status 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 606379] Review Request: rubygem-oniguruma - Bindings for the oniguruma regular expression 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=606379 Mamoru Tasaka changed: What|Removed |Added Flag||needinfo?(g...@zimt.uni-siege ||n.de) --- Comment #5 from Mamoru Tasaka 2010-07-04 10:41:34 EDT --- ping? -- 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 578269] Review Request: xgospel - An X11 client for Internet Go Server
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=578269 --- Comment #26 from Mamoru Tasaka 2010-07-04 10:42:41 EDT --- ping again? -- 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 577975] Review Request: kde-plasma-daisy - A versatile application launcher
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=577975 --- Comment #5 from nucleo 2010-07-04 10:01:03 EDT --- Ok. Will wait for Fedora-legal response. http://lists.fedoraproject.org/pipermail/legal/2010-July/001312.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 577975] Review Request: kde-plasma-daisy - A versatile application launcher
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=577975 --- Comment #4 from Julian Aloofi 2010-07-04 09:54:25 EDT --- Woops, totally forgot this review :) Well anyway, here are: Spec URL: http://julian.fedorapeople.org/kde-plasma-daisy/kde-plasma-daisy.spec SRPM URL: http://julian.fedorapeople.org/kde-plasma-daisy/kde-plasma-daisy-0.0.4.23-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 608973] Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.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=608973 --- Comment #9 from Julian Aloofi 2010-07-04 09:33:59 EDT --- Okay, by the way the BuildRoot tag is not needed anymore so you can just remove 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 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 Iain Arnell 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 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 --- Comment #3 from Iain Arnell 2010-07-04 09:20:45 EDT --- New Package CVS Request === Package Name: perl-Devel-PartialDump Short Description: Partial dumping of data structures, optimized for argument printing Owners: iarnell Branches: F-12 F-13 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611165] Review Request: perl-Devel-PartialDump - Partial dumping of data structures, optimized for argument printing
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=611165 Mamoru Tasaka changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #2 from Mamoru Tasaka 2010-07-04 09:12:54 EDT --- ! Two comments rpmlint may complain, however - BuildRoot is no longer needed (on Fedora: if you want to import this package also into EPEL, BuildRoot tag is needed there) - On Fedora 13+, writing %clean section explicitly is no longer needed (on F-12 and EPEL, writing %clean explicitly is still needed) + spec file generally okay + tarball matches the upstream + license matches + builds on koji (at least for dist-rawhide, dist-f13-updates-candidate) + can be installed (with yum --nogpgcheck localinstall) + needed requires (especially module compat one) explicitly written + check succeeds + %files entry okay + seems to work (use Devel::PartialDump; seems to succeed) + not still in Fedora yet, duplicate review request not found (on google) approving. --- This package (perl-Devel-PartialDump) is APPROVED by mtasaka --- -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608925] Review Request: python-iso8601 - Simple module to parse ISO 8601 dates
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=608925
Fabian Affolter changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||fab...@bernewireless.net
AssignedTo|nob...@fedoraproject.org|fab...@bernewireless.net
Flag||fedora-review+
--- Comment #1 from Fabian Affolter 2010-07-04
09:07:33 EDT ---
Package Review
==
Package:
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec
[x] Package meets the Packaging Guidelines
[x] Package successfully compiles and builds into binary RPMs on at least one
supported architecture
Tested on: F13/i386
[x] Rpmlint output:
Source RPM:
[...@laptop011 SRPMS]$ rpmlint python-iso8601-0.1.4-1.fc12.src.rpm
python-iso8601.src: W: spelling-error %description -l en_US datetime ->
date time, date-time, dateline
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
Binary RPM(s):
[...@laptop011 noarch]$ rpmlint python-iso8601-0.1.4-1.fc13.noarch.rpm
python-iso8601.noarch: W: spelling-error %description -l en_US datetime ->
date time, date-time, dateline
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[x] Package is not relocatable
[x] Buildroot is correct
master : %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
spec file: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license
License type: MIT
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc
[x] Spec file is legible and written in American English
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL
Upstream source: f63c17a1786b9fe48d05b43d61970f85
Build source:f63c17a1786b9fe48d05b43d61970f85
[-] Package is not known to require ExcludeArch
[x] Architecture independent packages have: BuildArch: noarch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly. %find_lang used for locales
[-] %{optflags} or RPM_OPT_FLAGS are honoured
[-] ldconfig called in %post and %postun if required
[x] %install starts with rm -rf %{buildroot} or $RPM_BUILD_ROOT
[-] Package must own all directories that it creates
[x] Package requires other packages for directories it uses
[x] Package does not own files or directories owned by other packages
[x] Package does not contain duplicates in %files
[x] Permissions on files are set properly. %defattr(-,root,root,-) is in every
%files section
[x] Package uses nothing in %doc for runtime
[x] Package has a %clean section, which contains rm -rf %{buildroot} or
$RPM_BUILD_ROOT
[-] Included tests passed successfully
[x] Package consistently uses macros
[x] Package contains code, or permissable content
[x] Included filenames are in UTF-8
[-] Large documentation files are in a -doc subpackage, if required
[-] Header files (.h) in -devel subpackage, if present
[-] Fully versioned dependency in subpackage, if present
[-] Static libraries (.a) in -static subpackage, if present
[-] Package requires pkgconfig, if .pc files are present
[-] Development .so files in -devel subpackage, if present
[-] Package does not contain any libtool archives (.la)
[-] -debuginfo subpackage is present and looks complete
[x] No pre-built binaries (.a, .so*, executable)
[-] Package contains a properly installed .desktop file if it is a GUI
application
[-] Follows desktop entry spec
[-] Valid .desktop Name
[-] Valid .desktop GenericName
[-] Valid .desktop Categories
[-] Valid .desktop StartupNotify
[-] .desktop file installed with desktop-file-install in %install
=== SUGGESTED ITEMS ===
[-] Timestamps preserved with cp and install
[-] Uses parallel make (%{?_smp_mflags})
[x] Latest version is packaged
[x] Package does not include license text files separate from upstream
[x] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available
[x] Package should compile and build into binary RPMs on all supported
architectures.
Tested: http://koji
[Bug 567563] Review request: tikiwiki - Website/Wiki/Groupware/CMS/Forum/Blog/Bug Tracker written in PHP
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=567563 Fabian Affolter changed: What|Removed |Added CC||fab...@bernewireless.net Summary|Review request: tikiwiki|Review request: tikiwiki - ||Website/Wiki/Groupware/CMS/ ||Forum/Blog/Bug Tracker ||written in PHP -- 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 605859] Review Request: python-eggtestinfo - Python setuptools plugin for adding test information to .egg-info
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=605859
Fabian Affolter changed:
What|Removed |Added
CC||fab...@bernewireless.net
--- Comment #1 from Fabian Affolter 2010-07-04
08:24:06 EDT ---
Some quick comments on your spec file:
[...@laptop011 SRPMS]$ rpmlint python-eggtestinfo-0.2.2-1.fc13.src.rpm
python-eggtestinfo.src: W: spelling-error Summary(en_US) setuptools -> setup
tools, setup-tools, setups
python-eggtestinfo.src: W: spelling-error Summary(en_US) plugin -> plug in,
plug-in, plugging
python-eggtestinfo.src: W: spelling-error %description -l en_US setuptools ->
setup tools, setup-tools, setups
python-eggtestinfo.src: W: spelling-error %description -l en_US plugin -> plug
in, plug-in, plugging
python-eggtestinfo.src: W: invalid-license Python Software Foundation License
python-eggtestinfo.src: W: invalid-license Zope Public License
1 packages and 0 specfiles checked; 0 errors, 6 warnings.
- Please fix the license tag.
- Using macros in the URL is not very useful for copy-and-paste
From my point of view the url can be http://pypi.python.org/pypi/eggtestinfo
- If you want to build this package for Fedora 12 please use this
%{python_sitelib} macros
https://fedoraproject.org/wiki/Packaging:Python#Macros
--
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 611175] Review Request: yash - Yet Another SHell
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=611175 --- Comment #7 from Iain Arnell 2010-07-04 08:13:06 EDT --- Changes are good for me. Still 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 608069] Tango standard library for D language of d1 specification
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=608069 --- Comment #3 from MERCIER Jonathan 2010-07-04 08:01:49 EDT --- (In reply to comment #1) > other directories appears there in the futur, should they be removed too ? rpmlint say nothing about this so i think is good -- 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 611175] Review Request: yash - Yet Another SHell
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=611175 --- Comment #6 from Mamoru Tasaka 2010-07-04 07:58:27 EDT --- Okay, then stay Japanese document files seperated. http://mtasaka.fedorapeople.org/Review_request/yash/yash.spec http://mtasaka.fedorapeople.org/Review_request/yash/yash-2.22-0.2.b0.fc.src.rpm * Sun Jul 4 2010 Mamoru Tasaka - 2.22-0.2.b0 - Handle %%_sysconfdir/shells - Move binary to /bin Summary is also trimmed. -- 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 611175] Review Request: yash - Yet Another SHell
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=611175
--- Comment #5 from Iain Arnell 2010-07-04 07:49:48 EDT ---
Thanks Ralf.
(In reply to comment #4)
> (In reply to comment #3)
> > MUSTFIX:
> > * Missing /etc/shells entry
>
> - Will add scriptlets at %post{,un} (borrowing what zsh does
> on %post{,un} - Well, with this yash will depend on bash on
> scriptlets, however I guess bash is installed on system anyway,
> and I don't know about lua)
Agreed - the zsh scriptlets are good to copy.
> > Consider:
> > * abandon the *-doc package and bundle the doc with the main package.
> > This separate doc-package makes not much sense
>
> - Well, it seems that size of yash{-doc} is (much) smaller
> than bash/zsh, will merge.
I actually liked that the (no offence intended, but useless to most people who
don't understand Japanese) docs were completely separate. I know there's some
way to exclude languages, but I can't for the life of me remember how. And if
the docs grow to include translations in future, they will likely become bigger
than the main rpm.
> > * Trim the description.
> - Will remove.
Agreed.
--
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 561985] Review Request: php-phpunit-phpdcd - Dead Code Detector (DCD) for PHP code
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=561985 Christof Damian changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #8 from Christof Damian 2010-07-04 07:25:12 EDT --- Package Change Request == Package Name: php-phpunit-phpdcd New Branches: EL-6 Owners: cdamian -- 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 231772] Review Request: Siege - Siege is an http regression testing and benchmarking utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=231772 Christof Damian changed: What|Removed |Added CC||chris...@damian.net Flag|fedora-cvs+ |fedora-cvs? --- Comment #5 from Christof Damian 2010-07-04 07:27:38 EDT --- Package Change Request == Package Name: siege New Branches: EL-6 Owners: cdamian -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551723] Review Request: php-pdepend-PHP-Depend - PHP_Depend design quality metrics for PHP package
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=551723 Christof Damian changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #13 from Christof Damian 2010-07-04 07:23:04 EDT --- Package Change Request == Package Name: php-pdepend-PHP-Depend New Branches: EL-6 Owners: cdamian -- 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 551724] Review Request: php-phpmd-PHP-PMD - PHPMD - PHP Mess Detector
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=551724 Christof Damian changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #15 from Christof Damian 2010-07-04 07:23:42 EDT --- Package Change Request == Package Name: php-phpmd-PHP-PMD New Branches: EL-6 Owners: cdamian -- 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 546020] Review Request: php-swift-Swift - Free Feature-rich PHP Mailer
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=546020 Christof Damian changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #8 from Christof Damian 2010-07-04 07:25:49 EDT --- Package Change Request == Package Name: php-swift-Swift New Branches: EL-6 Owners: cdamian -- 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 611175] Review Request: yash - Yet Another SHell
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=611175
--- Comment #4 from Mamoru Tasaka 2010-07-04
07:20:36 EDT ---
(In reply to comment #3)
> MUSTFIX:
> * Missing /etc/shells entry
- Will add scriptlets at %post{,un} (borrowing what zsh does
on %post{,un} - Well, with this yash will depend on bash on
scriptlets, however I guess bash is installed on system anyway,
and I don't know about lua)
> Consider:
> * abandon the *-doc package and bundle the doc with the main package.
> This separate doc-package makes not much sense
- Well, it seems that size of yash{-doc} is (much) smaller
than bash/zsh, will merge.
> * Trim the description. To me, reading
>
> "Actually, it is much
> more POSIX-compliant than other shell like bash and zsh." are shallow
> marketing
> slogans without substance.
- Will remove.
--
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 561981] Review Request: php-phpunit-PHP-TokenStream - Wrapper around PHP's tokenizer extension
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=561981 Christof Damian changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #8 from Christof Damian 2010-07-04 07:24:42 EDT --- Package Change Request == Package Name: php-phpunit-PHP-TokenStream New Branches: EL-6 Owners: cdamian -- 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
