[Bug 575005] Review Request: zinnia-tomoe - Online hand recognition system with machine learning
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=575005 --- Comment #57 from Fedora Update System upda...@fedoraproject.org 2010-07-27 01:57:17 EDT --- zinnia-0.06-8.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/zinnia-0.06-8.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 570731] Review Request: ibus-handwrite - IBus handwrite project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=570731 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2010-07-27 02:00:16 EDT --- ibus-handwrite-2.1.0-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/ibus-handwrite-2.1.0-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 614329] Review Request: emacs-common-w3m - W3m interface for Emacsen
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=614329 Akira TAGOH ta...@redhat.com changed: What|Removed |Added CC||peter...@redhat.com --- Comment #5 from Akira TAGOH ta...@redhat.com 2010-07-27 02:33:08 EDT --- *** Bug 595213 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 615129] Review Request: cjkuni-ukai-fonts - Chinese Unicode TrueType font in Kai face
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615129 --- Comment #6 from Peng Wu p...@redhat.com 2010-07-27 02:47:31 EDT --- New Package CVS Request === Package Name: cjkuni-ukai-fonts Short Description: Chinese Unicode TrueType font in Kai face Owners: pwu Branches: F-13 F-14 InitialCC: petersen i18n-team -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 614298] Review Request: emacs-apel - A Portable Emacs 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=614298
Parag AN(पराग) panem...@gmail.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #1 from Parag AN(पराग) panem...@gmail.com 2010-07-27 02:46:39 EDT
---
Review:
+ package builds in mock (rawhide i686).
koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=2353361
+ rpmlint output for SRPM and for RPM.
emacs-apel.src: W: no-buildroot-tag
emacs-apel-el.noarch: W: spelling-error Summary(en_US) Elisp - Elis, Lisp,
Elise
emacs-apel-el.noarch: W: spelling-error %description -l en_US elisp - lisp, e
lisp, Elise
emacs-apel-el.noarch: W: obsolete-not-provided apel
emacs-apel-el.noarch: W: no-documentation
3 packages and 0 specfiles checked; 0 errors, 5 warnings.
+ source files match upstream url (sha1sum)
089c18ae006df093aa2edaeb486bfaead6ac4918 apel-10.8.tar.gz
089c18ae006df093aa2edaeb486bfaead6ac4918 apel-10.8.tar.gz.srpm
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present which is not needed now.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Follow emacs addon packaging guidelines.
+ Not a GUI application
APPROVED.
Suggestions:
1)If package is needed only for F-13 and above then please follow
a) %clean not needed
b) cleaning of buildroot at start of %install also not needed
2) See http://koji.fedoraproject.org/koji/getfile?taskID=2353362name=build.log
I think its good to ask upstream to follow new style in *.el files.
3) installing commands should preserve timestamps. Use
make install PREFIX=$RPM_BUILD_ROOT%{_prefix} \
LISPDIR=$RPM_BUILD_ROOT%{_emacs_sitelispdir} INSTALL=install -p
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615128] Review Request: cjkuni-uming-fonts - Chinese Unicode TrueType font in Ming face
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615128 --- Comment #8 from Peng Wu p...@redhat.com 2010-07-27 02:45:53 EDT --- New Package CVS Request === Package Name: cjkuni-uming-fonts Short Description: Chinese Unicode TrueType font in Ming face Owners: pwu Branches: F-13 F-14 InitialCC: petersen i18n-team -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615128] Review Request: cjkuni-uming-fonts - Chinese Unicode TrueType font in Ming face
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615128 Peng Wu p...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615129] Review Request: cjkuni-ukai-fonts - Chinese Unicode TrueType font in Kai face
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615129 Peng Wu p...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616354] Review Request: geronimo-jpa - Java persistence API 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=616354 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #9 from Stanislav Ochotnicky socho...@redhat.com 2010-07-27 03:30:41 EDT --- Thanks, closing. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2353487 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531107 --- Comment #11 from Marcela Mašláňová mmasl...@redhat.com 2010-07-27 03:45:01 EDT --- SRPM can't be found. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 612671] Review Request: nodm - A display manager automatically starting an X session
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=612671 --- Comment #4 from Michel Alexandre Salim michael.silva...@gmail.com 2010-07-27 04:21:05 EDT --- Thanks, I'll take a look later today. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 614416] Review Request: python-scripttest - Helper to test command-line scripts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=614416 Martin Bacovsky mbaco...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #5 from Martin Bacovsky mbaco...@redhat.com 2010-07-27 04:40:55 EDT --- Package was built in F-12, F-13 and devel branches. Aslo is ready to pass the rebuild with py2.7. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616120] Review Request: spacecmd - Command-line interface to Spacewalk and Satellite 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=616120 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Miroslav Suchý msu...@redhat.com 2010-07-27 04:51:11 EDT --- New Package CVS Request === Package Name: spacecmd Short Description: Command-line interface to Spacewalk and Satellite servers Owners: msuchy Branches: F-13, F-12, EL-5, EL-6, EL-4 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618451
Chen Lei supercyp...@gmail.com changed:
What|Removed |Added
CC||supercyp...@gmail.com
--- Comment #2 from Chen Lei supercyp...@gmail.com 2010-07-27 04:55:30 EDT ---
one question：
Why not use %global module_install_path %{python_sitelib} instead of %global
module_install_path %{_libdir}/gdb-heap? Can gdb-heap work out of the box if
we add python modules to a standard site?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226473] Merge Review: system-switch-mail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226473 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Parag AN(पराग) panem...@gmail.com 2010-07-27 05:09:18 EDT --- Thanks for the update. I will approve this review. APPROVED. -- 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 226225] Merge Review: pam_krb5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226225 --- Comment #5 from Parag AN(पराग) panem...@gmail.com 2010-07-27 05:13:03 EDT --- Thanks. New spec looks good. Just minor thing, if adding my name please use just Parag AN as when seen plain text spec file in git it showed garbage characters in parenthesis :) Please commit in devel and build new package and will approve this review. -- 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 609011] Review Request: derby - Relational database implemented entirely in Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=609011 --- Comment #4 from Lubomir Rintel lkund...@v3.sk 2010-07-27 05:19:28 EDT --- .--. .''. / \ ( ____ ) /|o- o-|\ Oh noes ( |^^| ) people over the internets _ ) \ __ / / adding themselves to cc /##\ \_(__)_/ /laughing at java tools packaging /) )#\__// ( /( |##| |#\ \ (#| |##(_/##\_/\ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603245] Review Request: pyzmq - Software library for fast, message-based 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=603245
Bug 603245 depends on bug 603233, which changed state.
Bug 603233 Summary: Review Request: zeromq - Software library for fast,
message-based applications
https://bugzilla.redhat.com/show_bug.cgi?id=603233
What|Old Value |New Value
Status|ASSIGNED|CLOSED
Resolution||ERRATA
--- Comment #3 from Parag AN(पराग) panem...@gmail.com 2010-07-27 05:27:48 EDT
---
1)Python packages installing .so files don't need to follow
http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Shared_libraries
so remove that scriptlet from spec.
2) Please don't use more empty lines as I can see between %install and %check
3) also, I see python packaging guidelines now suggests using BR:python2-devel
4) Why do you need following in SPEC?
%global _use_internal_dependency_generator 0
%global __find_provides%{_rpmconfigdir}/find-provides | grep -v _zmq.so
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 592670] Review Request: mongoose - An easy-to-use self-sufficient web 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=592670 --- Comment #29 from Fedora Update System upda...@fedoraproject.org 2010-07-27 08:06:30 EDT --- mongoose-2.8-6.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/mongoose-2.8-6.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 592670] Review Request: mongoose - An easy-to-use self-sufficient web 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=592670 --- Comment #30 from Fedora Update System upda...@fedoraproject.org 2010-07-27 08:06:35 EDT --- mongoose-2.8-6.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/mongoose-2.8-6.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608069] Tango standard library for D language of d1 specification
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608069 --- Comment #27 from MERCIER Jonathan bioinfornat...@gmail.com 2010-07-27 08:40:11 EDT --- hello, I write here for everybody who has same question, but is last time (i no idea where is the best channel of communication) if you want learn D: - read source file (/usr/include/d) - contains documentation and unit test is usefull for learn how it works - irc: freenode = #d and #d.tango - book: * http://www.dsource.org/projects/tango/wiki/LearnToTangoWithD - tango API: http://www.dsource.org/projects/tango/docs/stable - tango example: http://www.dsource.org/projects/tango/wiki/Examples - tango tutorial: http://www.dsource.org/projects/tango/wiki/Tutorials - tango general: http://www.dsource.org/projects/tango -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618451 --- Comment #3 from Dave Malcolm dmalc...@redhat.com 2010-07-27 09:19:10 EDT --- My reasoning here is that gdb-heap uses the gdb module, which only exists in the python environment embedded inside /usr/bin/gdb, not in /usr/bin/python. So it's not going to work if you try to use it from the regular /usr/bin/python environment. Having said that, we could simply claim the gdbheap part of the namespace below python's site-packages (doing so might break pydoc and other tools that assume that all such modules are importable by /usr/bin/python, though). Does this make sense? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608069] Tango standard library for D language of d1 specification
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608069 --- Comment #28 from MERCIER Jonathan bioinfornat...@gmail.com 2010-07-27 09:42:31 EDT --- you can mail me if you are some other question i will ytry help you ;) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 510104] Review Request: epic5 - an ircII chat 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=510104 --- Comment #14 from manuel wolfshant wo...@nobugconsulting.ro 2010-07-27 09:58:17 EDT --- will close the bug in one week unless we have an update by then. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618668] New: Derelict is a collection of D bindings to C shared libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Derelict is a collection of D bindings to C shared libraries https://bugzilla.redhat.com/show_bug.cgi?id=618668 Summary: Derelict is a collection of D bindings to C shared libraries Product: Fedora Version: 13 Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: bioinfornat...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora new Package for D programming Feature who allow use openGL in D application. spec: http://bioinfornatics.fedorapeople.org/derelict.spec src.rpm: http://bioinfornatics.fedorapeople.org/derelict-2-1.20100407svn490.fc13.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 608069] Tango standard library for D language of d1 specification
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608069 --- Comment #29 from MERCIER Jonathan bioinfornat...@gmail.com 2010-07-27 10:16:14 EDT --- i add just add a new package for D user: https://bugzilla.redhat.com/show_bug.cgi?id=618668 this one allow use openGL in D application -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226813] Merge Review: zsh
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226813 James Antill james.ant...@redhat.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||CURRENTRELEASE -- 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 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531107 --- Comment #12 from David Fetter da...@fetter.org 2010-07-27 10:57:37 EDT --- Oops. sorry. Wrong URL. The correct URLs are: http://fetter.org/~shackle/perl-SQL-Tokenizer.spec http://fetter.org/~shackle/perl-SQL-Tokenizer-0.20-1.fc13.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 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618451
Tom spot Callaway tcall...@redhat.com changed:
What|Removed |Added
CC||tcall...@redhat.com
--- Comment #4 from Tom spot Callaway tcall...@redhat.com 2010-07-27
11:02:12 EDT ---
The license tag should be:
License: LGPLv2+ and Python
Also, you're using mixed macros ($RPM_BUILD_ROOT vs %{buildroot}). I would
suggest that you standardize on the latter.
I'm a little concerned about the forced dependency on
/usr/lib/debug/lib*/ld-2.11.2.so.debug, as technically, the package that
provides it (glibc-debuginfo) is not part of the normal repositories, so this
package will not be installable without the user consciously enabling the
debuginfo repo.
Perhaps a cleaner approach would be for the command to check on invocation for
presence of that library, and if not found, prompt the user to run the
appropriate debuginfo-install command, similar to gdb's behavior when it hits
missing debuginfo during a debug run.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226225] Merge Review: pam_krb5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226225 --- Comment #6 from Nalin Dahyabhai na...@redhat.com 2010-07-27 11:01:14 EDT --- Whoops, sorry about that. I can't read that script, so I tried to copy and paste from bugzilla. Should be all checked-in and built 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 618668] Derelict is a collection of D bindings to C shared libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618668 --- Comment #1 from MERCIER Jonathan bioinfornat...@gmail.com 2010-07-27 11:05:22 EDT --- $ rpmlint /home/builder/rpmbuild/SRPMS/derelict-2-1.20100407svn490.fc13.src.rpm derelict.src: W: spelling-error Summary(fr) It - T, Itô, Dt derelict.src: W: spelling-error Summary(fr) is - si, us, os derelict.src: W: spelling-error Summary(fr) of - off, if, or derelict.src: W: spelling-error Summary(fr) bindings - standings, buildings, poudings derelict.src: W: spelling-error Summary(fr) to - tau, go, t derelict.src: W: spelling-error Summary(fr) shared - shaker, shakeur, eschare derelict.src: W: spelling-error Summary(fr) libraries - libraires, librairies derelict.src: W: spelling-error %description -l en_US runtime - run time, run-time, runtish derelict.src: W: invalid-url Source0: derelict-20100407svn490.tar.xz 1 packages and 0 specfiles checked; 0 errors, 9 warnings. ___ $ rpmlint /home/builder/rpmbuild/SPECS/derelict.spec /home/builder/rpmbuild/SPECS/derelict.spec: W: invalid-url Source0: derelict-20100407svn490.tar.xz 0 packages and 1 specfiles checked; 0 errors, 1 warnings. ___ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 Chen Lei supercyp...@gmail.com changed: What|Removed |Added CC||supercyp...@gmail.com --- Comment #19 from Chen Lei supercyp...@gmail.com 2010-07-27 11:09:29 EDT --- Any news on this package? Some Meego 1.1 packages need libva 1.0 to be included 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 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #20 from Chen Lei supercyp...@gmail.com 2010-07-27 11:16:45 EDT --- Hi Adam, It seems Meego and debian already include this package in their repos(version 1.0.x). Can we lift FE-legal now or it there any FE-legal persons to clasify the patent issue? Debian package: http://packages.debian.org/source/sid/libva Meego package: http://repo.meego.com/MeeGo/builds/trunk/daily/core/repos/source/libva-1.0.3-1.5.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 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #21 from Adam Williamson awill...@redhat.com 2010-07-27 11:27:26 EDT --- We always go with Red Hat / Fedora Legal's take, not what other distros do, so no, we can't. Still waiting on a legal review of the package without i965 plugin. Note that the package is now in RPM Fusion - https://bugzilla.rpmfusion.org/show_bug.cgi?id=1321 . -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #22 from Chen Lei supercyp...@gmail.com 2010-07-27 11:42:41 EDT --- (In reply to comment #21) We always go with Red Hat / Fedora Legal's take, not what other distros do, so no, we can't. Still waiting on a legal review of the package without i965 plugin. Note that the package is now in RPM Fusion - https://bugzilla.rpmfusion.org/show_bug.cgi?id=1321 . -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers Actually, libva is inclued in all major linux distributions(debian/ubuntu opensuse mandriva gentoo arch) except Fedora. I'll post a request on Legal list for help soon :), this Review Request waits for a legal review for nearly one year. We need this package to be included in fedora not rpmfusion, otherwise we have to miss the whole meego MTF spin in Fedora. Can you help to contact spot or ajax? Is there any other one familiar with libva? Regards, Chen Lei -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606498] Review Request: hwloc - portable abstraction of hierarchical architectures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606498 --- Comment #28 from Fedora Update System upda...@fedoraproject.org 2010-07-27 11:50:56 EDT --- hwloc-1.0.2-1.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/hwloc-1.0.2-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606498] Review Request: hwloc - portable abstraction of hierarchical architectures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606498 --- Comment #26 from Fedora Update System upda...@fedoraproject.org 2010-07-27 11:50:45 EDT --- hwloc-1.0.2-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/hwloc-1.0.2-1.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606498] Review Request: hwloc - portable abstraction of hierarchical architectures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606498 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2010-07-27 11:50:51 EDT --- hwloc-1.0.2-1.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/hwloc-1.0.2-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 251110] Review Request: perl-IPC-Run3 - Run a subprocess in batch mode
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=251110 Ruediger Landmann r.landm...@redhat.com changed: What|Removed |Added CC||r.landm...@redhat.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Ruediger Landmann r.landm...@redhat.com 2010-07-27 11:57:28 EDT --- Package Change Request === Package Name: perl-IPC-Run3 New Branches: EL-5 Owners: rlandmann InitialCC: perl-sig I have discussed this with Ralf Corsépius, the package owner, and he is happy for me to maintain this branch. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618451
Dave Malcolm dmalc...@redhat.com changed:
What|Removed |Added
CC||tro...@redhat.com
--- Comment #5 from Dave Malcolm dmalc...@redhat.com 2010-07-27 12:03:59 EDT
---
(In reply to comment #4)
The license tag should be:
License: LGPLv2+ and Python
Fixed
Also, you're using mixed macros ($RPM_BUILD_ROOT vs %{buildroot}). I would
suggest that you standardize on the latter.
Fixed
I'm a little concerned about the forced dependency on
/usr/lib/debug/lib*/ld-2.11.2.so.debug, as technically, the package that
provides it (glibc-debuginfo) is not part of the normal repositories, so this
package will not be installable without the user consciously enabling the
debuginfo repo.
Perhaps a cleaner approach would be for the command to check on invocation for
presence of that library, and if not found, prompt the user to run the
appropriate debuginfo-install command, similar to gdb's behavior when it
hits
missing debuginfo during a debug run.
This code is autoloaded by gdb when the libc.so.debug is loaded by gdb (in
response to libc.so is loaded in the inferior process) i.e. when the DWARF
debug info for glibc is loaded by gdb. Hence if that file isn't present, the
code is not autoloaded or run, and thus isn't able to issue a warning message.
I've changed this to a dependency on:
Requires: glibc%{_isa} = %{glibc_version}
though this really ought to read:
Requires: glibc-debuginfo%{_isa} = %{glibc_version}
as that's what it really needs.
Without the debuginfo package this code not only is useless, but won't load or
run.
I don't know if it's possible to rig things up so that this autoloaded whenever
gdb starts up (CCing tromey for gdb advice), that way we could issue a message.
(Though it strikes me that this ought to be handled at the rpm metadata
level, not as a runtime failure, and that we ought to allow packages that
require debuginfo to require that debuginfo).
Updated specfile:
http://dmalcolm.fedorapeople.org/python-packaging/gdb-heap.spec
Updated SRPM:
http://dmalcolm.fedorapeople.org/python-packaging/gdb-heap-0.1-2.fc12.src.rpm
Specfile changes:
http://dmalcolm.fedorapeople.org/python-packaging/from-0.1-1-to-0.1-2.diff
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618451
--- Comment #6 from Chen Lei supercyp...@gmail.com 2010-07-27 12:20:54 EDT ---
(In reply to comment #5)
(In reply to comment #4)
I've changed this to a dependency on:
Requires: glibc%{_isa} = %{glibc_version}
though this really ought to read:
Requires: glibc-debuginfo%{_isa} = %{glibc_version}
as that's what it really needs.
Without the debuginfo package this code not only is useless, but won't load or
run.
I don't know if it's possible to rig things up so that this autoloaded
whenever
gdb starts up (CCing tromey for gdb advice), that way we could issue a
message.
(Though it strikes me that this ought to be handled at the rpm metadata
level, not as a runtime failure, and that we ought to allow packages that
require debuginfo to require that debuginfo).
Actually, Requires: glibc%{_isa} is useless for this package, gdb-heap depends
on python/gdb, python and gdb will pull in glibc as a dependency, I think
adding a README to %doc will be more helpful.
Also, I suggest you to install head modules to %{_datadir}/gdb-heap, arch
independent data should go into /usr/share in FHS. Many other packages(e.g.
ibus) already install their private python modules to %{_datadir}
See
http://www.pathname.com/fhs/pub/fhs-2.3.html#USRSHAREARCHITECTUREINDEPENDENTDATA
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618451
--- Comment #7 from Dave Malcolm dmalc...@redhat.com 2010-07-27 12:45:27 EDT
---
(In reply to comment #6)
Actually, Requires: glibc%{_isa} is useless for this package, gdb-heap
depends
on python/gdb, python and gdb will pull in glibc as a dependency, I think
adding a README to %doc will be more helpful.
The point of that Requires is to require a specific version of glibc.
Also, I suggest you to install head modules to %{_datadir}/gdb-heap, arch
independent data should go into /usr/share in FHS. Many other packages(e.g.
ibus) already install their private python modules to %{_datadir}
See
http://www.pathname.com/fhs/pub/fhs-2.3.html#USRSHAREARCHITECTUREINDEPENDENTDATA
Thanks - good idea.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618451 --- Comment #8 from Chen Lei supercyp...@gmail.com 2010-07-27 12:58:44 EDT --- Is it possible to read /usr/lib/debug/lib64/ld-linux-x86-64.so.2.debug instead of /usr/lib/debug/lib64/ld-2.12.90.so.debug and /usr/lib/debug/lib/ld-linux.so.2.debug instead of /usr/lib/debug/lib/ld-2.12.90.so.debug? /usr/lib/debug/lib64/ld-linux-x86-64.so.2.debug - /usr/lib/debug/lib64/ld-2.12.90.so.debug /usr/lib/debug/lib/ld-linux.so.2.debug - /usr/lib/debug/lib/ld-2.12.90.so.debug All current Fedora/RHEL release have /usr/lib/debug/lib64/ld-linux-x86-64.so.2.debug and /usr/lib/debug/lib/ld-linux.so.2.debug which is not restricted by glibc version. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618761] New: Review request: Gold Allocation Manager for HPC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review request: Gold Allocation Manager for HPC https://bugzilla.redhat.com/show_bug.cgi?id=618761 Summary: Review request: Gold Allocation Manager for HPC Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fed...@zaniyah.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.fedoraproject.org/~zaniyah/gold/gold.spec SRPM URL: http://people.fedoraproject.org/~zaniyah/gold/gold-2.1.12.2-1.src.rpm Other useful files also in http://people.fedoraproject.org/~zaniyah/gold -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618761] Review request: Gold Allocation Manager for HPC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618761 --- Comment #1 from Jessica Jones fed...@zaniyah.org 2010-07-27 13:00:24 EDT --- Gold is an open source accounting system developed by Pacific Northwest National Laboratory (PNNL) as part of the Department of Energy (DOE) Scalable Systems Software Project (SSS). It tracks resource usage on High Performance Computers and acts much like a bank, establishing accounts in order to pre-allocate user and project resource usage over specific nodes and timeframe. Gold provides balance and usage feedback to users, managers, and system administrators. SQLite is used by default, but Gold can be configured to use either MySQL or PostgreSQL instead. There is also a web interface that provides feedback and allows management in a more 'usable' format. This and the documentation are packaged separately. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #24 from Tom spot Callaway tcall...@redhat.com 2010-07-27 13:01:07 EDT --- (In reply to comment #22) I'll post a request on Legal list for help soon :), this Review Request waits for a legal review for nearly one year. We need this package to be included in fedora not rpmfusion, otherwise we have to miss the whole meego MTF spin in Fedora. What in meego depends on libva? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 Tom spot Callaway tcall...@redhat.com changed: What|Removed |Added CC||tcall...@redhat.com --- Comment #23 from Tom spot Callaway tcall...@redhat.com 2010-07-27 13:00:26 EDT --- At a minimum, the i965 code in this package is not acceptable for inclusion in Fedora. That begs the question of whether it would be better to separate it out or simply push it all to rpmfusion. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618761] Review request: Gold Allocation Manager for HPC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618761 --- Comment #2 from Jessica Jones fed...@zaniyah.org 2010-07-27 13:04:30 EDT --- I have also asked the upstream maintainer to make the changes I made via the two patches: http://www.supercluster.org/pipermail/gold-users/2010-July/000343.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618451] Review Request: gdb-heap - Extensions to gdb for debugging dynamic memory allocation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618451 --- Comment #9 from Dave Malcolm dmalc...@redhat.com 2010-07-27 13:18:11 EDT --- (In reply to comment #8) Is it possible to read /usr/lib/debug/lib64/ld-linux-x86-64.so.2.debug instead of /usr/lib/debug/lib64/ld-2.12.90.so.debug and /usr/lib/debug/lib/ld-linux.so.2.debug instead of /usr/lib/debug/lib/ld-2.12.90.so.debug? /usr/lib/debug/lib64/ld-linux-x86-64.so.2.debug - /usr/lib/debug/lib64/ld-2.12.90.so.debug /usr/lib/debug/lib/ld-linux.so.2.debug - /usr/lib/debug/lib/ld-2.12.90.so.debug I don't believe so, based on strace-ing gdb -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #25 from Chen Lei supercyp...@gmail.com 2010-07-27 13:18:53 EDT --- (In reply to comment #24) (In reply to comment #22) I'll post a request on Legal list for help soon :), this Review Request waits for a legal review for nearly one year. We need this package to be included in fedora not rpmfusion, otherwise we have to miss the whole meego MTF spin in Fedora. What in meego depends on libva? Thanks a lot, Tom. Meego netbook spin(based on moblin) currently don't have a dependency on libva, however Netbook MTF(multitouchframework) spin depends on it. libva is also used by clutter-gst, gnash and totem which is already included in fedora. i965 part is a drive and can be packaged seperately, there are some other 3rd party drive/backend available for libva. See http://freedesktop.org/wiki/Software/vaapi -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #27 from Bill Nottingham nott...@redhat.com 2010-07-27 13:40:26 EDT --- Siimlarly, it's not as if libva is a blocker for the inclusion of totem, gnash, clutter-gst, etc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618155] Review Request: python-SciMath - Scientific and Mathematical calculations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=618155
Thomas Spura toms...@fedoraproject.org changed:
What|Removed |Added
CC||toms...@fedoraproject.org
--- Comment #1 from Thomas Spura toms...@fedoraproject.org 2010-07-27
14:23:52 EDT ---
Why do you do this:
49 # non-executable-script
50 pushd %{buildroot}/%{python_sitearch}/enthought/units
51 chmod +x temperature.py area.py power.py substance.py unit_parser.py
energy.py \
52 force.py unit.py pressure.py SI.py
53 popd
Because of shebangs and rpmlint warnings?
$ rpmlint -I non-executable-script
non-executable-script:
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed. If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere
- Please remove the shebang instead...
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618059] Review Request: x509watch - Simple tool to list expiring or expired X.509 certificates
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618059 --- Comment #7 from Robert Scheck redhat-bugzi...@linuxnetz.de 2010-07-27 14:44:19 EDT --- Spec URL: http://labs.linuxnetz.de/bugzilla/x509watch.spec SRPM URL: http://labs.linuxnetz.de/bugzilla/x509watch-0.2.0-1.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 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #28 from Adam Williamson awill...@redhat.com 2010-07-27 15:01:56 EDT --- Right. They can use it, but they don't *need* it. If you install gstreamer-vaapi then gstreamer-based players like Totem will have libva available as a decoder, if you don't, they'll just use something else (non-accelerated, of course). There's no need to have a dependency anywhere afaik. If you want to build the vaapi support into gnash then you have to build gnash against libva, it's true, but then gnash sucks ass so I don't think anyone really cares about that. It's not really usable for most people. I haven't even bothered building it in my private repo, for that reason. I believe spot's correct that there's still no backend that's not encumbered in some way. Though given that we've accepted WebM, if someone were to write a backend for, say, Intel which did WebM decoding, we could presumably take that. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #29 from Adam Williamson awill...@redhat.com 2010-07-27 15:08:18 EDT --- btw, libva is in Mandriva because I put it there. The legal review process in Mandriva consists of Is anyone suing us right now? No? We're probably good, then. =) -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #30 from Tom spot Callaway tcall...@redhat.com 2010-07-27 15:15:55 EDT --- (In reply to comment #29) btw, libva is in Mandriva because I put it there. The legal review process in Mandriva consists of Is anyone suing us right now? No? We're probably good, then. =) Also, Debian/Ubuntu/Gentoo are known to simply ignore patent issues. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427484] Review Request: publican-RedHat - Red Hat theme
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=427484
Ruediger Landmann r.landm...@redhat.com changed:
What|Removed |Added
CC||r.landm...@redhat.com
--- Comment #10 from Ruediger Landmann r.landm...@redhat.com 2010-07-27
15:31:51 EDT ---
Thanks Nick and Paul
New spec and srpm are here:
http://rlandmann.fedorapeople.org/publican/publican.spec
http://rlandmann.fedorapeople.org/publican/publican-redhat-2.0-1.fc13.src.rpm
I have
* removed the .svn directories (oops -- very embarrassing!)
* specified Obsoletes: documentation-devel-%{brand} = 0.4-0
* specified Provides: documentation-devel-%{brand}-%{version}-%{release}
* fixed the mixed use of tabs and spaces that I could find
However, rpmlint still warns:
* obsolete-not-provided and unversioned-explicit-provides -- it seems that
it isn't parsing documentation-devel-%{brand}-%{version}-%{release}, but as
you can see, the obsolete is provided for, and the provides is indeed versioned
* it's still complaining about mixed use of spaces and tabs in the spec file:
mixed-use-of-spaces-and-tabs (spaces: line 22, tab: line 191) -- however,
there's certainly no spaces in line 22, and line 191 does not exist at all, so
I'm not sure what else to do here.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #32 from Adam Williamson awill...@redhat.com 2010-07-27 15:42:29 EDT --- Um, really? I already asked months ago if you could simply review the code apart from the i965 plugin, which I would drop if the rest of the code was OK. If you were waiting for me to build an SRPM before doing that (which seems a bit unnecessary) you could at least have *told* me :( -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427484] Review Request: publican-RedHat - Red Hat theme
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427484 --- Comment #11 from Ruediger Landmann r.landm...@redhat.com 2010-07-27 15:47:13 EDT --- (In reply to comment #10) New spec and srpm are here: Sorry -- Spec URL is: http://rlandmann.fedorapeople.org/publican/publican-redhat.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427483] Review Request: publican-JBoss - JBoss Theme
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427483 Ruediger Landmann r.landm...@redhat.com changed: What|Removed |Added CC||r.landm...@redhat.com --- Comment #10 from Ruediger Landmann r.landm...@redhat.com 2010-07-27 15:48:57 EDT --- Updated spec and SRPM in light of comments in Bug #427484 New files: http://rlandmann.fedorapeople.org/publican/publican-jboss.spec http://rlandmann.fedorapeople.org/publican/publican-jboss-1.9-1.fc13.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 427484] Review Request: publican-RedHat - Red Hat theme
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=427484 --- Comment #12 from Ruediger Landmann r.landm...@redhat.com 2010-07-27 15:55:13 EDT --- (In reply to comment #10) * it's still complaining about mixed use of spaces and tabs in the spec file: mixed-use-of-spaces-and-tabs (spaces: line 22, tab: line 191) -- however, there's certainly no spaces in line 22, and line 191 does not exist at all, so I'm not sure what else to do here. Ah! Found and fixed -- no more warnings :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #33 from Tom spot Callaway tcall...@redhat.com 2010-07-27 16:00:38 EDT --- Apologies. I just decided that today, previously, I was hopeful that ajax would have time to do another audit pass on the code without the i965 stuff, but I think I'm comfortable at this point with the libva bits as is, since any actual work would happen in the driver/backends. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #34 from Tom spot Callaway tcall...@redhat.com 2010-07-27 16:01:19 EDT --- (In reply to comment #33) Apologies. I just decided that today, previously, I was hopeful that ajax would have time to do another audit pass on the code without the i965 stuff, but I think I'm comfortable at this point with the libva bits as is, since any actual work would happen in the driver/backends. To be absolutely clear, when i say as is, I mean with the i965 stuff removed entirely. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #35 from Adam Williamson awill...@redhat.com 2010-07-27 16:24:37 EDT --- yep, I understand. I'll co-ordinate with the fusion maintainer and we'll submit a build with the 965 driver removed. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618852] New: Review Request: laughlin-backgrounds - Laughlin desktop backgrounds
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: laughlin-backgrounds - Laughlin desktop backgrounds https://bugzilla.redhat.com/show_bug.cgi?id=618852 Summary: Review Request: laughlin-backgrounds - Laughlin desktop backgrounds Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: martin.sour...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://mso.fedorapeople.org/packages/SPECS/laughlin-backgrounds.spec SRPM URL: http://mso.fedorapeople.org/packages/SRPMS/laughlin-backgrounds-13.91.0-1.fc12.src.rpm Description: Package containing Laughlin desktop backgrounds. The main package serves as metapackage pulling in both gnome and kde stuff, the -gnome and -kde subpackages are self-explanatory, -single subpackage serves for containing single screen images shared by both gnome and kde. As the -single one is needed by all the rest, I've put license and credits there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 597410] Review Request: php-deepend-Mockery - Mockery is a simple but flexible PHP mock object framewor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=597410 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2010-07-27 17:21:20 EDT --- php-deepend-Mockery-0.6.3-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/php-deepend-Mockery-0.6.3-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 618059] Review Request: x509watch - Simple tool to list expiring or expired X.509 certificates
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=618059 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Flag||fedora-review+ --- Comment #8 from manuel wolfshant wo...@nobugconsulting.ro 2010-07-27 19:26:13 EDT --- Re-review: New sha1sum: 7ce87e8f735a6ceced6a537d50f4faa6f710c4d8 x509watch [wo...@wolfy tmp]$ rpmlint x509watch*rpm 2 packages and 0 specfiles checked; 0 errors, 0 warnings. No new problems spotted + previous issues solved. = 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 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #11 from Mads Kiilerich m...@kiilerich.com 2010-07-27 19:56:51 EDT --- I have updated to 0.7.3 and verified that pylint has no relevant complaints. Spec URL: http://kiilerix.fedorapeople.org/freerdp.spec SRPM URL: http://kiilerix.fedorapeople.org/freerdp-0.7.3-1.fc13.src.rpm cwickert, I hope that you will comment on my comments above. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #36 from Chen Lei supercyp...@gmail.com 2010-07-27 22:17:20 EDT --- (In reply to comment #30) (In reply to comment #29) btw, libva is in Mandriva because I put it there. The legal review process in Mandriva consists of Is anyone suing us right now? No? We're probably good, then. =) Also, Debian/Ubuntu/Gentoo are known to simply ignore patent issues. Correct, Debian ignored many decoder patents, but they are also aware of encoder patents. If the main part libva is patent free, I think it's not dangerous for Fedora to include it, because one year ago FESCo permitted a similar package - libvdpau[1] to go into fedora which is useless without Nvidia binary driver. [1]https://fedorahosted.org/fesco/ticket/238 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617632] Review Request: swift - OpenStack Object Storage (swift)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617632 Silas Sewell si...@sewell.ch changed: What|Removed |Added Depends on||618909 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615669] Review Request: perl-Crypt-Cracklib - perl module to interact with libcrack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615669 --- Comment #11 from Kevin Fenzi ke...@tummy.com 2010-07-27 23:37:53 EDT --- Please do add a changelog entry and bump Release while making changes for the review. This prevents confusion between submitter and reviewer as to what they are looking at. ;) Can you add pointers to the current spec/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 617400] Review Request: pfHandle - wrapper for tools to manage postfix mail queue
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617400 --- Comment #4 from Kevin Fenzi ke...@tummy.com 2010-07-27 23:59:12 EDT --- OK - Package meets naming and packaging guidelines OK - Spec file matches base package name. OK - Spec has consistant macro usage. See below - Meets Packaging Guidelines. See below - License OK - License field in spec matches See below - License file included in package OK - Spec in American English OK - Spec is legible. OK - Sources match upstream md5sum: 52a127b0616657d05edc0cb21e91d172 pfHandle-1.tar.gz 52a127b0616657d05edc0cb21e91d172 pfHandle-1.tar.gz.orig OK - BuildRequires correct OK - Package has %defattr and permissions on files is good. See below - Package has a correct %clean section. OK - Package has correct buildroot OK - Package is code or permissible content. OK - Packages %doc files don't affect runtime. OK - Package has rm -rf RPM_BUILD_ROOT at top of %install OK - Package compiles and builds on at least one arch. OK - Package has no duplicate files in %files. OK - Package doesn't own any directories other packages own. OK - Package owns all the directories it creates. OK - Package obey's FHS standard (except for 2 exceptions) OK - No rpmlint output. - final provides and requires are sane. SHOULD Items: OK - Should build in mock. OK - Should build on all supported archs OK - Should have dist tag OK - Should package latest version OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin Issues: 1. The only indicator of what the package license is is the launchpad site. ;) Might be good to include a COPYING file and explicitly state in the README that it's GPL v2 only. 2. You should nuke the [ $RPM_BUILD_ROOT != / ] from the clean section. 3. rpmlint says: 2 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615669] Review Request: perl-Crypt-Cracklib - perl module to interact with libcrack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615669 --- Comment #12 from Carl Thompson fed...@red-dragon.com 2010-07-28 00:15:04 EDT --- SPEC: http://red-dragon.com/perl-Crypt-Cracklib.spec SRPM: http://red-dragon.com/perl-Crypt-Cracklib-1.5-2.fc13.src.rpm 1) re enabled the internal dependancy generator 2) removed the $RPM_BUILD_ROOT test from install and clean 3) added %check section with make test 4) included the Changes and README as %doc files 5) Many perl packages sit in Crypt so can't properly own to this package 6) rpmlint says [redra...@gkar SPECS]$ rpmlint perl-Crypt-Cracklib.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. [redra...@gkar x86_64]$ rpmlint perl-Crypt-Cracklib-1.5-2.fc13.x86_64.rpm perl-Crypt-Cracklib.x86_64: W: private-shared-object-provides /usr/lib64/perl5/auto/Crypt/Cracklib/Cracklib.so Cracklib.so()(64bit) 1 packages and 0 specfiles checked; 0 errors, 1 warnings. 7) this is a new spec just for this package koji build on rawhide passed no failures koji build on f13 passed no failures -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615669] Review Request: perl-Crypt-Cracklib - perl module to interact with libcrack
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=615669
--- Comment #13 from Kevin Fenzi ke...@tummy.com 2010-07-28 00:50:38 EDT ---
1, 2, 3, all look good.
on 4:
You can replace:
%doc %{_defaultdocdir}/perl-Crypt-Cracklib-%{version}/Changes
%doc %{_defaultdocdir}/perl-Crypt-Cracklib-%{version}/README
with just:
%doc Changes README
5. I think this falls under:
https://fedoraproject.org/wiki/Packaging/Guidelines#Multiple_packages_own_files_in_a_common_directory_but_none_of_them_needs_to_require_the_others.
I see currently all of the following own this dir:
perl-Crypt-OpenSSL-PKCS10-0:0.06-15.fc14.x86_64
perl-Crypt-OpenSSL-Random-0:0.04-12.fc14.x86_64
perl-Crypt-DES-0:2.05-11.fc14.x86_64
perl-Crypt-OpenSSL-X509-0:0.7-7.fc14.x86_64
perl-Crypt-Eksblowfish-0:0.008-2.fc14.x86_64
perl-Crypt-Twofish-0:2.14-2.fc14.x86_64
perl-Crypt-OpenSSL-RSA-0:0.25-12.fc14.x86_64
perl-Crypt-SSLeay-0:0.57-17.fc14.x86_64
perl-Crypt-OpenSSL-AES-0:0.02-11.fc14.x86_64
perl-Crypt-Rijndael-0:1.09-2.fc14.x86_64
perl-Crypt-Blowfish-0:2.10-10.fc14.x86_64
crypto-utils-0:2.4.1-27.x86_64
perl-Crypt-OpenSSL-Bignum-0:0.04-10.fc14.x86_64
perl-Crypt-OpenSSL-DSA-0:0.13-16.fc14.x86_64
perl-Crypt-SMIME-0:0.09-4.fc14.x86_64
6 and 7: ok.
I see you have a
Epoch: 1
here. Why?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615669] Review Request: perl-Crypt-Cracklib - perl module to interact with libcrack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615669 --- Comment #14 from Carl Thompson fed...@red-dragon.com 2010-07-28 01:09:30 EDT --- SPEC: http://red-dragon.com/perl-Crypt-Cracklib.spec SRPM: http://red-dragon.com/perl-Crypt-Cracklib-1.5-3.fc13.src.rpm 4) updated %doc in files section 5) added the Crypt directory (amazingly this corrected the .so error) Epoch was removed, no idea why I plugged that one in. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617632] Review Request: swift - OpenStack Object Storage (swift)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617632 --- Comment #1 from Silas Sewell si...@sewell.ch 2010-07-28 01:14:12 EDT --- SRPM: http://github.com/downloads/silas/rpms/swift-1.0.2-2.fc14.src.rpm Updatesd * Added swift user and setup basic permissions * Rewrote init stuff (doesn't use swift-init anymore) * Decided against default etc scripts as it would require bad assumptions about filesystem layout and ring setup * Added ticket to get eventlet updated From my end this package is now ready for review. rpmlint swift.noarch: W: non-standard-uid /var/run/swift swift ...(repeats): Acceptable warning because we added swift user 8 packages and 0 specfiles checked; 0 errors, 6 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617632] Review Request: swift - OpenStack Object Storage (swift)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617632 --- Comment #2 from Silas Sewell si...@sewell.ch 2010-07-28 01:34:58 EDT --- SRPM: http://github.com/downloads/silas/rpms/swift-1.0.2-3.fc14.src.rpm Fix init return value. rpmlint swift.noarch: W: non-standard-uid /var/run/swift swift ...(repeats): see above 8 packages and 0 specfiles checked; 0 errors, 6 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
