[Bug 684239] Review Request: valadoc - Documentation generator for Vala source
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684239 --- Comment #6 from Tim Lauridsen t...@rasmil.dk 2011-03-13 04:10:13 EDT --- Still missing some build requirements checking for LIBGVC... yes checking for GLIB... yes checking for GMODULE... yes checking for LIBVALA... yes checking for LIBGEE... no configure: error: Package requirements (gee-1.0 = 0.5) were not met: No package 'gee-1.0' found Consider adjusting the PKG_CONFIG_PATH environment variable if you Taking a dive into configure.in and it look like you need: libgee-devel gdk-pixbuf2-devel -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 681393] Review Request: perl-Nagios-Plugin-WWW-Mechanize - Login to a web page as a user and get data as a Nagios plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=681393 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Steve Traylen steve.tray...@cern.ch 2011-03-13 04:25:34 EDT --- Looks GOOD. 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 624705] Review Request: pam_script - execute scripts from within pam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624705 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added Status Whiteboard||BuildFails Flag|fedora-review? | --- Comment #5 from Steve Traylen steve.tray...@cern.ch 2011-03-13 04:34:55 EDT --- Build still fails with same error, removing myself from review and updating white board to reflect. Please update whiteboard on fix. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684511] New: Review Request: perl-DBIx-Class-Cursor-Cached - Cursor class with built-in caching support
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-DBIx-Class-Cursor-Cached - Cursor class with built-in caching support https://bugzilla.redhat.com/show_bug.cgi?id=684511 Summary: Review Request: perl-DBIx-Class-Cursor-Cached - Cursor class with built-in caching support Product: Fedora Version: rawhide Platform: Unspecified URL: http://search.cpan.org/dist/DBIx-Class-Cursor-Cached/ OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: iarn...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~iarnell/review/perl-DBIx-Class-Cursor-Cached.spec SRPM URL: http://fedorapeople.org/~iarnell/review/perl-DBIx-Class-Cursor-Cached-1.001000-1.fc16.src.rpm Description: This module provides a DBIx cursor class with built-in caching support. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2907991 *rt-0.10_01 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624705] Review Request: pam_script - execute scripts from within pam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624705 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added Status|ASSIGNED|NEW CC|steve.tray...@cern.ch | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624705] Review Request: pam_script - execute scripts from within pam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624705 --- Comment #6 from Marcus Moeller marcus.moel...@gmx.ch 2011-03-13 04:42:03 EDT --- Sorry for not responding earlier. Going to check that next week and report back. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 678855] Review Request: rpyc - A Transparent, Symmetrical Python Library for Distributed-Computing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=678855 --- Comment #2 from Erez Shinan e...@redhat.com 2011-03-13 04:46:41 EDT --- Fixed, the corrected package+spec are available under the same links. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684239] Review Request: valadoc - Documentation generator for Vala source
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684239 --- Comment #7 from Raul Gutierrez S. r...@rieder.net.py 2011-03-13 05:43:51 EDT --- (In reply to comment #6) libgee-devel gdk-pixbuf2-devel Updated. 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 565949] Review Request: perl-VOMS-Lite - Perl extension for VOMS Attribute certificate creation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=565949 --- Comment #21 from Steve Traylen steve.tray...@cern.ch 2011-03-13 07:15:11 EDT --- Hi Marcella, I completely forgot you had commented on this thanks Updated packages reflecting your comments. http://cern.ch/straylen/rpms/perl-VOMS-Lite/perl-VOMS-Lite.spec http://cern.ch/straylen/rpms/perl-VOMS-Lite/perl-VOMS-Lite-0.11-1.fc14.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 570318] Review Request: gnuplot44 - A program for plotting mathematical expressions and data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=570318 Steve Traylen steve.tray...@cern.ch changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||CURRENTRELEASE Last Closed||2011-03-13 07:24:10 --- Comment #16 from Steve Traylen steve.tray...@cern.ch 2011-03-13 07:24:10 EDT --- I'm just going to live with gnuplot versions in RHEL6 and ignore RHEL4 and 5. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 505154] Tracker: Review Requests for Science and Technology related packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505154 Bug 505154 depends on bug 570318, which changed state. Bug 570318 Summary: Review Request: gnuplot44 - A program for plotting mathematical expressions and data https://bugzilla.redhat.com/show_bug.cgi?id=570318 What|Old Value |New Value Resolution||CURRENTRELEASE Status|ON_QA |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 680657] Review Request: mpdas - An MPD audioscrobbling 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=680657 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #3 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-03-13 07:48:24 EDT --- @Jason, Ok, currently the license of this md5.cpp file is not mentioned in the spec file. That has to be done. However I am unable to find out what license that file is under. The text seems to match MIT license but not word for word. @sanjay.an...@gmail.com , can you have a look at that file as well? It is md5.cpp -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652648] Review Request: erlang-skerl - Skein hash function for Erlang, via NIFs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652648 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #5 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-03-13 07:54:07 EDT --- Hello, Is there any update 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 684475] Review Request: wmblob - Dockapp which shows funny moving `blobs'
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684475 --- Comment #2 from Mario Blättermann mari...@gnome.org 2011-03-13 07:59:37 EDT --- Spec URL: http://dl.dropbox.com/u/19373040/Fedora/SPECS/wmblob.spec SRPM URL: http://dl.dropbox.com/u/19373040/Fedora/wmblob-1.0.3-2.fc14.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2908300 I've changed the file encoding and tarball permissions as recommended. The enchant outputs from rpmlint are due to a missing German dictionary on your system. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652648] Review Request: erlang-skerl - Skein hash function for Erlang, via NIFs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652648 --- Comment #6 from Peter Lemenkov lemen...@gmail.com 2011-03-13 08:08:58 EDT --- (In reply to comment #5) Hello, Is there any update from upstream? Not yet. It seems that adding licensing headers is not among the upstream's top priority tasks. I'll ping them again next week. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652543] Review Request: erlang-riak_client - Erlang client for Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652543 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2011-03-13 08:11:11 EDT --- erlang-riak_client-1.1.0-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/erlang-riak_client-1.1.0-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 652543] Review Request: erlang-riak_client - Erlang client for Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652543 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652543] Review Request: erlang-riak_client - Erlang client for Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652543 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||RAWHIDE Last Closed||2011-03-13 08:12: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 652629] Review Request: erlang-riak_kv - Riak Key/Value Store
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652629 Bug 652629 depends on bug 652543, which changed state. Bug 652543 Summary: Review Request: erlang-riak_client - Erlang client for Riak https://bugzilla.redhat.com/show_bug.cgi?id=652543 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|MODIFIED Resolution||RAWHIDE Status|MODIFIED|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 652543] Review Request: erlang-riak_client - Erlang client for Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652543 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2011-03-13 08:11:18 EDT --- erlang-riak_client-1.1.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/erlang-riak_client-1.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 652543] Review Request: erlang-riak_client - Erlang client for Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652543 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2011-03-13 08:11:03 EDT --- erlang-riak_client-1.1.0-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/erlang-riak_client-1.1.0-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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||lakshminaras2...@gmail.com AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684475] Review Request: wmblob - Dockapp which shows funny moving `blobs'
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=684475
Martin Gieseking martin.giesek...@uos.de changed:
What|Removed |Added
CC||martin.giesek...@uos.de
--- Comment #3 from Martin Gieseking martin.giesek...@uos.de 2011-03-13
08:57:47 EDT ---
Some further notes on the latest spec:
- please use a more neutral and less biased %description, e.g. the one from
dockapps.org:
wmblob shows some blobs moving around. It does nothing useful, it's just
a toy.
- replace the iconv lines with the following loop to ensure a correct
conversion
and to preserve the timestamps:
for f in ChangeLog README doc/wmblob.1 doc/how_it_works; do
iconv -f iso-8859-1 -t utf8 $f $f.tmp \
touch -r $f $f.tmp \
mv $f.tmp $f
done
- replace the make statement with this one to apply the %optflags during the
build:
make CFLAGS=%{optflags} `pkg-config --cflags gtk+-2.0` %{?_smp_mflags}
configure.ac looks pretty messy and could need some maintenance. ;)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488
Lakshmi Narasimhan lakshminaras2...@gmail.com changed:
What|Removed |Added
Flag||fedora-review+
--- Comment #1 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-03-13
09:30:47 EDT ---
[+]MUST: rpmlint must be run on every package. The output should be posted in
the review.
rpmlint -i *.rpm erlang-basho_stats.spec
erlang-basho_stats.i686: E: explicit-lib-dependency erlang-stdlib
You must let rpm find the library dependencies by itself. Do not put unneeded
explicit Requires: tags. (False positive)
erlang-basho_stats.i686: E: no-binary
The package should be of the noarch architecture because it doesn't contain
any binaries. They are arch independent by erlang vm searches in %{libdir}
#652585
erlang-basho_stats.i686: W: only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.
erlang-basho_stats.i686: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.
erlang-basho_stats.src: W: invalid-url Source0:
basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
The value should be a valid, public HTTP, HTTPS, or FTP URL.
erlang-basho_stats.spec: W: invalid-url Source0:
basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
The value should be a valid, public HTTP, HTTPS, or FTP URL.
2 packages and 1 specfiles checked; 2 errors, 4 warnings.
[+]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
[+]MUST: The package must meet the Packaging Guidelines.
Naming-Yes
Version-release - Matches
License - OK, Apache software license v2.0
No prebuilt external bits - OK, there is one(rebar) in the sources but
is not used or distributed
Spec legibity - OK
Package template - OK
Arch support - OK
Libexecdir - OK
rpmlint - yes
changelogs - OK
Source url tag - OK, not provided as the current source url might
change (#652585)
Buildroot is ignored - present anyway. OK
%clean is ignored - present anyway. OK
Build Requires list - OK
Summary and description - OK
API documentation - OK, none provided.
[+]MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
Licensed with Apache software license version 2. All source files (.erl) have
license headers.
[+]MUST: The License field in the package spec file must match the actual
license.
[+]MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
No license text included with the package.
[+]MUST: The spec file must be written in American English.
[+]MUST: The spec file for the package MUST be legible.
[+]MUST: The sources used to build the package must match the upstream
source,as provided in the spec URL. Reviewers should use md5sum for this task.
md5sum basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
7e819c11daf275d07b5d3413619fa480
basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
md5sum
erlang-basho_stats-1.0.1-1.fc15.src/basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
7e819c11daf275d07b5d3413619fa480
erlang-basho_stats-1.0.1-1.fc15.src/basho-basho_stats-basho_stats-1.0.1-0-gfbb957b.tar.gz
[+]MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
Built on x86_64 and i686
[+]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.
None listed.
[+]MUST: All build dependencies must be listed in BuildRequires.
[NA]MUST: The spec file MUST handle locales properly using the %find_lang macro
No locale files.
[NA]MUST: Packages stores shared library files must call ldconfig in %post and
%postun.
[+]MUST: Packages must NOT bundle copies of system libraries.
Checked with rpmquery --list
[NA]MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review.
[+]MUST: A package must own all directories that it creates.
Checked with rpmquery --whatprovdes
[+]MUST: A Fedora package must not list a file more than once in the spec
file's %files listings.
[+]MUST: Permissions on files must be set properly.
The beam files are not marked as executable.
[+]MUST: Each package must consistently use macros.
[+]MUST: The package must contain code, or permissible content.
[+]MUST: Large documentation files must go in a -doc
[Bug 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 --- Comment #2 from Peter Lemenkov lemen...@gmail.com 2011-03-13 09:36:29 EDT --- Thanks a lot! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Peter Lemenkov lemen...@gmail.com 2011-03-13 09:38:01 EDT --- New Package SCM Request === Package Name: erlang-basho_stats Short Description: Basic Erlang statistics library Owners: peter Branches: f14 f15 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 634052] Review Request: ghc-ConfigFile - configuration file 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=634052 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added CC||lakshminaras2...@gmail.com AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684239] Review Request: valadoc - Documentation generator for Vala source
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684239 --- Comment #8 from Tim Lauridsen t...@rasmil.dk 2011-03-13 09:49:34 EDT --- Now it builds in mock, but there is some issues. [!] : MUST - Header files in -devel subpackage, if present. valadoc-0.2.1-1.fc16.i686.rpm : /usr/include/valadoc-1.0.h [!] : MUST - Rpmlint output is silent. rpmlint valadoc-debuginfo-0.2.1-1.fc16.i686.rpm valadoc-debuginfo.i686: W: no-version-in-last-changelog 1 packages and 0 specfiles checked; 0 errors, 1 warnings. rpmlint valadoc-0.2.1-1.fc16.i686.rpm valadoc.i686: W: spelling-error %description -l en_US libvala - Libava, Liberal, liberal valadoc.i686: E: description-line-too-long C Valadoc is a documentation generator for generating API documentation from Vala source valadoc.i686: W: no-version-in-last-changelog valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/libvaladoc.so valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/libvaladoc.a valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/valadoc/plugins/devhelp/libdoclet.a valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/pkgconfig/valadoc-1.0.pc valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/valadoc/plugins/html/libdoclet.a valadoc.i686: W: devel-file-in-non-devel-package /usr/lib/valadoc/plugins/gtkdoc/libdoclet.a valadoc.i686: W: devel-file-in-non-devel-package /usr/include/valadoc-1.0.h 1 packages and 0 specfiles checked; 1 errors, 9 warnings. rpmlint valadoc-0.2.1-1.fc16.src.rpm valadoc.src: W: spelling-error %description -l en_US libvala - Libava, Liberal, liberal valadoc.src: E: description-line-too-long C Valadoc is a documentation generator for generating API documentation from Vala source valadoc.src: W: no-version-in-last-changelog valadoc.src:7: W: mixed-use-of-spaces-and-tabs (spaces: line 7, tab: line 1) valadoc.src: W: invalid-url Source0: valadoc-0.2.1.tar.gz 1 packages and 0 specfiles checked; 1 errors, 4 warnings. [!] : MUST - Development .so files in -devel subpackage, if present. valadoc-0.2.1-1.fc16.i686.rpm : /usr/lib/libvaladoc.so valadoc-0.2.1-1.fc16.i686.rpm : /usr/lib/valadoc/plugins/devhelp/libdoclet.so valadoc-0.2.1-1.fc16.i686.rpm : /usr/lib/valadoc/plugins/gtkdoc/libdoclet.so valadoc-0.2.1-1.fc16.i686.rpm : /usr/lib/valadoc/plugins/html/libdoclet.so You need to make a -devel sub package with the .h, .so .pc files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684239] Review Request: valadoc - Documentation generator for Vala source
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684239 --- Comment #9 from Tim Lauridsen t...@rasmil.dk 2011-03-13 10:14:07 EDT --- http://fedoraproject.org/wiki/Packaging:Guidelines#Devel_Packages -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 522821] Review Request: bluetile - A modern tiling window manager with a gentle learning curve
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=522821 Lakshmi Narasimhan lakshminaras2...@gmail.com changed: What|Removed |Added CC||lakshminaras2...@gmail.com AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 634052] Review Request: ghc-ConfigFile - configuration file 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=634052
Lakshmi Narasimhan lakshminaras2...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review+
--- Comment #3 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-03-13
10:37:48 EDT ---
[+]MUST: rpmlint must be run on every package. The output should be posted in
the review.
ghc-ConfigFile.i686: W: spelling-error %description -l en_US config - con fig,
con-fig, configure
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile.i686: W: spelling-error %description -l en_US parsable -
parable, parables, passable
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile.src: W: spelling-error %description -l en_US config - con fig,
con-fig, configure
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile.src: W: spelling-error %description -l en_US parsable -
parable, parables, passable
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile-devel.i686: W: spelling-error %description -l en_US config -
con fig, con-fig, configure
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile-devel.i686: W: spelling-error %description -l en_US parsable -
parable, parables, passable
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile-prof.i686: E: devel-dependency ghc-ConfigFile-devel
Your package has a dependency on a devel package but it's not a devel package
itself.
ghc-ConfigFile-prof.i686: W: spelling-error %description -l en_US config - con
fig, con-fig, configure
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile-prof.i686: W: spelling-error %description -l en_US parsable -
parable, parables, passable
The value of this tag appears to be misspelled. Please double-check.
ghc-ConfigFile-prof.i686: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.
ghc-ConfigFile-prof.i686: W: devel-file-in-non-devel-package
/usr/lib/ghc-6.12.3/ConfigFile-1.0.6/libHSConfigFile-1.0.6_p.a
A development file (usually source code) is located in a non-devel package. If
you want to include source code in your package, be sure to create a
development package.
4 packages and 1 specfiles checked; 1 errors, 10 warnings.
[+]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
[+]MUST: The package must meet the Packaging Guidelines.
Naming-Yes
Version-release - Matches
License - OK, LGPL v2+
No prebuilt external bits - OK
Spec legibity - OK
Package template - OK , cabal2spec 0.22.4
Arch support - OK
Libexecdir - OK
rpmlint - yes
changelogs - OK
Source url tag - OK, validated.
Buildroot is ignored - OK not present
%clean is ignored - OK not present
Build Requires list - OK
Summary and description - OK
API documentation - OK,in devel package
[+]MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
License is LGPL v2
[+]MUST: The License field in the package spec file must match the actual
license.
[+]MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
COPYING contains the license text and is included.
[+]MUST: The spec file must be written in American English.
[+]MUST: The spec file for the package MUST be legible.
[+]MUST: The sources used to build the package must match the upstream
source,as provided in the spec URL. Reviewers should use md5sum for this task.
md5sum ConfigFile-1.0.6.tar.gz
fbdef31a2989723cc25ba92622ff0e42 ConfigFile-1.0.6.tar.gz
md5sum ghc-ConfigFile-1.0.6-3.fc14.src/ConfigFile-1.0.6.tar.gz
fbdef31a2989723cc25ba92622ff0e42
ghc-ConfigFile-1.0.6-3.fc14.src/ConfigFile-1.0.6.tar.gz
[+]MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
Built on i686
[+]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.
Consider putting sparcv9 in ExclusiveArch
[+]MUST: All build dependencies must be listed in BuildRequires.
[NA]MUST: The spec file MUST handle locales properly using the %find_lang macro
[NA]MUST: Packages stores shared library files must call ldconfig in %post and
%postun.
[+]MUST:
[Bug 683150] Review Request: yad - Display graphical dialogs from shell scripts or command line
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=683150 Raghu Udiyar raghusidda...@gmail.com changed: What|Removed |Added CC||raghusidda...@gmail.com --- Comment #2 from Raghu Udiyar raghusidda...@gmail.com 2011-03-13 10:59:02 EDT --- Hi, Just a few comments : - BuildRoot tag is not required any more : http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag - %clean section not required for F13 and above http://fedoraproject.org/wiki/Packaging:Guidelines#.25clean - Use desktop-file-validate : http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage - Upstream seems to have released a new version (0.9.1), please update the package to this release. - Run rpmlint and fix the errors/warnings it throws. Here's what I got : $ rpmlint yad.spec ../RPMS/i686/yad-* ../SRPMS/yad-0.9.0-1.fc14.src.rpm yad.spec: W: invalid-url Source0: http://yad.googlecode.com/files/yad-0.9.0.tar.xz HTTP Error 404: Not Found yad.i686: W: spelling-error Summary(en_US) dialogs - dialog, dialog s, dialings yad.i686: W: spelling-error %description -l en_US dialogs - dialog, dialog s, dialings yad.i686: W: no-manual-page-for-binary yad-icon-browser yad-devel.i686: W: no-documentation yad-devel.i686: W: no-manual-page-for-binary yad-icon-browser yad.src: W: spelling-error Summary(en_US) dialogs - dialog, dialog s, dialings yad.src: W: spelling-error %description -l en_US dialogs - dialog, dialog s, dialings yad.src: W: invalid-url Source0: http://yad.googlecode.com/files/yad-0.9.0.tar.xz HTTP Error 404: Not Found 4 packages and 1 specfiles checked; 0 errors, 9 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 634052] Review Request: ghc-ConfigFile - configuration file 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=634052 --- Comment #4 from Lakshmi Narasimhan lakshminaras2...@gmail.com 2011-03-13 11:00:37 EDT --- Can sparcv9 be added to the Exclusive Arch field while generating template spec file by cabal2spec? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 682545] Review Request: nutrika - Nutrient content calculator and intake analyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=682545 Raghu Udiyar raghusidda...@gmail.com changed: What|Removed |Added CC||raghusidda...@gmail.com --- Comment #10 from Raghu Udiyar raghusidda...@gmail.com 2011-03-13 11:06:01 EDT --- Hello Péter, Please post the new SPEC/SRPM along with the rpmlint output and the koji build. Or is it the same link above? I see the changes, but the changelog and release haven't been updated. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675689] Review Request: perl-Test-Log-Dispatch - Test what you are logging
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675689 --- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 11:26:00 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 675646] Review Request: perl-Log-Any-Adapter - Tell Log::Any where to send its logs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675646 --- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 11:25:47 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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 --- Comment #4 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 11:27:22 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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-03-13 12:12:16 EDT --- erlang-basho_stats-1.0.1-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/erlang-basho_stats-1.0.1-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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-03-13 12:12:09 EDT --- erlang-basho_stats-1.0.1-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/erlang-basho_stats-1.0.1-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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 --- Comment #5 from Fedora Update System upda...@fedoraproject.org 2011-03-13 12:12:02 EDT --- erlang-basho_stats-1.0.1-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/erlang-basho_stats-1.0.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 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680488] Review Request: erlang-basho_stats - Basic Erlang statistics 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=680488 Peter Lemenkov lemen...@gmail.com changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||RAWHIDE Last Closed||2011-03-13 12:13:21 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652598] Review Request: erlang-riak_core - Distributed systems infrastructure used by Riak
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652598 Bug 652598 depends on bug 680488, which changed state. Bug 680488 Summary: Review Request: erlang-basho_stats - Basic Erlang statistics library https://bugzilla.redhat.com/show_bug.cgi?id=680488 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|MODIFIED Resolution||RAWHIDE Status|MODIFIED|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 629744] Review Request: sparkleshare - sharing work made easy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=629744 --- Comment #2 from Alex Hudson (Fedora Address) fed...@alexhudson.com 2011-03-13 12:32:42 EDT --- Hi Eric, apologies for not updating this sooner. I stopped updating the SparkleShare package after the notification-over-IRC stuff got merged, because I didn't really like the idea of it silently spilling information out into a public forum without people knowing. I'll review the latest code but I suspect that is still the case. However, I'll have a chat with Hylke and if it is, see if there's something we can do about it - e.g. writing some sort of alternative system. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676308] Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=676308
Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:
What|Removed |Added
CC||mtas...@ioa.s.u-tokyo.ac.jp
--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2011-03-13
12:57:07 EDT ---
If you are still interested in importing this package into
Fedora, I will have a look at this one. Instead I will appreciate
it if you would review one of my review requests (e.g. bug 678678)
Some notes:
* Explicit version-specific Requires
- Please consider if you really have to write explicit
version-specific Requires like rubygems = 1.2.
* Usually when packages shipped on currently supported
Fedora branches all satisfy such version requirements,
we don't add such explicit version Requires:
https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires
* Remove obsolete stuff
- rm -rf %{buildroot} at the top of %install is no longer needed
(if you intend to import this only on Fedora).
! Directly installing gem archive into %buildroot
- I always recommend to install gem archive once under %_builddir
to make it sure that no additional files are generated or files
are not modified after installation is done.
* For example, some testsuites generate additional executables
or log files or modify files, which frequently confuses %files
entry or leads to rpmbuild failure.
And because of the same reason, it is recommended that %buildroot
is touched only in %install section (i.e. executing %check under
%buildroot is discouraged).
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566410] Review Request: snmpcheck - An utility to get information via SNMP protocols
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=566410 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Status|ASSIGNED|NEW AssignedTo|fab...@bernewireless.net|nob...@fedoraproject.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566410] Review Request: snmpcheck - An utility to get information via SNMP protocols
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=566410 Fabian Affolter fab...@bernewireless.net 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 676308] Review Request: rubygem-net-scp - A pure Ruby implementation of the SCP client protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676308 --- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2011-03-13 14:01:01 EDT --- * By the way, I also recommend to add -V option to gem install because it shows some debuging information. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 669726] Review Request: rubygem-escape - HTML/URI/shell escaping 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=669726
Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:
What|Removed |Added
CC||mtas...@ioa.s.u-tokyo.ac.jp
--- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2011-03-13
14:07:20 EDT ---
If you are still interested in importing this package into Fedora,
I will review this package (although there is almost nothing to check
for this package...)
* rm -rf %{buildroot} at the top of %install is no longer needed
(on Fedora)
By the way I would appreciate it if you would review one of my
review requests (e.g. bug 678676)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 683150] Review Request: yad - Display graphical dialogs from shell scripts or command line
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=683150 --- Comment #3 from Elder Marco elderma...@gmail.com 2011-03-13 14:17:34 EDT --- Hi Raghu, thanks. Updated to new version (0.9.1). There is no clean section or BuildRoot tag any more. Spec URL (updated): http://eldermarco.fedorapeople.org/files/specs/yad.spec SRPM URL: http://eldermarco.fedorapeople.org/files/srpms/yad-0.9.1-1.fc14.src.rpm Changelog: * Sun Mar 13 2011 Elder Marco elderma...@fedoraproject.org - 0.9.1-1 - New upstream release - Added desktop-file-utils as BuildRequires. - Removed clean section and BuildRoot tag (not required any more). - Removed Encoding key from .desktop file. Running rpmlint, I got: - $ rpmlint yad.spec ../RPMS/i686/yad-0.9.1-1.fc14.i686.rpm ../RPMS/i686/yad-devel-0.9.1-1.fc14.i686.rpm ../SRPMS/yad-0.9.1-1.fc14.src.rpm yad.spec: W: invalid-url Source0: http://yad.googlecode.com/files/yad-0.9.1.tar.xz HTTP Error 404: Not Found yad.i686: W: spelling-error Summary(en_US) dialogs - dialog, dialog s, dialings yad.i686: W: spelling-error %description -l en_US zenity - lenity, zenith, amenity yad.i686: W: spelling-error %description -l en_US dialogs - dialog, dialog s, dialings yad-devel.i686: W: no-documentation yad-devel.i686: W: no-manual-page-for-binary yad-icon-browser yad.src: W: spelling-error Summary(en_US) dialogs - dialog, dialog s, dialings yad.src: W: spelling-error %description -l en_US zenity - lenity, zenith, amenity yad.src: W: spelling-error %description -l en_US dialogs - dialog, dialog s, dialings yad.src: W: invalid-url Source0: http://yad.googlecode.com/files/yad-0.9.1.tar.xz HTTP Error 404: Not Found 3 packages and 1 specfiles checked; 0 errors, 10 warnings. -- - The URL is correct. I don't know why this happens.. - The correct is 'dialog/dialogs' and 'zenity'. There is no spelling error. - The upstream doesn't provides any documentation for the devel package. So, there is no manual for yad-icon-browser. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684323] NetworkManager08 - temporary alternative until all frontends are ported
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684323 --- Comment #4 from Colin Walters walt...@redhat.com 2011-03-13 14:22:40 EDT --- I made a git repository with empty RPM packages simulating the situation here. http://fedorapeople.org/gitweb?p=walters/public_git/testpkgs.git;a=summary Basically, f14/nm.spec should mirror NetworkManager from Fedora 14. Then f15/ has the split packages. I've verified that this works: make yum install f14/nm.rpm # yum install f15/nm08.rpm # replaces f14 NM yum install f15/nm.rpm # replaces nm08 What still needs analysis here is whether yum will if you're running the KDE fork, know to install nm08 on upgrade. I think explicit Requires may need to be added somewhere there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684475] Review Request: wmblob - Dockapp which shows funny moving `blobs'
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684475 --- Comment #4 from Mario Blättermann mari...@gnome.org 2011-03-13 14:32:15 EDT --- Spec URL: http://dl.dropbox.com/u/19373040/Fedora/SPECS/wmblob.spec SRPM URL: http://dl.dropbox.com/u/19373040/Fedora/wmblob-1.0.3-3.fc14.src.rpm Thanks for your hints. (In reply to comment #3) configure.ac looks pretty messy and could need some maintenance. ;) Yes, of course, but upstream is dead. It works as it works. In my mind, it's senseless to patch configure.ac without any chance to get it ever fixed in the sources. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=662257 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:53:25 EDT --- php-pear-CodeGen-PECL-1.1.3-3.el5 has been pushed to the Fedora EPEL 5 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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:53:58 EDT --- globus-gridmap-callout-error-0.3-1.el5 has been pushed to the Fedora EPEL 5 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 675798] Review Request: liblzf - Small data compression 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=675798 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:52:04 EDT --- liblzf-3.6-2.el6 has been pushed to the Fedora EPEL 6 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 675798] Review Request: liblzf - Small data compression 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=675798 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:52:28 EDT --- liblzf-3.6-2.el4 has been pushed to the Fedora EPEL 4 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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:52:52 EDT --- globus-gridmap-callout-error-0.3-1.el4 has been pushed to the Fedora EPEL 4 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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:53:13 EDT --- globus-gridmap-callout-error-0.3-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 675798] Review Request: liblzf - Small data compression 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=675798 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:52:40 EDT --- liblzf-3.6-2.el5 has been pushed to the Fedora EPEL 5 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 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=662257 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|php-pear-CodeGen-PECL-1.1.3 |php-pear-CodeGen-PECL-1.1.3 |-3.fc13 |-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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||globus-gridmap-callout-erro ||r-0.3-1.el4 Resolution||ERRATA Last Closed||2011-03-13 14:52:58 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675798] Review Request: liblzf - Small data compression 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=675798 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|liblzf-3.6-2.el6|liblzf-3.6-2.el4 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675798] Review Request: liblzf - Small data compression 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=675798 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|liblzf-3.6-2.fc14 |liblzf-3.6-2.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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|globus-gridmap-callout-erro |globus-gridmap-callout-erro |r-0.3-1.el4 |r-0.3-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 675798] Review Request: liblzf - Small data compression 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=675798 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|liblzf-3.6-2.el4|liblzf-3.6-2.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 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=662257 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2011-03-13 14:54:11 EDT --- php-pear-CodeGen-PECL-1.1.3-3.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 662257] Review Request: php-pear-CodeGen-PECL - Tool to generate PECL extensions from an XML description
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=662257 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|php-pear-CodeGen-PECL-1.1.3 |php-pear-CodeGen-PECL-1.1.3 |-3.el5 |-3.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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|globus-gridmap-callout-erro |globus-gridmap-callout-erro |r-0.3-1.el6 |r-0.3-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673839] Review Request: boost141 - The free peer-reviewed portable C++ source libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673839 Robert Scheck red...@linuxnetz.de changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #28 from Robert Scheck red...@linuxnetz.de 2011-03-13 14:57:23 EDT --- Zarafa upstream decided to backport some Zarafa 7 features into the stable 6.40.x series. Thus boost141 is now required on EPEL 4, too. Package Change Request == Package Name: boost141 New Branches: el4 Owners: robert -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 632874] Review Request: freedup - Links substantially identical, duplicate files to save file system space
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632874 --- Comment #3 from David Riches davi...@ultracar.co.uk 2011-03-13 15:07:40 EDT --- Finally all fixed: http://dcr226.fedorapeople.org/SRPMS/freedup-1.5-8.fc14.src.rpm http://dcr226.fedorapeople.org/SPECS/freedup.spec rpmlint reports no errors f13 koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2909167 f14 koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2909174 f15 koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2909180 [david@drlaptop SPECS]$ rpmlint ../RPMS/i686/freedup-debuginfo-1.5-8.fc14.i686.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [david@drlaptop SPECS]$ rpmlint ../RPMS/i686/freedup-1.5-8.fc14.i686.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [david@drlaptop SPECS]$ [david@drlaptop SPECS]$ rpmlint ../SRPMS/freedup-1.5-8.fc14.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [david@drlaptop SPECS]$ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608319] Review Request: memaker - An avatar creator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608319 Ankur Sinha sanjay.an...@gmail.com changed: What|Removed |Added Status Whiteboard|StalledSubmitter| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 524119] Review Request: nmon - Nigel's performance MONitor for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=524119 --- Comment #22 from Ankur Sinha sanjay.an...@gmail.com 2011-03-13 16:06:46 EDT --- (In reply to comment #20) Thanks d. johnson for the spec file. I shall take a look over the next couple of days hope to get back to you over this coming weekend. I have a man page ready to put in - I chose not to use the wiki as that is for nmon, and this package is actually from nmon for linux. I didn't want to pollute the manpage with any options that might not be available in the linux version event though they ultimately come from the same source. However I think I may now take a better look at the nmon wiki and see if there is some useful content. Dylan Ping Dylan? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680657] Review Request: mpdas - An MPD audioscrobbling 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=680657 --- Comment #4 from Ankur Sinha sanjay.an...@gmail.com 2011-03-13 16:15:59 EDT --- (In reply to comment #3) @Jason, Ok, currently the license of this md5.cpp file is not mentioned in the spec file. That has to be done. However I am unable to find out what license that file is under. The text seems to match MIT license but not word for word. @sanjay.an...@gmail.com , can you have a look at that file as well? It is md5.cpp It looks like a variant of the BSD license. This one in particular is pretty close to the license statement of md5.cpp : http://fedoraproject.org/wiki/Licensing/BSD#Hybrid_BSD_.28half_BSD.2C_half_zlib.29 Jason, how do you suggest I proceed? 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 678634] Review Request: Saaghar - A Cross-Platform Persian Poetry Software
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=678634
Raghu Udiyar raghusidda...@gmail.com changed:
What|Removed |Added
CC||raghusidda...@gmail.com
--- Comment #1 from Raghu Udiyar raghusidda...@gmail.com 2011-03-13 16:21:16
EDT ---
Hello Hedayat,
Few things to note :
- Consider using lower case for the spec and package name :
http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Case_Sensitivity
- BuildRoot and %clean are not required any more :
http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag
- Add some description for the patches - are they Fedora specific? Can the
patches be integrated into upstream? See :
http://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment
- Use desktop-file-validate :
http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage
- Looks like a typo in the release field :
Release:1.1051%{?dist}
- You can omit the %dir in the files section :
%dir %{_datadir}/saaghar
HTH
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680657] Review Request: mpdas - An MPD audioscrobbling 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=680657 --- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 16:27:54 EDT --- Well, the first barrier here is that, as is often the case when you have one piece of code with an odd license, this is a bundled library. Is this one of the md5 libraries we've already approved for bundling? https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries If so you'll at minimum need to add the Provides: bundled(md5-whatever) bit. If not, FPC will need to review the situation. As for the license issue, of course the license of the actual files present in the binary package are significant (as told in the URLs I've already provided). If it is not obvious how the licenses combine, you will need to get an opinion from the legal folks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 684239] Review Request: valadoc - Documentation generator for Vala source
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=684239 --- Comment #10 from Raul Gutierrez S. r...@rieder.net.py 2011-03-13 16:35:05 EDT --- I updated the spec file according to latest comment. Since this packages is written in Vala, I've also added the .vapi files to the devel package. I've added --disable-static to ./configure too, according to the wiki's recommendation. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 457709] Review Request: perizia-sfd-fonts - English asymmetric font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=457709 Ankur Sinha sanjay.an...@gmail.com changed: What|Removed |Added Summary|Review Request: |Review Request: |perizia-fonts - English |perizia-sfd-fonts - English |asymmetric font |asymmetric font --- Comment #18 from Ankur Sinha sanjay.an...@gmail.com 2011-03-13 16:57:03 EDT --- Hello, SPEC: http://ankursinha.fedorapeople.org/perizia/perizia-sfd-fonts.spec SRPM: http://ankursinha.fedorapeople.org/perizia/perizia-sfd-fonts-20081017-1.fc14.src.rpm It's been quite a while since I packaged a font. Could you please check if the fontconfig stuff is alright. rpmlint gives: [ankur@ankur noarch]$ rpmlint perizia-sfd-fonts-20081017-1.fc14.noarch.rpm perizia-sfd-fonts.noarch: W: invalid-url URL: http://hiran.in/hiran.in/blog/thanks-perizia-is-now-a-font HTTP Error 404: Not Found 1 packages and 0 specfiles checked; 0 errors, 1 warnings. 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 680666] Review Request: rssdler - A utility to automatically download enclosures and other objects linked to from various types of RSS feeds
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680666 --- Comment #2 from Ankur Sinha sanjay.an...@gmail.com 2011-03-13 17:00:03 EDT --- (In reply to comment #1) Hi Ankur A note: Perhaps you don't need neither BuildRoot tag nor %clean section Greetings Ugh! I'll correct that. (The spec template that vim uses hasn't been updated apparently). Thanks Sergio. 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 643140] Review Request: autokey - A desktop automation utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643140 --- Comment #15 from Patrice FERLET meta...@gmail.com 2011-03-13 17:10:52 EDT --- This 404 error is strange, could you try to do: spectool -gf youspecfile It must download tarball on current directory... if not: can you try to download your source tarball with wget ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 681093] Review Request: wdiff - A front-end to GNU diff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=681093 --- Comment #22 from Fedora Update System upda...@fedoraproject.org 2011-03-13 17:18:50 EDT --- wdiff-0.6.5-5.fc13 has been pushed to the Fedora 13 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 650667] Review Request: hitori - Hitori game 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=650667 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|hitori-0.2.6-3.fc14 |hitori-0.2.6-4.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 681093] Review Request: wdiff - A front-end to GNU diff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=681093 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|wdiff-0.6.5-5.fc15 |wdiff-0.6.5-5.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|globus-gridmap-callout-erro |globus-gridmap-callout-erro |r-0.3-1.el5 |r-0.3-1.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 --- Comment #18 from Fedora Update System upda...@fedoraproject.org 2011-03-13 17:18:30 EDT --- globus-gridmap-callout-error-0.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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2011-03-13 17:17:08 EDT --- globus-gridmap-callout-error-0.3-1.fc13 has been pushed to the Fedora 13 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 643679] Review Request: globus-gridmap-callout-error - Globus Toolkit - Globus Gridmap Callout Errors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643679 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|globus-gridmap-callout-erro |globus-gridmap-callout-erro |r-0.3-1.fc13|r-0.3-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 681093] Review Request: wdiff - A front-end to GNU diff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=681093 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2011-03-13 17:21:12 EDT --- wdiff-0.6.5-5.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 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674207 --- Comment #21 from Fedora Update System upda...@fedoraproject.org 2011-03-13 17:21:28 EDT --- python-cloudfiles-1.7.8-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 681093] Review Request: wdiff - A front-end to GNU diff
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=681093 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|wdiff-0.6.5-5.fc13 |wdiff-0.6.5-5.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 674207] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=674207 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|python-cloudfiles-1.7.8-1.f |python-cloudfiles-1.7.8-1.f |c15 |c14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680657] Review Request: mpdas - An MPD audioscrobbling 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=680657
--- Comment #6 from Ankur Sinha sanjay.an...@gmail.com 2011-03-13 17:28:09
EDT ---
(In reply to comment #5)
Well, the first barrier here is that, as is often the case when you have one
piece of code with an odd license, this is a bundled library. Is this one of
the md5 libraries we've already approved for bundling?
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries
If so you'll at minimum need to add the Provides: bundled(md5-whatever) bit.
If not, FPC will need to review the situation.
As for the license issue, of course the license of the actual files present in
the binary package are significant (as told in the URLs I've already
provided).
If it is not obvious how the licenses combine, you will need to get an
opinion
from the legal folks.
Hey,
It appears to be the Peter Deutsch's version: bundled(md5-deutsch) version.
I've added a virtual provides.
SPEC:http://ankursinha.fedorapeople.org/mpdas/mpdas.spec
SRPM:http://ankursinha.fedorapeople.org/mpdas/mpdas-0.3.0-2.fc14.src.rpm
rpmlint outputs: (They don't look major, I'll look into them as the review
progresses)
[ankur@ankur SPECS]$ rpmlint mpdas.spec ../SRPMS/mpdas-0.3.0-2.fc14.src.rpm
../RPMS/x86_64/mpdas-0.3.0-2.fc14.x86_64.rpm
mpdas.spec:15: W: unversioned-explicit-provides bundled(md5-deutsch)
mpdas.spec:29: W: rpm-buildroot-usage %build CXXFLAGS=$RPM_OPT_FLAGS
PREFIX=$RPM_BUILD_ROOT%{_prefix} MANPREFIX=$RPM_BUILD_ROOT%{_mandir}
mpdas.spec:15: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 15)
mpdas.src: W: spelling-error Summary(en_US) audioscrobbling - audiological,
audiocassette, audiovisuals
mpdas.src: W: spelling-error Summary(en_US) clien - cline, lien, client
mpdas.src: W: spelling-error %description -l en_US libmpd - libido, limpid,
Librium
mpdas.src: W: spelling-error %description -l en_US libcurl - lib curl,
lib-curl, liberal
mpdas.src:15: W: unversioned-explicit-provides bundled(md5-deutsch)
mpdas.src:29: W: rpm-buildroot-usage %build CXXFLAGS=$RPM_OPT_FLAGS
PREFIX=$RPM_BUILD_ROOT%{_prefix} MANPREFIX=$RPM_BUILD_ROOT%{_mandir}
mpdas.src:15: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 15)
mpdas.x86_64: W: spelling-error Summary(en_US) audioscrobbling - audiological,
audiocassette, audiovisuals
mpdas.x86_64: W: spelling-error Summary(en_US) clien - cline, lien, client
mpdas.x86_64: W: spelling-error %description -l en_US libmpd - libido, limpid,
Librium
mpdas.x86_64: W: spelling-error %description -l en_US libcurl - lib curl,
lib-curl, liberal
2 packages and 1 specfiles checked; 0 errors, 14 warnings.
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 643140] Review Request: autokey - A desktop automation utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643140 --- Comment #16 from Raghu Udiyar raghusidda...@gmail.com 2011-03-13 18:36:46 EDT --- Yes, spectool works fine, not sure why rpmlint fails : $ spectool -gf autokey.spec --2011-03-14 04:03:37-- http://autokey.googlecode.com/files/autokey_0.71.2.tar.gz Resolving autokey.googlecode.com... 209.85.153.82 Connecting to autokey.googlecode.com|209.85.153.82|:80... connected. HTTP request sent, awaiting response... 200 OK Length: 235998 (230K) [application/x-gzip] Saving to: “./autokey_0.71.2.tar.gz” 100%[===] 235,998 66.1K/s in 3.5s 2011-03-14 04:03:41 (66.1 KB/s) - “./autokey_0.71.2.tar.gz” saved [235998/235998] -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 524119] Review Request: nmon - Nigel's performance MONitor for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=524119 --- Comment #23 from Dylan Swift dylan.sw...@gmail.com 2011-03-13 19:04:16 EDT --- Ping Ankur, gosh I'm so sorry folks, I hadn't realised so much time had passed! (lots of work trips, and a new baby consumed all my spare cycles) Let me see if I can't get something together in the next week. Dylan -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 678634] Review Request: Saaghar - A Cross-Platform Persian Poetry Software
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=678634
--- Comment #2 from Hedayat Vatankhah heday...@gmail.com 2011-03-13 19:09:27
EDT ---
(In reply to comment #1)
Hello Hedayat,
Hi! Thanks a lot for reviewing this package :)
Few things to note :
- Consider using lower case for the spec and package name :
http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Case_Sensitivity
When I was selecting the name I checked various placing in the code and the
project and I conclude that upstream prefers Saaghar to saaghar, so I decided
to go with the former (e.g. the archive file name and the name of Debian
packages generated by upstream).
- BuildRoot and %clean are not required any more :
http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag
OK, removed.
- Add some description for the patches - are they Fedora specific? Can the
patches be integrated into upstream? See :
http://fedoraproject.org/wiki/Packaging:Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment
Oops, sorry for forgetting that. Added.
- Use desktop-file-validate :
http://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage
Added (was a little misinterpretation :P).
- Looks like a typo in the release field :
Release:1.1051%{?dist}
Nope. A little description added to the first patch. This is in fact a minor
version used by upstream for the version with Saaghar-upstream_update.patch
patch applied (the version is directly extracted from a version macro in source
files).
- You can omit the %dir in the files section :
%dir %{_datadir}/saaghar
I don't want to add the whole directory to the main package. There is a DB
inside %{_datadir}/saaghar which is included in Saaghar-data sub-package. But I
wanted the main package to own the directory.
HTH
Thanks again.
Updated SRPM and SPEC files are here:
SPEC: http://hedayat.fedorapeople.org/reviews/Saaghar/0.7.2-2/Saaghar.spec
SRPM:
http://hedayat.fedorapeople.org/reviews/Saaghar/0.7.2-2/Saaghar-0.7.2-2.1051.fc14.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 682963] Review Request: perl-JSON-Util - Easy and portable encode/decode of JSON
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=682963 Ruediger Landmann r.landm...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ruediger Landmann r.landm...@redhat.com 2011-03-13 19:16:07 EDT --- Thanks Iain! New Package SCM Request === Package Name: perl-JSON-Util Short Description: Easy and portable encode/decode of JSON Owners: rlandmann Branches: f14 f15 el6 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 682949] Review Request: perl-Test-Fixme - Check code for FIXMEs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=682949 Ruediger Landmann r.landm...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ruediger Landmann r.landm...@redhat.com 2011-03-13 19:17:13 EDT --- Thanks Iain! New Package SCM Request === Package Name: perl-Test-Fixme Short Description: Check code for FIXMEs Owners: rlandmann Branches: f14 f15 el6 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518441] Review Request: ike - Shrew Soft VPN Client For Linux
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=518441
Ruediger Landmann r.landm...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #33 from Ruediger Landmann r.landm...@redhat.com 2011-03-13
20:46:42 EDT ---
Thanks Andrew -- looks good now. Please go ahead and make your SCM request.
Thanks for your patience with the process and sorry that this has dragged on
for so long.
ACCEPT
- = N/A
/ = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[/] Rpmlint output is clean:
$ rpmlint SPECS/ike.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
$ rpmlint SRPMS/ike-2.1.7-4.fc14.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
$ rpmlint RPMS/x86_64/ike-2.1.7-4.fc14.x86_64.rpm
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libith.so
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/liblog.so
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libip.so
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libike.so
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libpfk.so
ike.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libidb.so
ike.x86_64: W: service-default-enabled /etc/rc.d/init.d/iked
1 packages and 0 specfiles checked; 0 errors, 7 warnings.
$ rpmlint RPMS/x86_64/ike-debuginfo-2.1.7-4.fc14.x86_64.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[/] Package is named according to the Package Naming Guidelines.
[/] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[/] Package meets the Packaging Guidelines including the Language specific
items
[/] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[/] License field in the package spec file matches the actual license.
License type: Sleepycat
[/] 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.
[/] Spec file is legible and written in American English.
[/] Sources used to build the package matches the upstream source, as provided
in the spec URL.
$ md5sum SOURCES/ike-2.1.7-release.tbz2
bc86e101809fc750013e18480c8c1040 SOURCES/ike-2.1.7-release.tbz2
$ md5sum ~/Download/ike-2.1.7-release.tbz2
bc86e101809fc750013e18480c8c1040
/home/rlandmann/Download/ike-2.1.7-release.tbz2
[/] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested: http://koji.fedoraproject.org/koji/taskinfo?taskID=2909578
[/] Package is not known to require ExcludeArch
[/] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly (with the %find_lang macro)
[/] ldconfig called in %post and %postun if required.
[/] Package does not bundle copies of system libraries
[/] Package is not relocatable.
[/] Package must own all directories that it creates.
[/] Package does not contain duplicates in %files.
[/] Permissions on files are set properly
[/] %files section includes a %defattr(...) line
[/] Package consistently uses macros.
[-] Large documentation files are in a -doc subpackage, if required.
[/] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -static subpackage, if present.
[-] Development .so files in -devel subpackage, if present.
.so files discussed above
[-] -devel packages require base package with full versioning.
[/] Package does not contain any libtool archives (.la).
[/] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[/] Package does not own files or directories owned by other packages.
[/] Filenames are valid UTF-8
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673839] Review Request: boost141 - The free peer-reviewed portable C++ source libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673839 --- Comment #29 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 21:04:49 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 682963] Review Request: perl-JSON-Util - Easy and portable encode/decode of JSON
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=682963 --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 21:05:22 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 682949] Review Request: perl-Test-Fixme - Check code for FIXMEs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=682949 --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2011-03-13 21:05:05 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
