[Bug 695058] Review Request: transgui - An App to remotely control a Transmission Bit-Torrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=695058 Martin Gieseking changed: What|Removed |Added CC||martin.giesek...@uos.de --- Comment #19 from Martin Gieseking 2011-11-12 02:53:35 EST --- Would you like to update the package to the latest upstream release? There's also a source tarball available now. Just two quick comments: - Currently, the debuginfo package is empty. It's probably because of the fpc flag -g- applied in the Makefiles. You should patch them appropriately. - Also, drop the Encoding entry from the .desktop file as it's deprecated. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751925] Review Request: python-tables - Hierarchical datasets in Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751925 Brendan Jones changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|brendan.jones...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 739417] Review Request: php53-php-gettext - Gettext emulation in PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=739417 Fedora Update System changed: What|Removed |Added Status|ON_QA |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 750013] Review Request: clucene09 - A C++ port of Lucene
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750013 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System 2011-11-11 22:31:18 EST --- clucene09-0.9.21b-2.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673661] Review Request: R-ALL - Data of T- and B-cell Acute Lymphocytic Leukemia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673661 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System 2011-11-11 22:27:29 EST --- Package R-ALL-1.4.11-1.fc15: * should fix your issue, * was pushed to the Fedora 15 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing R-ALL-1.4.11-1.fc15' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-15798 then log in and leave karma (feedback). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746415] Review Request: ghc-MonadCatchIO-transformers - Exception handling with IO monad transformers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746415 Fedora Update System changed: What|Removed |Added Fixed In Version|ghc-MonadCatchIO-transforme |ghc-MonadCatchIO-transforme |rs-0.2.2.3-1.fc14 |rs-0.2.2.3-1.fc16 --- Comment #16 from Fedora Update System 2011-11-11 22:28:42 EST --- ghc-MonadCatchIO-transformers-0.2.2.3-1.fc16 has been pushed to the Fedora 16 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #48 from Fedora Update System 2011-11-11 22:27:57 EST --- wicd-kde-0.2.3-3.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746415] Review Request: ghc-MonadCatchIO-transformers - Exception handling with IO monad transformers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746415 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-MonadCatchIO-transforme ||rs-0.2.2.3-1.fc15 Resolution||ERRATA Last Closed||2011-11-11 22:25:53 --- Comment #14 from Fedora Update System 2011-11-11 22:25:53 EST --- ghc-MonadCatchIO-transformers-0.2.2.3-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746415] Review Request: ghc-MonadCatchIO-transformers - Exception handling with IO monad transformers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746415 Fedora Update System changed: What|Removed |Added Fixed In Version|ghc-MonadCatchIO-transforme |ghc-MonadCatchIO-transforme |rs-0.2.2.3-1.fc15 |rs-0.2.2.3-1.fc14 --- Comment #15 from Fedora Update System 2011-11-11 22:26:03 EST --- ghc-MonadCatchIO-transformers-0.2.2.3-1.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 --- Comment #6 from Jon Ciesla 2011-11-11 20:30:00 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 753182] Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753182 --- Comment #3 from Jon Ciesla 2011-11-11 20:25:30 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 732146] Review Request: simfqt - C++ Simulated Fare Quote System 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=732146 --- Comment #5 from Jon Ciesla 2011-11-11 20:20:53 EST --- Git done (by process-git-requests). Not accepting new f14 branches. Also, , etc, not a valid FAS account. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749757] Review Request: zita-rev1 - Proaudio reverb for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749757 --- Comment #10 from Jon Ciesla 2011-11-11 20:23:18 EST --- Git done (by process-git-requests). Thibault, please take ownership of vreview BZs. 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 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753181 --- Comment #4 from Jon Ciesla 2011-11-11 20:24:27 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 491550] Review Request: libass - Portable library for SSA/ASS subtitles rendering
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491550 --- Comment #11 from Jon Ciesla 2011-11-11 20:19:34 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 753354] New: Review Request: strongswan - IKEv1 and IKEv2 daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: strongswan - IKEv1 and IKEv2 daemon https://bugzilla.redhat.com/show_bug.cgi?id=753354 Summary: Review Request: strongswan - IKEv1 and IKEv2 daemon 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/strongswan.spec SRPM URL: http://data.pavlix.net/fedora/strongswan-4.6.0-1.fc15.src.rpm Description: The strongSwan 4.6 branch supports both the IKEv1 and IKEv2 key exchange protocols in conjunction with the native NETKEY IPsec stack of the Linux kernel. Please test with permissive selinux for now. I appreciate any help making selinux happy with this package. Strongswan is not Openswan. The main difference that prevents me from using Openswan is that Openswan doesn't work well with IPv6 (just try to use road warrior mode). Therefore I believe there are use cases for Strongswan in Fedora. Strongswan and Racoon2 are the only two implementation that fully implement IPv6 and IKEv2 that I know. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 732146] Review Request: simfqt - C++ Simulated Fare Quote System 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=732146 Denis Arnaud changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Denis Arnaud 2011-11-11 18:04:20 EST --- Thank you very much for that review, Martin! I shall simply remove the INSTALL file from the source, i.e. upstream, as the build system is now based on CMake, and INSTALL is no longer relevant. --- New Package SCM Request === Package Name: simfqt Short Description: C++ Simulated Fare Quote System Library Owners: denisarnaud Branches: f14 f15 f16 el4 el5 el6 InitialCC: denisarnaud --- -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #33 from Pavel Simerda 2011-11-11 17:59:55 EST --- (In reply to comment #28) > (In reply to comment #26) > > Is there a reason we would ship both this and ipsec-tools? > > Ah, I overlooked the --disable-kinkd option, so just the iked.8 might come in > useful. Please feel free to test without --disable-kinkd. It didn't compile for me, so I disabled it for the beginning. Of course I would be happy to have it enabled too. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #32 from Pavel Simerda 2011-11-11 17:57:54 EST --- (In reply to comment #26) > Is there a reason we would ship both this and ipsec-tools? There definitely is. Racoon2 and Racoon are two very different projects. Racoon is well known and used in some environments, ipsec howto includes Racoon configuration. Racoon2 supports IKEv2 which Racoon doesn't. Therefore, both of them are useful. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491550] Review Request: libass - Portable library for SSA/ASS subtitles rendering
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491550 Martin Sourada changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Martin Sourada 2011-11-11 17:48:16 EST --- Package Change Request == Package Name: libass New Branches: el5 el6 Owners: mso 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 752909] Review Request: libverto - Main loop abstraction 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=752909 Nathaniel McCallum changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2011-11-11 17:17:59 --- Comment #8 from Nathaniel McCallum 2011-11-11 17:17:59 EST --- 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 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 #31 from Tomas Mraz 2011-11-11 17:14:40 EST --- I have no problem with shipping both racoon2 and ipsec-tools. Ipsec-tools seem to be more stable base on the other hand racoon2 has multiple new features that ipsec-tools do not have. So both have their own opportunities to be used. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751344] Review Request: sesame - Red Hat MRG management system 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=751344 --- Comment #2 from Trevor McKay 2011-11-11 16:54:02 EST --- Spec URL: http://tmckay.fedorapeople.org/sesame.spec SRPM URL: http://tmckay.fedorapeople.org/sesame-1.1-1.fc15.src.rpm Updated to use systemd. Note, tried to put a source tarball for sesame under https://fedorahosted.org/releases/c/u/cumin/, since the source for sesame is actually part of the cumin source tree and sesame does not have a fedora project of its own. However, the system wouldn't let me. Thoughts, short of making sesame its own fedora project? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 732146] Review Request: simfqt - C++ Simulated Fare Quote System 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=732146
Martin Gieseking changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||martin.giesek...@uos.de
AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
Flag||fedora-review+
--- Comment #3 from Martin Gieseking 2011-11-11
16:27:33 EST ---
Hi Denis,
the package looks fine. Just remove the INSTALL file from the docs. It's added
to the package because you install it with "make install" into
%{_docdir}/%{name}-%{version}. Files in this directory are considered a part of
the package docs, and are added even if not listed in %files. The easiest way
to fix this is to insert
rm -rf $RPM_BUILD_ROOT%{_docdir}/%{name}-%{version}/
after the mv statement in %install.
Please fix this before you check in the package.
$ rpmlint ./simfqt-*.rpm
simfqt.x86_64: W: install-file-in-docs /usr/share/doc/simfqt-0.1.1/INSTALL
5 packages and 0 specfiles checked; 0 errors, 1 warnings.
-
key:
[+] OK
[.] OK, not applicable
[X] needs work
-
[+] 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.
[+] MUST: The package must be licensed with a Fedora approved license.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The 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.
$ md5sum simfqt-0.1.1.tar.bz2*
85c626f3d7c5183bc983eff3fc69c691 simfqt-0.1.1.tar.bz2
85c626f3d7c5183bc983eff3fc69c691 simfqt-0.1.1.tar.bz2.upstream
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied.
[.] MUST: The spec file MUST handle locales properly.
[+] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[+] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[+] MUST: Large documentation files must go in a -doc subpackage.
[.] MUST: Files in %doc must not affect the runtime of the application.
[+] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[+] MUST: If a package contains library files with a suffix, then .so files
without suffix must go in a -devel package.
[+] MUST: devel packages must require the base package using a fully versioned
dependency.
[+] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.
EPEL <= 5 only:
[+] MUST: The spec file must contain a valid BuildRoot field.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot}.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot}.
[+] MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
[+] SHOULD: The reviewer should test that the package functions as described.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[+] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[+] SHOULD: If the package has
[Bug 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753181 Tom "spot" Callaway changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Tom "spot" Callaway 2011-11-11 15:46:33 EST --- New Package SCM Request === Package Name: R-RCurl Short Description: General network (HTTP/FTP) client interface for R Owners: spot 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 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 Tom "spot" Callaway changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Tom "spot" Callaway 2011-11-11 15:48:13 EST --- New Package SCM Request === Package Name: R-Rcompression Short Description: R package for in-memory compression Owners: spot 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 753182] Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753182 Tom "spot" Callaway changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Tom "spot" Callaway 2011-11-11 15:47:29 EST --- New Package SCM Request === Package Name: R-biomaRt Short Description: R interface to BioMart databases Owners: spot 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 751172] Review Request: cumin - management console for Red Hat MRG grid
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751172 --- Comment #4 from Trevor McKay 2011-11-11 15:19:03 EST --- Spec URL: http://tmckay.fedorapeople.org/cumin.spec SRPM URL: http://tmckay.fedorapeople.org/cumin-0.1.5137-1.fc15.src.rpm Updated to use systemd. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #30 from Matěj Cepl 2011-11-11 15:12:40 EST --- (In reply to comment #29) > Well I'm just getting started and my reviews are "informal", as I am not yet > sponsored. Just practising, let me know if it's not helping or you have some > advice, thanks! Sure, make a complete review here, please, (https://fedoraproject.org/wiki/Packaging:ReviewGuidelines) and let me know. I cannot sponsor you, but certainly every done review is helpful. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #36 from Fedora Update System 2011-11-11 14:55:15 EST --- dpm-dsi-1.8.2-4.el5 has been pushed to the Fedora EPEL 5 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System 2011-11-11 14:55:29 EST --- python-straight-plugin-1.1.1-0.7.2010.git57ef11c.el6 has been pushed to the Fedora EPEL 6 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #29 from Ben Thompson 2011-11-11 14:55:25 EST --- (In reply to comment #27) > (In reply to comment #25) > > I've had a look at previous packages and the remaining errors and warnings > > seem > > OK to me. There are a few man pages you've forgotten to add to the spec > > file: > > racoon2-20100526a/iked/iked.8 > > racoon2-20100526a/kinkd/kinkd.8 > > Ben, do you want to take over review of this bug? Well I'm just getting started and my reviews are "informal", as I am not yet sponsored. Just practising, let me know if it's not helping or you have some advice, 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 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 Flag|needinfo?(b...@d0g.co.uk)| --- Comment #28 from Ben Thompson 2011-11-11 14:52:49 EST --- (In reply to comment #26) > Is there a reason we would ship both this and ipsec-tools? Ah, I overlooked the --disable-kinkd option, so just the iked.8 might come in useful. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713461] Review Request: tomahawk - The Social Music Player
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713461 Magnus Tuominen changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|magnus.tuomi...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 Flag||needinfo?(b...@d0g.co.uk) --- Comment #27 from Matěj Cepl 2011-11-11 14:33:00 EST --- (In reply to comment #25) > I've had a look at previous packages and the remaining errors and warnings > seem > OK to me. There are a few man pages you've forgotten to add to the spec file: > racoon2-20100526a/iked/iked.8 > racoon2-20100526a/kinkd/kinkd.8 Ben, do you want to take over review 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 673660] Review Request: R-RCurl - General network (HTTP/FTP/...) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673660 Bug 673660 depends on bug 673658, which changed state. Bug 673658 Summary: Review Request: R-Rcompression - In-memory decompression for GNU zip and bzip2 formats. https://bugzilla.redhat.com/show_bug.cgi?id=673658 What|Old Value |New Value Resolution||DUPLICATE Status|NEW |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673658] Review Request: R-Rcompression - In-memory decompression for GNU zip and bzip2 formats.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673658 Pierre-YvesChibon changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE Last Closed||2011-11-11 12:40:38 --- Comment #7 from Pierre-YvesChibon 2011-11-11 12:40:38 EST --- *** This bug has been marked as a duplicate of bug 753185 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673660] Review Request: R-RCurl - General network (HTTP/FTP/...) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673660 Pierre-YvesChibon changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE Last Closed||2011-11-11 13:07:52 --- Comment #1 from Pierre-YvesChibon 2011-11-11 13:07:52 EST --- *** This bug has been marked as a duplicate of bug 753181 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753182] Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753182
Pierre-YvesChibon changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review+
--- Comment #1 from Pierre-YvesChibon 2011-11-11 13:53:54
EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at least
one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[-]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package matches the upstream source, as
provided in the spec URL.
MD5SUM this package : a2d53e989f1aee40668b3acbc0b7a188
MD5SUM upstream package : a2d53e989f1aee40668b3acbc0b7a188
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[-]: SHOULD Package does not include license text files separate from upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
Checked on x86_64
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[x]: SHOULD Spec use %global instead of %define.
R
[x]: MUST Package contains the mandatory BuildRequires.
[x]: MUST The package has the standard %build section.
[x]: MUST The package owns the created directory.
[x]: MUST Package have the default element marked as %doc :
[x]: MUST Package requires R-core.
[x]: SHOULD The %check macro is present
[x]: SHOULD Latest version is packaged.
This package is approved
Two remarks:
- I trust you want to change the URL (It's bad to copy/paste when you have
R2spec to do the work for you ;-))
- You might want to change the Source0 url to:
http://www.bioconductor.org/packages/release/bioc/src/contrib/%{packname}_%{version}.tar.gz
It'll make it more stable accross Bioconductor releases.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mail
[Bug 411421] Review Request: d-feet - D-Bus debugger
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=411421 --- Comment #17 from Jon Ciesla 2011-11-11 12:33:34 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 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753188
Pierre-YvesChibon changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review?
--- Comment #1 from Pierre-YvesChibon 2011-11-11 13:47:19
EST ---
The sig.h, portimpl.h, hmmstats.h and cheapcgi.h contain the header:
/*
* Copyright (C) 2000 Jim Kent. This source code may be freely used *
* for personal, academic, and non-profit purposes. Commercial use *
* permitted only by explicit agreement with Jim Kent (jim_k...@pacbell.net) *
*/
I will proceed with the review and we'll see if we can figure this licensing
issue.
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
C/C++
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contains kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at least
one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[-]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
[x]: MUST Sources used to build the package matches the upstream source, as
provided in the spec URL.
MD5SUM this package : 130af70245801e131b0ebbda8fd83fde
MD5SUM upstream package : 130af70245801e131b0ebbda8fd83fde
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[-]: SHOULD Package does not include license text files separate from upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
Checked on x86_64
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
files.
[x]: SHOULD Spec use %global instead of %define.
R
[x]: MUST Package contains the mandatory BuildRequires.
[x]: MUST The package has the standard %build section.
[x]: MUST The package owns the created directory.
[x]: MUST Pack
[Bug 749757] Review Request: zita-rev1 - Proaudio reverb for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749757 Brendan Jones changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Brendan Jones 2011-11-11 13:43:33 EST --- New Package SCM Request === Package Name: zita-rev1 Short Description: Proaudio reverb for JACK Owners: bsjones 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 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753181
Pierre-YvesChibon changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr
Flag||fedora-review+
--- Comment #2 from Pierre-YvesChibon 2011-11-11 13:39:03
EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
C/C++
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contains kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at least
one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[-]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
$ rpmlint /home/pierrey/rpmbuild/SRPMS/R-RCurl-1.7.0-1.fc16.src.rpm
/home/pierrey/rpmbuild/RPMS/x86_64/R-RCurl-1.7.0-1.fc16.x86_64.rpm
/home/pierrey/rpmbuild/RPMS/x86_64/R-RCurl-debuginfo-1.7.0-1.fc16.x86_64.rpm
R-RCurl.src: W: spelling-error %description -l en_US ldap -> lap, leap, dapple
R-RCurl.x86_64: W: spelling-error %description -l en_US ldap -> lap, leap,
dapple
R-RCurl.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/R/library/RCurl/examples/post.c
R-RCurl.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/R/library/RCurl/examples/curl.c
R-RCurl.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/R/library/RCurl/enums/Renums.c
3 packages and 0 specfiles checked; 0 errors, 5 warnings.
This is safe to ignore according to the guideline
[x]: MUST Sources used to build the package matches the upstream source, as
provided in the spec URL.
MD5SUM this package : a2d53e989f1aee40668b3acbc0b7a188
MD5SUM upstream package : a2d53e989f1aee40668b3acbc0b7a188
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[x]: SHOULD Package functions as described.
[-]: SHOULD Package does not include license text files separate from upstream.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[-]: SHOULD Package should compile and build into binary rpms on all supported
architectures.
Checked on x86_64
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try
[Bug 752909] Review Request: libverto - Main loop abstraction 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=752909 --- Comment #7 from Jon Ciesla 2011-11-11 13:36:22 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 752909] Review Request: libverto - Main loop abstraction 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=752909 Stephen Gallagher changed: What|Removed |Added Flag|fedora-review? |fedora-review+, fedora-cvs? --- Comment #5 from Stephen Gallagher 2011-11-11 13:31:03 EST --- Package is approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752909] Review Request: libverto - Main loop abstraction 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=752909 --- Comment #6 from Stephen Gallagher 2011-11-11 13:32:44 EST --- New Package SCM Request === Package Name: libverto Short Description: Main loop abstraction library Owners: npmccallum Branches: 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 753262] New: Review Request: qxmpp-dev - Qt XMPP Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: qxmpp-dev - Qt XMPP Library https://bugzilla.redhat.com/show_bug.cgi?id=753262 Summary: Review Request: qxmpp-dev - Qt XMPP Library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: nlmin...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: https://raw.github.com/Ignotus/qxmpp-fedora/7f46c9159093e9228d35d829db4da91aa736c8a9/qxmpp-dev.spec SRPM URL: https://github.com/Ignotus/qxmpp-fedora/blob/7f46c9159093e9228d35d829db4da91aa736c8a9/qxmpp-dev-0.3.45.1-3.fc16.src.rpm?raw=true Description: QXmpp is a cross-platform C++ XMPP client library. It is based on Qt and C++. QXmpp is pretty intuitive and easy to use. It uses Qt extensively. Qt is the only third party library it is dependent on. Users need to a have working knowledge of C++ and Qt basics (Signals and Slots and Qt data types). The underlying TCP socket and the XMPP RFCs (RFC3920 and RFC3921) have been encapsulated into classes and functions. Therefore the user would not be bothered with these details. But it is always recommended to the advanced users to read and enjoy the low level details. This package's the fork of QXmpp. It introduces some new features to QXmpp core, like in-band registration and extended API. Some additional XEPs were also introduced, like XHTML-IM, Attention requests, Delivery receipts, etc. It's used for LeechCraft Internet Client (http://leechcraft.org/). SPEC file for it's already completed too https://github.com/Ignotus/leechcraft-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 752909] Review Request: libverto - Main loop abstraction 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=752909 --- Comment #4 from Nathaniel McCallum 2011-11-11 13:23:03 EST --- Spec: http://npmccallum.fedorapeople.org/libverto/libverto.spec SRPM: http://npmccallum.fedorapeople.org/libverto/libverto-0.2.1-2.fc16.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3508201 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753262] Review Request: qxmpp-dev - Qt XMPP 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=753262 Minh Ngo changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753185
Pierre-YvesChibon changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag|fedora-review? |fedora-review+
--- Comment #4 from Pierre-YvesChibon 2011-11-11 13:11:59
EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
C/C++
[x]: MUST Header files in -devel subpackage, if present.
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contains kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
[x]: MUST Development .so files in -devel subpackage, if present.
Generic
[x]: MUST Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at least
one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).(EPEL6 & Fedora < 13)
[x]: MUST Sources contain only permissible code or content.
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the
beginning of %install. (EPEL5)
[x]: MUST If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.
rpmlint R-Rcompression-0.93.2-3.fc16.x86_64.rpm
R-Rcompression.x86_64: W: spelling-error %description -l en_US zlib ->
lib, glib, z lib
R-Rcompression.x86_64: W: spelling-error %description -l en_US
uncompressing -> uncompromising, compressing, uncompressed
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
rpmlint R-Rcompression-0.93.2-3.fc16.src.rpm
R-Rcompression.src: W: spelling-error %description -l en_US zlib ->
lib, glib, z lib
R-Rcompression.src: W: spelling-error %description -l en_US
uncompressing -> uncompromising, compressing, uncompressed
R-Rcompression.src:48: W: macro-in-comment %{_bindir}
R-Rcompression.src:48: W: macro-in-comment %{packname}
1 packages and 0 specfiles checked; 0 errors, 4 warnings.
rpmlint R-Rcompression-debuginfo-0.93.2-3.fc16.x86_64.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[x]: MUST Sources used to build the package matches the upstream source, as
provided in the spec URL.
/home/pierrey/repos/gitrepo/FedoraReview/src/753185/Rcompression_0.93-2.tar.gz
:
MD5SUM this package : 2b7ff21879491d0a62e5182f403ca9fc
MD5SUM upstream package : 2b7ff21879491d0a62e5182f403ca9fc
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package
[Bug 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753181 Pierre-YvesChibon changed: What|Removed |Added CC||pin...@pingoured.fr --- Comment #1 from Pierre-YvesChibon 2011-11-11 13:07:52 EST --- *** Bug 673660 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 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 --- Comment #3 from Tom "spot" Callaway 2011-11-11 12:58:51 EST --- And this package has the correct BuildRequires and drops the unnecessary sampleData/ directory full of junk: Spec URL: http://spot.fedorapeople.org/R-Rcompression.spec SRPM URL: http://spot.fedorapeople.org/R-Rcompression-0.93.2-3.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 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 Pierre-YvesChibon changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr 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 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 --- Comment #9 from Fedora Update System 2011-11-11 12:58:44 EST --- python-straight-plugin-1.1.1-0.7.2010.git57ef11c.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.2010.git57ef11c.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 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 --- Comment #8 from Fedora Update System 2011-11-11 12:58:37 EST --- python-straight-plugin-1.1.1-0.7.2010.git57ef11c.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.2010.git57ef11c.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 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 --- Comment #10 from Fedora Update System 2011-11-11 12:58:53 EST --- python-straight-plugin-1.1.1-0.7.2010.git57ef11c.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-straight-plugin-1.1.1-0.7.2010.git57ef11c.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 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 Fedora Update System changed: What|Removed |Added Status|NEW |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 730306] Review Request: jboss-interceptors-api_1.1 - Interceptors 1.1 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=730306 Vladimir Kostadinov changed: What|Removed |Added Status|NEW |ASSIGNED CC||vladimir.kostadinov@gmail.c ||om AssignedTo|nob...@fedoraproject.org|vladimir.kostadinov@gmail.c ||om Flag||fedora-review? --- Comment #1 from Vladimir Kostadinov 2011-11-11 12:49:53 EST --- I'll review it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 Pierre-YvesChibon changed: What|Removed |Added CC||pin...@pingoured.fr --- Comment #2 from Pierre-YvesChibon 2011-11-11 12:40:38 EST --- *** Bug 673658 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 752909] Review Request: libverto - Main loop abstraction 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=752909 --- Comment #3 from Nathaniel McCallum 2011-11-11 12:34:46 EST --- 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 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 --- Comment #1 from Tom "spot" Callaway 2011-11-11 12:27:54 EST --- Okay, this package now has the bundled copy of minizip removed, it now uses the system package: Spec URL: http://spot.fedorapeople.org/R-Rcompression.spec SRPM URL: http://spot.fedorapeople.org/R-Rcompression-0.93.2-2.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 753240] New: Review Request: colorhug-client - Tools for the Hughski Colorimeter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: colorhug-client - Tools for the Hughski Colorimeter https://bugzilla.redhat.com/show_bug.cgi?id=753240 Summary: Review Request: colorhug-client - Tools for the Hughski Colorimeter Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rhug...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://people.freedesktop.org/~hughsient/temp/colorhug-client.spec SRPM URL: http://people.freedesktop.org/~hughsient/temp/colorhug-client-0.1.0-1.fc16.src.rpm Koij Build: [not available, you need to build with libgusb in updates-testing] Description: The Hughski ColorHug colorimeter is a low cost open-source hardware sensor used to calibrate screens. This package includes the client tools which allows the user to upgrade the firmware on the sensor or to access the sensor from command line scripts. rpmlint Output: colorhug-client.x86_64: W: spelling-error %description -l en_US colorimeter -> cyclometer colorhug-client.src: W: spelling-error %description -l en_US colorimeter -> cyclometer 3 packages and 0 specfiles checked; 0 errors, 2 warnings. (these are clearly bogus...) NOTE: I'm the owner of Hughski Ltd. and I want to get the package with the client tools in distributions *before* the hardware is sold so that users don't have to muck about with "configure && make" from a tarball when they get the device. The device is 100% open source, with GPL firmware, GPL client code and GPL bootloader. If you want to know more about the company or the device please don't hesitate to ask. 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 411421] Review Request: d-feet - D-Bus debugger
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=411421 John (J5) Palmieri changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #16 from John (J5) Palmieri 2011-11-11 12:11:20 EST --- Package Change Request == Package Name: d-feet New Branches: el6 Owners: johnp -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 739263] Review Request: sugar-bounce - Fast paced 3D action 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=739263
--- Comment #4 from Brendan Jones 2011-11-11
11:38:14 EST ---
Your package contains a C library which must be compiled in the %build section
and installed in the appropriate location. Being compiled that means the
package is architecture specific - so you have to remove the BuildArch tag.
You will be doing something along the lines of this in your build and prep
section and make sure the files are installed in the correct place.
%prep
%setup -q -n Bounce-%{version}
sed -i -e 's|-fPIC -O2|%{optflags}|' pongc/Makefile
%build
# build C binaries
pushd pongc
make
popd
For reference:
http://fedoraproject.org/wiki/Packaging:SugarActivityGuidelines#Architecture-specific_Activities
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752447] Review Request: plasma-mobile - A Plasma Active mobile workspace
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752447 Rex Dieter changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Rex Dieter 2011-11-11 11:35:50 EST --- Looking good. naming: ok sources: ok md5sum *.bz2 ec5560f243a861ea3eedcfc4103cf6d6 plasma-mobile-0.2.tar.bz2 licensing: ok macros: ok scriptlets: ok 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 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 Bill Nottingham changed: What|Removed |Added CC||tm...@redhat.com --- Comment #26 from Bill Nottingham 2011-11-11 11:26:39 EST --- Is there a reason we would ship both this and ipsec-tools? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673661] Review Request: R-ALL - Data of T- and B-cell Acute Lymphocytic Leukemia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673661 --- Comment #8 from Fedora Update System 2011-11-11 10:57:51 EST --- R-ALL-1.4.11-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/R-ALL-1.4.11-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 673661] Review Request: R-ALL - Data of T- and B-cell Acute Lymphocytic Leukemia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673661 --- Comment #7 from Fedora Update System 2011-11-11 10:57:36 EST --- R-ALL-1.4.11-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/R-ALL-1.4.11-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 673661] Review Request: R-ALL - Data of T- and B-cell Acute Lymphocytic Leukemia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673661 Fedora Update System changed: What|Removed |Added Status|NEW |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 747448] Review Request: joda-convert - Java library for conversion to and from standard string formats
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747448 Tomas Radej changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753188 Pierre-YvesChibon changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753182] Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753182 Pierre-YvesChibon changed: What|Removed |Added CC||pin...@pingoured.fr AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753188] New: Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks https://bugzilla.redhat.com/show_bug.cgi?id=753188 Summary: Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://spot.fedorapeople.org/R-rtracklayer.spec SRPM URL: http://spot.fedorapeople.org/R-rtracklayer-1.14.1-1.fc16.src.rpm Description: Extensible framework for interacting with multiple genome browsers (currently UCSC built-in) and manipulating annotation tracks in various formats (currently GFF, BED, bedGraph, BED15, WIG, and BigWig built-in). The user may export/import tracks to/from the supported browsers, as well as query and modify the browser state, such as the current viewport. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753185] Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753185 Tom "spot" Callaway changed: What|Removed |Added Blocks||753181 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753181 Tom "spot" Callaway changed: What|Removed |Added Depends on||753185 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753185] New: Review Request: R-Rcompression - R Package for in-memory compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: R-Rcompression - R Package for in-memory compression https://bugzilla.redhat.com/show_bug.cgi?id=753185 Summary: Review Request: R-Rcompression - R Package for in-memory compression Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://spot.fedorapeople.org/R-Rcompression.spec SRPM URL: http://spot.fedorapeople.org/R-Rcompression-0.93.2-1.fc16.src.rpm Description: This package is a basic R interface to the zlib and bzip2 facilities for compressing and uncompressing data that are in memory rather than in files. Koji Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3507587 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753181] New: Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: R-RCurl - General network (HTTP/FTP) client interface for R https://bugzilla.redhat.com/show_bug.cgi?id=753181 Summary: Review Request: R-RCurl - General network (HTTP/FTP) client interface for R Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://spot.fedorapeople.org/R-RCurl.spec SRPM URL: http://spot.fedorapeople.org/R-RCurl-1.7.0-1.fc16.src.rpm Description: The package allows one to compose general HTTP requests and provides convenient functions to fetch URIs, get & post forms, etc. and process the results returned by the Web server. This provides a great deal of control over the HTTP/FTP/... connection and the form of the request while providing a higher-level interface than is available just using R socket connections. Additionally, the underlying implementation is robust and extensive, supporting FTP/FTPS/TFTP (uploads and downloads), SSL/HTTPS, telnet, dict, ldap, and also supports cookies, redirects, authentication, etc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753181] Review Request: R-RCurl - General network (HTTP/FTP) client interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753181 Tom "spot" Callaway changed: What|Removed |Added Blocks||753182 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753182] Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753182 Tom "spot" Callaway changed: What|Removed |Added Depends on||753181 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753182] New: Review Request: R-biomaRt - R Interface to BioMart databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: R-biomaRt - R Interface to BioMart databases https://bugzilla.redhat.com/show_bug.cgi?id=753182 Summary: Review Request: R-biomaRt - R Interface to BioMart databases Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://spot.fedorapeople.org/R-biomaRt.spec SRPM URL: http://spot.fedorapeople.org/R-biomaRt-2.10.0-1.fc16.src.rpm Description: In recent years a wealth of biological data has become available in public data repositories. Easy access to these valuable data resources and firm integration with data analysis is needed for comprehensive bioinformatics data analysis. biomaRt provides an interface to a growing collection of databases implementing the BioMart software suite (http://www.biomart.org). The package enables retrieval of large amounts of data in a uniform way without the need to know the underlying database schemas or write complex SQL queries. Examples of BioMart databases are Ensembl, COSMIC, Uniprot, HGNC, Gramene, Wormbase and dbSNP mapped to Ensembl. These major databases give biomaRt users direct access to a diverse set of data and enable a wide range of powerful online queries from gene annotation to database mining. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752451] Review Request: kde-artwork-active - Artwork for Plasma Active
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752451 Radek Novacek changed: What|Removed |Added Status|NEW |ASSIGNED CC||rnova...@redhat.com AssignedTo|nob...@fedoraproject.org|rnova...@redhat.com Flag||fedora-review? --- Comment #1 from Radek Novacek 2011-11-11 09:44:19 EST --- This package should probably be noarch, because it doesn't contains any architecture specific files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 739263] Review Request: sugar-bounce - Fast paced 3D action 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=739263 --- Comment #3 from Kalpa Welivitigoda 2011-11-11 09:34:51 EST --- Hi Brendan Would you please explain it in detail. btw, build needs in the INSTALL are all listed in the spec. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719908] Review Request: rubygem-multi_json - A gem to provide swappable JSON backends
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719908 --- Comment #6 from Vít Ondruch 2011-11-11 09:28:44 EST --- Great, the OkJson was granted exception for bundling. So here are the latest packages, which reflect your comments: Spec URL: http://people.redhat.com/vondruch/rubygem-multi_json.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-multi_json-1.0.3-2.fcf17.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3507536 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749757] Review Request: zita-rev1 - Proaudio reverb for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749757 Thibault North changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Thibault North 2011-11-11 09:26:01 EST --- Thanks for these updates. Adding the "should" items: # SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. NA # SHOULD: The description and summary sections in the package spec file should contain translations for supported Non-English languages, if available. NA # SHOULD: The reviewer should test that the package builds in mock. NO:as clxclient-devel didn't reach the repos yet, mock fails. But rpmbuild -ba works with this dependency, so I am pretty confident that there won't be any issues on that side. # SHOULD: The package should compile and build into binary rpms on all supported architectures. OK # SHOULD: The reviewer should test that the package functions as described. A package should not segfault instead of running, for example. OK # SHOULD: If scriptlets are used, those scriptlets must be sane. This is vague, and left up to the reviewers judgement to determine sanity. NA # SHOULD: Usually, subpackages other than devel should require the base package using a fully versioned dependency. NA # SHOULD: The placement of pkgconfig(.pc) files depends on their usecase, and this is usually for development purposes, so should be placed in a -devel pkg. A reasonable exception is that the main pkg itself is a devel tool not installed in a user runtime, e.g. gcc or gdb. NA # SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin consider requiring the package which provides the file instead of the file itself. NA # SHOULD: your package should contain man pages for binaries/scripts. If it doesn't, work with upstream to add them where they make sense. Doc added. >I'm not getting the URL timeouts here so I think it may be a false positive Yes. All issues have been fixed, therefore the package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #34 from Fedora Update System 2011-11-11 08:31:16 EST --- dpm-dsi-1.8.2-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/dpm-dsi-1.8.2-4.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 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 #35 from Fedora Update System 2011-11-11 08:31:24 EST --- dpm-dsi-1.8.2-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/dpm-dsi-1.8.2-4.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 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 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 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 #33 from Fedora Update System 2011-11-11 08:27:58 EST --- dpm-dsi-1.8.2-4.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/dpm-dsi-1.8.2-4.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 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 #47 from Fedora Update System 2011-11-11 08:25:26 EST --- wicd-kde-0.2.3-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/wicd-kde-0.2.3-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 742166] Review Request: wmctrl - X Window Manager command-line tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=742166 --- Comment #3 from Jon Ciesla 2011-11-11 08:27:56 EST --- Looks like this is already unorphaned and you own it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752938] Review Request: python-straight-plugin - Python plugin loader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752938 --- Comment #7 from Jon Ciesla 2011-11-11 08:25:50 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 744433] Review Request: libquvi - A cross-platform library for parsing flash media stream
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744433 --- Comment #7 from Jon Ciesla 2011-11-11 08:22:07 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 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 Fedora Update System changed: What|Removed |Added Status|NEW |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 #46 from Fedora Update System 2011-11-11 08:22:02 EST --- wicd-kde-0.2.3-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/wicd-kde-0.2.3-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 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 Jim Meyering changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #29 from Jim Meyering 2011-11-11 07:47:49 EST --- 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 675234] Review Request: duply - Wrapper for duplicity
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675234 --- Comment #7 from Thomas Moschny 2011-11-11 07:21:31 EST --- Spec URL: http://thm.fedorapeople.org/duply/duply.spec SRPM URL: http://thm.fedorapeople.org/duply/duply-1.5.5.4-1.fc15.src.rpm %changelog * Fri Nov 11 2011 Thomas Moschny <..> - 1.5.5.4-1 - Update to 1.5.5.4. - Rename license file to LICENSE. - Remove %%defattr directive in %%files. As the package is intended to go to EPEL5, BuildRoot stuff must remain. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744349] Review Request: python-rtslib - python lib to configure TCM/LIO kernel target
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744349 --- Comment #1 from Tomasz Torcz 2011-11-11 07:23:07 EST --- Review follows: What needs work: - usual s/define/global - version in changelog differs from Version: python-rtslib.noarch: W: incoherent-version-in-changelog 1.9.9-7 ['1.99.1.git644eece-7.fc16', '1.99.1.git644eece-7' - my checkout produces different md5sum, could you check: 7c9c0220bba3773639b3e33ffa8d9d22 ../SOURCES/rtslib-1.99.1.git644eece.tar.gz ace89d9b619277b52e7bde5760418502 rtslib-1.99.1.git644eece.tar.gz What's good: - *pyo, *pyc and egg-info files are properly included - name is fine (module version) - license is fine - all files are owned by package - macro use is consistent - it builds - permissions are fine -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751316] Review Request: felix-gogo-parent - Parent package for Felix Gogo
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751316 Alexander Kurtakov changed: What|Removed |Added Blocks|652183(FE-JAVASIG) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693370] Review Request: elementary-gtk-theme - elementary GTK theme
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693370 --- Comment #9 from Christoph Wickert 2011-11-11 06:24:15 EST --- Sorry it took so long. The layout of the new package is good, however - the base name should probably be named elementary-theme or egtk. The letter is the upstream name but I prefer elementary-theme. - You are not installing any files but only create empty folders - There are unowned files: Checking for unpackaged file(s): /usr/lib/rpm/check-files /home/chris/linux/rpmbuild/BUILDROOT/elementary-2.1-2.fc15.x86_64 warning: Installed (but unpackaged) file(s) found: /usr/share/themes/elementary/gtk-3.0 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716299] Review Request: clipit - lightweight, fully featured GTK+ clipboard manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716299 --- Comment #16 from Christoph Wickert 2011-11-11 06:14:27 EST --- Sorry it took so long. I have now sponsored you. Please continue with the SCM request. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
