[Bug 737293] Review Request: python-django - A high-level Python Web framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=737293 --- Comment #17 from Bohuslav "Slavek" Kabrda 2012-03-14 02:47:25 EDT --- (In reply to comment #16) > Weird, those are different from the build errors I get from a local mock > setup. > > My suggestion is adding an optional --with build option for running the test > suite, until we resolve these test problems; until then, maintainers can make > sure that a local rpmbuild rebuild (which currently works for 1.3.1-7) at > least > passes. When the tests are fixed we can flip the switch to make building with > tests the default (and maybe make it non-default for unbranched Rawhide so we > can quickly test new versions). > > Bohuslav, any other change we need to make for the review, apart from the test > suite as already discussed? Thanks. Hi guys, I've been ill for a few days, so if you let me catch up a bit, I'll try to have a look at this as soon as I have some time. 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 801439] Review Request: rubygem-netrc - Library to read and write netrc files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801439 Bohuslav "Slavek" Kabrda changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #7 from Bohuslav "Slavek" Kabrda 2012-03-14 02:31:11 EDT --- Package Change Request == Package Name: rubygem-netrc New Branches: f17 Owners: vondruch bkabrda -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 800720] Review Request: resteasy - Framework for RESTful Web services and Java applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=800720 --- Comment #3 from Ade Lee 2012-03-14 01:28:24 EDT --- Based on feedback from jhernand, I have updated the spec file and srpm. The new spec file and srpm can be downloaded from the same links as above: Spec URL: http://vakwetu.fedorapeople.org/resteasy/resteasy.spec SRPM Url: http://vakwetu.fedorapeople.org/resteasy/resteasy-2.3.2-1.fc17.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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=772432 Huzaifa S. Sidhpurwala changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Huzaifa S. Sidhpurwala 2012-03-14 00:51:33 EDT --- Package Change Request == Package Name: gnome-applet-sensors New Branches: f17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 797706] Review Request: ghc-aeson - Fast JSON parsing and encoding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=797706 Shakthi Kannan changed: What|Removed |Added Status|MODIFIED|VERIFIED Flag||fedora-review+ --- Comment #5 from Shakthi Kannan 2012-03-14 00:38:46 EDT --- 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 797706] Review Request: ghc-aeson - Fast JSON parsing and encoding
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=797706 --- Comment #4 from Jens Petersen 2012-03-14 00:11:51 EDT --- Ok thanks for posting the fedora-review output. BTW I just tried with 0.1.3 and it gives only: Issues: [!]: MUST If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. See: http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text [!]: MUST Development (unversioned) .so files in -devel subpackage, if present. Note: ghc-aeson-0.6.0.0-1.fc18.i686.rpm : /usr/lib/ghc-7.0.4/aeson-0.6.0.0/libHSaeson-0.6.0.0-ghc7.0.4.so See: http://fedoraproject.org/wiki/Packaging/Guidelines#DevelPackages As you can see ghc-aeson does include the license file so it is false positive: maybe fedora-review only looks in the spec file? The second one seems new to me and while valid generally I think it can be waived here since ghc only created one single .so. Indeed the SONAME of ghc shared libraries is the full filename. In this context it might be more natural to write the filename something like libHSaeson.so.0.6.0.0-ghc7.0.4. Anyway this is obviously true for every ghc built shared library already in Fedora so I suggest to waive this also since the libraries are only used by ghc itself. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803149] New: Review Request: pyrasite - Code injection and monitoring of running Python processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pyrasite - Code injection and monitoring of running Python processes https://bugzilla.redhat.com/show_bug.cgi?id=803149 Summary: Review Request: pyrasite - Code injection and monitoring of running Python processes Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lmac...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://lmacken.fedorapeople.org/rpms/pyrasite.spec SRPM URL: http://lmacken.fedorapeople.org/rpms/pyrasite-2.0-0.1.beta3.fc16.src.rpm Description: Pyrasite uses gdb to inject code into a running Python process. It is comprised of a command-line tool, a Python API, and a graphical interface. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803149] Review Request: pyrasite - Code injection and monitoring of running Python processes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803149 Luke Macken changed: What|Removed |Added Depends on||803148 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803148] Review Request: python-pycallgraph - A module that creates call graphs for Python programs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803148 Luke Macken changed: What|Removed |Added Blocks||803149 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803148] New: Review Request: python-pycallgraph - A module that creates call graphs for Python programs
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-pycallgraph - A module that creates call graphs for Python programs https://bugzilla.redhat.com/show_bug.cgi?id=803148 Summary: Review Request: python-pycallgraph - A module that creates call graphs for Python programs Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lmac...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://lmacken.fedorapeople.org/rpms/python-pycallgraph.spec SRPM URL: http://lmacken.fedorapeople.org/rpms/python-pycallgraph-0.5.1-1.fc16.src.rpm Description: Python Call Graph uses GraphViz to generate call graphs from one execution of your Python code. It's very easy to use and can point out possible problems with your code execution. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728837] Review Request: xml2dict - Use attributes of dictionary to access xml elements.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=728837 --- Comment #23 from Yuguang Wang 2012-03-13 23:13:35 EDT --- ping, I made a koji scratch build here: [yuwang@yuwang ~]$ koji build --scratch dist-rawhide /home/yuwang/rpmbuild/SRPMS/xml2dict-0-0.3.2008.6.src.rpm Uploading srpm: /home/yuwang/rpmbuild/SRPMS/xml2dict-0-0.3.2008.6.src.rpm [] 100% 00:00:01 5.32 KiB 3.68 KiB/sec Created task: 3892856 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=3892856 Watching tasks (this may be safely interrupted)... 3892856 build (dist-rawhide, xml2dict-0-0.3.2008.6.src.rpm): open (ppc12.phx2.fedoraproject.org) 3892857 buildArch (xml2dict-0-0.3.2008.6.src.rpm, noarch): open (x86-12.phx2.fedoraproject.org) 3892856 build (dist-rawhide, xml2dict-0-0.3.2008.6.src.rpm): open (ppc12.phx2.fedoraproject.org) -> closed 0 free 1 open 1 done 0 failed 3892857 buildArch (xml2dict-0-0.3.2008.6.src.rpm, noarch): open (x86-12.phx2.fedoraproject.org) -> closed 0 free 0 open 2 done 0 failed 3892856 build (dist-rawhide, xml2dict-0-0.3.2008.6.src.rpm) completed successfully http://koji.fedoraproject.org/koji/taskinfo?taskID=3892856 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 799810] Review Request: python-picloud - PiCloud client-side 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=799810
--- Comment #11 from Amit Saha 2012-03-13 22:08:41 EDT
---
(In reply to comment #10)
> Please look at these examples:
>
> http://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs
>
> Your format is the one without the dash, but you're missing the release
> number.
>
> * Fri Mar 9 2012 Amit Saha 2.4.2
>
> should be
>
> * Fri Mar 9 2012 Amit Saha 2.4.2-2
>
> %{_bindir}/ must be %{_bindir}/*, otherwise you're trying to own /usr/bin.
>
> You could also consider to patch setup.py. That'd get you rid of the error
> message while installing and you don't have to install manpage and bash
> completion file on your own.
>
> On second thoughts: The bash completion should probably not be considered
> configuration at all. I looked through a couple of bash completion files (yum,
> mock, git, bash-completion) and none of them labels it as configuration. I
> guess you should do the same.
>
> rpm -qf /etc/bash_completion.d/mock.bash
> rpm -qc mock
> ...
>
> If you incorporate these changes, the package is pretty fine from my point of
> view. Nevertheslles I can't take the review, because you need a sponsor.
>
> For the reviewer: The package now builds in Mock.
Many thanks for the comments. I have incorporated these changes, and checked in
Mock. It seems to be fine. Updated files:
SPEC file:
http://amitksaha.fedorapeople.org/contribs/picloud_packaging/python-picloud.spec
SRPM:
http://amitksaha.fedorapeople.org/contribs/picloud_packaging/python-picloud-2.4.2-3.fc16.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 801947] Review Request: papaki - Library for scanning annotations in Java 5+ 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=801947 Ricardo Arguello changed: What|Removed |Added Priority|unspecified |low -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 801651] Review Request: jboss-jacc-1.4-api - JBoss Java Authorization Contract for Containers 1.4 API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801651 --- Comment #7 from Fedora Update System 2012-03-13 19:28:32 EDT --- jboss-jacc-1.4-api-1.0.2-0.1.20120310git7976d2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/jboss-jacc-1.4-api-1.0.2-0.1.20120310git7976d2.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 801651] Review Request: jboss-jacc-1.4-api - JBoss Java Authorization Contract for Containers 1.4 API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801651 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] New: Review Request: whenjobs - Replacement for cron with dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: whenjobs - Replacement for cron with dependencies https://bugzilla.redhat.com/show_bug.cgi?id=803089 Summary: Review Request: whenjobs - Replacement for cron with dependencies Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rjo...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- NB: This requires F17+ to build! Spec URL: http://oirase.annexia.org/reviews/whenjobs/whenjobs.spec SRPM URL: http://oirase.annexia.org/reviews/whenjobs/whenjobs-0.7.0-1.src.rpm Description: Whenjobs is a powerful but simple cron replacement. Two key advantages over cron are a simpler syntax for writing rules and a powerful dependency system that lets one job depend on variables set when other jobs run (allowing, for example, one job to run only when another job has finished successfully). A Koji scratch build is here: http://koji.fedoraproject.org/koji/taskinfo?taskID=3892254 rpmlint says: whenjobs.src: W: spelling-error Summary(en_US) cron -> corn, con, crone whenjobs.src: W: spelling-error %description -l en_US cron -> corn, con, crone Nonsense, of course. whenjobs.x86_64: E: devel-dependency ocaml-camlp4-devel We really need the camlp4 packages. whenjobs.x86_64: W: spelling-error Summary(en_US) cron -> corn, con, crone whenjobs.x86_64: W: spelling-error %description -l en_US cron -> corn, con, crone More non-spelling-mistakes. whenjobs.x86_64: W: unstripped-binary-or-object /usr/sbin/whenjobsd whenjobs.x86_64: W: ocaml-mixed-executable /usr/sbin/whenjobsd whenjobs.x86_64: W: unstripped-binary-or-object /usr/bin/whenjobs whenjobs.x86_64: W: ocaml-mixed-executable /usr/bin/whenjobs These binaries are unstripped because they contain bytecode. 'strip' would remove the bytecode. whenjobs.x86_64: W: only-non-binary-in-usr-lib /usr/lib contains binaries, but rpmlint doesn't recognise them as such. whenjobs.x86_64: W: non-conffile-in-etc /etc/prelink.conf.d/whenjobs.conf This is not a conf file, but an instruction to prelink to stop it destroying the bytecode binaries. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750898] Review Request: icaro - A robot automation language
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750898 --- Comment #6 from Guillermo Gómez 2012-03-13 18:00:49 EDT --- (In reply to comment #2) > I'll review it (and will sponsor you). AFAIK LL is already sponsored, he just need to be reviewed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803082] Review Request: pogo - Probably the simplest and fastest audio player for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803082 Mario Blättermann changed: What|Removed |Added Alias||pogo --- Comment #1 from Mario Blättermann 2012-03-13 17:58:11 EDT --- Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3892079 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803082] New: Review Request: pogo - Probably the simplest and fastest audio player for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pogo - Probably the simplest and fastest audio player for Linux https://bugzilla.redhat.com/show_bug.cgi?id=803082 Summary: Review Request: pogo - Probably the simplest and fastest audio player for Linux Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mario.blaetterm...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/pogo.spec SRPM URL: http://mariobl.fedorapeople.org/Review/SRPMS/pogo-0.5-1.fc16.src.rpm Description: Pogo's elementary-inspired design uses the screen-space very efficiently. It is especially well-suited for people who organize their music by albums on the harddrive. The main interface components are a directory tree and a playlist that groups albums in an innovative way. Supported file formats include Ogg Vorbis, MP3, FLAC, Musepack, Wavpack, and MPEG-4 AAC. Pogo is a fork of Decibel Audio Player. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 760154] Review Request: xcb-util-keysyms - Standard X key constants and keycodes conversion on top of libxcb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=760154 Mohamed El Morabity changed: What|Removed |Added Blocks||802952 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ 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=773011 --- Comment #4 from Richard Shaw 2012-03-13 17:40:04 EDT --- (In reply to comment #1) > X license ( ) OK, text in %doc, matches source Should be GPLv2 or LGPLv2, I > think. Forgot to comment on this. I think since this is only an end user binary (script) that doesn't provide a library of any kind that it's only GPLv2, but I'm not a license guru either. Richard -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 794715] Review Request: commons-ognl - Object Graph Navigation 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=794715 --- Comment #8 from Andy Grimm 2012-03-13 17:40:58 EDT --- SPEC: http://downloads.eucalyptus.com/devel/packages/fedora-17/SPECS/apache-commons-ognl.spec SRPM: http://downloads.eucalyptus.com/devel/packages/fedora-17/sources/apache-commons-ognl-3.0.2-1.20120313svn1102435.fc17.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 803057] New: Review Request: perl-Test-Valgrind - Generate suppressions, analyze and test any command with valgrind
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-Test-Valgrind - Generate suppressions, analyze and test any command with valgrind https://bugzilla.redhat.com/show_bug.cgi?id=803057 Summary: Review Request: perl-Test-Valgrind - Generate suppressions, analyze and test any command with valgrind Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@city-fan.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Test-Valgrind/branches/fedora/perl-Test-Valgrind.spec SRPM URL: http://www.city-fan.org/~paul/extras/perl-Test-Valgrind/perl-Test-Valgrind-1.13-2.fc18.src.rpm Description: The Test::Valgrind::* API lets you run Perl code through the memcheck tool of the valgrind memory debugger, to test for memory errors and leaks. The Test::Valgrind module itself is a front-end to this API. If they aren't available yet, it will first generate suppressions for the current perl interpreter and store them in the portable flavor of ~/.perl/Test-Valgrind/suppressions/$VERSION. The actual run will then take place, and tests will be passed or failed according to the result of the analysis. The complete API is much more versatile than this. By declaring an appropriate Test::Valgrind::Command class, you can run any executable (that is, not only Perl scripts) under valgrind, generate the corresponding suppressions on-the-fly and convert the analysis result to TAP output so that it can be incorporated into your project's test suite. If you're not interested in producing TAP, you can output the results in whatever format you like (for example HTML pages) by defining your own Test::Valgrind::Action class. Dependencies not yet in Fedora: perl-Env-Sanctify (Bug #802377) perl-Perl-Destruct-Level (Bug #802865) For the EPEL-5 build, I need perl-File-HomeDir updating to at least 0.86 (Bug #803044), plus perl-Test-Kwalitee, which I can't build until perl-Sub-Exporter has been updated to at least version 0.979 (Bug #73). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802377] Review Request: perl-Env-Sanctify - Lexically scoped sanctification of %ENV
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802377 Paul Howarth changed: What|Removed |Added Blocks||803057 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803057] Review Request: perl-Test-Valgrind - Generate suppressions, analyze and test any command with valgrind
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803057 Paul Howarth changed: What|Removed |Added Depends on||802377, 802865 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802865] Review Request: perl-Perl-Destruct-Level - Allows you to change perl's internal destruction level
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802865 Paul Howarth changed: What|Removed |Added Blocks||803057 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785416] Review Request: python-xappy - A Python module providing an easy-to-use layer on top of the Xapian search engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=785416 --- Comment #3 from Luke Macken 2012-03-13 17:09:55 EDT --- Spec URL: http://lmacken.fedorapeople.org/rpms/python-xappy.spec SRPM URL: http://lmacken.fedorapeople.org/rpms/python-xappy-0.6.0-0.1.svn624.fc16.src.rpm * Tue Mar 13 2012 Luke Macken - 0.6.0-0.2.svn624 - Fix a typo in the license - Shorten the summary - Require xapian-bindings-python - Remove shebangs - Add svn snapshot instructions - Remove the buildroot tag, buildroot cleaning, and defattr - Include the license -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785416] Review Request: python-xappy - A Python module providing an easy-to-use layer on top of the Xapian search engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=785416 --- Comment #5 from Luke Macken 2012-03-13 17:11:19 EDT --- Actual SRPM URL (part 2): http://lmacken.fedorapeople.org/rpms/python-xappy-0.6.0-0.2.svn624.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 785416] Review Request: python-xappy - A Python module providing an easy-to-use layer on top of the Xapian search engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=785416 --- Comment #4 from Luke Macken 2012-03-13 17:10:27 EDT --- Actual SRPM URL: http://lmacken.fedorapeople.org/rpms/python-xappy-0.6.0-0.12svn624.fc16.sc.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 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ 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=773011 --- Comment #3 from Richard Shaw 2012-03-13 17:00:20 EDT --- Spec URL: http://hobbes1069.fedorapeople.org/api-sanity-checker/api-sanity-checker.spec SRPM URL: http://hobbes1069.fedorapeople.org/api-sanity-checker/api-sanity-checker-1.12.9-2.fc16.src.rpm Added the man page! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 794715] Review Request: commons-ognl - Object Graph Navigation 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=794715
--- Comment #7 from Juan Hernández 2012-03-13
16:53:41 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[!] Rpmlint output:
$ rpmlint commons-ognl-3.0.2-1.fc17.src.rpm
commons-ognl.src: W: invalid-url Source0: commons-ognl-3.0.2.tar.xz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
Can't check binary packages as build fails.
[!] Package is named according to the Package Naming Guidelines[1].
As pointed out by Alexander Kurtakov other packages from Apache commons are
named apache-commons-whatever, so this one should be apache-commons-ognl.
Being a post release the version and release tags should be:
3.0.2-1.20120313svn1102435
[x] Spec file name must match the base package name, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines[2].
[!] Package successfully compiles and builds into binary rpms.
See http://koji.fedoraproject.org/koji/taskinfo?taskID=3891615. The relevant
error is the following in build.log:
The repository system is offline but the artifact
net.java.dev.jna:jna:jar:3.2.2 is not available in the local repository.
This can be resolved adding "BuildRequires: jna".
[x] Buildroot definition is not present
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type: ASL 2.0
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package: 33b6d3507a5fd8e952c6d156b891d022
MD5SUM upstream package: ee9905892d509ef4bf2cec826eaa2fa3
Compared the sources in this package with the upstream sources using diff and
there is no difference.
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates or must require other
packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[x] Permissions on files are set properly.
[x] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[-] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[x] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[x] Javadoc subpackages have Require: jpackage-utils
[x] Package uses %global not %define
[x] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[-] All filenames in rpm packages must be valid UTF-8.
[x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x] If package contains pom.xml files install it (including depmaps) even when
building with ant
[x] pom files has correct add_maven_depmap
=== Maven ===
[x] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why
it's needed in a comment
[x] Package DOES NOT use %update_maven_depmap in %post/%postun
[x] Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[!] Latest version is packaged.
Version in the package is revision 1239219 and latest in the upstream
repository is 1102435.
[!] Reviewer should test that the package builds in mock.
Tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=3891615
=== Issues ===
1. The rpmlint warning about t
[Bug 785727] Review Request: ocaml-camlimages - OCaml image processing 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=785727 --- Comment #16 from Fedora Update System 2012-03-13 16:50:31 EDT --- ocaml-camlimages-4.0.1-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/ocaml-camlimages-4.0.1-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 785727] Review Request: ocaml-camlimages - OCaml image processing 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=785727 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785727] Review Request: ocaml-camlimages - OCaml image processing 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=785727 --- Comment #15 from Fedora Update System 2012-03-13 16:50:20 EDT --- ocaml-camlimages-4.0.1-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/ocaml-camlimages-4.0.1-2.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785727] Review Request: ocaml-camlimages - OCaml image processing 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=785727 --- Comment #14 from Fedora Update System 2012-03-13 16:49:22 EDT --- ocaml-camlimages-4.0.1-2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/ocaml-camlimages-4.0.1-2.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ 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=773011 --- Comment #2 from Richard Shaw 2012-03-13 16:40:00 EDT --- (In reply to comment #1) > Good: > > - rpmlint checks return: > > api-sanity-checker.src: W: summary-ended-with-dot C An automatic generator of > basic unit tests for a shared C/C++ library. > Summary ends with a dot. > > Fix. Cut-n-paste bites me again! Fixed. > api-sanity-checker.noarch: W: no-manual-page-for-binary api-sanity-checker > Each executable in standard binary directories should have a man page. > > If available. It's not, but I may be able to come up with a help2man 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 715127] Review Request: abcMIDI - ABC to/from MIDI conversion utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=715127 --- Comment #13 from Volker Fröhlich 2012-03-13 16:37:11 EDT --- How's it going? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803018 Richard Shaw changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #2 from Richard Shaw 2012-03-13 16:28:30 EDT --- +: OK -: must be fixed =: should be fixed (at your discretion) ?: Question or clairification needed N: not applicable MUST: [+] rpmlint output: shown in comment. [+] follows package naming guidelines [+] spec file base name matches package name [+] package meets the packaging guidelines [+] package uses a Fedora approved license: LGPLv3+ [+] license field matches the actual license. [+] license file is included in %doc: COPYING [+] spec file is in American English [+] spec file is legible [+] sources match upstream: md5 sum matches (83be32a820b5d5211431b0d6f56181a9) [+] package builds on at least one primary arch: Tested F16 x86_64 [N] appropriate use of ExcludeArch [+] all build requirements in BuildRequires [N] spec file handles locales properly [N] ldconfig in %post and %postun [+] no bundled copies of system libraries [+] no relocatable packages [N] package owns all directories that it creates [+] no files listed twice in %files [+] proper permissions on files [+] consistent use of macros [+] code or permissible content [N] large documentation in -doc [+] no runtime dependencies in %doc [N] header files in -devel [N] static libraries in -static [N] .so in -devel [N] -devel requires main package [+] package contains no libtool archives [N] package contains a desktop file, uses desktop-file-install/validate [+] package does not own files/dirs owned by other packages [+] all filenames in UTF-8 SHOULD: [+] query upstream for license text [N] description and summary contains available translations [+] package builds in mock [+] package builds on all supported arches: Tested x86_64 [?] package functions as described: Not tested [+] sane scriptlets [N] subpackages require the main package [N] placement of pkgconfig files [N] file dependencies versus package dependencies [N] package contains man pages for binaries/scripts I'm guessing it's ok but what is the purpose of Source1? I'm assuming it's some sort of signature file but it's never installed anywhere so it's only in the SRPM. *** 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 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ 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=773011 --- Comment #1 from Jon Ciesla 2012-03-13 16:27:52 EDT --- Good: - rpmlint checks return: api-sanity-checker.src: W: summary-ended-with-dot C An automatic generator of basic unit tests for a shared C/C++ library. Summary ends with a dot. Fix. api-sanity-checker.src: W: invalid-url URL: http://forge.ispras.ru/projects/api-sanity-autotest The value should be a valid, public HTTP, HTTPS, or FTP URL. Worked for my browser. api-sanity-checker.noarch: W: no-manual-page-for-binary api-sanity-checker Each executable in standard binary directories should have a man page. If available. - package meets naming guidelines - package meets packaging guidelines X license ( ) OK, text in %doc, matches source Should be GPLv2 or LGPLv2, I think. - spec file legible, in am. english - source matches upstream - package compiles on devel (x86_64) - no missing BR - no unnecessary BR - no locales - not relocatable - owns all directories that it creates - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - no need for .desktop file Otherwise, it looks good. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 773011] Review Request: api-sanity-checker - An automatic generator of basic unit tests for a shared C/C++ 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=773011 Jon Ciesla changed: What|Removed |Added Status|NEW |ASSIGNED CC||limburg...@gmail.com AssignedTo|nob...@fedoraproject.org|limburg...@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 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803018 Richard Shaw changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #1 from Richard Shaw 2012-03-13 16:16:59 EDT --- The spec looks good although it's got a lot of the stuff that isn't necessary anymore unless you're going to build for EL5... BuildRoot: rm -rf $RPM_BUILD_ROOT in %install %clean %defattr in %files Obviously not a showstopper... full review to follow. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803018 Richard Shaw changed: What|Removed |Added CC||hobbes1...@gmail.com AssignedTo|nob...@fedoraproject.org|hobbes1...@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 794715] Review Request: commons-ognl - Object Graph Navigation 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=794715 Juan Hernández changed: What|Removed |Added Status|NEW |ASSIGNED CC||juan.hernan...@redhat.com AssignedTo|nob...@fedoraproject.org|juan.hernan...@redhat.com Flag||fedora-review? --- Comment #6 from Juan Hernández 2012-03-13 16:07:16 EDT --- I am taking this for review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 794715] Review Request: commons-ognl - Object Graph Navigation 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=794715 Alexander Kurtakov changed: What|Removed |Added Blocks|182235(FE-Legal)| --- Comment #5 from Alexander Kurtakov 2012-03-13 16:04:31 EDT --- I'm dropping the FE-LEGAL block as the codebase seems to be properly licensed with already approved license we no longer need the legal team opinion. FWIW, all other apache commons packages are named apache-commons-smth so it would be good if this one follows the same name scheme at least for consistency. About versioning the guidelines are pretty clear see https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 799896] Review Request: python-django-tracking - Django site visitor tracking, including basic blacklisting
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=799896
Cédric OLIVIER changed:
What|Removed |Added
CC||cedric.oliv...@free.fr
--- Comment #2 from Cédric OLIVIER 2012-03-13 15:10:05
EDT ---
As Django, could you use your own %find_lang rather than define locale in
%files section :
# Handling locale files
# This is adapted from the %%find_lang macro, which cannot be directly
# used since Django locale files are not located in %%{_datadir}
#
# The rest of the packaging guideline still apply -- do not list
# locale files by hand!
(cd $RPM_BUILD_ROOT && find . -name 'django*.mo') | %{__sed} -e 's|^.||' |
%{__sed} -e \
's:\(.*/locale/\)\([^/_]\+\)\(.*\.mo$\):%lang(\2) \1\2\3:' \
>> %{name}.lang
You can find more about this "problem" here :
https://bugzilla.redhat.com/show_bug.cgi?id=584866
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802929] New: Review Request: jboss-iiop-client - JBoss IIOP Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: jboss-iiop-client - JBoss IIOP Client https://bugzilla.redhat.com/show_bug.cgi?id=802929 Summary: Review Request: jboss-iiop-client - JBoss IIOP Client Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mgold...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://goldmann.fedorapeople.org/package_review/jboss-iiop-client/1/jboss-iiop-client.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/jboss-iiop-client/1/jboss-iiop-client-1.0.0-1.fc17.src.rpm Description: Client library for EJB applications working against JBoss AS using the IIOP protocol -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802929] Review Request: jboss-iiop-client - JBoss IIOP Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802929 Marek Goldmann changed: What|Removed |Added Depends on||730234 Blocks||652183(FE-JAVASIG) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730234] Review Request: jboss-ejb-3.1-api - EJB 3.1 API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730234 Marek Goldmann changed: What|Removed |Added Blocks||802929 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 718395] Review Request: libmusicbrainz4-4.0.0 - Library for accessing MusicBrainz servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=718395 Fedora Update System changed: What|Removed |Added Fixed In Version||libmusicbrainz4-4.0.0-1.fc1 ||7 Resolution|NEXTRELEASE |ERRATA --- Comment #15 from Fedora Update System 2012-03-13 14:42:07 EDT --- libmusicbrainz4-4.0.0-1.fc17 has been pushed to the Fedora 17 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 801695] Review Request: stax-ex - StAX API extensions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801695 Fedora Update System changed: What|Removed |Added Fixed In Version||stax-ex-1.7-1.fc17 Resolution|RAWHIDE |ERRATA --- Comment #8 from Fedora Update System 2012-03-13 14:45:19 EDT --- stax-ex-1.7-1.fc17 has been pushed to the Fedora 17 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 784784] Review Request: ghc-base-unicode-symbols - Unicode alternatives to common Haskell operators and functions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784784 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-base-unicode-symbols-0. ||2.2.3-1.fc17 Resolution||ERRATA Last Closed||2012-03-13 14:44:56 --- Comment #9 from Fedora Update System 2012-03-13 14:44:56 EDT --- ghc-base-unicode-symbols-0.2.2.3-1.fc17 has been pushed to the Fedora 17 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 803018] New: Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format https://bugzilla.redhat.com/show_bug.cgi?id=803018 Summary: Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: limburg...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- SRPM: http://fedorapeople.org/~limb/review/lziprecover/lziprecover-1.13-1.fc16.src.rpm SPEC: http://fedorapeople.org/~limb/review/lziprecover/lziprecover.spec Description: Lziprecover is a data recovery tool and decompressor for files in the lzip compressed data format (.lz) able to repair slightly damaged files, recover badly damaged files from two or more copies, extract undamaged members from multi-member files, decompress files and test integrity of files. Lziprecover is able to recover or decompress files produced by any of the compressors in the lzip family; lzip, plzip, minilzip/lzlib, clzip and pdlzip. This recovery capability contributes to make the lzip format one of the best options for long-term data archiving. Recently split from the lzip package, already in Fedora. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803018] Review Request: lziprecover - Data recovery tool and decompressor for files in the lzip compressed format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=803018 Jon Ciesla changed: What|Removed |Added Blocks||802973 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 799089] Review Request: dyninst - An API for Run-time Code Generation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=799089
Frank Ch. Eigler changed:
What|Removed |Added
Flag||needinfo?(wco...@redhat.com
||)
--- Comment #3 from Frank Ch. Eigler 2012-03-13 15:19:10 EDT
---
ReviewGuidelines MUST items failed:
MUST: packaging guidelines:
- bundled librares, as below
- make DESTDIR=%{buildroot} should work
- %configure should be used
MUST: not bundle system libraries
- boost is bundled, and is even installed into dyninst-devel.
ReviewGuidelines SHOULD items failed:
/usr/bin/parseThat should have a man page
Other items in Packaging:ReviewGuidelines and Packaging:Guidelines appear OK.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 800738] Review Request: avro - Apache Avro is a data serialization system
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=800738
--- Comment #2 from Marek Goldmann 2012-03-13 14:35:00
EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Rpmlint output:
$ rpmlint SPECS/avro.spec
SPECS/avro.spec: W: invalid-url Source0: avro-1.6.2-CLEAN.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
$ rpmlint SRPMS/avro-1.6.2-2.fc17.src.rpm
avro.src: I: enchant-dictionary-not-found en_US
avro.src: W: invalid-url Source0: avro-1.6.2-CLEAN.tar.xz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
$ rpmlint RPMS/noarch/avro-1.6.2-2.fc17.noarch.rpm
avro.noarch: I: enchant-dictionary-not-found en_US
avro.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[x] Package is named according to the Package Naming Guidelines[1].
[X] Spec file name must match the base package name, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines[2].
[x] Package successfully compiles and builds into binary rpms.
[x] Buildroot definition is not present
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type: ASL 2.0
[!] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
See #1.
[-] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package: 7fb386f43c9029a27e2110bc9e309a1b
MD5SUM upstream package: b90d7cb26ec37f655769d42373e40a22
SVN export.
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates or must require other
packages for directories it uses.
[X] Package does not contain duplicates in %files.
[x] File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[x] Permissions on files are set properly.
[x] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[x] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[x] Javadoc subpackages have Require: jpackage-utils
[x] Package uses %global not %define
[x] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[x] If source tarball includes bundled jar/class files these need to be
removed prior to building
[x] All filenames in rpm packages must be valid UTF-8.
[x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x] If package contains pom.xml files install it (including depmaps) even when
building with ant
[x] pom files has correct add_maven_depmap
=== Maven ===
[x] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[x] If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why
it's needed in a comment
[x] Package DOES NOT use %update_maven_depmap in %post/%postun
[x] Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[x] Latest version is packaged.
[x] Reviewer should test that the package builds in mock.
Tested on:
locally
=== Issues ===
1. Please inclcude the LICENSE.txt file in main and javadocs package.
=== Final Notes ===
1. Please remove also the .dll files from source tarbal.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraprojec
[Bug 784799] Review Request: ghc-monad-control - Lift control operations through monad transformers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784799 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-monad-control-0.3.1-1.f ||c17 Resolution||ERRATA Last Closed||2012-03-13 14:35:26 Bug 784799 depends on bug 784769, which changed state. Bug 784769 Summary: Review Request: ghc-transformers-base - Haskell monad transformer lifting library https://bugzilla.redhat.com/show_bug.cgi?id=784769 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #8 from Fedora Update System 2012-03-13 14:35:26 EDT --- ghc-monad-control-0.3.1-1.fc17 has been pushed to the Fedora 17 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 734275] Review Request: aqemu - A QT graphical interface to QEMU and KVM
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=734275 Fedora Update System changed: What|Removed |Added Fixed In Version||aqemu-0.8.2-7.fc17 Resolution|NEXTRELEASE |ERRATA --- Comment #34 from Fedora Update System 2012-03-13 14:32:47 EDT --- aqemu-0.8.2-7.fc17 has been pushed to the Fedora 17 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 800890] Review Request: mojarra - JSF Reference Implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=800890 Fedora Update System changed: What|Removed |Added Fixed In Version||mojarra-2.1.7-2.fc17 Resolution|RAWHIDE |ERRATA --- Comment #6 from Fedora Update System 2012-03-13 14:31:55 EDT --- mojarra-2.1.7-2.fc17 has been pushed to the Fedora 17 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 788159] Review Request: ghc-conduit - Streaming data processing 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=788159 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-conduit-0.2.2-1.fc17 Resolution||ERRATA Last Closed||2012-03-13 14:29:26 --- Comment #8 from Fedora Update System 2012-03-13 14:29:26 EDT --- ghc-conduit-0.2.2-1.fc17 has been pushed to the Fedora 17 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 784156] Review Request: uwsgi - Fast, self-healing, application container server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=784156 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||uwsgi-1.0.4-1.fc17 Resolution||ERRATA Last Closed||2012-03-13 14:29:55 --- Comment #27 from Fedora Update System 2012-03-13 14:29:55 EDT --- uwsgi-1.0.4-1.fc17 has been pushed to the Fedora 17 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 743612] Review Request: lbdb - collect email addresses from several sources and offer them in mutt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=743612 Fedora Update System changed: What|Removed |Added Fixed In Version||lbdb-0.38-2.fc17 Resolution|NEXTRELEASE |ERRATA --- Comment #10 from Fedora Update System 2012-03-13 14:31:17 EDT --- lbdb-0.38-2.fc17 has been pushed to the Fedora 17 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 802909] Review Request: jboss-interceptor - JBoss EJB Interceptor 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=802909 Marek Goldmann changed: What|Removed |Added Depends on||730234 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730234] Review Request: jboss-ejb-3.1-api - EJB 3.1 API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730234 Marek Goldmann changed: What|Removed |Added Blocks||802909 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802909] New: Review Request: jboss-interceptor - JBoss EJB Interceptor 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: jboss-interceptor - JBoss EJB Interceptor Library https://bugzilla.redhat.com/show_bug.cgi?id=802909 Summary: Review Request: jboss-interceptor - JBoss EJB Interceptor Library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mgold...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://goldmann.fedorapeople.org/package_review/jboss-interceptor/1/jboss-interceptor.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/jboss-interceptor/1/jboss-interceptor-2.0.0-1.fc17.src.rpm Description: JBoss EJB 3.1 Common Interceptor 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 802909] Review Request: jboss-interceptor - JBoss EJB Interceptor 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=802909 Marek Goldmann changed: What|Removed |Added Blocks||652183(FE-JAVASIG) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 737286] Review Request: salt - A parallel remote execution system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=737286 --- Comment #40 from Fedora Update System 2012-03-13 14:02:53 EDT --- salt-0.9.7-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/salt-0.9.7-2.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 737286] Review Request: salt - A parallel remote execution system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=737286 --- Comment #39 from Fedora Update System 2012-03-13 14:02:05 EDT --- salt-0.9.7-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/salt-0.9.7-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 737286] Review Request: salt - A parallel remote execution system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=737286 --- Comment #38 from Fedora Update System 2012-03-13 14:01:18 EDT --- salt-0.9.7-2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/salt-0.9.7-2.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 800738] Review Request: avro - Apache Avro is a data serialization system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=800738 Marek Goldmann changed: What|Removed |Added Status|NEW |ASSIGNED CC||mgold...@redhat.com AssignedTo|nob...@fedoraproject.org|mgold...@redhat.com Flag||fedora-review? --- Comment #1 from Marek Goldmann 2012-03-13 13:53:18 EDT --- Taking 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 798715] Review Request: luminance-hdr - A graphical tool for creating and tone-mapping HDR images
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=798715 --- Comment #24 from Jon Ciesla 2012-03-13 13:38:24 EDT --- 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 798715] Review Request: luminance-hdr - A graphical tool for creating and tone-mapping HDR images
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=798715 Franco Comida 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 730233] Review Request: jboss-jaxrpc-1.1-api - Java API for XML-Based RPC (JAX-RPC) 1.1
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730233 Marek Goldmann changed: What|Removed |Added Blocks|652183(FE-JAVASIG) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730233] Review Request: jboss-jaxrpc-1.1-api - Java API for XML-Based RPC (JAX-RPC) 1.1
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730233 --- Comment #18 from Fedora Update System 2012-03-13 13:31:28 EDT --- jboss-jaxrpc-1.1-api-1.0.1-0.1.20120309gita3c227.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/jboss-jaxrpc-1.1-api-1.0.1-0.1.20120309gita3c227.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 730233] Review Request: jboss-jaxrpc-1.1-api - Java API for XML-Based RPC (JAX-RPC) 1.1
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=730233 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 769919] Review Request: hydra - Very fast network log-on cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=769919 --- Comment #17 from Athmane Madjoudj 2012-03-13 13:11:57 EDT --- Upstream responded that he'll include the patchs/fixes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802388] Review Request: perl-Language-Prolog-Sugar - Syntactic sugar for Prolog term constructors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802388 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2012-03-13 13:09:31 EDT --- perl-Language-Prolog-Types-0.10-1.fc17, perl-Language-Prolog-Yaswi-0.19-1.fc17, perl-Language-Prolog-Sugar-0.06-1.fc17 has been pushed to the Fedora 17 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 802443] Review Request: perl-Language-Prolog-Yaswi - Yet another interface to SWI-Prolog
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802443 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2012-03-13 13:09:37 EDT --- perl-Language-Prolog-Types-0.10-1.fc17, perl-Language-Prolog-Yaswi-0.19-1.fc17, perl-Language-Prolog-Sugar-0.06-1.fc17 has been pushed to the Fedora 17 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 801865] Review Request: jboss-transaction-spi - JBoss Transaction SPI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801865 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System 2012-03-13 13:09:59 EDT --- jboss-transaction-spi-7.0.0-2.fc17 has been pushed to the Fedora 17 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 801614] Review Request: jboss-connector-1.6-api - Java EE Connector Architecture 1.6 API classes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801614 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2012-03-13 13:09:54 EDT --- jboss-connector-1.6-api-1.0.1-0.1.20120310git9dc9a5.fc17 has been pushed to the Fedora 17 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 796332] Review Request: paranamer - Method parameter name access
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=796332 Mattia Verga changed: What|Removed |Added Status|NEW |CLOSED CC||mattia.ve...@tiscali.it Resolution||DUPLICATE Flag||fedora-review- Last Closed||2012-03-13 12:53:21 --- Comment #1 from Mattia Verga 2012-03-13 12:53:21 EDT --- Closing, duplicate of #795801 *** This bug has been marked as a duplicate of bug 795801 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 795801] Review Request: paranamer - Library for accessing non-private method parameter names at run-time
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=795801 --- Comment #10 from Mattia Verga 2012-03-13 12:53:21 EDT --- *** Bug 796332 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 799976] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=799976 Juan Hernández changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2012-03-13 12:44:45 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802862] New: Review Request: drupal6-votingapi - Voting API module for Drupal6
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: drupal6-votingapi - Voting API module for Drupal6 https://bugzilla.redhat.com/show_bug.cgi?id=802862 Summary: Review Request: drupal6-votingapi - Voting API module for Drupal6 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: whe...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://jknife.fedorapeople.org/SPECS/drupal6-votingapi.spec SRPM URL: http://jknife.fedorapeople.org/SRPMS/drupal6-votingapi-2.3-3.el6.src.rpm Description: VotingAPI for Drupal 6 helps developers who want to use a standardized API and schema for storing, retrieving, and tabulating votes for Drupal content. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802865] New: Review Request: perl-Perl-Destruct-Level - Allows you to change perl's internal destruction level
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-Perl-Destruct-Level - Allows you to change perl's internal destruction level https://bugzilla.redhat.com/show_bug.cgi?id=802865 Summary: Review Request: perl-Perl-Destruct-Level - Allows you to change perl's internal destruction level Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@city-fan.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://subversion.city-fan.org/repos/cfo-repo/perl-Perl-Destruct-Level/branches/fedora/perl-Perl-Destruct-Level.spec SRPM URL: http://www.city-fan.org/~paul/extras/perl-Perl-Destruct-Level/perl-Perl-Destruct-Level-0.02-2.fc18.src.rpm Description: This module allows you to change perl's internal destruction level. The default value of the destruct level is 0; it means that perl won't bother destroying all of its internal data structures and lets the OS do the cleanup for it at exit. For perls built with debugging support (-DDEBUGGING), an environment variable PERL_DESTRUCT_LEVEL allows you to control the destruction level. This module enables you to modify it on non-debugging perls too. Note that some embedded environments might extend the meaning of the destruction level for their own purposes: mod_perl does that, for example. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 787020] Review Request: trafficserver - Apache Traffic Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=787020 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #16 from Fedora Update System 2012-03-13 12:23:10 EDT --- trafficserver-3.0.3-2.el6 has been pushed to the Fedora EPEL 6 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 758734] Review Request: fatrat-subtitlesearch - FatRat plugin enabling OpenSubtitles.org and Sublight.si 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=758734 Volker Fröhlich changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|volke...@gmx.at 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 799976] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=799976 --- Comment #6 from Jon Ciesla 2012-03-13 11:34:19 EDT --- 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 799976] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=799976 Juan Hernández changed: What|Removed |Added Blocks|652183(FE-JAVASIG) | Flag||fedora-cvs? --- Comment #5 from Juan Hernández 2012-03-13 11:30:39 EDT --- New Package SCM Request === Package Name: hibernate-validator Short Description: Bean Validation (JSR 303) Reference Implementation Owners: jhernand Branches: f17 InitialCC: goldmann, arg -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 799976] Review Request: hibernate-validator - Bean Validation (JSR 303) Reference Implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=799976
Andy Grimm changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #4 from Andy Grimm 2012-03-13 11:20:19 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Rpmlint output:
hibernate-validator.noarch: W: invalid-url URL:
http://www.hibernate.org/subprojects/validator.html HTTP Error 403: Forbidden
hibernate-validator.src: W: invalid-url URL:
http://www.hibernate.org/subprojects/validator.html HTTP Error 403: Forbidden
hibernate-validator.src: W: invalid-url Source0:
hibernate-validator-4.2.0.Final.tar.xz
hibernate-validator-javadoc.noarch: W: invalid-url URL:
http://www.hibernate.org/subprojects/validator.html HTTP Error 403: Forbidden
3 packages and 0 specfiles checked; 0 errors, 4 warnings.
this is normal for jboss packages' URLs
[x] Package is named according to the Package Naming Guidelines[1].
[x] Spec file name must match the base package name, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines[2].
[x] Package successfully compiles and builds into binary rpms.
[x] Buildroot definition is not present
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type: ASL 2.0
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
Git source, unpacked tarball matches
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates or must require other
packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[x] Permissions on files are set properly.
[x] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[x] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[x] Javadoc subpackages have Require: jpackage-utils
[x] Package uses %global not %define
[x] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[x] All filenames in rpm packages must be valid UTF-8.
[x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x] If package contains pom.xml files install it (including depmaps) even when
building with ant
[x] pom files has correct add_maven_depmap
=== Maven ===
[x] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why
it's needed in a comment
[x] Package DOES NOT use %update_maven_depmap in %post/%postun
[x] Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[x] Latest version is packaged.
[x] Reviewer should test that the package builds in mock.
Tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=3888606
=== Issues ===
None noted.
*** APPROVED ***
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
__
[Bug 225749] Merge Review: fetchmail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225749 Michal Hlavinka changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Flag|fedora-review? |fedora-review+ Last Closed||2012-03-13 10:41:50 --- Comment #5 from Michal Hlavinka 2012-03-13 10:41:50 EDT --- verified, everything looks ok now -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226252] Merge Review: perl-DBD-Pg
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226252 --- Comment #6 from Jon Ciesla 2012-03-13 10:39:43 EDT --- Done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226252] Merge Review: perl-DBD-Pg
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226252 Petr Pisar changed: What|Removed |Added CC||ppi...@redhat.com Flag||fedora-cvs? --- Comment #5 from Petr Pisar 2012-03-13 10:19:40 EDT --- Package Change Request == Package Name: perl-DBD-Pg Branches: f15 f16 f17 Owners: InitialCC: perl-sig Please add perl-sig user with watch* permissions only to all Fedora branches. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 801003] Review Request: slf4j-jboss-logmanager - SLF4J backend for JBoss LogManager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801003 --- Comment #5 from Jon Ciesla 2012-03-13 10:19:05 EDT --- 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 801003] Review Request: slf4j-jboss-logmanager - SLF4J backend for JBoss LogManager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=801003 Asaf Shakarchi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Asaf Shakarchi 2012-03-13 10:08:53 EDT --- New Package SCM Request === Package Name: slf4j-jboss-logmanager Short Description: SLF4J backend for JBoss LogManager Owners: asaf Branches: f17 InitialCC: goldmann -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 769919] Review Request: hydra - Very fast network log-on cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=769919 --- Comment #16 from Athmane Madjoudj 2012-03-13 09:46:46 EDT --- (In reply to comment #15) > (I need to investigate more on this because it does not > seem to find a valid user/pass against mysql server 5.5.x) Nevermind, I had the same issue with a package from other distro (it's an issue with hydra-mysql itself). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771252] Review Request: cinnamon - Window management and application launching for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771252 --- Comment #35 from Rahul Sundaram 2012-03-13 09:42:31 EDT --- @Christoph Wickert, The quote doesn't mean what you think it does. We don't do code review as part of the review process clearly and there is no real history of even checking for functionality. If you want this to change, that is a reasonable position but any claim otherwise is overreaching. The checklist for instance focuses only on packaging policy. The worst that could happen is that the package gets abandoned after a while but that isn't a real problem. It happens routinely anyway. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802388] Review Request: perl-Language-Prolog-Sugar - Syntactic sugar for Prolog term constructors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802388 --- Comment #5 from Fedora Update System 2012-03-13 09:42:33 EDT --- perl-Language-Prolog-Types-0.10-1.fc17, perl-Language-Prolog-Yaswi-0.19-1.fc17, perl-Language-Prolog-Sugar-0.06-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/FEDORA-2012-3608/perl-Language-Prolog-Yaswi-0.19-1.fc17,perl-Language-Prolog-Sugar-0.06-1.fc17,perl-Language-Prolog-Types-0.10-1.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802443] Review Request: perl-Language-Prolog-Yaswi - Yet another interface to SWI-Prolog
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802443 --- Comment #6 from Fedora Update System 2012-03-13 09:42:39 EDT --- perl-Language-Prolog-Types-0.10-1.fc17, perl-Language-Prolog-Yaswi-0.19-1.fc17, perl-Language-Prolog-Sugar-0.06-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/FEDORA-2012-3608/perl-Language-Prolog-Yaswi-0.19-1.fc17,perl-Language-Prolog-Sugar-0.06-1.fc17,perl-Language-Prolog-Types-0.10-1.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 795801] Review Request: paranamer - Library for accessing non-private method parameter names at run-time
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=795801 --- Comment #9 from Fedora Update System 2012-03-13 09:43:24 EDT --- paranamer-2.4.1-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/paranamer-2.4.1-1.fc17 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 802388] Review Request: perl-Language-Prolog-Sugar - Syntactic sugar for Prolog term constructors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=802388 --- Comment #6 from Fedora Update System 2012-03-13 09:45:23 EDT --- perl-Language-Prolog-Types-0.10-1.fc16, perl-Language-Prolog-Yaswi-0.19-1.fc16, perl-Language-Prolog-Sugar-0.06-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/perl-Language-Prolog-Yaswi-0.19-1.fc16,perl-Language-Prolog-Sugar-0.06-1.fc16,perl-Language-Prolog-Types-0.10-1.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
