[Bug 252108] Review Request: python-html5lib - A python based HTML5 parser/tokenizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=252108 --- Comment #34 from Rahul Sundaram 2011-06-30 02:32:55 EDT --- Ping again. I would like to move forward here since this package is a dependency for something I am working on. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 704445] Review Request: python-timelib - Parse english textual date descriptions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=704445 --- Comment #5 from Fabian Affolter 2011-06-30 01:57:54 EDT --- Any news from upstream? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691114] Review Request: python-msgpack - A MessagePack (de)serializer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691114 --- Comment #12 from Fedora Update System 2011-06-30 01:41:28 EDT --- python-msgpack-0.1.9-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/python-msgpack-0.1.9-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691114] Review Request: python-msgpack - A MessagePack (de)serializer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691114 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 691114] Review Request: python-msgpack - A MessagePack (de)serializer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691114 --- Comment #11 from Fedora Update System 2011-06-30 01:31:57 EDT --- python-msgpack-0.1.9-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-msgpack-0.1.9-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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717502 --- Comment #6 from German Ruiz 2011-06-30 00:16:52 EDT --- Here the files with the corrections... SPEC: http://lletelier.fedorapeople.org/i4uc/i4uc.spec SRPM: http://lletelier.fedorapeople.org/i4uc/i4uc-0.5.6-1.fc15.src.rpm RPM_F15_32bits: http://lletelier.fedorapeople.org/i4uc/i4uc-0.5.6-1.fc15.i686.rpm RPM_F15_64bits: http://lletelier.fedorapeople.org/i4uc/i4uc-0.5.6-1.fc15.x86_64.rpm rpmlint outputs: rpmlint -i SPECS/i4uc.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. rpmlint -i RPMS/i686/i4uc-0.5.6-1.fc15.i686.rpm i4uc.i686: W: no-manual-page-for-binary i4uc-gtk Each executable in standard binary directories should have a man page. 1 packages and 0 specfiles checked; 0 errors, 1 warnings. rpmlint -i SRPMS/i4uc-0.5.6-1.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint -i SPECS/i4uc.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. rpmlint -i SRPMS/i4uc-0.5.6-1.fc15.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint -i RPMS/x86_64/i4uc-0.5.6-1.fc15.x86_64.rpm i4uc.x86_64: W: no-manual-page-for-binary i4uc-gtk Each executable in standard binary directories should have a man page. 1 packages and 0 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 716352] Review Request: toothchart - A PHP script which graphically shows how a baby's primary teeth have erupted
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716352 --- Comment #5 from Ankur Sinha 2011-06-29 22:11:17 EDT --- Thank you Mario! :D -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716352] Review Request: toothchart - A PHP script which graphically shows how a baby's primary teeth have erupted
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716352 Ankur Sinha changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Ankur Sinha 2011-06-29 22:13:04 EDT --- New Package SCM Request === Package Name: toothchart Short Description: A PHP script which graphically shows how a baby's primary teeth have erupted Owners: ankursinha Branches: f14 f15 InitialCC: susmit mrceresa -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716384] Review Request: filo - Useful FILe and stream Operations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=716384
Ben Boeckel changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||maths...@gmail.com
AssignedTo|nob...@fedoraproject.org|maths...@gmail.com
Flag||fedora-review?
--- Comment #1 from Ben Boeckel 2011-06-29 20:35:51 EDT ---
I'll take this. Looks like some interesting tools (though shuffle is "sort -R")
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[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] Spec file is legible and written in American English.
[x] Spec file lacks Packager, Vendor, PreReq tags.
[x] Spec uses macros instead of hard-coded directory names.
[x] Package consistently uses macros.
[x] Macros in Summary, %description expandable at SRPM build time.
[x] PreReq is not used.
[-] Requires correct, justified where necessary.
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[-] Package use %makeinstall only when ``make install DESTDIR=...'' doesn't
work.
[-] The spec file handles locales properly.
[-] Changelog in prescribed format.
[x] Rpmlint output is silent.
filo.x86_64: W: no-manual-page-for-binary stats
filo.x86_64: W: no-manual-page-for-binary shuffle
filo.x86_64: W: no-manual-page-for-binary groupBy
3 packages and 0 specfiles checked; 0 errors, 3 warnings.
Manpages would be nice to have.
[!] License field in the package spec file matches the actual license.
I see no indication of the license in the package. Contact upstream to ship a
LICENSE or COPYING file.
[-] 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.
[-] License file installed when any subpackage combination is installed.
[!] 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]
No licensing specified.
[!] Sources contain only permissible code or content.
No license.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package : d8b495af3a5b68f8796374a2ce15c44d
arq5x-filo-f56efe3.tar.gz
MD5SUM upstream package : d8b495af3a5b68f8796374a2ce15c44d
arq5x-filo-f56efe3.tar.gz.orig
[x] Compiler flags are appropriate.
[x] %build honors applicable compiler flags or justifies otherwise.
Flags seem to be set correctly, but not all compile commands are in the logs.
[-] ldconfig called in %post and %postun if required.
[-] Package must own all directories that it creates.
[x] Package does not own files or directories owned by other packages.
[-] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
chmod -x would probably be better for the source files (getting upstream to
remove the perms would also be nice).
[x] Each %files section contains %defattr.
[x] No %config files under /usr.
[-] %config files are marked noreplace or the reason is justified.
[-] Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application. [5]
[-] Package contains a valid .desktop file.
[x] Package contains code, or permissable content.
[x] File names are valid UTF-8.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[!] Package contains no bundled libraries.
Seems to ship with a bundled gzstream
http://www.cs.unc.edu/Research/compgeom/gzstream/
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -static subpackage, if present.
[x] Package contains no static executables.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[-] Package does not contain any libtool archives (.la).
[x] Useful -debuginfo package or justification otherwise.
[x] Rpath absent or only used for internal libs.
[x] Package does not generate any conflict.
[x] Package does not contains kernel modules.
[x] Package is not relocatable.
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
[x] Package is not known to require ExcludeArch.
[x] Package installs prope
[Bug 717595] Review Request: italc - intelligent teaching and learning with computers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717595 Ben Boeckel changed: What|Removed |Added CC||maths...@gmail.com --- Comment #1 from Ben Boeckel 2011-06-29 19:39:06 EDT --- Bug #459065 might be of interest here. Not sure which one should become the main 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 683591] Review Request: tclap - Templatized Command Line Argument Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=683591 Ben Boeckel changed: What|Removed |Added CC||maths...@gmail.com --- Comment #1 from Ben Boeckel 2011-06-29 19:31:06 EDT --- I'd follow the lead of eigen2[1] with this and make a -devel package with an empty base package. [1]http://pkgs.fedoraproject.org/gitweb/?p=eigen2.git;a=tree -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717473] Review Request: SeqAn - Biological sequence analysis 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=717473 --- Comment #4 from Adam Huffman 2011-06-29 18:46:39 EDT --- Yes, I added that to fix the rpmlint complaint about executable files. Will take a closer look. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717473] Review Request: SeqAn - Biological sequence analysis 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=717473 --- Comment #3 from Volker Fröhlich 2011-06-29 18:41:02 EDT --- The tests aren't executed. I found out, what causes my build troubles: + rm -rf SeqAn-1.3 rm: cannot remove `SeqAn-1.3/apps/stellar/examples/reads.fasta': Permission denied rm: cannot remove `SeqAn-1.3/apps/stellar/examples/NC_001477.fasta': Permission denied rm: cannot remove `SeqAn-1.3/apps/stellar/examples/NC_001474.fasta': Permission denied The examples directory ends up with 644, although it isn't like that in the tarball. I happens while installing: + chmod 0644 /builddir/build/BUILD/SeqAn-1.3/apps/stellar/README /builddir/build/BUILD/SeqAn-1.3/apps/stellar/examples [...] -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on 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 #5 from Volker Fröhlich 2011-06-29 18:24:51 EDT
---
OK, you got me wrong about the "place". Whenever you publish a new version of
your package, increment the release number. It makes it easier for the
reviewer.
You can drop "prefix=%{_prefix} docdir=share/doc/%{name}-%{version}" on the
make call. Having it only on "make install" works. "all" is the default build
target, so you can also omit that.
These are, of course, no blockers.
Concerning the docs, you could just make it:
make install DESTDIR=%{buildroot} prefix=%{_prefix}
# Install it with doc macro later
rm -rf %{buildroot}%{_docdir}
The %doc macro should take care of it. There may be better solutions though.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716400] Review Request: MACS - Model-based Analysis for ChIP-Seq
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716400 --- Comment #1 from Adam Huffman 2011-06-29 18:25:30 EDT --- Update for 1.4.1 release: http://verdurin.fedorapeople.org/reviews/MACS/MACS.spec http://verdurin.fedorapeople.org/reviews/MACS/MACS-1.4.1-1.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 701332] Review Request: ghc-blaze-html - Fast HTML combinator library for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=701332 --- Comment #10 from Fedora Update System 2011-06-29 18:02:35 EDT --- ghc-blaze-html-0.4.1.3-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 701332] Review Request: ghc-blaze-html - Fast HTML combinator library for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=701332 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-blaze-html-0.4.1.3-1.fc ||14 Resolution||ERRATA Last Closed||2011-06-29 17:59: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 710751] Review Request: php-pecl-oauth - PHP OAuth extension
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=710751 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||php-pecl-oauth-1.1.0-6.fc15 Resolution||ERRATA Last Closed||2011-06-29 18:02:59 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710751] Review Request: php-pecl-oauth - PHP OAuth extension
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=710751 --- Comment #11 from Fedora Update System 2011-06-29 18:02:54 EDT --- php-pecl-oauth-1.1.0-6.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 701332] Review Request: ghc-blaze-html - Fast HTML combinator library for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=701332 Fedora Update System changed: What|Removed |Added Fixed In Version|ghc-blaze-html-0.4.1.3-1.fc |ghc-blaze-html-0.4.1.3-1.fc |14 |15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702998] Review Request: pyttsx - Cross platform text-to-speech
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702998 Fedora Update System changed: What|Removed |Added Fixed In Version|pyttsx-1.0-1.fc14 |pyttsx-1.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710848] Review Request: python-myhdl - A python hardware description and verification 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=710848 --- Comment #12 from Fedora Update System 2011-06-29 18:00:40 EDT --- python-myhdl-0.7-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702998] Review Request: pyttsx - Cross platform text-to-speech
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702998 --- Comment #12 from Fedora Update System 2011-06-29 18:01:57 EDT --- pyttsx-1.0-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710848] Review Request: python-myhdl - A python hardware description and verification 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=710848 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||python-myhdl-0.7-1.fc14 Resolution||ERRATA Last Closed||2011-06-29 17:58:34 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710848] Review Request: python-myhdl - A python hardware description and verification 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=710848 Fedora Update System changed: What|Removed |Added Fixed In Version|python-myhdl-0.7-1.fc14 |python-myhdl-0.7-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710848] Review Request: python-myhdl - A python hardware description and verification 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=710848 --- Comment #11 from Fedora Update System 2011-06-29 17:58:29 EDT --- python-myhdl-0.7-1.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713741] Review Request: libgcal - A library to access google calendar events and contacts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713741 --- Comment #11 from Fedora Update System 2011-06-29 17:58:02 EDT --- libgcal-0.9.6-5.fc15 has been pushed to the Fedora 15 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713741] Review Request: libgcal - A library to access google calendar events and contacts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713741 Fedora Update System changed: What|Removed |Added Fixed In Version|libgcal-0.9.6-5.fc14|libgcal-0.9.6-5.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 713741] Review Request: libgcal - A library to access google calendar events and contacts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713741 --- Comment #10 from Fedora Update System 2011-06-29 17:56:39 EDT --- libgcal-0.9.6-5.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 701332] Review Request: ghc-blaze-html - Fast HTML combinator library for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=701332 --- Comment #9 from Fedora Update System 2011-06-29 17:59:40 EDT --- ghc-blaze-html-0.4.1.3-1.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713741] Review Request: libgcal - A library to access google calendar events and contacts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713741 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||libgcal-0.9.6-5.fc14 Resolution||ERRATA Last Closed||2011-06-29 17:56:49 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702998] Review Request: pyttsx - Cross platform text-to-speech
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702998 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pyttsx-1.0-1.fc14 Resolution||ERRATA Last Closed||2011-06-29 17:57:09 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714036] Review Request: perl-PPI-PowerToys - Handy collection of small PPI-based 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=714036 Fedora Update System changed: What|Removed |Added Fixed In Version|perl-PPI-PowerToys-0.14-1.f |perl-PPI-PowerToys-0.14-1.f |c16 |c15 Resolution|RAWHIDE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642995] Review Request: rubygem-deltacloud-client - Deltacloud REST 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=642995 Fedora Update System changed: What|Removed |Added Fixed In Version|rubygem-deltacloud-client-0 |rubygem-deltacloud-client-0 |.1.0-2.fc13 |.3.1-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642995] Review Request: rubygem-deltacloud-client - Deltacloud REST 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=642995 --- Comment #19 from Fedora Update System 2011-06-29 17:55:27 EDT --- rubygem-deltacloud-client-0.3.1-1.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 702998] Review Request: pyttsx - Cross platform text-to-speech
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=702998 --- Comment #11 from Fedora Update System 2011-06-29 17:57:04 EDT --- pyttsx-1.0-1.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 698362] Review Request: writetype - Light word processor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=698362 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||writetype-1.2.130-6.fc15 Resolution||ERRATA Last Closed||2011-06-29 17:52:31 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711047] Review Request: naev - 2d action, RPG space game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711047 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System 2011-06-29 17:54:14 EDT --- naev-data-0.5.0-4.fc15, naev-0.5.0-3.fc15 has been pushed to the Fedora 15 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711048] Review Request: naev-data - Data files for NAEV game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711048 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System 2011-06-29 17:54:19 EDT --- naev-data-0.5.0-4.fc15, naev-0.5.0-3.fc15 has been pushed to the Fedora 15 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 698362] Review Request: writetype - Light word processor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=698362 Fedora Update System changed: What|Removed |Added Fixed In Version|writetype-1.2.130-6.fc15|writetype-1.2.130-6.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717345] csync2-1.34 - cluster synchronization tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717345 --- Comment #5 from Volker Fröhlich 2011-06-29 17:51:44 EDT --- The URL works fine for my rpmlint. What about the other comments? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714036] Review Request: perl-PPI-PowerToys - Handy collection of small PPI-based 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=714036 --- Comment #8 from Fedora Update System 2011-06-29 17:54:25 EDT --- perl-PPI-PowerToys-0.14-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 698362] Review Request: writetype - Light word processor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=698362 --- Comment #16 from Fedora Update System 2011-06-29 17:52:25 EDT --- writetype-1.2.130-6.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 698362] Review Request: writetype - Light word processor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=698362 --- Comment #17 from Fedora Update System 2011-06-29 17:52:38 EDT --- writetype-1.2.130-6.fc14 has been pushed to the Fedora 14 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717750] Review Request: lttv - Linux Trace Toolkit Viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717750 --- Comment #1 from Yannick Brosseau 2011-06-29 16:51:05 EDT --- New srpm: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/lttv-0.12.38-1.fc15.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717748] Review Request: UST - LTTng Userspace Tracer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717748 --- Comment #1 from Yannick Brosseau 2011-06-29 16:44:06 EDT --- I've an new SRPMS: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/ust-0.14-2.fc15.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 700427] Review Request: jopt-simple - A Java command line parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=700427 --- Comment #6 from Karel Klíč 2011-06-29 16:43:05 EDT --- Spec URL: http://kklic.fedorapeople.org/jopt-simple.spec SRPM URL: http://kklic.fedorapeople.org/jopt-simple-3.3-1.fc15.src.rpm * Wed Jun 29 2011 Karel Klíč - 3.3-1 - Use maven3 instead of maven2 to build the package. - Updated to upstream final 3.3 release. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717337] Review Request: URCU - Userspace RCU 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=717337 --- Comment #3 from Yannick Brosseau 2011-06-29 16:38:39 EDT --- (In reply to comment #2) > Some comments: Thank you for the feedbacks, I've posted an update spec and srpm: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SPECS/liburcu.spec http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/liburcu-0.6.3-1.fc15.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717768] Review Request: iwhd - image warehouse daemon
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=717768
--- Comment #1 from Chris Lalancette 2011-06-29 16:37:52
EDT ---
[clalance@localhost SPECS]$ rpmlint iwhd.spec
iwhd.spec: W: invalid-url Source0: iwhd-0.96.1.9e86.tar.gz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.
The Source line should be a full URL, so should be replaced by something like:
Source0: http://people.redhat.com/meyering/iwhd/%{name}-%{version}.tar.gz
[ OK ] MUST: rpmlint must be run on every package
[ OK ] MUST: The package must be named according to the Package Naming
Guidelines
[ OK ] MUST: The spec file name must match the base package %{name} [...]
[ OK ] MUST: The package must meet the Packaging Guidelines
[ OK ] MUST: The package must be licensed with a Fedora approved license
and meet the Licensing Guidelines
[ OK ] MUST: The License field in the package spec file must match the
actual license
[ OK ] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of
the license(s) for the package must be included in %doc
[ OK ] MUST: The spec file must be written in American English.
[ OK ] MUST: The spec file for the package MUST be legible.
[ ] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for
this task. If no upstream URL can be specified for this package,
please see the Source URL Guidelines for how to deal with this.
[ OK ] MUST: The package MUST successfully compile and build into binary
rpms on at least one primary architecture
[ N/A ] MUST: If the package does not successfully compile, build or work on
an architecture, then those architectures should be listed in the
spec in ExcludeArch. Each architecture listed in ExcludeArch MUST
have a bug filed in bugzilla, describing the reason that the package
does not compile/build/work on that architecture. The bug number MUST
be placed in a comment, next to the corresponding ExcludeArch line
[ OK ] MUST: All build dependencies must be listed in BuildRequires, except
for any that are listed in the exceptions section of the Packaging
Guidelines ; inclusion of those as BuildRequires is optional. Apply
common sense.
[ OK ] MUST: The spec file MUST handle locales properly. This is done by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly
forbidden
[ N/A ] MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's
default paths, must call ldconfig in %post and %postun.
[ N/A ] MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the
rationalization for relocation of that specific package. Without
this, use of Prefix: /usr is considered a blocker.
[ OK ] MUST: A package must own all directories that it creates. If it does
not create a directory that it uses, then it should require a package
which does create that directory.
[ OK ] MUST: A package must not contain any duplicate files in the %files
listing.
[ OK ] MUST: Permissions on files must be set properly. Executables should
be set with executable permissions, for example. Every %files section
must include a %defattr(...) line.
[ OK ] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[ FAIL ] MUST: Each package must consistently use macros. - clalance: minor,
but Source0 can be changed to use %{name}-%{version}.
[ OK ] MUST: The package must contain code, or permissable content.
[ OK ] MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but
is not restricted to size. Large can refer to either size or
quantity).
[ OK ] MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the
program must run properly if it is not present.
[ N/A ] MUST: Header files must be in a -devel package.
[ N/A ] MUST: Static libraries must be in a -static package.
[ ] MUST: Packages containing pkgconfig(.pc) files must 'Requires:
pkgconfig' (for directory ownership and usability).
[ N/A ] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix)
must go in a -devel package.
[ N/A ] MUST: In the vast majority of cases, de
[Bug 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=707199 --- Comment #9 from Matt Domsch 2011-06-29 16:18:30 EDT --- Created attachment 510531 --> https://bugzilla.redhat.com/attachment.cgi?id=510531 spec file patch My suggestions for how to fix many of the items in the above 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 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=707199 --- Comment #10 from Matt Domsch 2011-06-29 16:19:21 EDT --- Created attachment 510533 --> https://bugzilla.redhat.com/attachment.cgi?id=510533 rpmlint.txt After above patch is applied, still get a lot of warnings and errors from rpmlint, mostly from the content of the initscripts. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=707199
--- Comment #8 from Matt Domsch 2011-06-29 16:14:01 EDT
---
MUST: rpmlint must be run on the source rpm and all binary rpms
the build produces. The output should be posted in the review.[1]
= done for spec only; once spec is updated to match guidelines
more closely, will perform this step.
MUST: The package must be named according to the Package Naming
Guidelines .
MUST: The spec file name must match the base package %{name}, in
the format %{name}.spec unless your package has an exemption. [2]
= Per convention, this package should be called 'openstack-nova'.
Name tag must be changed to this.
MUST: The package must meet the Packaging Guidelines .
= in review here
MUST: The package must be licensed with a Fedora approved license
=nd meet the Licensing Guidelines .
* Apache 2.0 is acceptable.
MUST: The License field in the package spec file must match the
actual license. [3]
= OK
MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of
the license(s) for the package must be included in %doc.[4]
= needs work. LICENSE file present in two of the 10 subpackages
only. It must be added to the rest.
MUST: The spec file must be written in American English. [5]
= OK
MUST: The spec file for the package MUST be legible. [6]
= OK
MUST: The sources used to build the package must match the
upstream source, as provided in the spec URL. Reviewers should use
md5sum for this task. If no upstream URL can be specified for this
package, please see the Source URL Guidelines for how to deal with
this.
= Needs work. URL there doesn't work, must provide an explicit
bzr-versioned URL it seems.
MUST: The package MUST successfully compile and build into binary
rpms on at least one primary architecture. [7]
= OK on x86_64 (building noarch)
MUST: If the package does not successfully compile, build or work
on an architecture, then those architectures should be listed in
the spec in ExcludeArch. Each architecture listed in ExcludeArch
= N/A
MUST have a bug filed in bugzilla, describing the reason that the
package does not compile/build/work on that architecture. The bug
number MUST be placed in a comment, next to the corresponding
ExcludeArch line. [8]
= N/A
MUST: All build dependencies must be listed in BuildRequires,
except for any that are listed in the exceptions section of the
Packaging Guidelines ; inclusion of those as BuildRequires is
optional. Apply common sense.
= OK. Some BRs are missing that would be beneficial but which do
not block the build, including python-carrot, python-mox,
python-suds, m2crypto, bpython, python-memcached, python-migrate
MUST: The spec file MUST handle locales properly. This is done by
using the %find_lang macro. Using %{_datadir}/locale/* is strictly
forbidden.[9]
= N/A
MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's
default paths, must call ldconfig in %post and %postun. [10]
= N/A
MUST: Packages must NOT bundle copies of system libraries.[11]
= OK
MUST: If the package is designed to be relocatable, the packager
must state this fact in the request for review, along with the
rationalization for relocation of that specific package. Without
this, use of Prefix: /usr is considered a blocker. [12]
= N/A
MUST: A package must own all directories that it creates. If it
does not create a directory that it uses, then it should require a
package which does create that directory. [13]
= OK
MUST: A Fedora package must not list a file more than once in the
spec file's %files listings. (Notable exception: license texts in
specific situations)[14]
= needs work. %{_datarootdir}/nova/setup_iptables.sh appeared
twice by nature of directory is listed, as well as this specific file.
MUST: Permissions on files must be set properly. Executables
should be set with executable permissions, for example. [15]
= needs work. ajaxterm py[co] files are group writeable.
MUST: Each package must consistently use macros. [16]
= OK
MUST: The package must contain code, or permissable content. [17]
= OK
MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement,
but is not restricted to size. Large can refer to either size or
quantity). [18]
= needs work. -doc subpackage exists. Not set to build
automaticaly though, which it must. Dro
[Bug 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=707199 Matt Domsch changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717768] Review Request: iwhd - image warehouse daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717768 Chris Lalancette changed: What|Removed |Added Status|NEW |ASSIGNED CC||clala...@redhat.com AssignedTo|nob...@fedoraproject.org|clala...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 683463] 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=683463 --- Comment #31 from Jan-Frode Myklebust 2011-06-29 16:01:41 EDT --- uid:gid 176:176 has been reserved for ats in setup-2.8.35-1.fc16/rawhide, ref: bz#715266. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717768] New: Review Request: iwhd - image warehouse daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: iwhd - image warehouse daemon https://bugzilla.redhat.com/show_bug.cgi?id=717768 Summary: Review Request: iwhd - image warehouse daemon Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: meyer...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://people.redhat.com/meyering/iwhd/iwhd.spec SRPM URL: http://people.redhat.com/meyering/iwhd/iwhd-0.96.1.9e86-1.fc15.src.rpm Description: The image warehouse daemon is part of Aeolus (http://www.aeolusproject.org/). iwhd is the component responsible for storing and moving virtual-machine images -- and potentially other large objects -- in a federated-cloud environment. Note that the .spec in the src.rpm has a URL: pointing to the gitweb URL. In the spec at the people.redhat.com URL above I've changed it to point to the latest stable compressed tarball. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=707199 Matt Domsch changed: What|Removed |Added Flag||needinfo?(ogelbukh@mirantis ||.com) --- Comment #7 from Matt Domsch 2011-06-29 15:43:11 EDT --- Oleg, the spec looks like a good starting point. There are definitely some things to fix, but nothing impossible yet... You will need to join the Fedora packagers team to be able to maintain this package in Fedora. https://fedoraproject.org/wiki/PackageMaintainers/Join for how. I will sponsor you. Please clear needinfo when you've got a FAS account created, and tell us your account name. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688315] Review Request: rubygem-little-plugger - gem based plugin management
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=688315 Chris Lalancette changed: What|Removed |Added Blocks||688322 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688322] Review Request: rubygem-webby - Awesome static website creation and management!
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=688322 Chris Lalancette changed: What|Removed |Added Depends on||688318, 688315, 688316, ||688317 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688318] Review Request: rubygem-directory_watcher - Rubygem for watching files in a directory and generating events when they change
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=688318 Chris Lalancette changed: What|Removed |Added Blocks||688322 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688317] Review Request: rubygem-loquacious - Descriptive configuration files for Ruby written in Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=688317 Chris Lalancette changed: What|Removed |Added Blocks||688322 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 688316] Review Request: rubygem-logging - A flexible and extendable logging library for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=688316 Chris Lalancette changed: What|Removed |Added Blocks||688322 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717345] csync2-1.34 - cluster synchronization tool
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=717345
--- Comment #4 from Luis Bazan 2011-06-29
15:31:03 EDT ---
http://lbazan.fedorapeople.org/csync2.spec
http://lbazan.fedorapeople.org/csync2-1.34-5.fc15.src.rpm
$ rpmlint -v csync2-1.34-5.fc15.src.rpm
csync2.src: I: checking
csync2.src: I: checking-url http://oss.linbit.com/csync2/ (timeout 10 seconds)
csync2.src: I: checking-url http://oss.linbit.com/csync2/csync2-1.34.tar.gz
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint -v csync2.spec
csync2.spec: I: checking-url http://oss.linbit.com/csync2/csync2-1.34.tar.gz
(timeout 10 seconds)
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
change to %{_mandir}/man1/csync2.1.*
remove make clean
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 716756] Review Request: perl-HTML-FormatText-WithLinks - HTML to text conversion with links as footnotes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716756 Mario Blättermann changed: What|Removed |Added Status|NEW |ASSIGNED CC||mari...@freenet.de AssignedTo|nob...@fedoraproject.org|mari...@freenet.de Flag||fedora-review? --- Comment #2 from Mario Blättermann 2011-06-29 15:32:06 EDT --- Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3170492 $ rpmlint -v -i * perl-HTML-FormatText-WithLinks.src: I: checking perl-HTML-FormatText-WithLinks.src: I: checking-url http://search.cpan.org/dist/HTML-FormatText-WithLinks/ (timeout 10 seconds) perl-HTML-FormatText-WithLinks.src: I: checking-url http://www.cpan.org/authors/id/S/ST/STRUAN/HTML-FormatText-WithLinks-0.14.tar.gz (timeout 10 seconds) perl-HTML-FormatText-WithLinks.noarch: I: checking perl-HTML-FormatText-WithLinks.noarch: I: checking-url http://search.cpan.org/dist/HTML-FormatText-WithLinks/ (timeout 10 seconds) perl-HTML-FormatText-WithLinks.spec: I: checking-url http://www.cpan.org/authors/id/S/ST/STRUAN/HTML-FormatText-WithLinks-0.14.tar.gz (timeout 10 seconds) 2 packages and 1 specfiles checked; 0 errors, 0 warnings. Looks fine so far. Formal review will 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 707199] Review Request: openstack-nova - OpenStack Compute (nova)
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=707199
Matt Domsch changed:
What|Removed |Added
CC||matt_dom...@dell.com
AssignedTo|nob...@fedoraproject.org|matt_dom...@dell.com
--- Comment #6 from Matt Domsch 2011-06-29 15:27:18 EDT
---
rpmlint on the spec file is the first step. These are trivially fixed.
$ rpmlint SPECS/openstack-nova.spec
SPECS/openstack-nova.spec:55: W: hardcoded-packager-tag "Mirantis
SPECS/openstack-nova.spec:380: W: deprecated-grep [u'fgrep']
SPECS/openstack-nova.spec:430: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:431: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:450: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:451: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:467: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:483: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:499: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:515: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:566: W: macro-in-comment %{name}
SPECS/openstack-nova.spec:384: W: mixed-use-of-spaces-and-tabs (spaces: line 7,
tab: line 384)
SPECS/openstack-nova.spec: W: invalid-url Source0:
http://nova.openstack.org/tarballs/nova.tar.gz HTTP Error 404: Not Found
In addition I have started other packaging guidelines related cleanups, and
will post a spec file patch shortly.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717337] Review Request: URCU - Userspace RCU 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=717337
Veeti Paananen changed:
What|Removed |Added
CC||veeti.paana...@rojekti.fi
--- Comment #2 from Veeti Paananen 2011-06-29
15:25:27 EDT ---
Some comments:
1. The spec file should be named after the package (so it should be
"liburcu.spec").
2. The License should be "LGPLv2+" instead.
3. The Release tag should be in the format "1%{?dist}".
4. The build should be done with "make %{?_smp_mflags}".
5. The libraries seem to include rpaths
(http://fedoraproject.org/wiki/Packaging:Guidelines#Beware_of_Rpath).
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717750] New: Review Request: lttv - Linux Trace Toolkit Viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: lttv - Linux Trace Toolkit Viewer https://bugzilla.redhat.com/show_bug.cgi?id=717750 Summary: Review Request: lttv - Linux Trace Toolkit Viewer Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: yannick.bross...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SPECS/lttv.spec SRPM URL: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/lttv-0.12.38-1.src.rpm Description: LTTV is a modular trace viewer. It can perform analysis on traces of a Linux kernel instrumented with LTTng and UST. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717750] Review Request: lttv - Linux Trace Toolkit Viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717750 Yannick Brosseau changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717337] Review Request: URCU - Userspace RCU 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=717337 Yannick Brosseau changed: What|Removed |Added Blocks||717748 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717748] Review Request: UST - LTTng Userspace Tracer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717748 Yannick Brosseau changed: What|Removed |Added Depends on||717337 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717748] Review Request: UST - LTTng Userspace Tracer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717748 Yannick Brosseau changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717748] New: Review Request: UST - LTTng Userspace Tracer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: UST - LTTng Userspace Tracer https://bugzilla.redhat.com/show_bug.cgi?id=717748 Summary: Review Request: UST - LTTng Userspace Tracer Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: yannick.bross...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SPECS/ust.spec SRPM URL: http://www.dorsal.polymtl.ca/~ybrosseau/fedora/SRPMS/ust-0.14-2.src.rpm Description: The userspace tracer is designed to provide detailed information about userspace activity. Like the kernel tracer, performance is the main goal. Tracing does not require system calls or traps. UST instrumentation points may be added in any userspace code including signal handlers and libraries. from: http://lttng.org/ust This package depends on liburcu, which is review request 717337 https://bugzilla.redhat.com/show_bug.cgi?id=717337 (This series is my first fedora packages, so I'll need a sponsor) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717741] New: Review Request: VMDKstream - Python tool to convert raw disk images to stream-optimized VMDK files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: VMDKstream - Python tool to convert raw disk images to stream-optimized VMDK files https://bugzilla.redhat.com/show_bug.cgi?id=717741 Summary: Review Request: VMDKstream - Python tool to convert raw disk images to stream-optimized VMDK files Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: clala...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://people.redhat.com/clalance/VMDKstream/VMDKstream.spec SRPM URL: http://people.redhat.com/clalance/VMDKstream/VMDKstream-0.2-1.fc14.src.rpm Description: While adding support for VMWare ESX/vSphere to the Image Factory project we discovered that we needed to support the "stream-optimized" variant of VMDK. This module is a first draft attempt to do so. It is known to work on ESXi 4.1. This format is only useful if you are attempting to import virtual machines into ESX using the vSphere SOAP API and HTTP POST uploads of image files. (In which case, it is required.) [clalance@localhost SPECS]$ rpmlint ../SRPMS/VMDKstream-0.2-1.fc14.src.rpm VMDKstream.src: W: spelling-error %description -l en_US vSphere -> v Sphere, sphere, spherule VMDKstream.src: W: invalid-url Source0: VMDKstream-0.2.tar.gz 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [clalance@localhost SPECS]$ rpmlint VMDKstream.spec VMDKstream.spec: W: invalid-url Source0: VMDKstream-0.2.tar.gz 0 packages and 1 specfiles checked; 0 errors, 1 warnings. (upstream VMDKstream doesn't do tarball releases, so there is no URL to point to for the source) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697780 Fedora Update System changed: What|Removed |Added Fixed In Version|rubygem-daemon_controller-0 |rubygem-daemon_controller-0 |.2.6-1.fc13 |.2.6-1.el6 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 697780] Review Request: rubygem-daemon_controller - A library for implementing daemon management capabilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=697780 --- Comment #19 from Fedora Update System 2011-06-29 14:26:04 EDT --- rubygem-daemon_controller-0.2.6-1.el6 has been pushed to the Fedora EPEL 6 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 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System 2011-06-29 14:26:47 EDT --- packagedb-cli-1.0.0-3.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 716352] Review Request: toothchart - A PHP script which graphically shows how a baby's primary teeth have erupted
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716352 Mario Blättermann changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Mario Blättermann 2011-06-29 14:15:44 EDT --- OK, looks well now. That's why: PACKAGE APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 --- Comment #13 from Fedora Update System 2011-06-29 14:06:50 EDT --- python26-argparse-1.2.1-3.el5, packagedb-cli-1.0.0-4.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python26-argparse-1.2.1-3.el5,packagedb-cli-1.0.0-4.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 708765] Review Request: Frogr - Flickr Remote Organizer 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=708765 --- Comment #52 from Martin Gieseking 2011-06-29 13:50:33 EDT --- Since the reviewing process seems to have stalled, I asked Toshio Kuratomi (a member of the FPC) how to continue with the package. He agreed to go on without requesting a bundled library exception. The library may also be linked statically as long as no third-party project uses it. Once someone submits a package in the future that depends on flicksoup (despite upstream's decision to keep it private), we have to revisit Frogr too, and split out flicksoup to a separate (sub-)package. So, sorry for having been a bit overcautious. If there's anybody who wants to take the review, he/she can continue without further hassle. :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711047] Review Request: naev - 2d action, RPG space game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711047 --- Comment #10 from Jonathan Dieter 2011-06-29 13:44:57 EDT --- Spot, thanks much for this review! Naev has been built for F14, F15 and EL6 now. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711047] Review Request: naev - 2d action, RPG space game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711047 --- Comment #9 from Fedora Update System 2011-06-29 13:38:13 EDT --- naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711048] Review Request: naev-data - Data files for NAEV game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711048 --- Comment #8 from Fedora Update System 2011-06-29 13:38:18 EDT --- naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/naev-data-0.5.0-4.fc15,naev-0.5.0-3.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711048] Review Request: naev-data - Data files for NAEV game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711048 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 711047] Review Request: naev - 2d action, RPG space game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711047 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 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 --- Comment #12 from Jon Ciesla 2011-06-29 13:30:10 EDT --- Git done (by process-git-requests). No worries! 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 713920] Review Request: CSparse - Sparse Matrix software 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=713920 Tom "spot" Callaway changed: What|Removed |Added Status|NEW |CLOSED Resolution||WONTFIX Last Closed||2011-06-29 13:21:56 --- Comment #4 from Tom "spot" Callaway 2011-06-29 13:21:56 EDT --- Its not a different package, so this is not necessary. Good catch. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713925] Review Request: naev - 2D space trading and combat game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713925 Bug 713925 depends on bug 713920, which changed state. Bug 713920 Summary: Review Request: CSparse - Sparse Matrix software library https://bugzilla.redhat.com/show_bug.cgi?id=713920 What|Old Value |New Value Resolution||WONTFIX Status|NEW |CLOSED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 Pierre-YvesChibon changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Pierre-YvesChibon 2011-06-29 13:09:53 EDT --- Indeed sorry about that. Package Change Request == Package Name: packagedb-cli New Branches: el5 Owners: pingou -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 --- Comment #10 from Jon Ciesla 2011-06-29 13:03:30 EDT --- Since this has already been created, submit this as a Package Change request. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 --- Comment #4 from Fedora Update System 2011-06-29 13:02:23 EDT --- python26-argparse-1.2.1-3.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python26-argparse-1.2.1-3.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 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 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 716352] Review Request: toothchart - A PHP script which graphically shows how a baby's primary teeth have erupted
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716352 --- Comment #3 from Ankur Sinha 2011-06-29 12:51:45 EDT --- (In reply to comment #2) > - > key: > > [+] OK > [.] OK, not applicable > [X] needs work > - > [X] MUST: The package must be licensed with a Fedora approved license. > According to the license included in the package, it should be GPLv2 > (without +). The source itself doesn't say anything about the license. > > [X] MUST: The License field in the package spec file must match the actual > license. Hi Mario, I think I had referred to the site for the license: http://sourceforge.net/projects/toothchart/ I've corrected it now to the license file included in the source. New spec/srpm: http://ankursinha.fedorapeople.org/toothchart/toothchart.spec http://ankursinha.fedorapeople.org/toothchart/toothchart-0.02.0-0.1beta.fc15.src.rpm Thanks, Ankur -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 Pierre-YvesChibon changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #9 from Pierre-YvesChibon 2011-06-29 12:41:31 EDT --- New Package SCM Request === Package Name: packagedb-cli Short Description: A CLI for pkgdb Owners: pingou Branches: el5 InitialCC: Since we will have a python26-argparse in epel5, I am requesting the el5 branch as well -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 --- Comment #3 from Jon Ciesla 2011-06-29 12:41:42 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 701079] Review Request: wmSun - Rise/Set time of Sun in a dockapp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=701079 Ankur Sinha changed: What|Removed |Added Status|NEW |ASSIGNED CC||sanjay.an...@gmail.com Alias||wmSun Flag||fedora-review? --- Comment #3 from Ankur Sinha 2011-06-29 12:42:40 EDT --- I'll take this one :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 Pierre-YvesChibon changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Pierre-YvesChibon 2011-06-29 12:34:36 EDT --- Thanks for the review, I will make the changes when I import the package. New Package SCM Request === Package Name: python26-argparse Short Description: Optparse inspired command line parser for Python Owners: pingou Branches: el5 InitialCC: toshio -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 Toshio Ernie Kuratomi changed: What|Removed |Added Status|NEW |ASSIGNED CC||a.bad...@gmail.com AssignedTo|nob...@fedoraproject.org|a.bad...@gmail.com Flag||fedora-review+ --- Comment #1 from Toshio Ernie Kuratomi 2011-06-29 12:22:26 EDT --- Good: * Named appropriately * Proper license * Spec file is legible * Source matches upstream * No locale files present * No shared libraries * Not relocatable * Owns all directories and files it creates and nothing more * Consistently uses macros * Code not content * Not a GUI app * All filenames valid UTF-8 * No scriptlets * builds in koji * rpmlint just has false positives on spelling Needswork: * LICENSE.txt file present but not included (now fixed in python-argparse as well) - NEWS.txt could be included as well (run dos2unix on it to fix EOL as well) Fix the inclusion of the LICENSE file and this is APPROVED Remember to retire and dead.package in devel after creating this (since we only need it in 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 497441] Review Request: mumble - Voice chat application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497441 --- Comment #91 from Ortal 2011-06-29 11:56:55 EDT --- *** Bug 717708 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 716844] Review Request: django-recaptcha - A Django application for adding ReCAPTCHA to a form
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716844 --- Comment #13 from Jon Ciesla 2011-06-29 11:48:08 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 716844] Review Request: django-recaptcha - A Django application for adding ReCAPTCHA to a form
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=716844 Jitesh Shah changed: What|Removed |Added Flag||fedora-cvs? --- Comment #12 from Jitesh Shah 2011-06-29 11:37:41 EDT --- Thanks! New Package SCM Request === Package Name: django-recaptcha Short Description: A Django application for adding ReCAPTCHA to a form Owners: jiteshs Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
