[Bug 630288] Review Request: ghc-enumerator - Implementation of Oleg Kiselyov's left-fold enumerators
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630288 Lakshmi Narasimhan changed: What|Removed |Added CC||lakshminaras2...@gmail.com AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630283] Review Request: ghc-neither - Either with monad and applicative instances
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630283 Lakshmi Narasimhan changed: What|Removed |Added CC||lakshminaras2...@gmail.com AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 595011] Review Request: sshdfilter - Filter for SSH ports
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=595011 --- Comment #21 from David Highley 2010-11-07 23:52:13 EST --- I have not heard any feed back on the last changes. So I'm thinking I will see if I can put this on the wish list and see if some one who has maintainer rights is interested in 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 650180] Review Request: tkabber - Client for the Jabber instant messaging system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650180 Dmitrij S. Kryzhevich changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||NEXTRELEASE Last Closed||2010-11-07 23:03:27 --- Comment #10 from Dmitrij S. Kryzhevich 2010-11-07 23:03:27 EST --- According #10 for Contributor from https://fedoraproject.org/wiki/Package_Review_Process, closed 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 650181] Review Request: tkabber-plugins - Additional plugins for tkabber
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650181 Bug 650181 depends on bug 650180, which changed state. Bug 650180 Summary: Review Request: tkabber - Client for the Jabber instant messaging system https://bugzilla.redhat.com/show_bug.cgi?id=650180 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|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 650180] Review Request: tkabber - Client for the Jabber instant messaging system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650180 --- Comment #9 from Fedora Update System 2010-11-07 22:48:52 EST --- tkabber-0.11.1-2.svn1948.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/tkabber-0.11.1-2.svn1948.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650180 Fedora Update System 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 650180] Review Request: tkabber - Client for the Jabber instant messaging system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650180 --- Comment #8 from Fedora Update System 2010-11-07 22:47:37 EST --- tkabber-0.11.1-2.svn1948.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/tkabber-0.11.1-2.svn1948.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 593800] Review Request: python-keyring - keyring module 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=593800 --- Comment #43 from Ratnadeep Debnath 2010-11-07 22:42:39 EST --- I added "Group:" entry for the gnome and kde subpackages. I submitted it for koji scratch build and it failed. https://koji.fedoraproject.org/koji/taskinfo?taskID=2583854 The build.log contains only: Mock Version: 1.1.4 Mock Version: 1.1.4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 607584] Review Request: wordgroupz - A vocabulary building 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=607584 --- Comment #38 from Ratnadeep Debnath 2010-11-07 22:36:27 EST --- @Mamoru Tasaka The el-5 build failed for wordgroupz. Please check http://koji.fedoraproject.org/koji/getfile?taskID=2585997 Error: + /usr/bin/python setup.py install --skip-build --root usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...] or: setup.py --help [cmd1 cmd2 ...] or: setup.py --help-commands or: setup.py cmd --help error: option --root requires argument error: Bad exit status from /var/tmp/rpm-tmp.22660 (%install) RPM build errors: Bad exit status from /var/tmp/rpm-tmp.22660 (%install) But --root is having $RPM_BUILD_ROOT as argument. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 578024] Review Request: ingres - Relational DBMS Server and Utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=578024 Cristian Ciupitu changed: What|Removed |Added CC||cristian.ciup...@yahoo.com Flag||needinfo?(jeremy.hankinson@ ||ingres.com) --- Comment #33 from Cristian Ciupitu 2010-11-07 22:16:04 EST --- Disclaimer: I'm not a packager. I wasn't able to download your SPEC ("get: Access failed: 550 Failed to open file. (ingres.spec)"), but I've looked a bit at the rpmlint output. You have a lot of non-standard-uid/gid warnings, including for files under /usr, e.g. /usr/libexec/ingres/utility/iiask4it. Is it really necessarily for these files to be owned by someone else besides root? Also, is to me or there's no reason for the files under /var/lib/ingres/files/ucharmaps/ to change? If so, you should try moving them somewhere under /usr/share. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239043] Review Request: libdc1394 - IEEE 1394 based Digital Camera control 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=239043 --- Comment #47 from Jason Tibbitts 2010-11-07 22:07:55 EST --- If you have the means to maintain and properly test an el6 branch of your package, surely you could just run 'yum search' on your test machine. -- 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 639816] Review Request: cmake-fedora - CMake helper modules for fedora developers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639816 --- Comment #25 from Fedora Update System 2010-11-07 21:37:39 EST --- cmake-fedora-0.3.1-1.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/cmake-fedora-0.3.1-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 639816] Review Request: cmake-fedora - CMake helper modules for fedora developers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639816 --- Comment #24 from Fedora Update System 2010-11-07 21:37:25 EST --- cmake-fedora-0.3.1-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/cmake-fedora-0.3.1-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 650767] Review Request: aiki - A CMS Creator Web 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=650767 Steven Garcia changed: What|Removed |Added Platform|All |noarch 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 650767] New: Review Request: aiki - A CMS Creator Web Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: aiki - A CMS Creator Web Application https://bugzilla.redhat.com/show_bug.cgi?id=650767 Summary: Review Request: aiki - A CMS Creator Web Application Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: webwha...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://aikiframework.org/files/pkgs/0.4.5/aiki.spec SRPM URL: http://aikiframework.org/files/pkgs/0.4.5/aiki-0.4.5-1.fc13.src.rpm Description: Aiki Framework is a web application written in PHP that allows users to easily create and work with content management systems. This is my first Fedora Core package and I need a sponsor. I'm one of the upstream developers of this package and have already done all of the following tasks: Read and followed the Packaging Guidelines and Package Naming Guidelines. Read and followed the Licensing Guidelines Licensed the package of which the text is included in %doc of the spec file Identified all existing licenses in the package and listed them in the license field of the spec file License: AGPLv3 and GPLv2 and LGPLv2+ and zlib and BSD Verified all licenses appear in Good Licenses of http://fedoraproject.org/wiki/Licensing#SoftwareLicenses Included PACKAGE-LICENSING in %doc which lists all files under different licenses Added comments for every license in the %files section above the files they pertain to Created a Red Hat Bugzilla Account Joined and read the fedora devel-annou...@lists.fedoraproject.org mailing list Verified the package is NOT in the Fedora Package Database for packages already in the repository Verified the package is NOT in the Review Tracker for packages under review Verified the package is NOT in the retired package list Verified the package does NOT contain forbidden items Verified it is NOT proprietary Verified it is NOT legally encumbered Verified it does NOT violate United States laws Read Other Submissions for package review requests Created and uploaded RPM Spec and SRPM files http://aikiframework.org/files/pkgs/0.4.5/ Created a review request Put the name of the package and brief summary in the 'Review Summary' field Put a description of the package in the 'Review Description' field. Included the URLs to the SRPM and SPEC files. Explained that this is my first package and I need a sponsor. Edited my review request bug and added FE-NEEDSPONSOR in the 'Bug blocks' field Informed the upstream developers Created an account in the Fedora Account System Set account name as fosdevel Set Full name as Steven Garcia Set Email Set Telephone number Set Country US Set IRC nick as fosdevel Set PGP key Set Public SSH Key Signed the CLA Downloaded a client-side certificate Installed the Client Tools (Koji) Ran fedora-packager-setup Imported my certificate (fedora-browser-cert.p12) into my firefox browser Imported Fedora and Koji Server certificates into my firefox browser Ran successfull koji scratch builds https://koji.fedoraproject.org/koji/tasks?owner=fosdevel&state=all $ koji build --scratch dist-f12 aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585711 $ koji build --scratch dist-f12-updates-candidate aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585715 $ koji build --scratch dist-f13 aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585741 $ koji build --scratch dist-f13-kde aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585761 $ koji build --scratch dist-f13-updates-candidate aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585818 $ koji build --scratch dist-f14 aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585825 $ koji build --scratch dist-f14-gobject aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585833 $ koji build --scratch dist-f14-kde aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585839 $ koji build --scratch dist-f14-updates-candidate aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585843 $ koji build --scratch dist-f15 aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585845 $ koji build --scratch dist-olpc4 aiki-0.4.5-1.fc13.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=2585847 Verified each Package Review Guideline Ran rpmlint on spec, srpm and rpm files $ rpmlint SPECS/aiki.spec SRPMS/aiki-0.4.5-1.fc13.src.rpm RPMS/noarch/aiki-0.4.5-1.fc13.
[Bug 595011] Review Request: sshdfilter - Filter for SSH ports
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=595011 --- Comment #20 from Rafael Aquini 2010-11-07 19:41:07 EST --- David, There has been almost two months since your last update to this bug. Are you still interested in packaging sshdfilter? If so, please consider posting updates here soon. If you are getting trouble in be sponsored, I can help you pointing out what have to be done prior to your sponsorship. While I do not possess such power, I'm still willing to help you get sponsored. Best regards -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650744] New: Review Request: spor - Store file modes (permission/ownership) recursively
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: spor - Store file modes (permission/ownership) recursively https://bugzilla.redhat.com/show_bug.cgi?id=650744 Summary: Review Request: spor - Store file modes (permission/ownership) recursively Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: aqu...@linux.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Hello folks, I've just finished packaging this little & very simple software, that helped me, and, at least, one more folk in our projects -- http://mamchenkov.net/wordpress/2005/04/27/subversion-and-file-permissions/ I'm looking forward your reviews to this work! Cheers!! Spec URL: http://aquini.fedorapeople.org/spor/spor.spec SRPM URL: http://aquini.fedorapeople.org/spor/spor-1.0-0.fc14.src.rpm Description: Spor recursively walks into a given directory, storing file mode & ownership information in a flat-file database for future retrievals. It was firstly intended to use with backup & version control scripts, and it provides a simple and safe method to save and restore particular meta-data information of a given 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 635453] Review Request: deafshell - Deaf Shell is used for system accounts dedicated to ssh port forwarding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635453 --- Comment #5 from François Cami 2010-11-07 19:26:36 EST --- I've updated the program summary and added CFLAGS. Thank you. However, the source tarball is not a snapshot release, it is related to the 0.1 git tag of the upstream git repository. See https://github.com/deafshell/deafshell/tree/v0.1 - therefore, I don't think I should change the naming to the changeset, though I agree github's tarball generation messes things up. If this is not acceptable to you, I will revert to using the git hash. I normally use %{_smp_mflags} but there is only a single C file to compile. I will update the package if upstream ever becomes more than that. Spec URL: http://fcami.fedorapeople.org/srpms/deafshell.spec SRPM URL: http://fcami.fedorapeople.org/srpms/deafshell-0.1-3.fc14.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 239043] Review Request: libdc1394 - IEEE 1394 based Digital Camera control 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=239043 --- Comment #46 from Tim Niemueller 2010-11-07 18:56:08 EST --- *sigh* Is there a list of the packages which are now in RHEL? I'd really wish they would use the same infrastructure for such packages to a) allow contributions from non-RH people and b) make it more obvious what's happening. A note to the former maintainer that the package will be in core and thus no longer in EPEL would have been nice as well. In general I'm happy how RH is doing things, but this procedure clearly would benefit from some improvements and more communication. https://fedorahosted.org/fedora-infrastructure/ticket/2465 -- 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 650643] Review Request: cegui06 - CEGUI library 0.6 for apps which need this specific version
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650643 Steven Garcia changed: What|Removed |Added CC||webwha...@gmail.com --- Comment #1 from Steven Garcia 2010-11-07 18:07:43 EST --- Hello I'm not a sponsor, but I have a few suggestions reviewing you package. rpmlint SPECS/cegui06.spec SRPMS/cegui06-0.6.2-1.fc14.src.rpm SPECS/cegui06.spec:9: W: macro-in-comment %{version} SPECS/cegui06.spec: W: invalid-url Source0: CEGUI-0.6.2.tar.gz cegui06.src: W: spelling-error %description -l en_US codecs -> codes, coders, code's cegui06.src: W: spelling-error %description -l en_US xml -> XML, cml, ml cegui06.src:9: W: macro-in-comment %{version} cegui06.src: W: invalid-url Source0: CEGUI-0.6.2.tar.gz 1 packages and 1 specfiles checked; 0 errors, 6 warnings. Change the comments above the Source0 field on how to generate the tarball used in the srpm Use a link such as: http://downloads.sourceforge.net/crayzedsgui/CEGUI-0.6.2b.tar.gz I would remove the macro %{version}, from your comment Change the spelling of xml to XML in your %description -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 531544] Review Request: python-trml2pdf - Tiny RML2PDF is a tool to easily create PDF documents without programming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531544 Cristian Ciupitu changed: What|Removed |Added Flag||needinfo?(a.bad...@gmail.co ||m) --- Comment #10 from Cristian Ciupitu 2010-11-07 18:02:47 EST --- (In reply to comment #9) > Needswork: > * License is LGPLv2+. Please update the spec file to reflect that. I didn't see any mention of "or later", so I used LGPLv2 to be safe. I'll address the rest of the issues later. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 547621] Review Request: django-googlecharts - A suite of template tags for Django to assist in generating charts using Google's Chart API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=547621 Cristian Ciupitu changed: What|Removed |Added CC||cristian.ciup...@yahoo.com Flag||needinfo?(acturne...@gmail. ||com) --- Comment #1 from Cristian Ciupitu 2010-11-07 17:34:49 EST --- I'm getting HTTP 404 errors. Are the links still 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 625602] Review Request:libbluray - Library to access Blu-Ray disks for video playback
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=625602 Alex Lancaster changed: What|Removed |Added Flag||needinfo?(xav...@bachelot.o ||rg) --- Comment #15 from Alex Lancaster 2010-11-07 17:20:59 EST --- Thanks for the rawhide build, any chance we could get builds and updates for f13 and f14? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 467655] Review Request: yafaray - a raytracer for Blender.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467655 --- Comment #84 from Paulo Roma Cavalcanti 2010-11-07 17:07:18 EST --- The problem with yafaray+blender was gettext. Upgrading to gettext 0.18.1.1 on my systems running F10/F12, avoids crashing, when using the qt style GTK+. https://bugzilla.redhat.com/show_bug.cgi?id=650471 However, I have a new .src.rpm here: Spec URL:http://orion.lcg.ufrj.br/RPMS/SPECS/yafaray.spec SRPM URL:http://roma.fedorapeople.org/srpms/yafaray-0.1.1-2.fc12.src.rpm I was not able to address the soname issue, because yafaray is compiled using scons and it is difficul to access the ldd flags. I put a comment in the spec file where the soname option should be passed, but the compilation fails in this case. But I am really not sure what is the exact syntax. Maybe someone else has a suggestion on how to proceed ... #YF_SHLINKFLAGS = "-Wl,-soname,libyafaraycore.so.1.0" This is the rpmlint output: yafaray.x86_64: W: spelling-error Summary(en_US) raytracing -> ray tracing, ray-tracing, tracing yafaray.x86_64: W: spelling-error %description -l en_US raytracing -> ray tracing, ray-tracing, tracing yafaray.x86_64: W: spelling-error %description -l en_US Raytracing -> Ray tracing, Ray-tracing, Tracing yafaray.x86_64: W: invalid-license LGPLv2.1 yafaray.x86_64: W: no-soname /usr/lib64/libyafaraycore.so yafaray.x86_64: W: no-manual-page-for-binary yafaray-xml 1 packages and 0 specfiles checked; 0 errors, 6 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 531544] Review Request: python-trml2pdf - Tiny RML2PDF is a tool to easily create PDF documents without programming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531544 --- Comment #9 from Toshio Ernie Kuratomi 2010-11-07 16:54:11 EST --- Please up the Release number whenever you make a new spec file. Good: * Package named appropriately * License file is included * specfile is readable * tarball matches what's on satchmo but see the information in the Notes section. * Builds in koji * No locales to handle * No shared libraries * No bundled libraries * Not relocatable * Package owns all directories it creates * No files listed multiple times * Permissions set properly * Packages used consistently * Code not content * Nothing in %doc is used at runtime * No GUI * All filenames are valid UTF-8 Needswork: * License is LGPLv2+. Please update the spec file to reflect that. Note: * Removing the python_sitelib definition is fine as long as you are not planning on building on older releases. If you do want to build this on F13 or less or EL-4 or EL-5 you will need to add it back in. * Upstream seems dead as you note but there's several projects that have copies: - satchmo (where you got the tarball from) http://www.satchmoproject.com/snapshots/ - kraft: http://sourceforge.net/projects/kraft/files/ - template2pdf: http://code.google.com/p/template2pdf/ It would be very nice if you could contact these groups and get them to revive the project in some shape or form. For instance, everyone could contirbute to the template2pdf project and make that module the new upstream Currently the tarballs for these projects do not match but the differences are minor and not in the code. satchmo has a setup.py file, kraft is saved as tar.bz2, template2pdf has bundled the module in their code. * Realize that currently, as there is no upstream for this package, you are putting yourself on the hook for fixing anything wrong in the code, bugs, security issues, etc. This can be alleviated if you get those other projects to help out in creating a new canonical upstream. rpmlint: python-trml2pdf.src: W: no-cleaning-of-buildroot %clean python-trml2pdf.src: W: no-%clean-section Both of these are something that's needed in EPEL builds but not F-14. If you're not planning to build on EPEL-5 then you don't need to worry about this. python-trml2pdf.noarch: W: no-manual-page-for-binary trml2pdf This is not a requirement but if you want to write a man page for the script, it would be helpful for people looking to run it. It doesn't look like Debian has one so we'd have to write one rather than copy it. This is not a blocker. 3 packages and 0 specfiles checked; 0 errors, 5 warnings. Summary: So fixing the licensing issue is the only blocker to the package. Do take the things in the Note section under consideration. I also don't see any reviews from you. There's quite a few python-* reviews listed here: http://fedoraproject.org/PackageReviewStatus/NEW.html I know that the maintainer of the python-zope-* packages is quite active if you want a relatively quick turnaround time. If you'll just choose one review and let me know what you're doing, I'll check your review and then sponsor you so you can officially complete 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 630272] Review Request: ghc-tagsoup - Parsing HTML/XML documents 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=630272 --- Comment #14 from Fedora Update System 2010-11-07 16:28:51 EST --- ghc-tagsoup-0.11.1-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630208] Review Request: ghc-csv - CSV loader and dumper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630208 Fedora Update System changed: What|Removed |Added Fixed In Version|ghc-csv-0.1.1-1.fc14|ghc-csv-0.1.1-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 630208] Review Request: ghc-csv - CSV loader and dumper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630208 --- Comment #13 from Fedora Update System 2010-11-07 16:29:03 EST --- ghc-csv-0.1.1-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648254] Review Request: fawkes - Robot Software 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=648254 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System 2010-11-07 16:27:02 EST --- fawkes-0.4-5.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update fawkes'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/fawkes-0.4-5.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 630222] Review Request: ghc-colour - A model for human colour/color perception
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630222 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-colour-2.3.1-1.fc13 Resolution||ERRATA Last Closed||2010-11-07 16:26:55 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630272] Review Request: ghc-tagsoup - Parsing HTML/XML documents 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=630272 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-tagsoup-0.11.1-1.fc14 Resolution||ERRATA Last Closed||2010-11-07 16:28:55 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630222] Review Request: ghc-colour - A model for human colour/color perception
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630222 --- Comment #12 from Fedora Update System 2010-11-07 16:26:50 EST --- ghc-colour-2.3.1-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630222] Review Request: ghc-colour - A model for human colour/color perception
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630222 Fedora Update System changed: What|Removed |Added Fixed In Version|ghc-colour-2.3.1-1.fc13 |ghc-colour-2.3.1-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630208] Review Request: ghc-csv - CSV loader and dumper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630208 --- Comment #12 from Fedora Update System 2010-11-07 16:27:07 EST --- ghc-csv-0.1.1-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630208] Review Request: ghc-csv - CSV loader and dumper
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630208 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-csv-0.1.1-1.fc14 Resolution||ERRATA Last Closed||2010-11-07 16:27:13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630222] Review Request: ghc-colour - A model for human colour/color perception
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630222 --- Comment #13 from Fedora Update System 2010-11-07 16:27:25 EST --- ghc-colour-2.3.1-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650717] New: Review Request: PolicyKit-olpc - OLPC-specific PolicyKit overrides
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: PolicyKit-olpc - OLPC-specific PolicyKit overrides https://bugzilla.redhat.com/show_bug.cgi?id=650717 Summary: Review Request: PolicyKit-olpc - OLPC-specific PolicyKit overrides Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: d...@laptop.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://dev.laptop.org/~dsd/20101107/PolicyKit-olpc.spec SRPM URL: http://dev.laptop.org/~dsd/20101107/PolicyKit-olpc-1.4-1.fc13.src.rpm Description: Software for the XO laptop requires some modification to the default policies installed by certain applications. This package provides OLPC-specific policy overrides. This package is already in Fedora but broke during fallout in the F12 cycle, and hence was blocked from being built. I've now finally fixed it, and was told on IRC that I have to go through the review process again in order to get it unblocked. Hopefully it's a simple review, since it was accepted before and its a trivial package. it follows the guidelines in "man pklocalauthority" -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650712] Review Request: xcm - X Color Management tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650712 --- Comment #1 from Nicolas Chauvet (kwizart) 2010-11-07 16:11:13 EST --- libXcm-0.3.0 isn't yet in f14 buildroot, so it will only work for f15. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #17 from Fedora Update System 2010-11-07 16:01:18 EST --- python-dulwich-0.6.2-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/python-dulwich-0.6.2-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 Fedora Update System 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 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 #16 from Fedora Update System 2010-11-07 16:01:06 EST --- python-dulwich-0.6.2-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/python-dulwich-0.6.2-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 650712] New: Review Request: xcm - X Color Management tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: xcm - X Color Management tools https://bugzilla.redhat.com/show_bug.cgi?id=650712 Summary: Review Request: xcm - X Color Management tools Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kwiz...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://kwizart.fedorapeople.org/review/xcm.spec SRPM URL: http://kwizart.fedorapeople.org/review/xcm-0.3.0-1.fc13.src.rpm Description: X Color Management tools http://koji.fedoraproject.org/koji/taskinfo?taskID=2585237 $ rpmlint -i /home/builder/rpmbuild/RPMS/x86_64/xcm-0.3.0-1.fc13.x86_64.rpm xcm.x86_64: W: no-manual-page-for-binary xcmevents Each executable in standard binary directories should have a man page. xcm.x86_64: W: no-manual-page-for-binary xcmedid Each executable in standard binary directories should have a man page. 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [buil...@liet SPECS]$ rpmlint -i /home/builder/rpmbuild/RPMS/x86_64/xcm-debuginfo-0.3.0-1.fc13.x86_64.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [buil...@liet SPECS]$ rpmlint -i /home/builder/rpmbuild/SRPMS/xcm-0.3.0-1.fc13.src.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 646139] Review Request: KXStitch - tool that creates cross stitch patterns
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=646139 --- Comment #4 from Martin Gieseking 2010-11-07 15:12:06 EST --- (In reply to comment #3) > thank you very much for your comments and remarks. I really appreciate your > help. You're welcome. > I updated the package and here are the new URLs OK, fine. The new package looks pretty clean now. Just two minor cosmetic issues: - I suggest to use the plural in the Summary (pattern => patterns). - Add a final dot to the %description. $ rpmlint /var/lib/mock/fedora-14-x86_64/result/*.rpm kxstitch.src: W: no-cleaning-of-buildroot %clean kxstitch.src: W: no-buildroot-tag kxstitch.src: W: no-%clean-section 3 packages and 0 specfiles checked; 0 errors, 3 warnings. These warnings can be ignored if you plan to maintain the package for Fedora only. However, EPEL <= 5 still requires the missing items mentioned above. > One thing is a little bit strange: I added the kernel-headers to the BRs > because otherwise the package could not be build in mock on my F13 machine. I > followed your advice and removed it and now it builds smoothly, however, I > tested it only on my F14 machine. Yes, these kind of things happen from time to time, and can't be reproduced later on. Anyway, the package builds fine for F13 too: http://koji.fedoraproject.org/koji/taskinfo?taskID=2584994 As you probably already know, joining the Fedora packager group isn't an automatic process but requires some initial work to convince a sponsor to approve you. It's important to familiarize yourself with the packaging guidelines and show it by doing some informal reviews of other packager's submissions. That's a crucial thing because you're allowed to formally review, packages once you're sponsored. And, of course, you shouldn't mess up your own packages. So you should know what you're doing. :) If you don't have a sponsor yet, I can sponsor you if you're willing to informally review two or three package submissions. Just choose an uncommented review request ticket (e.g. bug #634025) and comment on the packages according to the review guidelines: https://fedoraproject.org/wiki/Packaging:ReviewGuidelines There are a lot of further things to consider, but these are the most important ones. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648466] Review Request: php-phpunit-PHP-CodeBrowser - PHP_CodeBrowser for integration in Hudson and CruiseControl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648466 Christof Damian changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Christof Damian 2010-11-07 13:48:46 EST --- (In reply to comment #4) > > * APPROVED * New Package SCM Request === Package Name: php-phpunit-PHP-CodeBrowser Short Description: PHP_CodeBrowser for integration in Hudson and CruiseControl Owners: cdamian Branches: f13 f14 el5 el6 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648466] Review Request: php-phpunit-PHP-CodeBrowser - PHP_CodeBrowser for integration in Hudson and CruiseControl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648466 --- Comment #5 from Christof Damian 2010-11-07 13:45:27 EST --- (In reply to comment #3) > > I hope the doc location is to your liking :-) > NO ! > I will prefer a better solution. But this is the standard one used in fedora. Very well, I will change it once Fedora provides a better standard. > This package have optional deps : > "pear/Text_Highlighter" (version >= 0.7.1) > "channel://pear.phpunit.de/PHPUnit" (version >= 3.4.0) > "channel://pear.phpundercontrol.org/phpUnderControl" (version >= 0.5.0) > "pear/PHP_CodeSniffer" (version >= 1.2.0) > "pear/PhpDocumentor" (version >= 1.4.3) > > With pear package, I think optional dep can be make "mandatory" (ie Requires > in > spec, when available in the repo.) as packages are small, and don't eat memory > if not used. OK, I think we got them except of phpUnderControl. thanks for the review and your help. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 627180] Review Request: django-paste - dpaste is a code pastebin application using Django.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=627180 François Cami changed: What|Removed |Added Status|ASSIGNED|NEW AssignedTo|f...@fcami.net |nob...@fedoraproject.org --- Comment #9 from François Cami 2010-11-07 13:19:10 EST --- Realistically speaking, I will not have the time to review this in the very near future. Sorry about 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 526805] Review Request: lua-loop - Class models for Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=526805 --- Comment #3 from Michel Alexandre Salim 2010-11-07 13:18:03 EST --- Spec URL: http://salimma.fedorapeople.org/specs/funpl/lua-loop.spec SRPM URL: http://salimma.fedorapeople.org/specs/funpl/lua-loop-2.3-0.2.beta.fc14.src.rpm Updated: - scripts now installed in %%{_bindir} I contemplated giving them a loop- prefix, but then LuaRocks install them unprefixed, and they are generic enough to be usable to interoperate between C code and *any* Lua module, not just loop. One would assume that if there is any naming conflict, someone would have fixed the Rocks specification by now! - remove unneeded dependencies -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 646139] Review Request: KXStitch - tool that creates cross stitch patterns
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=646139 --- Comment #3 from Golo Fuchert 2010-11-07 12:49:45 EST --- Martin, thank you very much for your comments and remarks. I really appreciate your help. I updated the package and here are the new URLs SPEC URL: http://golotop.de/kxstitch.spec SRPM URL: http://golotop.de/kxstitch-0.8.4.1-3.fc14.src.rpm One thing is a little bit strange: I added the kernel-headers to the BRs because otherwise the package could not be build in mock on my F13 machine. I followed your advice and removed it and now it builds smoothly, however, I tested it only on my F14 machine. Most likely I made something wrong, but I will investigate on this further and try it again on my F13 machine. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 499992] Review Request: mingw32-webkitgtk - MinGW Windows web content engine 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=42 Erik van Pienbroek changed: What|Removed |Added Status Whiteboard|NotReady| --- Comment #11 from Erik van Pienbroek 2010-11-07 12:30:47 EST --- New Spec URL: http://www.ftd4linux.nl/contrib/mingw32-webkitgtk.spec New SRPM URL: http://www.ftd4linux.nl/contrib/mingw32-webkitgtk-1.2.5-1.fc14.src.rpm * Sun Nov 7 2010 Erik van Pienbroek - 1.2.5-1 - Update to 1.2.5 - Automatically generate debuginfo subpackage - Use %global instead of %define - Dropped the manual strip command - Dropped old patches -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650667] New: Review Request: hitori - Hitori game for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: hitori - Hitori game for GNOME https://bugzilla.redhat.com/show_bug.cgi?id=650667 Summary: Review Request: hitori - Hitori game for GNOME Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mari...@gnome.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.filehosting.org/file/details/180311/hitori.spec SRPM URL: http://www.filehosting.org/file/details/180288/hitori-0.2.5-1.fc14.src.rpm Description: Hitori is a small application written to allow one to play the eponymous puzzle game, which is similar in theme to more popular puzzles such as Sudoku. It is my first package for Fedora, but not my first package at all. I have some basic experience with building RPM's for Mandriva, which were published by the Mandrivauser.de project. See ftp://ftp.mandrivauser.de/rpm/GPL/2009.0/SRPMS/release/. Some of these packages are from me, mostly GTK based software, but also WindowMaker dockapps ans some other stuff. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 Mads Kiilerich changed: What|Removed |Added Flag||fedora-cvs? --- Comment #27 from Mads Kiilerich 2010-11-07 13:09:37 EST --- New Package SCM Request === Package Name: freerdp Short Description: remote desktop protocol client Owners: kiilerix Branches: f14 el4 el5 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 Levente Farkas changed: What|Removed |Added Flag|needinfo?(lfar...@lfarkas.o | |rg) | --- Comment #30 from Levente Farkas 2010-11-07 12:14:32 EST --- i no longer believe it's possible to add gstreamer to fedora as part of the mingw project. the main problem here is that gstreamer alone are not usable. it requires a lots of other media library which also should have to added. and that'd be a huge job (for which i don't have enough time). and it's not possible that some lib are compiled with mingw while others with msvc:-( i'd rather recommend ossbuild for everybody http://code.google.com/p/ossbuild/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 512323] Review Request: mingw32-gc - MinGW Windows port of the gc 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=512323 Nicolas Chauvet (kwizart) changed: What|Removed |Added Status|NEW |CLOSED Resolution||NEXTRELEASE Last Closed||2010-11-07 11:24:46 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239043] Review Request: libdc1394 - IEEE 1394 based Digital Camera control 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=239043 --- Comment #45 from Nicolas Chauvet (kwizart) 2010-11-07 10:58:57 EST --- They'll have to open an infrastructure ticket to get rid of the branch they should not have requested. @timn, you need to request the branch to be removed here: https://fedorahosted.org/fedora-infrastructure/ -- 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 239043] Review Request: libdc1394 - IEEE 1394 based Digital Camera control 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=239043 Nicolas Chauvet (kwizart) changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs- --- Comment #44 from Nicolas Chauvet (kwizart) 2010-11-07 10:48:25 EST --- I don't understand what's happened here, but libdc1394 is part from base in el6 It shouldn't be requested for el6 @Jason, can you remove this branch ? -- 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 648175] Review Request: flterm - Firmware download program
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648175 --- Comment #5 from Jason Tibbitts 2010-11-07 10:45:44 EST --- We are not yet accepting f15 branches. That won't happen until f15 branches from rawhide. Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650180] Review Request: tkabber - Client for the Jabber instant messaging system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650180 --- Comment #7 from Jason Tibbitts 2010-11-07 10:46:18 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648465] Review Request: php-pear-Console-CommandLine - A full featured command line options and arguments parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648465 --- Comment #5 from Jason Tibbitts 2010-11-07 10:46:02 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 633082] Review Request: mediawiki115 - EPEL mediawiki 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=633082 --- Comment #5 from Jason Tibbitts 2010-11-07 10:44:05 EST --- Please re-raise the fedora-cvs flag when you provide an SCM request for us to process. https://fedoraproject.org/wiki/Package_SCM_admin_requests -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 633085] Review Request: mediawiki116 - Mediawiki package for EPEL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=633085 --- Comment #4 from Jason Tibbitts 2010-11-07 10:44:36 EST --- Please re-raise the fedora-cvs flag when you provide an SCM request for us to process. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #15 from Jason Tibbitts 2010-11-07 10:41:24 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 440681] Review Request: luadoc - Documentation Generator Tool for Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=440681 --- Comment #11 from Jason Tibbitts 2010-11-07 10:40:12 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 502690] Review Request: mingw32-dirac - MinGW Windows dirac compression 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=502690 --- Comment #7 from Jason Tibbitts 2010-11-07 10:41:05 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 449994] Review Request: lua-expat - SAX XML parser based on the Expat 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=449994 --- Comment #9 from Jason Tibbitts 2010-11-07 10:40:25 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 451120] Review Request: lua-lpeg - Parsing Expression Grammars for Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451120 --- Comment #8 from Jason Tibbitts 2010-11-07 10:40:48 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630264] Review Request: ghc-bytestring-trie - An efficient finite map from (byte)strings to values
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630264 --- Comment #3 from Jason Tibbitts 2010-11-07 10:42:51 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 440678] Review Request: lua-socket - Network support for the Lua 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=440678 --- Comment #7 from Jason Tibbitts 2010-11-07 10:39:44 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 607584] Review Request: wordgroupz - A vocabulary building 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=607584 --- Comment #37 from Jason Tibbitts 2010-11-07 10:41:46 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 440677] Review Request: lua-posix - A POSIX library for Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=440677 --- Comment #9 from Jason Tibbitts 2010-11-07 10:39:29 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 239043] Review Request: libdc1394 - IEEE 1394 based Digital Camera control 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=239043 --- Comment #43 from Jason Tibbitts 2010-11-07 10:39:12 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 440680] Review Request: lua-logging - A simple API to use logging features in Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=440680 --- Comment #8 from Jason Tibbitts 2010-11-07 10:39:57 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648466] Review Request: php-phpunit-PHP-CodeBrowser - PHP_CodeBrowser for integration in Hudson and CruiseControl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648466 Remi Collet changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Remi Collet 2010-11-07 10:25:08 EST --- === FORMAL REVIEW === -=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 including the PHP specific items [x] Package successfully compiles and builds into binary rpms on at least one supported architecture. Tested on: F-14 x86_64 [x] Rpmlint output: php-phpunit-PHP-CodeBrowser.spec: I: checking-url http://pear.phpunit.de/get/PHP_CodeBrowser-0.9.1.tgz (timeout 10 seconds) php-phpunit-PHP-CodeBrowser.src: I: checking php-phpunit-PHP-CodeBrowser.src: I: checking-url http://github.com/mayflowergmbh/PHP_CodeBrowser (timeout 10 seconds) php-phpunit-PHP-CodeBrowser.src: I: checking-url http://pear.phpunit.de/get/PHP_CodeBrowser-0.9.1.tgz (timeout 10 seconds) php-phpunit-PHP-CodeBrowser.noarch: I: checking php-phpunit-PHP-CodeBrowser.noarch: I: checking-url http://github.com/mayflowergmbh/PHP_CodeBrowser (timeout 10 seconds) php-phpunit-PHP-CodeBrowser.noarch: W: no-manual-page-for-binary phpcb 2 packages and 1 specfiles checked; 0 errors, 1 warnings. [x] Package is not relocatable. [x] Buildroot is correct Used : %{_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: BSD [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. md5sum : 2ad66f8dc7e3ce4c7673c07a96b7bdcb PHP_CodeBrowser-0.9.1.tgz [x] Package is not known to require ExcludeArch [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. [-] ldconfig called in %post and %postun if required. [x] Package must own all directories that it creates. [x] Package requires other packages for directories it uses. [x] Package does not contain duplicates in %files. [x] Permissions on files are set properly. [x] Package has a %clean section, which contains rm -fR $RPM_BUILD_ROOT. [x] Package consistently uses macros. [x] Package contains code, or permissable content. [-] Large documentation files are in a -doc subpackage, if required. [x] Package uses nothing in %doc for runtime. [-] Header files in -devel subpackage, if present. [-] Static libraries in -devel subpackage, if present. [-] Package requires pkgconfig, if .pc files are present. [-] Development .so files in -devel subpackage, if present. [-] Fully versioned dependency in subpackages, if present. [-] Package does not contain any libtool archives (.la). [-] Package contains a properly installed %{name}.desktop file if it is a GUI application. [x] Package does not own files or directories owned by other packages. [x] Final requires /bin/sh /usr/bin/pear /usr/bin/php php-channel(pear.phpunit.de) php-common >= 5.2.6 php-pear(Console_CommandLine) >= 1.1.3 php-pear(Log) >= 1.12.1 php-pear(pear.phpunit.de/File_Iterator) >= 1.2.1 [x] Final provides php-pear(pear.phpunit.de/PHP_CodeBrowser) = 0.9.1 php-phpunit-PHP-CodeBrowser = 0.9.1-2.fc14 === SUGGESTED ITEMS === [x] Latest version is packaged. [x] Package does not include license text files separate from upstream. [-] Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x] Reviewer should test that the package builds in mock. Tested on: f14 i386 [-] Package should compile and build into binary rpms on all supported architectures. [x] Package functions as described. [x] Scriptlets must be sane, if used. [-] The placement of pkgconfig(.pc) files is correct. [-] File based requires are sane. [-] %check is present and the tests pass Koji scratch build : http://koji.fedoraproject.org/koji/taskinfo?taskID=2584398 * APPROVED * -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. _
[Bug 648466] Review Request: php-phpunit-PHP-CodeBrowser - PHP_CodeBrowser for integration in Hudson and CruiseControl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648466 --- Comment #3 from Remi Collet 2010-11-07 10:24:39 EST --- > I hope the doc location is to your liking :-) NO ! I will prefer a better solution. But this is the standard one used in fedora. This package have optional deps : "pear/Text_Highlighter" (version >= 0.7.1) "channel://pear.phpunit.de/PHPUnit" (version >= 3.4.0) "channel://pear.phpundercontrol.org/phpUnderControl" (version >= 0.5.0) "pear/PHP_CodeSniffer" (version >= 1.2.0) "pear/PhpDocumentor" (version >= 1.4.3) With pear package, I think optional dep can be make "mandatory" (ie Requires in spec, when available in the repo.) as packages are small, and don't eat memory if not used. This is the packager choice. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650643] New: Review Request: cegui06 - CEGUI library 0.6 for apps which need this specific version
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: cegui06 - CEGUI library 0.6 for apps which need this specific version https://bugzilla.redhat.com/show_bug.cgi?id=650643 Summary: Review Request: cegui06 - CEGUI library 0.6 for apps which need this specific version Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: hdego...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://people.fedoraproject.org/~jwrdegoede/cegui06.spec SRPM URL: http://people.fedoraproject.org/~jwrdegoede/cegui06-0.6.2-1.fc14.src.rpm Description: Crazy Eddie's GUI System is a free library providing windowing and widgets for graphics APIs / engines. This package contains the older version 0.6 for apps which cannot be easily ported to 0.7. As such this version has been build without additional image codecs or xml parsers. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 Flag||fedora-cvs? --- Comment #14 from Fabian Affolter 2010-11-07 09:46:27 EST --- New Package SCM Request === Package Name: python-dulwich Short Description: A python implementation of the Git file formats and protocols Owners: fab Branches: F-14 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 526805] Review Request: lua-loop - Class models for Lua
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=526805 --- Comment #2 from Michel Alexandre Salim 2010-11-07 12:52:06 EST --- Thanks for the review! EL-5 and EL-6 have reasonable Lua stacks, so (In reply to comment #1) > Just going over the old review tickets. This one builds fine and rpmlint is > silent. As expected for a ticket this old, there are some lines in the spec > which are unnecessary on modern Fedora (BuildRoot, first line of %install, the > entire %clean section) which I would recommend removing unless you plan to > target EPEL with the same spec. > EL-5 and -6 have decent Lua stacks (though -5 is a bit buggy right now), so it'd be nice to have this package there, yes. > I see a comment "# for checks" but I don't see any checks enabled and I don't > see anything that's obviously a test suite. Is there one? > Will investigate -- it's been a while! I suspect it comes from another Lua spec, for a package that does have a test suite. > Do you think that preloader.lua and precompiler.lua, which aren't in the > package-specific directory, are problematic? They don't conflict with > anything > but the names do seem quite generic. I don't know much about Lua, however, so > I'll defer to you. I'll compare it with the packaging done for LuaRocks (Lua's equivalent of CPAN) and get back to you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #26 from Christoph Wickert 2010-11-07 08:49:59 EST --- (In reply to comment #24) > I get 1e64... for > http://downloads.sourceforge.net/freerdp/freerdp-0.8.1.tar.gz > no matter what I do. How/where do you get 8a26...? I used spectool and the result I got was indeed 8a26... Must have been a bad download. I have tried 3 times and always got 1e64... > AFAICS the following says that new packages shouldn't require it explicitly. > http://fedoraproject.org/wiki/Packaging:Guidelines#Pkgconfig_Files > (http://fedoraproject.org/wiki/PackagingDrafts/PkgconfigAutoRequires) That page is still a draft and it only says that *do*not*need* it but not that you *should*not require it. As it is stil required for EPEL 4 and 5 and someone already indicated he wants to maintain this for EPEL, I suggest to add it. Anyway, not a blocker, please go ahead with the SCM admin request. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #25 from Steve Traylen 2010-11-07 08:35:56 EST --- > > FIX - SHOULD: -devel package contains a pkgconfig(.pc) files must 'Requires: > > pkgconfig'. > > AFAICS the following says that new packages shouldn't require it explicitly. > http://fedoraproject.org/wiki/Packaging:Guidelines#Pkgconfig_Files > (http://fedoraproject.org/wiki/PackagingDrafts/PkgconfigAutoRequires) you require it explicitly for EPEL4 and 5 only where it is not computed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #24 from Mads Kiilerich 2010-11-07 07:54:09 EST --- (In reply to comment #23) Thanks for the review. But ... please comment on this: > FIX - MUST: sources do not match the upstream source by MD5 > > - Upstream: 8a265ce267ea6508d30db29fd3c3c037 > - SRPM: 1e64b766874966004c07db12fe73dde8 I get 1e64... for http://downloads.sourceforge.net/freerdp/freerdp-0.8.1.tar.gz no matter what I do. How/where do you get 8a26...? > FIX - SHOULD: -devel package contains a pkgconfig(.pc) files must 'Requires: > pkgconfig'. AFAICS the following says that new packages shouldn't require it explicitly. http://fedoraproject.org/wiki/Packaging:Guidelines#Pkgconfig_Files (http://fedoraproject.org/wiki/PackagingDrafts/PkgconfigAutoRequires) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 649662] Review Request: python-tilecache - A web map tile caching system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=649662 --- Comment #10 from Michel Alexandre Salim 2010-11-07 07:46:34 EST --- Oh, and you'd want to own the /etc/TileCache directory too, since you created it and put files in it. %dir %{_sysconfdir}/TileCache should do it. You do *not* want to just simply say %{_sysconfdir}/TileCache since that will recursively include the files underneath it, and you're already handling them separately (to mark them %config(noreplace) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641572] Review Request: celt071 - Celt version 0.7.1 for mumble compatibility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641572 Leszek Matok changed: What|Removed |Added CC||l...@lac.pl --- Comment #9 from Leszek Matok 2010-11-07 07:43:33 EST --- Quick hack: rpm -i celt071-0.7.1-1.fc14.i686.rpm (built from above srpm) ln -s /usr/lib/libcelt071.so.0.0.0 /usr/lib/libcelt0.so.0.0.0 ... and Mumble seems to work Mumble searches for: celt.so.0.6.1 celt.so.0.6.1.so celt.so.0.6.2 celt.so.0.6.2.so celt.so.0.7.0 celt.so.0.7.0.so libcelt0.so.0.0.0 libcelt0.so.0.0.0.so libcelt0.so.0.6.1 libcelt0.so.0.6.1.so libcelt0.so.0.6.2 libcelt0.so.0.6.2.so libcelt0.so.0.7.0 libcelt0.so.0.7.0.so libcelt.so.0.0.0 libcelt.so.0.0.0.so libcelt.so.0.6.1 libcelt.so.0.6.1.so libcelt.so.0.6.2 libcelt.so.0.6.2.so libcelt.so.0.7.0 libcelt.so.0.7.0.so (not in this order, plus it searches for every name in many directories; not absence of 0.7.1) Since CELT 0.8.1 installs /usr/lib/libcelt0.so.1.0.0, I don't see a problem with installing this one as /usr/lib/libcelt0.so.0.0.0 out of the box for the time being. The fact that Mumble looks for 3 different versions of CELT in the system hints me that maybe it already supports multiple simultaneous CELT versions? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 649662] Review Request: python-tilecache - A web map tile caching system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=649662 --- Comment #9 from Michel Alexandre Salim 2010-11-07 07:39:27 EST --- (In reply to comment #7) > Please check the updated spec and source file, waiting for your review > comments. > > Spec URL: http://viji.fedorapeople.org/SPECS/python-tilecache.spec > SRPM URL: > http://viji.fedorapeople.org/SRPMS/python-tilecache-2.11-3.fc13.src.rpm > > #002 Please find rpmlint output, resolved all errors. > > [mockbu...@dev06 SPECS]$ rpmlint python-tilecache.spec > > python-tilecache.spec:59: W: macro-in-%changelog %clean > python-tilecache.spec: W: no-cleaning-of-buildroot %clean > python-tilecache.spec: W: no-buildroot-tag > python-tilecache.spec: W: no-%clean-section > 0 packages and 1 specfiles checked; 0 errors, 4 warnings. > > > [mockbu...@dev06 SPECS]$ rpmlint > /home/mockbuild/rpmbuild/RPMS/noarch/python-tilecache-2.11-3.fc13.noarch.rpm > > python-tilecache.noarch: W: spelling-error %description -l en_US pluggable -> > plug gable, plug-gable, plugged This can be ignored, the spell checker is rather atrocious when it comes to technical terms > python-tilecache.noarch: W: spurious-executable-perm > /usr/share/doc/python-tilecache-2.11/Example.py.txt Need to be fixed. A user should not be able to accidentally execute a file that's supposed to be documentation! Looks like the file is already marked executable in the source tarball, so you can fix it in the %prep stage; just chmod -x it there > python-tilecache.noarch: W: no-manual-page-for-binary tilecache_seed.py > python-tilecache.noarch: W: no-manual-page-for-binary tilecache.fcgi > python-tilecache.noarch: W: no-manual-page-for-binary tilecache_http_server.py > python-tilecache.noarch: W: no-manual-page-for-binary > tilecache_install_config.py > python-tilecache.noarch: W: no-manual-page-for-binary tilecache_clean.py > python-tilecache.noarch: W: no-manual-page-for-binary tilecache.cgi Can be ignored (but if you want to borrow manpages from, say, Debian (where they normally add missing manpages) that's fine too. > 1 packages and 0 specfiles checked; 0 errors, 8 warnings. > > > [mockbu...@dev06 SPECS]$ rpmlint > /home/mockbuild/rpmbuild/SRPMS/python-tilecache-2.11-3.fc13.src.rpm > > python-tilecache.src: W: spelling-error %description -l en_US pluggable -> > plug > gable, plug-gable, plugged Ignore > python-tilecache.src:59: W: macro-in-%changelog %clean A good practice is to double-comment the macros when you talk about them in %changelog -- replace %clean with %%clean > python-tilecache.src: W: no-cleaning-of-buildroot %clean > python-tilecache.src: W: no-buildroot-tag > python-tilecache.src: W: no-%clean-section Those are fine (on newer Fedora versions) > 1 packages and 0 specfiles checked; 0 errors, 5 warnings. > > #007 changelog and release details are updated properly. As Shakthi said, adding a newline between changelog entries would be greatly appreciated. Which editor do you use for editing your specfiles? I use Emacs, and there are several commands that are a lifesavers: 1. M-x rpm-add-change-log-entry Inserts a new %%changelog entry with the correct date stamp, author name and email, and version info (you might have to customize name & email using M-x customize-group, then entering group name 'rpm-spec' 2. M-x rpm-increase-release-tag increments the release tag 3. C-x h (select all) followd by M-x untabify converts all TABs in the specfile to spaces (so many specs suffer from a mix of tabs and spaces, making diff outputs hard to read) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 649662] Review Request: python-tilecache - A web map tile caching system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=649662 --- Comment #8 from Shakthi Kannan 2010-11-07 07:22:23 EST --- #007 The .spec file should be readable. Kindly give a newline between the changelog entries. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650604] New: Review Request: perl-PDF-Haru - Perl interface to Haru Free PDF Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-PDF-Haru - Perl interface to Haru Free PDF Library https://bugzilla.redhat.com/show_bug.cgi?id=650604 Summary: Review Request: perl-PDF-Haru - Perl interface to Haru Free PDF Library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: robinlee.s...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://cheeselee.fedorapeople.org/SPECS/perl-PDF-Haru.spec SRPM URL: http://cheeselee.fedorapeople.org/SRPMS/perl-PDF-Haru-1.00-1.fc14.src.rpm Description: Perl interface to Haru Free PDF Library.It supports the following features: 1. Generating PDF files with lines, text, images. 2. Outline, text annotation, link annotation. 3. Compressing document with deflate-decode. 4. Embedding PNG, Jpeg images. 5. Embedding Type1 font and TrueType font. 6. Creating encrypted PDF files. 7. Using various character sets (ISO8859-1~16, MSCP1250~8, KOI8-R). 8. Supporting CJK fonts and encodings. $ rpmlint ./RPMS/i686/perl-PDF-Haru-{,*}1.00-1.fc14.i686.rpm ./SRPMS/perl-PDF-Haru-1.00-1.fc14.src.rpm SPECS/perl-PDF-Haru.spec perl-PDF-Haru.i686: W: spelling-error %description -l en_US encodings -> encoding, encoding s, encodes perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/jpeg_demo.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/text_demo.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/text_demo2.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/line_demo.pl perl-PDF-Haru.i686: W: spelling-error %description -l en_US encodings -> encoding, encoding s, encodes perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/jpeg_demo.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/text_demo.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/text_demo2.pl perl-PDF-Haru.i686: W: spurious-executable-perm /usr/share/doc/perl-PDF-Haru-1.00/examples/line_demo.pl perl-PDF-Haru.src: W: spelling-error %description -l en_US encodings -> encoding, encoding s, encodes 4 packages and 1 specfiles checked; 0 errors, 11 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 650602] Review Request: clamsap - Virus Scan Adapter (VSA) for ClamAV
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650602 Markus Strehle changed: What|Removed |Added Group|sap, rhel_beta | Version|rawhide |el5 URL||http://sourceforge.net/proj ||ects/clamsap/ CC||markus.stre...@sap.com, ||nathan...@gnat.ca, ||n...@fedoraproject.org, ||redhat-bugzi...@linuxnetz.d ||e, st...@silug.org Component|Package Review |clamav Platform|All |x86_64 AssignedTo|nob...@fedoraproject.org|st...@silug.org Product|Fedora |Fedora EPEL -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650602] Review Request: clamsap - Virus Scan Adapter (VSA) for ClamAV
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650602 --- Comment #1 from Markus Strehle 2010-11-07 06:54:17 EST --- if possible I would use opensuse.org to provide the RPMs, see http://download.opensuse.org/repositories/home:/ehhle/ I already ask OpenSuse to make the clamsap RPMs avaiable in security Project. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650602] New: Review Request: clamsap - Virus Scan Adapter (VSA) for ClamAV
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: clamsap - Virus Scan Adapter (VSA) for ClamAV https://bugzilla.redhat.com/show_bug.cgi?id=650602 Summary: Review Request: clamsap - Virus Scan Adapter (VSA) for ClamAV Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: markus.stre...@sap.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Group: sap, rhel_beta Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://sourceforge.net/projects/clamsap/ SRPM URL: http://download.opensuse.org/repositories/home:/ehhle/ Description: This package provides two shared libraries which link SAP (NW-VSI) and ClamAV. The library clamsap links directly to libclamav engine library, the library clamdsap uses the clamd scan daemon to scan for viruses. Both libraries enable a SAP system to perform antivirus scans. An example would be a ClamAV integration in SAP NetWeaver 730 (java and ABAP) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 649662] Review Request: python-tilecache - A web map tile caching system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=649662 --- Comment #7 from viji 2010-11-07 06:41:28 EST --- Please check the updated spec and source file, waiting for your review comments. Spec URL: http://viji.fedorapeople.org/SPECS/python-tilecache.spec SRPM URL: http://viji.fedorapeople.org/SRPMS/python-tilecache-2.11-3.fc13.src.rpm #002 Please find rpmlint output, resolved all errors. [mockbu...@dev06 SPECS]$ rpmlint python-tilecache.spec python-tilecache.spec:59: W: macro-in-%changelog %clean python-tilecache.spec: W: no-cleaning-of-buildroot %clean python-tilecache.spec: W: no-buildroot-tag python-tilecache.spec: W: no-%clean-section 0 packages and 1 specfiles checked; 0 errors, 4 warnings. [mockbu...@dev06 SPECS]$ rpmlint /home/mockbuild/rpmbuild/RPMS/noarch/python-tilecache-2.11-3.fc13.noarch.rpm python-tilecache.noarch: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged python-tilecache.noarch: W: spurious-executable-perm /usr/share/doc/python-tilecache-2.11/Example.py.txt python-tilecache.noarch: W: no-manual-page-for-binary tilecache_seed.py python-tilecache.noarch: W: no-manual-page-for-binary tilecache.fcgi python-tilecache.noarch: W: no-manual-page-for-binary tilecache_http_server.py python-tilecache.noarch: W: no-manual-page-for-binary tilecache_install_config.py python-tilecache.noarch: W: no-manual-page-for-binary tilecache_clean.py python-tilecache.noarch: W: no-manual-page-for-binary tilecache.cgi 1 packages and 0 specfiles checked; 0 errors, 8 warnings. [mockbu...@dev06 SPECS]$ rpmlint /home/mockbuild/rpmbuild/SRPMS/python-tilecache-2.11-3.fc13.src.rpm python-tilecache.src: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged python-tilecache.src:59: W: macro-in-%changelog %clean python-tilecache.src: W: no-cleaning-of-buildroot %clean python-tilecache.src: W: no-buildroot-tag python-tilecache.src: W: no-%clean-section 1 packages and 0 specfiles checked; 0 errors, 5 warnings. #007 changelog and release details are updated properly. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 Christian Krause changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #13 from Christian Krause 2010-11-07 05:38:14 EST --- I have tested the latest package: all reported issues including the filtering of the provides are fixed now. -> 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 502690] Review Request: mingw32-dirac - MinGW Windows dirac compression 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=502690 --- Comment #6 from Nicolas Chauvet (kwizart) 2010-11-07 05:29:13 EST --- New Package SCM Request === Package Name: mingw32-dirac Short Description: MinGW Windows dirac compression library Owners: kwizart Branches: el5 el6 f13 f14 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 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 Christoph Wickert changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #23 from Christoph Wickert 2010-11-07 04:59:33 EST --- Sorry it took so long, this dropped of my radar. (In reply to comment #16) > I settled on freerdp providing xfreerdp, freerdp-libs with the dynamic > libraries, freerdp-plugins for "optional" plugins to libfreerdpchanman from > freerdp-libs, and freerdp-devel. Sounds good to me. REVIEW FOR 89b239734db4e419925af122ef5d0301 freerdp-0.8.1-1.fc14.src.rpm OK - MUST: $ rpmlint /var/lib/mock/fedora-14-x86_64/result/*.rpm freerdp.src: W: spelling-error %description -l en_US xrdp -> xref, hardpan, Oxnard freerdp.src: W: spelling-error %description -l en_US rdesktop -> desktop, r desktop, copydesk freerdp.x86_64: W: spelling-error %description -l en_US xrdp -> xref, hardpan, Oxnard freerdp.x86_64: W: spelling-error %description -l en_US rdesktop -> desktop, r desktop, copydesk freerdp-devel.x86_64: W: spelling-error %description -l en_US libs -> lobs, lib, lis freerdp-devel.x86_64: W: no-documentation freerdp-libs.x86_64: W: spelling-error %description -l en_US libfreerdp -> liberticide, subfreezing freerdp-libs.x86_64: W: spelling-error %description -l en_US libfreerdpchanman freerdp-libs.x86_64: W: spelling-error %description -l en_US plugins -> plug ins, plug-ins, plugging freerdp-libs.x86_64: W: spelling-error %description -l en_US libfreerdpkbd freerdp-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libfreerdp.so.0.0.0 e...@glibc_2.2.5 freerdp-plugins.x86_64: W: no-documentation 6 packages and 0 specfiles checked; 0 errors, 12 warnings. - Spelling errors can be ignored, the spelling is correct. - No documentation can be ignored, the docs are in another subpackage - shared-lib-calls-exit should be fixe upstream. Not a blocker. OK - MUST: named according to the Package Naming Guidelines OK - MUST: spec file name matches the base package %{name} OK - MUST: package meets the Packaging Guidelines OK - MUST: Fedora approved license and meets the Licensing Guidelines (GPLv2+) OK - MUST: License field in spec file matches the actual license OK - MUST: license file included in %doc OK - MUST: spec is in American English OK - MUST: spec is legible FIX - MUST: sources do not match the upstream source by MD5 - Upstream: 8a265ce267ea6508d30db29fd3c3c037 - SRPM: 1e64b766874966004c07db12fe73dde8 OK - MUST: successfully compiles and builds into binary rpms on x86_64 N/A - MUST: If the package does not successfully compile, build or work on an architecture, then those architectures should be listed in the spec in ExcludeArch. OK - MUST: all build dependencies are listed in BuildRequires. N/A - MUST: handles locales properly with %find_lang OK - MUST: Every binary RPM package (or subpackage) which stores shared library files (not just symlinks) in any of the dynamic linker's default paths, must call ldconfig in %post and %postun (freerdp-libs). OK - MUST: Package does not bundle copies of system libraries. N/A - MUST: If the package is designed to be relocatable, the packager must state this fact in the request for review, along with the rationalization for relocation of that specific package. OK - MUST: owns all directories that it creates OK - MUST: no duplicate files in the %files listing OK - MUST: Permissions on files are set properly, includes %defattr(...) OK - MUST: consistently uses macros OK - MUST: package contains code, or permissable content N/A - MUST: Large documentation files should go in a -doc subpackage OK - MUST: Files included as %doc do not affect the runtime of the application OK - MUST: Header files are in -devel package N/A - MUST: Static libraries must be in a -static package OK - MUST: library files that end in .so are in the -devel package. OK - MUST: devel packages requires the freerdp-libs package using a fully versioned dependency OK - MUST: The package does not contain any .la libtool archives. OK - The package contains a GUI application, but as xfreerdp cannot be called without arguments, the desktop file is useless OK - MUST: package does not own files or directories already owned by other packages. OK - Should: at the beginning of %install, the package runs rm -rf $RPM_BUILD_ROOT. OK - MUST: all filenames valid UTF-8 SHOULD Items: OK - SHOULD: Source package includes license text(s) as a separate file. N/A - SHOULD: The description and summary sections in the package spec file should contain translations for supported Non-English languages, if available. OK - SHOULD: builds in mock. OK - SHOULD: compiles and builds into binary rpms on all supported architectures (tested in koji). OK - SHOULD: functions as described. OK - S
[Bug 597307] Review Request: fastx_toolkit - Tools to process short-reads FASTA/FASTQ files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=597307 --- Comment #10 from Jussi Lehtola 2010-11-07 04:41:14 EST --- rpmlint output now at: fastx_toolkit.src: W: spelling-error %description -l en_US preprocessing -> reprocessing, p reprocessing, preprocessed fastx_toolkit.src: W: spelling-error %description -l en_US preprocess -> reprocess, p reprocess, procession fastx_toolkit.x86_64: W: spelling-error %description -l en_US preprocessing -> reprocessing, p reprocessing, preprocessed fastx_toolkit.x86_64: W: spelling-error %description -l en_US preprocess -> reprocess, p reprocess, procession fastx_toolkit.x86_64: W: no-manual-page-for-binary fasta_clipping_histogram.pl fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_quality_trimmer fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_quality_stats fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_quality_filter fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_trimmer fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_quality_boxplot_graph.sh fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_barcode_splitter.pl fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_quality_converter fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_nucleotide_distribution_line_graph.sh fastx_toolkit.x86_64: W: no-manual-page-for-binary fasta_formatter fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_collapser fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_renamer fastx_toolkit.x86_64: W: no-manual-page-for-binary fasta_nucleotide_changer fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_reverse_complement fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_to_fasta fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_nucleotide_distribution_graph.sh fastx_toolkit.x86_64: W: no-manual-page-for-binary fastq_masker fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_artifacts_filter fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_uncollapser fastx_toolkit.x86_64: W: no-manual-page-for-binary fastx_clipper fastx_toolkit-debuginfo.x86_64: W: spelling-error Summary(en_US) fastx -> fast, fasts, fast x fastx_toolkit-debuginfo.x86_64: W: spelling-error %description -l en_US fastx -> fast, fasts, fast x fastx_toolkit-galaxy.x86_64: W: spelling-error Summary(en_US) fastx -> fast, fasts, fast x fastx_toolkit-galaxy.x86_64: W: spelling-error %description -l en_US fastx -> fast, fasts, fast x fastx_toolkit-galaxy.x86_64: W: spelling-error %description -l en_US metagenomic -> meta genomic, meta-genomic, metagenesis fastx_toolkit-galaxy.x86_64: W: no-documentation 4 packages and 0 specfiles checked; 0 errors, 30 warnings. These are OK. Most of the issues have been fixed. However, some still remain: NEEDSWORK: - The -galaxy package contains autotools Makefiles. Get rid of them. - The license is AGPLv3+ as in libgtextutils. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 598511] Review Request: libgtextutils - Assaf Gordon text utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=598511 Jussi Lehtola changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Jussi Lehtola 2010-11-07 04:32:05 EST --- rpmlint now stands at libgtextutils.src: W: spelling-error Summary(en_US) Assaf -> Assad, Assam, Assai libgtextutils.src: W: spelling-error %description -l en_US fastx -> fast, fasts, fast x libgtextutils.x86_64: W: spelling-error Summary(en_US) Assaf -> Assad, Assam, Assai libgtextutils.x86_64: W: spelling-error %description -l en_US fastx -> fast, fasts, fast x libgtextutils-devel.x86_64: W: no-documentation 4 packages and 0 specfiles checked; 0 errors, 5 warnings. The shlib issue and the compilation flags have been fixed. (In reply to comment #8) > The query I still have is over the licensing. Rpmlint complains that the > value > you suggested - AGPLv3+ - is not valid. The choice seems to be: "AGPLv3" or > "AGPLv3 with exceptions" Well, rpmlint is not the canonical source for these kinds of issues. It's a tool targeted at all distributions using RPM, so it doesn't handle Fedora specific stuff well. The Fedora Licensing page at http://fedoraproject.org/wiki/Licensing does as well have only AGPLv1, AGPLv3 and AGPLv3 with exceptions, but all other GNU licenses seem to have the + versions that indicate that also later versions are accepted. The license of libgtextutils contains This program is free software: you can redistribute it and/or modify it under the terms of the GNU Affero General Public License as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version. so I would really put AGPLv3+, although it makes rpmlint cry. Please fix this issue before git import. The package has been APPROVED PS. One tiny nit-pick jumped to my eye: the empty line after %description is IMHO unnecessary; you don't have one in -devel. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 650181] Review Request: tkabber-plugins - Additional plugins for tkabber
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=650181 Peter Lemenkov changed: What|Removed |Added Status|NEW |ASSIGNED CC||lemen...@gmail.com AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com Flag||fedora-review? --- Comment #1 from Peter Lemenkov 2010-11-07 02:33:39 EST --- taking this for 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 593800] Review Request: python-keyring - keyring module 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=593800 --- Comment #42 from Mamoru Tasaka 2010-11-07 01:51:33 EST --- (In reply to comment #41) > > error: Group field must be present in package: python-keyring-gnome Try to add "Group:" entry for each subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 484386] Review Request: gri - A language for scientific illustration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484386 Orion Poplawski changed: What|Removed |Added Flag||needinfo?(my...@yahoo.com) --- Comment #24 from Orion Poplawski 2010-11-07 00:35:55 EDT --- D - are you still interested in pursuing 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 593800] Review Request: python-keyring - keyring module 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=593800 --- Comment #41 from Ratnadeep Debnath 2010-11-07 00:14:51 EDT --- @Mamoru Tasaka Please refer to https://koji.fedoraproject.org/koji/taskinfo?taskID=2583736 The EL-5 build is failing. Blow is the build log: Mock Version: 1.1.5 Mock Version: 1.1.5 Mock Version: 1.1.5 ENTER do(['bash', '--login', '-c', 'rpmbuild -bs --target x86_64 --nodeps builddir/build/SPECS/python-keyring.spec'], False, '/var/lib/mock/dist-5E-epel-build-920083-135491/root/', None, 86400, True, 0, 497, 497, None, logger=) Executing command: ['bash', '--login', '-c', 'rpmbuild -bs --target x86_64 --nodeps builddir/build/SPECS/python-keyring.spec'] error: Group field must be present in package: python-keyring-gnome Building target platforms: x86_64 Building for target x86_64 Child returncode was: 1 EXCEPTION: Command failed. See logs for output. # ['bash', '--login', '-c', 'rpmbuild -bs --target x86_64 --nodeps builddir/build/SPECS/python-keyring.spec'] Traceback (most recent call last): File "/usr/lib/python2.6/site-packages/mock/trace_decorator.py", line 70, in trace result = func(*args, **kw) File "/usr/lib/python2.6/site-packages/mock/util.py", line 345, in do raise mock.exception.Error, ("Command failed. See logs for output.\n # %s" % (command,), child.returncode) Error: Command failed. See logs for output. # ['bash', '--login', '-c', 'rpmbuild -bs --target x86_64 --nodeps builddir/build/SPECS/python-keyring.spec'] LEAVE do --> EXCEPTION RAISED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641572] Review Request: celt071 - Celt version 0.7.1 for mumble compatibility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641572 Peque changed: What|Removed |Added CC||peque_...@hotmail.com --- Comment #8 from Peque 2010-11-06 22:49:36 EDT --- Mumble will work if you have libcelt0.so.0.0.0 in /usr/lib (32 bits) or in /usr/lib64 (32 bits). You can get this lib compiling the sources from http://www.celt-codec.org/downloads/ (Older development releases --> celt-0.7.1.tar.gz). Files will be installed by default in /usr/local/lib, so you'll need to copy the lib. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #22 from Mads Kiilerich 2010-11-06 22:02:03 EDT --- Updated to 0.8.1 Spec URL: http://kiilerix.fedorapeople.org/freerdp.spec SRPM URL: http://kiilerix.fedorapeople.org/freerdp-0.8.1-1.fc14.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2583728 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review