[Bug 752311] New: Review Request: gitso - A front-end to reverse VNC connections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: gitso - A front-end to reverse VNC connections https://bugzilla.redhat.com/show_bug.cgi?id=752311 Summary: Review Request: gitso - A front-end to reverse VNC connections Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cedric.oliv...@free.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://cedric.olivier.free.fr/rpms/gitso/gitso.spec SRPM URL: http://cedric.olivier.free.fr/rpms/gitso/gitso-0.6-1.fc15.src.rpm Description: Gitso is a front-end to reverse VNC connections. It is meant to be a simple two-step process that connects one person to another screen. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751609] Review Request: ghc-fingertree - Haskell package provides generic finger-tree structure
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751609 --- Comment #2 from Jens Petersen 2011-11-08 20:53:17 EST --- Hi Heherson, thanks for the submission. I think I can review this - basically it looks ok but can you simplify/cleanup the description to avoid all the rpmlint warnings? http://koji.fedoraproject.org/koji/taskinfo?taskID=3498753 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #19 from Ben Thompson 2011-11-08 19:26:03 EST --- Informal review, I'm not sponsored yet. === Can you add changelog entries to the spec file? Also getting the following when building: yacc -d cfparse.y make[1]: yacc: Command not found -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751564 --- Comment #17 from Joe VLcek 2011-11-08 19:18:24 EST --- The latest changes can be found here: https://github.com/aeolusproject/audrey/tree/joev After applying the patch in attachment 532333 "make rpms" fails. It seems the man page is created: /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8/audrey.8 But then gets compressed to: /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8/audrey.8.gz Tail of "make rpms": + cp audrey_startup.py /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/bin/audrey + cp audrey.8 /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8 + ls -aFlR /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8 /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8: total 12 drwxr-xr-x. 2 jvlcek jvlcek 4096 Nov 8 19:15 ./ drwxr-xr-x. 3 jvlcek jvlcek 4096 Nov 8 19:15 ../ -rw-r--r--. 1 jvlcek jvlcek 1071 Nov 8 19:15 audrey.8 + /usr/lib/rpm/find-debuginfo.sh --strict-build-id /home/jvlcek/rpmbuild/BUILD/aeolus-audrey-agent-0.4.0 + /usr/lib/rpm/check-buildroot + /usr/lib/rpm/redhat/brp-compress + /usr/lib/rpm/redhat/brp-strip-static-archive /usr/bin/strip + /usr/lib/rpm/brp-python-bytecompile /usr/bin/python 1 + /usr/lib/rpm/redhat/brp-python-hardlink + /usr/lib/rpm/redhat/brp-java-repack-jars Processing files: aeolus-audrey-agent-0.4.0-7.fc15.noarch error: File not found: /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8/audrey.8 Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.cIYiNm + umask 022 + cd /home/jvlcek/rpmbuild/BUILD + cd aeolus-audrey-agent-0.4.0 + DOCDIR=/home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/doc/aeolus-audrey-agent-0.4.0 + export DOCDIR + /bin/mkdir -p /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/doc/aeolus-audrey-agent-0.4.0 + cp -pr COPYING /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/doc/aeolus-audrey-agent-0.4.0 + exit 0 RPM build errors: File not found: /home/jvlcek/rpmbuild/BUILDROOT/aeolus-audrey-agent-0.4.0-7.fc15.x86_64/usr/share/man/man8/audrey.8 make: *** [rpms] Error 1 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744864] Review Request: sugar-portfolio - Portfolio is a simple tool for generating slide show from starred Journal entries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744864 Peter Robinson changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|pbrobin...@gmail.com Flag||fedora-review? --- Comment #2 from Peter Robinson 2011-11-08 19:12:24 EST --- I'll review this -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747659] Review Request: proxool - Java connection pool 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=747659
Ben Thompson changed:
What|Removed |Added
CC||b...@d0g.co.uk
--- Comment #1 from Ben Thompson 2011-11-08 18:57:07 EST ---
This is an informal review, don't have a sponsor yet.
=== generic ===
OK - MUST - Package successfully compiles and builds into binary rpms on at
least one supported architecture.
OK - MUST - Permissions on files are set properly.
OK - MUST - Package does not contain duplicates in %files.
OK - MUST - Spec file lacks Packager, Vendor, PreReq tags.
OK - MUST - If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
OK - MUST - Package is named according to the Package Naming Guidelines.
OK - MUST - Rpmlint output is silent.
CHECK - MUST - Sources used to build the package matches the upstream source,
as provided in the spec URL.
proxool-0.9.1-source.tar.gz :
MD5SUM this package : f3e1d02b53c12e15282002fad50b9ef5
MD5SUM upstream package : f3e1d02b53c12e15282002fad50b9ef5
proxool.pom :
MD5SUM this package : 553a3091f43900e8ac6af9d03c6c83c2
MD5SUM upstream package : upstream source not found
OK - MUST - Spec file name must match the spec package %{name}, in the format
%{name}.spec.
OK - MUST - File names are valid UTF-8.
OK - SHOULD - Reviewer should test that the package builds in mock.
OK - SHOULD - Dist tag is present.
OK - SHOULD - SourceX is a working URL.
=== java related ===
OK - MUST - Packages have proper BuildRequires/Requires on jpackage-utils
OK - MUST - Fully versioned dependency in subpackages, if present.
OK - MUST - Javadoc documentation files are generated and included in -javadoc
subpackage
CHECK - MUST - Javadoc subpackages have Requires: jpackage-utils
OK - MUST - Javadocs are placed in %{_javadocdir}/%{name} (no -%{version}
symlink)
OK - MUST - Old add_to_maven_depmap macro is not being used
OK - MUST - Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
OK - MUST - Package DOES NOT use %update_maven_depmap in %post/%postun
OK - MUST - Packages use %{_mavenpomdir} instead of %{_datadir}/maven2/poms
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749320] Review Request: xinput_calibrator - A generic touchscreen calibration 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=749320 --- Comment #8 from Fedora Update System 2011-11-08 18:56:26 EST --- xinput_calibrator-0.7.5-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/xinput_calibrator-0.7.5-2.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749320] Review Request: xinput_calibrator - A generic touchscreen calibration 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=749320 --- Comment #9 from Fedora Update System 2011-11-08 18:58:06 EST --- xinput_calibrator-0.7.5-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/xinput_calibrator-0.7.5-2.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749320] Review Request: xinput_calibrator - A generic touchscreen calibration 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=749320 --- Comment #7 from Fedora Update System 2011-11-08 18:53:37 EST --- xinput_calibrator-0.7.5-2.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/xinput_calibrator-0.7.5-2.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 749320] Review Request: xinput_calibrator - A generic touchscreen calibration 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=749320 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 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #18 from Matěj Cepl 2011-11-08 18:10:56 EST --- http://koji.fedoraproject.org/koji/taskinfo?taskID=3498367 builds correctly for Rawhide on all architectures. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #17 from Pavel Simerda 2011-11-08 17:45:18 EST --- Created attachment 532415 --> https://bugzilla.redhat.com/attachment.cgi?id=532415 specfile -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Pavel Simerda changed: What|Removed |Added Attachment #532412|0 |1 is obsolete|| Attachment #532413|0 |1 is obsolete|| --- Comment #16 from Pavel Simerda 2011-11-08 17:44:28 EST --- Created attachment 532414 --> https://bugzilla.redhat.com/attachment.cgi?id=532414 SRPM v6 (fixed $ -> %) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Pavel Simerda changed: What|Removed |Added Attachment #532410|0 |1 is obsolete|| Attachment #532411|0 |1 is obsolete|| --- Comment #14 from Pavel Simerda 2011-11-08 17:38:28 EST --- Created attachment 532412 --> https://bugzilla.redhat.com/attachment.cgi?id=532412 SRPM with lib/lib64 fixed -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 --- Comment #10 from Fedora Update System 2011-11-08 17:37:08 EST --- perl-Net-Lite-FTP-0.61-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-Net-Lite-FTP-0.61-3.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #15 from Pavel Simerda 2011-11-08 17:39:28 EST --- Created attachment 532413 --> https://bugzilla.redhat.com/attachment.cgi?id=532413 specfile with lib/lib64 fixed -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 --- Comment #9 from Fedora Update System 2011-11-08 17:30:09 EST --- perl-Net-Lite-FTP-0.61-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-Net-Lite-FTP-0.61-3.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 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 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Pavel Simerda changed: What|Removed |Added Attachment #532408|0 |1 is obsolete|| Attachment #532409|0 |1 is obsolete|| --- Comment #12 from Pavel Simerda 2011-11-08 17:25:26 EST --- Created attachment 532410 --> https://bugzilla.redhat.com/attachment.cgi?id=532410 SRPM with flex -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746761] Review Request: mongo-java-driver - A Java driver for MongoDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746761 Alexander Kurtakov changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #13 from Pavel Simerda 2011-11-08 17:26:11 EST --- Created attachment 532411 --> https://bugzilla.redhat.com/attachment.cgi?id=532411 specfile with flex -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #11 from Pavel Simerda 2011-11-08 17:20:55 EST --- Created attachment 532409 --> https://bugzilla.redhat.com/attachment.cgi?id=532409 specfile with byacc dependency -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Pavel Simerda changed: What|Removed |Added Attachment #532406|0 |1 is obsolete|| --- Comment #10 from Pavel Simerda 2011-11-08 17:19:24 EST --- Created attachment 532408 --> https://bugzilla.redhat.com/attachment.cgi?id=532408 SRPM with byacc dependency -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746761] Review Request: mongo-java-driver - A Java driver for MongoDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746761 Jon VanAlten changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Jon VanAlten 2011-11-08 17:16:29 EST --- New Package SCM Request === Package Name: mongo-java-driver Short Description: A Java driver for MongoDB Owners: jvanalte Branches: f15 f16 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 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Matěj Cepl changed: What|Removed |Added CC||mc...@redhat.com Blocks||177841(FE-NEEDSPONSOR) AssignedTo|nob...@fedoraproject.org|mc...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #8 from Pavel Simerda 2011-11-08 17:07:14 EST --- Created attachment 532406 --> https://bugzilla.redhat.com/attachment.cgi?id=532406 Specfile with openssl dependecy added I'm adding a new specfile with openssl-devel dependency. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #9 from Pavel Simerda 2011-11-08 17:08:54 EST --- New SRPM: http://data.pavlix.net/fedora/racoon2-20100526a-2.fc15.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 752222] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=75 Pavel Simerda changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE Last Closed||2011-11-08 17:02:04 --- Comment #1 from Pavel Simerda 2011-11-08 17:02:04 EST --- *** This bug has been marked as a duplicate of bug 752223 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #7 from Pavel Simerda 2011-11-08 17:02:04 EST --- *** Bug 75 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 Ben Thompson changed: What|Removed |Added CC||b...@d0g.co.uk --- Comment #6 from Ben Thompson 2011-11-08 16:55:10 EST --- You seem to have posted this request twice: https://bugzilla.redhat.com/show_bug.cgi?id=75 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #5 from Matěj Cepl 2011-11-08 16:51:12 EST --- Created attachment 532402 --> https://bugzilla.redhat.com/attachment.cgi?id=532402 state.log -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #4 from Matěj Cepl 2011-11-08 16:50:57 EST --- Created attachment 532401 --> https://bugzilla.redhat.com/attachment.cgi?id=532401 mock_output.log -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #3 from Matěj Cepl 2011-11-08 16:50:17 EST --- Created attachment 532400 --> https://bugzilla.redhat.com/attachment.cgi?id=532400 root.log -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #2 from Matěj Cepl 2011-11-08 16:48:43 EST --- Created attachment 532398 --> https://bugzilla.redhat.com/attachment.cgi?id=532398 build log from the koji scratch build Unfortunately this package doesn't build in koji (http://koji.fedoraproject.org/koji/taskinfo?taskID=3498102). I will attach all logs from the build. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751564 --- Comment #16 from Joe VLcek 2011-11-08 16:42:41 EST --- (In reply to comment #11) > Hi Joe, > FYI, the optparse module has been deprecated since python2.7 > in favor of the new-in-2.7 argparse module. > > http://docs.python.org/library/optparse.html > > Depending on which version of Python is available in other target systems, you > may want to update. Seems this was done already with the oauth changes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750997] Review Request: Scilab - Numerical Analysis toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750997 --- Comment #3 from Clément DAVID 2011-11-08 16:34:51 EST --- Spec URL: http://davidcl.fedorapeople.org/scilab.spec SRPM URL: http://davidcl.fedorapeople.org/scilab-5.3.3-2.fc16.src.rpm Description: Scilab is a scientific software package for numerical computations providing a powerful open computing environment for engineering and scientific applications. Same rpmlint remarks than in the bug report. (In reply to comment #2) >flexdoc >= 0.5.2 (0.5.4 is in rawhide) >jrosetta-engine >= 1.0.4 (1.0.4 is in F16 updates testing) Updated flexdock and jrosetta minimal version checking I also use rpm auto dependencies instead of duplicate BuildRequires/Requires for native libs. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744347] Review Request: python-simpleparse - a parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744347 --- Comment #5 from Jon Ciesla 2011-11-08 16:22:08 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 744342] Review Request: python-configshell - Library for configuration shell development
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744342 Andy Grover changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Andy Grover 2011-11-08 16:20:59 EST --- New Package SCM Request === Package Name: python-configshell Short Description: A framework to implement simple but nice CLIs Owners: grover Branches: f16 InitialCC: ttorcz -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744342] Review Request: python-configshell - Library for configuration shell development
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744342 --- Comment #3 from Jon Ciesla 2011-11-08 16:21:31 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 744347] Review Request: python-simpleparse - a parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744347 Andy Grover changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 --- Comment #8 from Jon Ciesla 2011-11-08 16:18:16 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 744347] Review Request: python-simpleparse - a parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744347 --- Comment #4 from Andy Grover 2011-11-08 16:18:36 EST --- New Package SCM Request === Package Name: python-simpleparse Short Description: A simple and fast parser generator Owners: grover Branches: f16 InitialCC: ttorcz -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] New: Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: racoon2 - an implementation of key management system for IPsec https://bugzilla.redhat.com/show_bug.cgi?id=752223 Summary: Review Request: racoon2 - an implementation of key management system for IPsec Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pav...@pavlix.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://data.pavlix.net/fedora/racoon2.spec SRPM URL: http://data.pavlix.net/fedora/racoon2-20100526a-1.fc15.src.rpm Description: The Racoon2 project is a joint effort which provides an implementation of key management system for IPsec. The implementation is called Racoon2, a successor of Racoon, which was developed by the KAME project. It supports IKEv1, IKEv2, and KINK protocols. It works on FreeBSD, NetBSD, Linux, and Mac OS X. Racoon2 is provided under a BSD-style license. To support various environments that use IPsec, we will develop various functions. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752223] Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752223 --- Comment #1 from Pavel Simerda 2011-11-08 16:12:19 EST --- It's my first Fedora package. I'm sorry for any inconvenience. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 Sébastien Willmann changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Sébastien Willmann 2011-11-08 16:11:53 EST --- New Package SCM Request === Package Name: perl-Net-Lite-FTP Short Description: Perl FTP client with support for TLS Owners: wilqu Branches: f15 f16 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752222] New: Review Request: racoon2 - an implementation of key management system for IPsec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: racoon2 - an implementation of key management system for IPsec https://bugzilla.redhat.com/show_bug.cgi?id=75 Summary: Review Request: racoon2 - an implementation of key management system for IPsec Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pav...@pavlix.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://data.pavlix.net/fedora/racoon2.spec SRPM URL: http://data.pavlix.net/fedora/racoon2-20100526a-1.fc15.src.rpm Description: The Racoon2 project is a joint effort which provides an implementation of key management system for IPsec. The implementation is called Racoon2, a successor of Racoon, which was developed by the KAME project. It supports IKEv1, IKEv2, and KINK protocols. It works on FreeBSD, NetBSD, Linux, and Mac OS X. Racoon2 is provided under a BSD-style license. To support various environments that use IPsec, we will develop various functions. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 Pierre-YvesChibon changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | --- Comment #6 from Pierre-YvesChibon 2011-11-08 15:58:05 EST --- Removing the NEEDSPONSOR has this is not needed anymore -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744342] Review Request: python-configshell - Library for configuration shell development
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744342 Tomasz Torcz changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Tomasz Torcz 2011-11-08 15:14:14 EST --- %define could be replace by %global Review: - rpmlint is silent - *pyc, *pyo and egginfo files are properly included - md5sum of checkout matches provided sources - name is fine - license matches - it builds - do not bundle any system libraries - permissions are sane - macro use is OK Hi, I'm ttorcz and I approve 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 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749885 --- Comment #14 from Rex Dieter 2011-11-08 15:12:38 EST --- %changelog * Tue Nov 08 2011 Rex Dieter 1.0.0-0.5.20110904 - install/package qjdns Spec URL: http://rdieter.fedorapeople.org/rpms/iris/iris.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/iris/iris-1.0.0-0.5.20110904.fc16.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744347] Review Request: python-simpleparse - a parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744347 Tomasz Torcz changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Tomasz Torcz 2011-11-08 14:55:41 EST --- I find examples in %doc to be better - easy to find, and after all, those are examples, not to be dependent on. Rest of review: - *pyo and *pyc properly included - rpmlint silent (apart from wrong spelling suggestion) - package name is OK - license OK - source md5sum matches upstream - build fine on my machine - permissions are sane - macro use is consistent So, I approve this package. Thanks for packaging. BTW, my FAS account is "ttorcz". -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750997] Review Request: Scilab - Numerical Analysis toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750997 Mark Arrasmith changed: What|Removed |Added CC||arrasm...@math.wichita.edu --- Comment #2 from Mark Arrasmith 2011-11-08 14:37:53 EST --- For Fedora 16 and Scilab 5.3.3 the spec file from Clément DAVID needs additional dependencies ... flexdoc >= 0.5.2 (0.5.4 is in rawhide) jrosetta-engine >= 1.0.4 (1.0.4 is in F16 updates testing) I also want to say that you are all awesome for working on this. Just getting the dependences to build scilab is insane. So, many thanks for getting this available for Fedora. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744347] Review Request: python-simpleparse - a parser generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744347 Tomasz Torcz changed: What|Removed |Added CC||zdzi...@irc.pl AssignedTo|nob...@fedoraproject.org|zdzi...@irc.pl Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744342] Review Request: python-configshell - Library for configuration shell development
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744342 Tomasz Torcz changed: What|Removed |Added CC||zdzi...@irc.pl AssignedTo|nob...@fedoraproject.org|zdzi...@irc.pl Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749299] Review Request: lcgdm-dav - HTTP/DAV frontend to the DPM/LFC services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749299 Steve Traylen changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749232] Review Request: nagios-plugins-lcgdm - nagios probes for DPM / LFC nodes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749232 Steve Traylen changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749291] Review Request: dpm-xrootd - xroot interface to the Disk Pool Manager (DPM)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749291 Steve Traylen changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749132 Steve Traylen changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | --- Comment #16 from Steve Traylen 2011-11-08 13:55:10 EST --- You should now able to move onto: http://fedoraproject.org/wiki/Package_SCM_admin_requests Welcome. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746761] Review Request: mongo-java-driver - A Java driver for MongoDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746761 --- Comment #6 from Jon VanAlten 2011-11-08 13:57:30 EST --- (In reply to comment #5) Alex, Thanks again, and I will be sure to put changelog entries for any future changes. My account in FAS shows CLA signed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749132 Steve Traylen changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #15 from Steve Traylen 2011-11-08 13:44:59 EST --- Your 3 reviews look good and thorough. Thankyou. (In reply to comment #11) > Hi. > > Here are updated spec and source rpm. > Spec URL: http://rocha.web.cern.ch/rocha/fedora/dpm-dsi.spec > SRPM URL: http://rocha.web.cern.ch/rocha/fedora/dpm-dsi-1.8.2-2.src.rpm > > All fixed apart from: > > * LD_LIBRARY_PATH (would i be able to leave this one? it simplifies life for a > couple more months, or i can use a patch in the Fedora packaging) As discussed offline wrapping these statements with if [ -d /opt/lcg/lib ] ; then do what ever then I am happy. better still is a patch in the .spec file. Please do this before you import. > * no-documentation in dpm-dsi-devel: this was introduced by removing the %doc > from that rpm. It depends on dpm-dsi, so if i understood correctly the %doc is > not needed - i must be doing something strange to still get it This is fine, it's only a warning and is negated by the fact you pull in the main package. It's mainly a warning for you to check if there is any sensible documentation that should have included. Sources now match: $ diff -r --brief dpm-dsi-1.8.2 ../SPECS/dpm-dsi-1.8.2/ I think we are there. PACKAGE APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751820] Review Request: pius - A tool for signing and emailing all UIDs on a set of PGP keys.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751820
--- Comment #3 from Ricardo Rocha 2011-11-08 13:12:50
EST ---
Sorry about that, the issue is that i tried in a RHEL5 machine.
This is an informal review, for the purpose of my sponshorship, tracked here:
https://bugzilla.redhat.com/show_bug.cgi?id=749132
General comments:
1) I would put the actual name in the url, not using the %{name} tag (easier to
copy paste)
2) Package should probably be noarch, see below for more
3) Why do you package the spec file in the binary rpm?
4) It would be nice to get man pages.
5) And a wish, consider adding any tests to %check if you have them (not a must
of course).
+:ok, =:needs attention, -:needs fixing
MUST Items:
[-] MUST: rpmlint must be run on every package.
rpmlint is not silent:
W: summary-ended-with-dot C A tool for signing and email all UIDs on a set of
PGP keys.
- should be an obvious fix
W: spelling-error %description -l en_US keysigning -> key signing, key-signing,
designing
- key-signing?
E: no-binary
- stuff in %{_bindir} are python and perl scripts, should it be noarch?
W: no-manual-page-for-binary pius-keyring-mgr
W: no-manual-page-for-binary pius-party-worksheet
W: no-manual-page-for-binary pius
- please consider adding man pages for all commands
http://fedoraproject.org/wiki/Packaging:Guidelines#Man_pages
E: empty-debuginfo-package
- debuginfo rpm is empty, same as above, package should probably be noarch
# rpmlint -I empty-debuginfo-package
empty-debuginfo-package:
This debuginfo package contains no files. This is often a sign of binaries
being unexpectedly stripped too early during the build, rpmbuild not being
able to strip the binaries, the package actually being a noarch one but
erratically packaged as arch dependent, or something else. Verify what the
case is, and if there's no way to produce useful debuginfo out of it, disable
creation of the debuginfo package.
3 packages and 0 specfiles checked; 2 errors, 7 warnings.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}
[+] MUST: The package must meet the Packaging Guidelines. [FIXME?: covers this
list and more]
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
# md5sum pius-2.0.9.tar.bz2*
6ea4b2eabf50f2d40aabe21a22c46bc4 pius-2.0.9.tar.bz2
6ea4b2eabf50f2d40aabe21a22c46bc4 pius-2.0.9.tar.bz2_srcrpm
[=] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
- it builds fine, but see above regarding noarch
[+] 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.
[-] MUST: All build dependencies must be listed in BuildRequires
- i don't think you need python-devel in the BuildRequires (just python in
Requires)
[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.
[+] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.
[+] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review
[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example
- and %defattr not required anymore
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
- not required for >= F13
[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.
[+] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.
[+] MUST: Large documentation files should go in a doc subpackage.
[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[+] MUST: Header files must be in a -devel package.
[+] MUS
[Bug 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749132 --- Comment #14 from Ricardo Rocha 2011-11-08 13:13:56 EST --- Another information review (for sponsorship): https://bugzilla.redhat.com/show_bug.cgi?id=751820#c3 Package is pius. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711229] Review Request: ruby-spqr - easy QMF agent framework for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711229 Vít Ondruch changed: What|Removed |Added CC||vondr...@redhat.com --- Comment #4 from Vít Ondruch 2011-11-08 12:59:59 EST --- Hi, could you please enlighten me why did you packaged the library as Ruby library instead of RubyGem? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752169] New: Review Request: gnome-shell-theme-zukitwo - The Zukitwo theme for Gnome Shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: gnome-shell-theme-zukitwo - The Zukitwo theme for Gnome Shell https://bugzilla.redhat.com/show_bug.cgi?id=752169 Summary: Review Request: gnome-shell-theme-zukitwo - The Zukitwo theme for Gnome Shell Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: hal8...@hotmail.it QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://dl.dropbox.com/u/39458594/Fedora%20packages/gnome-shell-theme-zukitwo/gnome-shell-theme-zukitwo.spec SRPM URL: http://dl.dropbox.com/u/39458594/Fedora%20packages/gnome-shell-theme-zukitwo/gnome-shell-theme-zukitwo-0-1.d3df2ot.fc16.src.rpm Description: The Zukitwo theme for Gnome Shell, created by lassekongo83. This is the second package I submit for review (the first one, https://bugzilla.redhat.com/show_bug.cgi?id=751537 , was rejected due to legal reasons). Thus, I'm still seeking a sponsor. Rpmlint , executed on both the .spec and the .rpm files, returns neither errors nor warnings. I was a bit dubious about the entries "Version" and "Release" of the .spec file. I hope I've done the right thing in including the alphanumeric tag used by DeviantArt in the entry "Release". -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711230] Review Request: ruby-rhubarb - simple versioned object-graph persistence for ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711230 Vít Ondruch changed: What|Removed |Added CC||vondr...@redhat.com --- Comment #4 from Vít Ondruch 2011-11-08 12:58:41 EST --- Hi, could you please enlighten me why did you packaged the library as Ruby library instead of RubyGem? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752169] Review Request: gnome-shell-theme-zukitwo - The Zukitwo theme for Gnome Shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752169 Mattia Meneguzzo changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730232] Review Request: jboss-servlet-3.0-api - Java Servlet 3.0 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=730232 Ivan Afonichev changed: What|Removed |Added CC||ivan.afonic...@gmail.com --- Comment #4 from Ivan Afonichev 2011-11-08 11:57:44 EST --- Is there something that is absent on tomcat-servlet-3.0-api https://admin.fedoraproject.org/pkgdb/acls/name/tomcat ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751652] Review Request: rippit - The no-nonsense multimedia ripper.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751652
Ricardo Rocha changed:
What|Removed |Added
CC||rocha.po...@gmail.com
--- Comment #1 from Ricardo Rocha 2011-11-08 11:49:59
EST ---
Here's an informal review for the package.
Also part of my sponsorship process, tracked at:
https://bugzilla.redhat.com/show_bug.cgi?id=749132
General things:
1) Unless you're targeting EPEL, you can probably drop the BuildRoot.
http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag
+:ok, =:needs attention, -:needs fixing
MUST Items:
[-] MUST: rpmlint must be run on every package.
rpmlint is not silent:
rippit.x86_64: W: spelling-error %description -l en_US Bessl -> Bess, Bessel,
Bess l
rippit.x86_64: W: spelling-error %description -l en_US lossless -> loss less,
loss-less, loveless
- small things, i guess it's bells and loss-less.
rippit.src: W: invalid-url Source0: rippit-0.1.0.tar.bz2
- please provide the full url to the file, looking around i could find it at:
https://fedorahosted.org/releases/r/i/rippit/rippit-0.1.0.tar.bz2
rippit.x86_64: E: standard-dir-owned-by-package /usr/share/man/man1
- have a look at:
http://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Ownership
you should not own {%_mandir}/man1, better to explicitly:
%{_mandir}/man1/rippit.1*
rippit-debuginfo.x86_64: E: debuginfo-without-sources
# rpmlint -I debuginfo-without-sources
debuginfo-without-sources:
This debuginfo package appears to contain debug symbols but no source files.
This is often a sign of binaries being unexpectedly stripped too early during
the build, or being compiled without compiler debug flags (which again often
is a sign of distro's default compiler flags ignored which might have security
consequences), or other compiler flags which result in rpmbuild's debuginfo
extraction not working as expected. Verify that the binaries are not
unexpectedly stripped and that the intended compiler flags are used.
And indeed, from the log flags are not taken into account:
...
cd /builddir/build/BUILD/rippit-0.1.0/src && /usr/lib64/ccache/gcc -Wall
-I/bu
ilddir/build/BUILD/rippit-0.1.0/src -I/usr/include/gstreamer-0.10
-I/usr/include
/glib-2.0 -I/usr/lib64/glib-2.0/include -I/usr/include/libxml2
-I/usr/include/ne
on-o CMakeFiles/rippit.dir/rippit.c.o -c
/builddir/build/BUILD/rippit-0.1.
0/src/rippit.c
...
This is probably because you're explicitly setting CMAKE_C_FLAGS in your
CMakeLists.txt, please remove it.
3 packages and 0 specfiles checked; 2 errors, 1 warnings.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}
[=] MUST: The package must meet the Packaging Guidelines
Issues described in other parts of this review.
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[=] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
# md5sum rippit-0.1.0.tar.bz2*
09d96ada3d12c9881d634c2f0583181b rippit-0.1.0.tar.bz2
09d96ada3d12c9881d634c2f0583181b rippit-0.1.0.tar.bz2_srcrpm
Ok, but please add the proper url to the spec file.
[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[+] 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.
[+] MUST: All build dependencies must be listed in BuildRequires
[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.
[+] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.
[+] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review
[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must incl
[Bug 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749132 --- Comment #13 from Ricardo Rocha 2011-11-08 11:51:00 EST --- Pointer to an informal review (for sponshorship): https://bugzilla.redhat.com/show_bug.cgi?id=751652#c1 Package is rippit. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711313] Review Request: wicd-kde - a Wicd client built on the KDE Development Platform
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711313
--- Comment #33 from Kevin Kofler 2011-11-08 11:41:50
EST ---
Or I guess:
make install/fast DESTDIR=%{buildroot} -C %{_target_platform}
without the quotes is good enough (and consistent with the other uses of
%{buildroot}).
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711313] Review Request: wicd-kde - a Wicd client built on the KDE Development Platform
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=711313
--- Comment #32 from Kevin Kofler 2011-11-08 11:38:50
EST ---
Re the Conflicts, I don't care either way. The files don't conflict but the
functionality does. But in any case, that is an issue with the wicd core and
not related to this review.
Unfortunately, I see some issues remaining with the specfile:
> BuildRoot: %{_tmppath}/%{name}-%{version}
As Gregor already pointed out, please either remove this line or use one of the
lines from:
https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag
(The RPM in current versions of Fedora ignores BuildRoot entirely.)
> BuildRequires: qt-devel
Better use qt4-devel, which:
* also works on old versions where Qt 3 was the default,
* will also work on future versions where Qt 5 will be the default and
* allows specifying a version without an Epoch, i.e. these will work if some
future version of wicd-kde starts requiring Qt 4.8 for whatever reason:
BuildRequires: qt4-devel >= 4.8.0
or:
BuildRequires: qt-devel >= 1:4.8.0
but this will not:
BuildRequires: qt-devel >= 4.8.0
and the Epoch tends to get forgotten in such cases.
> BuildRequires: kdebase-devel
Are you sure you want kdebase-devel? I think you want kdelibs-devel, or better
kdelibs4-devel (same as for Qt above).
> %make_install -C %{_target_platform}
The %make_install macro should not be used in Fedora (except for broken legacy
tarballs which do not support DESTDIR nor INSTALL_ROOT, but all CMake-based
projects support DESTDIR), please use:
make install/fast DESTDIR="%{buildroot}" -C %{_target_platform}
instead. (For non-CMake projects, use just "install" instead of "install/fast".
"install/fast" is a CMake target which skips the checks for whether everything
to be installed is already built. We already run "make" in %build, so we can
rely on everything already being built by the time we get to %install.)
And you should use either %{buildroot} or ${RPM_BUILD_ROOT} consistently rather
than mixing the two.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751564
--- Comment #15 from Jim Meyering 2011-11-08 11:34:22 EST
---
Created attachment 532333
--> https://bugzilla.redhat.com/attachment.cgi?id=532333
generate man page
Joe,
Once you've added a --version option, apply the patch above.
Then, you'll have an automatically-generated man page.
However, please also change the minimal "Audrey Start"
description to be more like the 2-3-line description
in the RPM spec file. Otherwise, that two-word description
gets propagated to the man page like this:
AUDREY(8) System Administration Utilities AUDREY(8)
NAME
audrey - Aeolus startup agent
DESCRIPTION
usage: audrey [-h] [-e ENDPOINT] [-k OAUTH_KEY] [-s OAUTH_SECRET] [-p]
[-L {DEBUG,INFO,WARNING,ERROR,CRITICAL}] [--version]
Audrey Start
optional arguments:
-h, --help
show this help message and exit
-e ENDPOINT, --endpoint ENDPOINT
Config Server endpoint url
-k OAUTH_KEY, --key OAUTH_KEY
oAuth Key
-s OAUTH_SECRET, --secret OAUTH_SECRET
oAuth Secret
-p, --pwd
Log and look for configs in pwd
-L {DEBUG,INFO,WARNING,ERROR,CRITICAL}, --log_level {DEBUG,INFO,WARN-
ING,ERROR,CRITICAL}
Logging Level
--version
show program's version number and exit
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749885 --- Comment #13 from Kevin Kofler 2011-11-08 11:11:47 EST --- Ouch, 2 dependencies of the same program bundling the same library is a symbol conflict waiting to happen! This ought to be brought to upstream's attention. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749885 --- Comment #12 from Rex Dieter 2011-11-08 11:03:22 EST --- OK, ironically, another of tomahawk's bundled libs, jreen, also bundles qjdns, so looks like a subpkg is on the way. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743411] Review Request: drupal7-theme-ninesixty - Ninesixty theme for Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743411 --- Comment #7 from Fedora Update System 2011-11-08 10:29:48 EST --- drupal7-theme-ninesixty-1.0-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/drupal7-theme-ninesixty-1.0-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743411] Review Request: drupal7-theme-ninesixty - Ninesixty theme for Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743411 --- Comment #5 from Fedora Update System 2011-11-08 10:28:58 EST --- drupal7-theme-ninesixty-1.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/drupal7-theme-ninesixty-1.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743411] Review Request: drupal7-theme-ninesixty - Ninesixty theme for Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743411 --- Comment #6 from Fedora Update System 2011-11-08 10:29:17 EST --- drupal7-theme-ninesixty-1.0-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/drupal7-theme-ninesixty-1.0-1.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 743411] Review Request: drupal7-theme-ninesixty - Ninesixty theme for Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743411 --- Comment #4 from Fedora Update System 2011-11-08 10:26:59 EST --- drupal7-theme-ninesixty-1.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/drupal7-theme-ninesixty-1.0-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743411] Review Request: drupal7-theme-ninesixty - Ninesixty theme for Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743411 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 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 --- Comment #10 from Fedora Update System 2011-11-08 10:25:27 EST --- drupal7-theme-ninesixty-1.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/drupal7-theme-ninesixty-1.0-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 --- Comment #8 from Fedora Update System 2011-11-08 10:24:01 EST --- drupal7-diff-2.0-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/drupal7-diff-2.0-1.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 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 --- Comment #6 from Fedora Update System 2011-11-08 10:22:36 EST --- drupal7-diff-2.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/drupal7-diff-2.0-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 --- Comment #7 from Fedora Update System 2011-11-08 10:23:21 EST --- drupal7-diff-2.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/drupal7-diff-2.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743410] Review Request: drupal7-login_destination - Customize login landing page in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743410 --- Comment #5 from Fedora Update System 2011-11-08 10:19:47 EST --- drupal7-login_destination-1.0-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/drupal7-login_destination-1.0-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743410] Review Request: drupal7-login_destination - Customize login landing page in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743410 --- Comment #7 from Fedora Update System 2011-11-08 10:20:35 EST --- drupal7-login_destination-1.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/drupal7-login_destination-1.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 --- Comment #9 from Fedora Update System 2011-11-08 10:24:31 EST --- drupal7-diff-2.0-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/drupal7-diff-2.0-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743410] Review Request: drupal7-login_destination - Customize login landing page in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743410 --- Comment #6 from Fedora Update System 2011-11-08 10:20:08 EST --- drupal7-login_destination-1.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/drupal7-login_destination-1.0-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 743409] Review Request: drupal7-diff - Show diff-type changes in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743409 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 743410] Review Request: drupal7-login_destination - Customize login landing page in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743410 --- Comment #4 from Fedora Update System 2011-11-08 10:19:23 EST --- drupal7-login_destination-1.0-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/drupal7-login_destination-1.0-1.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 743410] Review Request: drupal7-login_destination - Customize login landing page in Drupal 7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743410 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 711313] Review Request: wicd-kde - a Wicd client built on the KDE Development Platform
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711313 --- Comment #31 from Minh Ngo 2011-11-08 10:18:42 EST --- @Gregor @Kevin Thank you for helping -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747050] Review Request: python-postman - cli for working with Amazon SES
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747050 --- Comment #19 from Fedora Update System 2011-11-08 10:11:59 EST --- python-postman-0.6.0-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/python-postman-0.6.0-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747050] Review Request: python-postman - cli for working with Amazon SES
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747050 --- Comment #18 from Fedora Update System 2011-11-08 10:10:26 EST --- python-postman-0.6.0-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-postman-0.6.0-1.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 751093] Review Request: jetty-artifact-remote-resources - Jetty toolchain artifact remote resources
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751093 --- Comment #10 from Jon Ciesla 2011-11-08 10:12: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 747050] Review Request: python-postman - cli for working with Amazon SES
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747050 --- Comment #20 from Fedora Update System 2011-11-08 10:12:40 EST --- python-postman-0.6.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-postman-0.6.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751564 --- Comment #14 from Jim Meyering 2011-11-08 10:01:20 EST --- sure. Thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751820] Review Request: pius - A tool for signing and emailing all UIDs on a set of PGP keys.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751820
--- Comment #2 from Praveen Kumar 2011-11-08
10:03:48 EST ---
(In reply to comment #1)
> I had a look at the package to try to do an informal review (as part of my
> sponsorship process). It seems to fail during build:
> ...
> + install -pdm 755 /usr/bin
> install: cannot change permissions of `/usr/bin': Operation not permitted
> error: Bad exit status from /var/tmp/rpm-tmp.10463 (%install)
> RPM build errors:
> Bad exit status from /var/tmp/rpm-tmp.10463 (%install)
> Child returncode was: 1
> EXCEPTION: Command failed. See logs for output.
>
> which looks consistent, you're trying to change permissions of /usr/bin. Maybe
> remove this line in the spec?
> ...
Look like you didn't put spec file and source in correct location, can you
please tell me process, how did you got this error.
> %install
> install -pdm 755 $RPM_BUILD_ROOT%{_bindir}
This is create directory tree in /home/YourUsername/rpmbuild/buildroot/usr/bin
so it's not creating problem
>
> I can have another look once it builds.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665733] Review Request: Coin3 - High-level 3D visualization 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=665733 --- Comment #5 from Richard Shaw 2011-11-08 09:37:39 EST --- Ralf, Can you respond to comment #3? I did a quick search: # repoquery --whatrequires Coin2 Coin2-devel-0:2.5.0-10.fc15.i686 Coin2-devel-0:2.5.0-10.fc15.x86_64 SIMVoleon-0:2.0.1-11.fc15.i686 SIMVoleon-0:2.0.1-11.fc15.x86_64 SoQt-0:1.5.0-2.fc15.i686 SoQt-0:1.5.0-2.fc15.x86_64 And it appears that only SoQt and SIMVoleon require Coin2 and I've successfully built them against Coin3. Is there a reason we can just make this package "Coin" and obsolete Coin2? Thanks, Richard -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752089] New: Review Request: rubygem-wirble - Handful of common Irb features, made easy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-wirble - Handful of common Irb features, made easy https://bugzilla.redhat.com/show_bug.cgi?id=752089 Summary: Review Request: rubygem-wirble - Handful of common Irb features, made easy Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dpie...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://mcpierce.fedorapeople.org/rpms/rubygem-wirble.spec SRPM URL: http://mcpierce.fedorapeople.org/rpms/rubygem-wirble-0.1.3-1.fc15.src.rpm Description: Handful of common Irb features, made easy. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 741494] Review Request: sugar-pukllanapac - A sliding puzzle game
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=741494
Thibault North changed:
What|Removed |Added
CC||thibault.no...@gmail.com
--- Comment #1 from Thibault North 2011-11-08
09:32:10 EST ---
Output of rpmlint -v:
sugar-pukllanapac.src: I: checking
sugar-pukllanapac.src: W: non-standard-group Sugar/Activities
sugar-pukllanapac.src: I: checking-url
http://wiki.sugarlabs.org/go/Activities/Pukllanapac (timeout 10 seconds)
sugar-pukllanapac.src: I: checking-url
http://download.sugarlabs.org/sources/honey/Pukllanapac/Pukllanapac-7.tar.bz2
(timeout 10 seconds)
sugar-pukllanapac.src: W: file-size-mismatch Pukllanapac-7.tar.bz2 = 27445,
http://download.sugarlabs.org/sources/honey/Pukllanapac/Pukllanapac-7.tar.bz2 =
27117
sugar-pukllanapac.noarch: I: checking
sugar-pukllanapac.noarch: W: non-standard-group Sugar/Activities
sugar-pukllanapac.noarch: I: checking-url
http://wiki.sugarlabs.org/go/Activities/Pukllanapac (timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 3 warnings.
A few comments:
- Upstream source doesn't match the tar archive of the source package
- You can remove the macros for %{__python} and just use python
- gettext is a buildrequire
- defattr is now optional
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 741494] Review Request: sugar-pukllanapac - A sliding puzzle game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=741494 Thibault North changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|thibault.no...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
