[Bug 749299] Review Request: lcgdm-dav - HTTP/DAV frontend to the DPM/LFC services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749299 --- Comment #5 from Ricardo Rocha rocha.po...@gmail.com 2011-11-10 03:26:39 EST --- Koji builds (successful): https://koji.fedoraproject.org/koji/taskinfo?taskID=3503751 (dist-5E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3503755 (dist-6E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3503759 (f16) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749291] Review Request: dpm-xrootd - xroot interface to the Disk Pool Manager (DPM)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749291 --- Comment #3 from Ricardo Rocha rocha.po...@gmail.com 2011-11-10 03:34:28 EST --- Koji builds (successful): https://koji.fedoraproject.org/koji/taskinfo?taskID=3503762 (dist-5E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3503766 (dist-6E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3503770 (f16) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 664221] Review Request: ghc-cautious-file - Provides ways to write a file cautiously
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=664221 Jens Petersen peter...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|peter...@redhat.com Flag||fedora-review? --- Comment #4 from Jens Petersen peter...@redhat.com 2011-11-10 04:27:07 EST --- http://packdeps.haskellers.com/reverse/cautious-file -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 731456] Review Request: jreen - 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=731456 --- Comment #24 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 2011-11-10 04:49:32 EST --- May be have worth package [q]jdns separately from upstream project? Then we can do not pack iris at all (it will not be used anymore). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 731456] Review Request: jreen - 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=731456 --- Comment #25 from Ruslan Nigmatullin euroeles...@yandex.ru 2011-11-10 05:01:52 EST --- As I suppose you've mostly finished splitting of jdns and iris into separate packages. Would new jdns be shared library? Is it possible to find it with, i.e. pkgconfig, just looking into /usr/lib/jdns.pc and /usr/lib/qjdns.pc? Or I have to write my own scripts to find this library? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 745510] Review Request: vdsm - Virtual Desktop Server 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=745510 --- Comment #14 from Federico Simoncelli fsimo...@redhat.com 2011-11-10 05:16:50 EST --- (In reply to comment #13) I am fixing the spec file to build unmodified on SUSE. On SUSE we require that a package either owns its directories or it BuildRequires the packages owning them. vdsm fails here: vdsm-4.9.1-0.4.g1bf3028.i586.rpm: directories not owned by a package: - /etc/pki - /etc/rwtab.d - /etc/sudoers.d I'm not sure if I can help you with these, the directories are owned by: filesystem, initscripts and sudo. As far as I remember I read somewhere that it's preferable to not add BuildRequires of packages that are automatically included by mock. Richard, what do you think? - /etc/vdsm - /usr/lib/python2.7/site-packages/sos - /usr/lib/python2.7/site-packages/sos/plugins - /usr/lib/vdsm/hooks - /var/lib/libvirt - /var/lib/libvirt/qemu I've no problem adding these BuildRequires. You can send a patch to vdsm-patc...@fedorahosted.org (cc me 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 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 #29 from Mattias Ellert mattias.ell...@fysast.uu.se 2011-11-10
05:18:56 EST ---
(In reply to comment #17)
Just one last question before i import. Trying a serious of scratch builds it
looks ok for EL5 and F16, but it fails in ppc for EL6, apparently due to
missing globus headers:
https://koji.fedoraproject.org/koji/taskinfo?taskID=3499253
Instead of adding a patch for this you could do:
make lib_m=${_lib}
in the specfile. This will work for all 64 bit architectures. Currently with
your patch, the makefile handles x86_64 and ppc64, which is sufficient for
primary architecture in Fedora and EPEL, but it will not work when someone
tries to do a build for a secondary Fedora 64 bit architecture like x390x and
sperc64.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 607405] Review Request: poppler-sharp - C Sharp Bindings for Poppler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=607405 Ismael Olea ism...@olea.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||ism...@olea.org Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 537640] Review Request: pdfmod - A simple application for modifying PDF documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=537640 Ismael Olea ism...@olea.org changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #30 from Ricardo Rocha rocha.po...@gmail.com 2011-11-10 05:55:44 EST --- Hi Mattias. Is it ok if i put that for the next update? I've started with the bodhi update with the current version. Should be fine no? 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 752771] New: Review Request: darkserver - A GNU build-id details service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: darkserver - A GNU build-id details service https://bugzilla.redhat.com/show_bug.cgi?id=752771 Summary: Review Request: darkserver - A GNU build-id details service Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kushal...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://kushal.fedorapeople.org/packages/darkserver.spec SRPM URL: http://kushal.fedorapeople.org/packages/darkserver-0.1-1.fc15.src.rpm Description: Service to provide details of GNU build-id -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752771] Review Request: darkserver - A GNU build-id details service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752771 Robert Scheck red...@linuxnetz.de changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|redhat-bugzilla@linuxnetz.d ||e 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 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
Mohamed El Morabity pikachu.2...@gmail.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #5 from Mohamed El Morabity pikachu.2...@gmail.com 2011-11-10
07:08:07 EST ---
Sorry for this late answer. Here is the review:
+:ok, =:needs attention, -:needs fixing
MUST Items:
[+] MUST: rpmlint must be run on every package.
[+] 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 and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
85c47035105b6b523a28793d4f6da9f1
[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.
[+] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.
[+] MUST: All build dependencies must be listed in BuildRequires
[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.
[+] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.
[+] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review
[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.
[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.
[+] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.
[+] MUST: Large documentation files should go in a doc subpackage.
[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[+] MUST: Header files must be in a -devel package.
[+] MUST: Static libraries must be in a -static package.
[+] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.
[+] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release}
[+] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.
[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.
This 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 #31 from Mattias Ellert mattias.ell...@fysast.uu.se 2011-11-10 07:09:59 EST --- Sure, the builds for the secondary architectures are way behind anyway at the moment, so they will probably not try to build this package any time soon. But eventually they will get to it. Mattias -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 607405] Review Request: poppler-sharp - C Sharp Bindings for Poppler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=607405 --- Comment #8 from Ismael Olea ism...@olea.org 2011-11-10 07:15:45 EST --- * rpmlint: $ rpmlint -iv /var/lib/mock/fedora-15-i386/result/poppler-sharp-0.0.2-2.fc15.src.rpm poppler-sharp.src: I: checking poppler-sharp.src: I: checking-url http://www.github.com/jacintos/poppler-sharp (timeout 10 seconds) poppler-sharp.src: I: checking-url http://github.com/downloads/jacintos/poppler-sharp/poppler-sharp-0.0.2.tar.gz (timeout 10 seconds) 1 packages and 0 specfiles checked; 0 errors, 0 warnings. fine $ rpmlint -iv /var/lib/mock/fedora-15-i386/result/poppler-sharp-0.0.2-2.fc15.i686.rpm poppler-sharp.i686: I: checking poppler-sharp.i686: I: checking-url http://www.github.com/jacintos/poppler-sharp (timeout 10 seconds) poppler-sharp.i686: E: no-binary The package should be of the noarch architecture because it doesn't contain any binaries. poppler-sharp.i686: W: only-non-binary-in-usr-lib There are only non binary files in /usr/lib so they should be in /usr/share. 1 packages and 0 specfiles checked; 1 errors, 1 warnings. here rpmlint not adheres to http://fedoraproject.org/wiki/Packaging:Mono, so it's fine as it is. [olea@patxuko rpmbuild]$ rpmlint -iv /var/lib/mock/fedora-15-i386/result/poppler-sharp-devel-0.0.2-2.fc15.i686.rpm poppler-sharp-devel.i686: I: checking poppler-sharp-devel.i686: I: checking-url http://www.github.com/jacintos/poppler-sharp (timeout 10 seconds) poppler-sharp-devel.i686: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. 1 packages and 0 specfiles checked; 0 errors, 1 warnings. if there is non trivial information to developers would be nice to add here; if not, it is fine AFAIK * I would suggest to modify translation from %description -l es Genera administración de enlaces a Poppler usando herramientas GAPI to Genera los vínculos administrados usando las herramientas GAPI * I understood, reading the linked issue 4 this modification is incompatible now: #libpoppler-glib fix https://github.com/jacintos/poppler-sharp/issues/4 sed -i 's/libpoppler-glib\.so\.6/libpoppler-glib\.so/g' sources/poppler-sharp.dll.config Correct me if I'm wrong. * gacutil is now required to install DLL in Fedora (see http://fedoraproject.org/wiki/Packaging:Mono#gacutil_in_a_spec_file) Please apply the suggested changes and use mock for next rebuilds for better control of potential collateral effects. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #32 from Steve Traylen steve.tray...@cern.ch 2011-11-10 07:20:47 EST --- (In reply to comment #30) I've started with the bodhi update with the current version. Should be fine no? Just for information you can edit an existing bodhi update with a new package... Unless it's in the 1 day or so window of being pushed to testing. Once there you can update. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749329] Review Request: pcfi - PDF Core Font Information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749329 --- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-11-10 08:32:29 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 530910] Review Request: hyena - Hyena is a library of GUI and non-GUI C# code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530910 --- Comment #14 from Ismael Olea ism...@olea.org 2011-11-10 08:34:06 EST --- Review of hyena-0.5-1.fc14.src.rpm from Claudio: * Compiling in F15 it needs mono-devel: BuildRequires: mono-devel * You should remove all the execution permisions for the files don't need it. * The setup -n parameter is redundant here * About %configure, I would not activate tests, as is not an in development version, but tests are not braking compilation so it's not necesary bad Please apply the suggested changes and use mock for next rebuilds for better control of potential collateral effects. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 530910] Review Request: hyena - Hyena is a library of GUI and non-GUI C# code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=530910 --- Comment #15 from Ismael Olea ism...@olea.org 2011-11-10 08:36:06 EST --- @Ian: I can take over the review process if you wish, since it's related to #537640 and #607405. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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
--- Comment #3 from Jaroslav Reznik jrez...@redhat.com 2011-11-10 08:47:32
EST ---
Spec URL:
http://rezza.hofyland.cz/fedora/packages/plasma-mobile/plasma-mobile.spec
SRPM URL:
http://rezza.hofyland.cz/fedora/packages/plasma-mobile/plasma-mobile-0.2-2.fc15.src.rpm
(In reply to comment #1)
A couple of concerns off the top of my head.
1. desktop-file-validate %check fails. (I'll attach a patch that fixes it for
me on f16), upstreamed these fixes already to git.kde.org
Ah, I had these patches locally, forgot to add to SPEC/SRPM - in your patch,
there's still one error, I'll take a look if I can upstream it
2. %{_qt4_plugindir}/inputmethods/plasmainputcontextplugin.so
will get multilib'd, so probably want to make -libs package for this item
Ok, added -libs.
3. file /usr/lib64/kde4/imports/org/kde/plasma/mobilecomponents is not owned
by any package. can probably just recursively own that dir, instead of
enumerating the items under it.
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 752829] New: Review Request: glue-validator - A valiation framework for GLUE 2.0 information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: glue-validator - A valiation framework for GLUE 2.0 information https://bugzilla.redhat.com/show_bug.cgi?id=752829 Summary: Review Request: glue-validator - A valiation framework for GLUE 2.0 information Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: laurence.fi...@cern.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://lfield.web.cern.ch/lfield/glue-validator.spec SRPM URL: http://lfield.web.cern.ch/lfield/glue-validator-0.0.3-2.el5.src.rpm Description: A valiation framework for GLUE 2.0 information -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752829] Review Request: glue-validator - A validation framework for GLUE 2.0 information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752829 laurence.fi...@cern.ch changed: What|Removed |Added Summary|Review Request: |Review Request: |glue-validator - A |glue-validator - A |valiation framework for |validation framework for |GLUE 2.0 information|GLUE 2.0 information -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752829] Review Request: glue-validator - A validation framework for GLUE 2.0 information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752829 --- Comment #1 from laurence.fi...@cern.ch 2011-11-10 09:41:42 EST --- This is my first package and I am seeking a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752836] New: Review Request: perl-Class-Field - Class Field Accessor Generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-Class-Field - Class Field Accessor Generator https://bugzilla.redhat.com/show_bug.cgi?id=752836 Summary: Review Request: perl-Class-Field - Class Field Accessor Generator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: berra...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://berrange.fedorapeople.org/review/perl-Class-Field/perl-Class-Field.spec SRPM URL: http://berrange.fedorapeople.org/review/perl-Class-Field/perl-Class-Field-0.15-1.fc15.src.rpm Description: perl-Class-Field is a Perl module for automatically generating class accessors. It is needed by perl-TAP-Formatter-JUnit NB: rpmlint generates a warning which is bogus since it refers to a programming language literal, not a normal english word $ rpmlint perl-Class-Field-0.15-1.fc15.src.rpm perl-Class-Field.src: W: spelling-error %description -l en_US const - cons, cont, cost -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752836] Review Request: perl-Class-Field - Class Field Accessor Generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752836 Daniel Berrange berra...@redhat.com changed: What|Removed |Added Blocks||752838 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752838] Review Request: perl-TAP-Formatter-JUnit - Harness output delegate for JUnit output
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752838 Daniel Berrange berra...@redhat.com changed: What|Removed |Added Depends on||752836 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752838] New: Review Request: perl-TAP-Formatter-JUnit - Harness output delegate for JUnit output
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-TAP-Formatter-JUnit - Harness output delegate for JUnit output https://bugzilla.redhat.com/show_bug.cgi?id=752838 Summary: Review Request: perl-TAP-Formatter-JUnit - Harness output delegate for JUnit output Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: berra...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://berrange.fedorapeople.org/review/perl-TAP-Formatter-JUnit/perl-TAP-Formatter-JUnit.spec SRPM URL:http://berrange.fedorapeople.org/review/perl-TAP-Formatter-JUnit/perl-TAP-Formatter-JUnit-0.08-1.fc15.src.rpm Description: This module provides JUnit output formatting for TAP::Harness (a replacement for Test::Harness. NB: this review first requires acceptance of bug 752836 for perl-Class-Field -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752829] Review Request: glue-validator - A validation framework for GLUE 2.0 information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752829 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added CC||steve.tray...@cern.ch Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751564
--- Comment #24 from Jim Meyering meyer...@redhat.com 2011-11-10 10:13:46 EST
---
Hi Joe,
In the spec, this looks wrong:
%if ! (0%{?fedora} 14 || 0%{?rhel} 6)
Requires: python-argparse
%endif
The problem is more apparent if you rewrite without the !:
%if (0%{?fedora} = 14 0%{?rhel} = 6)
Requires: python-argparse
%endif
Shouldn't it be || in place of here?
(i.e., in place of || in the original)
%if (0%{?fedora} = 14 || 0%{?rhel} = 6)
Requires: python-argparse
%endif
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 731456] Review Request: jreen - 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=731456 --- Comment #26 from Rex Dieter rdie...@math.unl.edu 2011-11-10 10:14:43 EST --- I've currently split it out into a libqjdns.so.1 shared library, yes, which includes a qjdns.pc pkgconfig file. (In a perfect world there'd also be a libjdns.so and jdns.pc for it's C-only bindings, but that would require more invasive changes) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751564 --- Comment #25 from Jim Meyering meyer...@redhat.com 2011-11-10 10:20:21 EST --- Hi Joe, I installed the new .src.rpm, then ran tar xf ~/rpmbuild/SOURCES/aeolus-audrey-agent-0.4.0.tar.gz cd aeolus-audrey-agent make which printed this: make: *** No rule to make target `audrey_startup.in.py', needed by\ `audrey_startup.py'. Stop. which means the new, required file is not included in the tarball. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 731456] Review Request: jreen - 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=731456 --- Comment #27 from Rex Dieter rdie...@math.unl.edu 2011-11-10 10:41:38 EST --- If anyone wants some intermediate/test iris/qjdns packages, let me know in bug #749885 (and what fedora release you're using) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 529441] Review Request: pdfbox - Java PDF 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=529441 Bug 529441 depends on bug 749329, which changed state. Bug 749329 Summary: Review Request: pcfi - PDF Core Font Information https://bugzilla.redhat.com/show_bug.cgi?id=749329 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749329] Review Request: pcfi - PDF Core Font Information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749329 Orion Poplawski or...@cora.nwra.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE Last Closed||2011-11-10 10:43:19 --- Comment #10 from Orion Poplawski or...@cora.nwra.com 2011-11-10 10:43:19 EST --- Checked in and built. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752858] Review Request: perl-Qt - Perl bindings for Qt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752858 --- Comment #1 from Iain Arnell iarn...@gmail.com 2011-11-10 10:57:17 EST --- Koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=3504521 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719152] Review Request: gappalib-coq - Coq support library for gappa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719152 Fedora Update System upda...@fedoraproject.org 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 749232] Review Request: nagios-plugins-lcgdm - nagios probes for DPM / LFC nodes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749232 --- Comment #5 from Ricardo Rocha rocha.po...@gmail.com 2011-11-10 10:55:51 EST --- Found an issue with the currently src rpm in F16 regarding byte-compiled python files. New spec and src rpms: Spec URL: http://rocha.web.cern.ch/rocha/fedora/nagios-plugins-lcgdm.spec SRPM URL: http://rocha.web.cern.ch/rocha/fedora/nagios-plugins-lcgdm-0.4.0-3.src.rpm Release number bumped, koji builds available (successful): https://koji.fedoraproject.org/koji/taskinfo?taskID=3504587 (dist-5E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3504609 (dist-6E-epel) https://koji.fedoraproject.org/koji/taskinfo?taskID=3504554 (f16) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752858] New: Review Request: perl-Qt - Perl bindings for Qt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-Qt - Perl bindings for Qt Alias: perl-Qt https://bugzilla.redhat.com/show_bug.cgi?id=752858 Summary: Review Request: perl-Qt - Perl bindings for Qt Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/Qt/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://fedorapeople.org/~iarnell/review/perl-Qt.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-Qt-0.96.0-1.fc17.src.rpm Description: This module provides Perl bindings for the Qt 4 libraries. *rt-0.10_02 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719152] Review Request: gappalib-coq - Coq support library for gappa
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719152 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2011-11-10 10:59:12 EST --- gappalib-coq-0.16.0-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/gappalib-coq-0.16.0-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 752452] Review Request: contour - A context sensitive user interface 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=752452 Jaroslav Reznik jrez...@redhat.com changed: What|Removed |Added Blocks||656997(kde-reviews) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 752452] Review Request: contour - A context sensitive user interface 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=752452 --- Comment #1 from Jaroslav Reznik jrez...@redhat.com 2011-11-10 11:20:22 EST --- Scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=3504789 rpmlint contour-0.1.1-1.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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
Kevin Kofler ke...@tigcc.ticalc.org changed:
What|Removed |Added
Blocks|177841(FE-NEEDSPONSOR) |
Flag||fedora-review+
--- Comment #35 from Kevin Kofler ke...@tigcc.ticalc.org 2011-11-10 11:35:26
EST ---
Pedantic comment of the day:
cp ${RPM_BUILD_ROOT}%{_kde4_datadir}/applications/kde4/%{name}.desktop \
${RPM_BUILD_ROOT}%{_kde4_datadir}/autostart/%{name}.desktop
We prefer cp -p (no clobbering of the timestamp) to plain cp. Though in this
case it doesn't matter because the file has already been edited by the
desktop-file-install --add-category command before. So this is only a SHOULD.
So it looks like all the issues found have been addressed. Thanks to everyone
who did preliminary reviews.
The package is APPROVED.
We can now proceed with the sponsoring process. Have you already signed up for
a FAS (Fedora Account System) account? What's your FAS account name?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on 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 #36 from Minh Ngo nlmin...@gmail.com 2011-11-10 11:49:01 EST --- What's your FAS account name? ignotusp -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728088] Review Request: rubygem-idn - Ruby Bindings for the GNU LibIDN 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=728088 --- Comment #11 from Mo Morsi mmo...@redhat.com 2011-11-10 11:48:41 EST --- OK thanks for the update - looks like you addressed all my feedback - rpmlint / koji builds are still green - as Vit mentioned the cp command and devel packages can probably be removed unless there is good reason to keep those there. I feel it might be useful to keep the cached gem in place though as it's good for reference and in order to keep the files provided by rpm and gem installs fairly consistent. None of this is a big deal either way though - Vit's other feedback is also good but not blockers as the package is now in compliance w/ Fedora guidelines. Feel free to implement his suggestions pre/post push. 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 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 #37 from Kevin Kofler ke...@tigcc.ticalc.org 2011-11-10 12:00:25 EST --- Congratulations, you are now sponsored! Please proceed with the remainder of the: https://fedoraproject.org/wiki/Join_the_package_collection_maintainers process. For any future package submissions, follow the: https://fedoraproject.org/wiki/New_package_process_for_existing_contributors process instead, as you are now an existing package maintainer. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748312] Review Request: ghc-cryptohash - Haskell crypto hashes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748312 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ghc-cryptohash-0.7.4-1.fc16 |ghc-cryptohash-0.7.4-1.fc14 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:24:02 EST --- ghc-cryptohash-0.7.4-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 751531] Review Request: perl-Env-Path - Advanced operations on path variables
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751531 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:28:24 EST --- perl-Env-Path-0.18-1.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 751533] Review Request: perl-MooseX-Has-Sugar - Sugar syntax for moose 'has' fields
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751533 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:26:26 EST --- perl-MooseX-Has-Sugar-0.05070419-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 749499] Review Request: ghc-regex-pcre - Haskell PCRE binding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749499 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:28:32 EST --- ghc-regex-pcre-0.94.2-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 689685] Review Request: anchorman - The recording studio in-a-box
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=689685 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||anchorman-0.0.1-1.fc14 Resolution||ERRATA Last Closed||2011-11-10 12:29:27 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:29:27 EST --- anchorman-0.0.1-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 748701] Review Request: ghc-wai-extra - Basic WAI handlers and middleware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748701 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ghc-wai-extra-0.4.3-1.fc16 |ghc-wai-extra-0.3.2.1-1.fc1 ||4 Bug 748701 depends on bug 746942, which changed state. Bug 746942 Summary: Review Request: ghc-blaze-builder-enumerator - Enumeratees for conversion of builders to bytestrings https://bugzilla.redhat.com/show_bug.cgi?id=746942 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:29:14 EST --- ghc-wai-extra-0.3.2.1-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 748701] Review Request: ghc-wai-extra - Basic WAI handlers and middleware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748701 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ghc-wai-extra-0.3.2.1-1.fc1 |ghc-wai-extra-0.4.3-1.fc15 |4 | --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:33:03 EST --- ghc-wai-extra-0.4.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 730593] Review Request: gnome-shell-theme-elementary - The Elementary gnome-shell 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=730593 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|gnome-shell-theme-elementar |gnome-shell-theme-elementar |y-3.2-1.fc16|y-3.0-1.fc15 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:39:57 EST --- gnome-shell-theme-elementary-3.0-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 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 #38 from Minh Ngo nlmin...@gmail.com 2011-11-10 12:39:13 EST --- How to create the git repository for wicd-kde? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730593] Review Request: gnome-shell-theme-elementary - The Elementary gnome-shell 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=730593 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||gnome-shell-theme-elementar ||y-3.2-1.fc16 Resolution||ERRATA Last Closed||2011-11-10 12:38:05 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:38:05 EST --- gnome-shell-theme-elementary-3.2-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 751522] Review Request: perl-DateTime-TimeZone-SystemV - System V and POSIX timezone strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751522 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:38:14 EST --- perl-DateTime-TimeZone-SystemV-0.005-1.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 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 Minh Ngo nlmin...@gmail.com changed: What|Removed |Added Flag||fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #40 from Minh Ngo nlmin...@gmail.com 2011-11-10 13:01:19 EST --- New Package SCM Request === Package Name: wicd-kde Short Description: A Wicd client built on the KDE Development Platform Owners: ignotusp Branches: f14 f15 f16 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730495] Review Request: perl-Net-Lite-FTP - Perl FTP client with support for TLS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730495 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:40:48 EST --- perl-Net-Lite-FTP-0.61-3.fc15 has been pushed to the Fedora 15 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 574545] Review Request: python26-mysqldb : Interface to MySQL for python26 on EPEL5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=574545 Ricardo Rocha rocha.po...@gmail.com changed: What|Removed |Added CC||rocha.po...@gmail.com AssignedTo|nob...@fedoraproject.org|rocha.po...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[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 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748312] Review Request: ghc-cryptohash - Haskell crypto hashes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748312 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ghc-cryptohash-0.7.4-1.fc14 |ghc-cryptohash-0.7.4-1.fc15 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:45:09 EST --- ghc-cryptohash-0.7.4-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 719150] Review Request: flocq - Formalization of floating point numbers for Coq
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719150 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||flocq-1.4.0-3.fc16 Resolution||ERRATA Last Closed||2011-11-10 12:45:35 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:45:35 EST --- flocq-1.4.0-3.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 719757] Review Request: apron - Numerical abstract domain 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=719757 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2011-11-10 12:45:48 EST --- apron-0.9.10-4.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 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 #39 from Kevin Kofler ke...@tigcc.ticalc.org 2011-11-10 12:50:11 EST --- By following yet another process… https://fedoraproject.org/wiki/Package_SCM_admin_requests but don't worry, you only have to post a message to this bug complying to the format (careful there, those requests are processed by scripts, so there's little to no tolerance for incorrectly-formatted requests) and set the fedora-cvs flag. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750997] Review Request: Scilab - Numerical Analysis toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750997 --- Comment #5 from Clément DAVID c.davi...@gmail.com 2011-11-10 13:29:06 EST --- Spec URL: http://davidcl.fedorapeople.org/scilab.spec SRPM URL: http://davidcl.fedorapeople.org/scilab-5.3.3-3.fc16.src.rpm Description: Scilab is a scientific software package for numerical computations providing a powerful open computing environment for engineering and scientific applications. Same rpmlint remarks than in the bug report. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3505218 Thanks for your review, I fixed these problems but also : - Remove dependency on atlas (need only lapack ABI) - Use noarch packages on doc and tests - add a default font for doc generation - scilab does not depends on scilab-doc anymore -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 588428] Review Request: rubygem-addressable - Improved URI/URL handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=588428 Mo Morsi mmo...@redhat.com changed: What|Removed |Added AssignedTo|shawn.st...@rogers.com |mmo...@redhat.com --- Comment #13 from Mo Morsi mmo...@redhat.com 2011-11-10 13:32:57 EST --- Package looks good. All feedback addressed. rpmlint / mock builds are green (again once rubygem-idn is installed manually). 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 752169] Review Request: gnome-shell-theme-zukitwo - The Zukitwo theme for Gnome Shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752169 Tim Lauridsen t...@rasmil.dk 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 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 #41 from Jon Ciesla l...@jcomserv.net 2011-11-10 13:41:05 EST --- Git done (by process-git-requests). Not currently accepting new packages for F-14. Kevin, please take ownership of review 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 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 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|ke...@tigcc.ticalc.org --- Comment #42 from Kevin Kofler ke...@tigcc.ticalc.org 2011-11-10 13:54:23 EST --- Not currently accepting new packages for F-14. FYI, Fedora 14 will reach its end of life in less than a month, and no new packages are accepted for Fedora 14. (Updates to existing packages are possible until the end of life. At that point, Fedora 14 will be completely dead as far as the Fedora Project is concerned.) Kevin, please take ownership of review BZs. Thanks! Whoops, 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 752909] New: 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. Summary: Review Request: libverto - Main loop abstraction library https://bugzilla.redhat.com/show_bug.cgi?id=752909 Summary: Review Request: libverto - Main loop abstraction library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: nathan...@natemccallum.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://npmccallum.fedorapeople.org/libverto/libverto.spec SRPM URL: http://npmccallum.fedorapeople.org/libverto/libverto-0.2.1-1.fc16.src.rpm Description: libverto provides a way for libraries to expose asynchronous interfaces without having to choose a particular event loop, offloading this decision to the end application which consumes the library. If you are packaging an application, not library, based on libverto, you should depend either on a specific implementation module or you can depend on the virtual provides 'libverto-module-base'. This will ensure that you have at least one module installed that provides io, timeout and signal functionality. Currently glib is the only module that does not provide these three because it lacks signal. However, glib will support signal in the future. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #43 from Kevin Kofler ke...@tigcc.ticalc.org 2011-11-10 13:56:44 EST --- (PS: Minh, next time, when you file a New Package SCM Request, please set the fedora-cvs flag to '?', not '+'. It gets set to '+' when the request is handled, as was done now with comment #41. I fixed that for you.) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #1 from Nathaniel McCallum nathan...@natemccallum.com 2011-11-10 14:01:40 EST --- Koji build against f17 available here: http://koji.fedoraproject.org/koji/taskinfo?taskID=3505386 Also, this package is a new dependency of mit krb5 1.10. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 --- Comment #5 from Christian Krause c...@plauener.de 2011-11-10 14:25:55 EST --- Peter, what's the status? ;-) If you don't find the time, please can you add a note and reset the review flag so that someone else can take over? 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 752938] New: 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. Summary: Review Request: python-straight.plugin - Python plugin loader https://bugzilla.redhat.com/show_bug.cgi?id=752938 Summary: Review Request: python-straight.plugin - Python plugin loader Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pin...@pingoured.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://pingou.fedorapeople.org/RPMs/python-straight.plugin.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/python-straight.plugin-1.1.1-1.57ef11c.fc16.src.rpm Description: straight.plugin is a Python plugin loader inspired by twisted.plugin with two important distinctions: - Fewer dependencies - Python 3 compatible The system is used to allow multiple Python packages to provide plugins within a namespace package, where other packages will locate and utilize. The plugins themselves are modules in a namespace package where the namespace identifies the plugins in it for some particular purpose or intent. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 --- Comment #7 from Jon Ciesla l...@jcomserv.net 2011-11-10 14:54:24 EST --- Excellent. 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 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 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added CC||a.bad...@gmail.com AssignedTo|nob...@fedoraproject.org|a.bad...@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 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 Robert Scheck red...@linuxnetz.de changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Robert Scheck red...@linuxnetz.de 2011-11-10 15:05:06 EST --- Thank you very much for the review! New Package SCM Request === Package Name: clucene09 Short Description: A C++ port of Lucene Owners: robert 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 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 --- Comment #9 from Jon Ciesla l...@jcomserv.net 2011-11-10 15:19:37 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 --- Comment #44 from Minh Ngo nlmin...@gmail.com 2011-11-10 15:20:02 EST --- 3505780 tagBuild (noarch): free 3505780 tagBuild (noarch): free - open (x86-06.phx2.fedoraproject.org) 3505780 tagBuild (noarch): open (x86-06.phx2.fedoraproject.org) - FAILED: ActionNotAllowed: tag requires admin permission 0 free 1 open 3 done 1 failed 3505695 build (f16, /wicd-kde:8f92e20460e571c7b5a3b177dd7e695d8b7065a0): open (x86-13.phx2.fedoraproject.org) - FAILED: ActionNotAllowed: tag requires admin permission 0 free 0 open 3 done 2 failed 3505695 build (f16, /wicd-kde:8f92e20460e571c7b5a3b177dd7e695d8b7065a0) failed http://koji.fedoraproject.org/koji/taskinfo?taskID=3505695 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #1 from Toshio Ernie Kuratomi a.bad...@gmail.com 2011-11-10 15:20:21 EST --- Talked on IRC and spec file has been updated. This review is based on the tarball from the srpm and the new spec file rpmlint: python-straight.plugin.noarch: W: spelling-error %description -l en_US plugins - plug ins, plug-ins, plugging [snip other spelling warnings] These are all false positives -- the words are common in computing. python-straight.plugin.src: W: invalid-url Source0: ironfroggy-straight.plugin-57ef11c.tar.gz There is a comment above the Source0 explaining how to retrieve the tarball from github so this is fine. Good: * License is BSD in source and spec * LICENSE file included * Spec file is legible * Sources match with upstream by following the github url to get the snapshot. * Package builds in koji for f16 * No localized files so no i18n concerns * Not a shared library package so no concerns there * No bundled libraries * Package owns the directories it creates and nothing more * Files listed once in the %files section * File permissions are correct * Macros used consistently * Code, not content * No large documentation files * Documentation is not used at runtime * Not a GUI app * No supbackages * All filenames are valid utf-8 Needswork: * Naming guidelines want . replaced with - so this should be named python-straight-plugin. (and spec file renamed to match). Cosmetic: * Could BuildRequire: python2-devel instead of python-devel * The description says python3 compatible but you aren't building a python3 subpackage. I'd either build the subpackage or remove that line from the description. Fix the naming and I'll approve this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 Terje Røsten terje...@phys.ntnu.no changed: What|Removed |Added CC||terje...@phys.ntnu.no Summary|Review Request: |Review Request: |python-straight.plugin -|python-straight-plugin - |Python plugin loader|Python plugin loader -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 brendan.jones...@gmail.com changed: What|Removed |Added CC||brendan.jones...@gmail.com Flag||fedora-review? --- Comment #2 from Brendan Jones brendan.jones...@gmail.com 2011-11-10 15:27:27 EST --- I'll take this 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 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751564 --- Comment #26 from Joe VLcek jvl...@redhat.com 2011-11-10 15:28:28 EST --- Hi Jim, Huge Thank you! for all the help with this. I believe I've address everything. The latest versions of the important documents can be found here: Please note it's now in: /v4 Spec URL: http://joev.fedorapeople.org/audrey-agent/v4/aeolus-audrey-agent.spec SRPM URL: http://joev.fedorapeople.org/audrey-agent/v4/aeolus-audrey-agent-0.4.0-9.fc15.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3505835 The upstream URL to the source is: https://github.com/aeolusproject/audrey -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2011-11-10 15:36:53 EST --- clucene09-0.9.21b-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/clucene09-0.9.21b-2.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 upda...@fedoraproject.org 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 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 thibault.no...@gmail.com changed: What|Removed |Added CC||thibault.no...@gmail.com Flag||fedora-review? --- Comment #4 from Thibault North thibault.no...@gmail.com 2011-11-10 15:38:22 EST --- I will take this one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750997] Review Request: Scilab - Numerical Analysis toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750997 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Rex Dieter rdie...@math.unl.edu 2011-11-10 15:59:19 EST --- excellent, otherwise looks good: naming: ok scriptlets: ok sources: ok $ md5sum *.gz 0211096464d0bb2e097154d62ef827bd scilab-5.3.3-src.tar.gz license: ok macros: 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 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749885 --- Comment #15 from Rex Dieter rdie...@math.unl.edu 2011-11-10 16:10:30 EST --- f16 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3506224 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #5 from Thibault North thibault.no...@gmail.com 2011-11-10
16:16:27 EST ---
rpmlint -v rpmbuild/{SRPMS,RPMS/x86_64}/zita*
zita-rev1.src: I: checking
zita-rev1.src: W: spelling-error %description -l en_US spatialiser - spatial,
spatiotemporal, specialistic
zita-rev1.src: I: checking-url http://kokkinizita.linuxaudio.org/ (timeout 10
seconds)
zita-rev1.src: I: checking-url
http://kokkinizita.linuxaudio.org/linuxaudio/downloads/zita-rev1-0.2.1.tar.bz2
(timeout 10 seconds)
zita-rev1.x86_64: I: checking
zita-rev1.x86_64: W: spelling-error %description -l en_US spatialiser -
spatial, spatiotemporal, specialistic
zita-rev1.x86_64: I: checking-url http://kokkinizita.linuxaudio.org/ (timeout
10 seconds)
zita-rev1.x86_64: W: no-manual-page-for-binary zita-rev1
zita-rev1-debuginfo.x86_64: I: checking
zita-rev1-debuginfo.x86_64: I: checking-url http://kokkinizita.linuxaudio.org/
(timeout 10 seconds)
3 packages and 0 specfiles checked; 0 errors, 3 warnings.
# MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review. OK
# MUST: The package must be named according to the Package Naming Guidelines OK
# MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. OK
# MUST: The package must meet the Packaging Guidelines OK
# MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines OK
# MUST: The License field in the package spec file must match the actual
license. OK
# MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc OK
# MUST: The spec file must be written in American English. OK
# MUST: The spec file for the package MUST be legible. OK
# MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this. OK
# MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. OK
# MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. NA
# MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense. OK
# MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.NA
# MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. NA
# MUST: Packages must NOT bundle copies of system libraries. OK
# MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. NA
# MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. OK
# MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. (Notable exception: license texts in specific situations)[14]
# MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. OK
# MUST: Each package must consistently use macros. OK
# MUST: The package must contain code, or permissable content. OK
# MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). NA
# MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present. OK
# MUST: Header files must be in a -devel package. NA
# MUST: Static libraries must be in a -static package. NA
# MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package. NA
# MUST: In the
[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 #2 from Pierre-YvesChibon pin...@pingoured.fr 2011-11-10 16:22:00 EST --- Thanks for your comments (here and on irc) and the review. I changed the name and added a python3 sub-package. I used 2to3 to generate the sub-package although it seems unnecessary but for the future, better safe than sorry. Spec URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin-1.1.1-0.2.2010git57ef11c.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 744430] Review Request: libquvi-scripts - Embedded lua scripts that libquvi uses for parsing the media details
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744430 Nicoleau Fabien nicoleau.fab...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-11-10 16:22:01 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 Bug 744433 depends on bug 744430, which changed state. Bug 744430 Summary: Review Request: libquvi-scripts - Embedded lua scripts that libquvi uses for parsing the media details https://bugzilla.redhat.com/show_bug.cgi?id=744430 What|Old Value |New Value Resolution||RAWHIDE Status|ASSIGNED|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 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 Nicoleau Fabien nicoleau.fab...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Nicoleau Fabien nicoleau.fab...@gmail.com 2011-11-10 16:25:51 EST --- Thank you (again :D) for this review Mohamed. New Package SCM Request === Package Name: libquvi Short Description: A cross-platform library for parsing flash media stream Owners: eponyme Branches: 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 751564] Review Request: The Aeolus Audrey Startup Agent
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=751564
--- Comment #27 from Jim Meyering meyer...@redhat.com 2011-11-10 16:34:54 EST
---
Joe, with this, I think I am done:
$ rpmlint aeolus-audrey-agent-0.4.0-9.fc15.src.rpm
aeolus-audrey-agent.src: W: invalid-license ASL2.0
aeolus-audrey-agent.src: W: invalid-url Source0:
aeolus-audrey-agent-0.4.0.tar.gz
Please fix #2: replace ASL2 by ASL 2.0
The first warning would be well enough addressed by the comments if you had
created/pushed a gpg-signed git tag and if the instructions said to use that.
# to build source tarball
# git clone g...@github.com:aeolusproject/audrey.git
# make dist
However, without that, the above clone instructions are not
enough to reproduce your source RPM, which is the goal, after all.
Once you've fixed these things, just put the tarball in
http://joev.fedorapeople.org/audrey-agent/ and use its URL
in the Source0: directive.
Now for the .rpm:
$ rpmlint
/h/j/rpmbuild/RPMS/noarch/aeolus-audrey-agent-0.4.0-9.fc16.noarch.rpm
aeolus-audrey-agent.noarch: E: explicit-lib-dependency python-httplib2
aeolus-audrey-agent.noarch: W: invalid-license ASL2.0
aeolus-audrey-agent.noarch: E: non-standard-executable-perm /usr/bin/audrey
0555L
1 packages and 0 specfiles checked; 2 errors, 1 warnings
Run rpmlint with the -i option and it'll give you more details.
You can address the non-standard-executable-perm error by using
install instead of cp. It will do what rpmlint wants: i.e.,
permissions of 0755, not 0555.
As I understand it, the python-httplib2 error is unwarranted
and thus ignorable.
Address the above and you'll have my approval.
Two MUST items are marked NO below because of the above.
=
From https://fedoraproject.org/wiki/Packaging:ReviewGuidelines
ok MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.[1]
ok MUST: The package must be named according to the Package Naming Guidelines
.
ok MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. [2] .
ok MUST: The package must meet the Packaging Guidelines .
ok MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
MUST: The License field in the package spec file must match the actual
license. [3]
ok MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.[4]
ok MUST: The spec file must be written in American English. [5]
ok MUST: The spec file for the package MUST be legible. [6]
NO (upstream source URL not specified)
__ MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
ok MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture. [7]
NA MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]
ok MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
ok MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
NA MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. [10]
ok MUST: Packages must NOT bundle copies of system libraries.[11]
NA MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. [12]
ok MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. [13]
ok MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)[14]
NO(0555 vs 0755, does this really
[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 sgall...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||sgall...@redhat.com AssignedTo|nob...@fedoraproject.org|sgall...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 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 #3 from Pierre-YvesChibon pin...@pingoured.fr 2011-11-10 16:40:39
EST ---
There is the new version with the correct use of %{__python3} on the build and
test
Spec URL: http://pingou.fedorapeople.org/RPMs/python-straight-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org/RPMs/python-straight-plugin-1.1.1-0.3.2010git57ef11c.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 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 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added Flag||fedora-review+ --- Comment #4 from Toshio Ernie Kuratomi a.bad...@gmail.com 2011-11-10 16:41:57 EST --- All problems have been fixed. 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 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 Pierre-YvesChibon pin...@pingoured.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Pierre-YvesChibon pin...@pingoured.fr 2011-11-10 16:45:25 EST --- New Package SCM Request === Package Name: python-straight-plugin Short Description: Python plugin loader Owners: pingou sochotnicky Branches: f15 f16 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 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 #2 from Stephen Gallagher sgall...@redhat.com 2011-11-10 16:46:09
EST ---
[sgallagh@sgallagh520 review]$ rpmlint ~/rpmbuild/RPMS/x86_64/*.rpm
libverto.x86_64: W: spelling-error %description -l en_US io - oi, Io, ii
libverto-devel.x86_64: W: no-documentation
libverto-glib.x86_64: W: summary-not-capitalized C glib module for libverto
libverto-glib.x86_64: W: no-documentation
libverto-glib-devel.x86_64: W: no-documentation
libverto-libev.x86_64: W: summary-not-capitalized C libev module for libverto
libverto-libev.x86_64: W: spelling-error %description -l en_US io - oi, Io, ii
libverto-libev.x86_64: W: no-documentation
libverto-libev-devel.x86_64: W: spelling-error %description -l en_US io - oi,
Io, ii
libverto-libev-devel.x86_64: W: no-documentation
libverto-libevent.x86_64: W: summary-not-capitalized C libevent module for
libverto
libverto-libevent.x86_64: W: no-documentation
libverto-libevent-devel.x86_64: W: no-documentation
libverto-tevent.x86_64: W: summary-not-capitalized C tevent module for libverto
libverto-tevent.x86_64: W: spelling-error %description -l en_US io - oi, Io,
ii
libverto-tevent.x86_64: W: no-documentation
libverto-tevent-devel.x86_64: W: no-documentation
11 packages and 0 specfiles checked; 0 errors, 17 warnings.
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release}
This is not met. The {?_isa} is missing.
So, fix that one minor issue and I will approve this.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
