[Bug 457535] Review Request: - myanmar3-unicode-fonts - Myanmar3 unicode font

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457535





--- Comment #6 from Minto Joseph [EMAIL PROTECTED]  2008-08-06 02:04:08 EDT 
---
New Package CVS Request
===
Package Name: myanmar3-unicode-fonts
Short Description: Myanmar3 unicode font
Owners: mintojoseph
Branches: devel only
InitialCC: fonts-sig
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363





--- Comment #40 from Hao Liu [EMAIL PROTECTED]  2008-08-06 02:08:53 EDT ---
Upload src rpm and spec for redhat-lsb3.1 to hliu.fedorapeople.org

1. Revise the License info
2. Remove the 2 requires provided by redhat-lsb3.1
3. Add comments explaining why hard-coded lib paths are kept.
4. Resolve some hard-coded path problems.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363





--- Comment #41 from Hao Liu [EMAIL PROTECTED]  2008-08-06 02:30:20 EDT ---
Update Comment#40:

The full urls for the 2 uploaded files are:

SPEC: [http://hliu.fedorapeople.org/redhat-lsb.spec]
SRPM: [http://hliu.fedorapeople.org/redhat-lsb-3.1-21.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458024] New: Review Request: sblim-sfcc - Small Footprint CIM Client Library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sblim-sfcc - Small Footprint CIM Client Library

https://bugzilla.redhat.com/show_bug.cgi?id=458024

   Summary: Review Request: sblim-sfcc - Small Footprint CIM
Client Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://linux.dell.com/files/fedora/sblim-sfcc/sfcc.spec
SRPM URL: http://linux.dell.com/files/fedora/sblim-sfcc/
SRPM: sblim-sfcc-2.1.0-0.src.rpm

Description: 
Hello! I have finished packaging sblim-sfcc and I would appreciate a review so
that I can get it into Fedora Extras.

Small Footprint CIM Broker(SFCB) is a lightweight CIM daemon that responds to
CIM client requests for system management data and/or performs system
management tasks. sfcb supports most of the standard CIM XML over http/https
protocol.It is highly modular, allowing functionality to be easily added,
removed or customized for different management applications.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458025] New: Review Request: wsmancli - Opensource Implementation of WS-Management - Command line Utility

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: wsmancli - Opensource Implementation of WS-Management 
- Command line Utility

https://bugzilla.redhat.com/show_bug.cgi?id=458025

   Summary: Review Request: wsmancli - Opensource Implementation
of WS-Management - Command line Utility
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://linux.dell.com/files/fedora/wsmancli/wsmancli.spec
SRPM URL: http://linux.dell.com/files/fedora/wsmancli/
SRPM: wsmancli-2.0.0-1.src.rpm

Description:

Hello! I have finished packaging sblim-sfcc and I would appreciate a review so
that I can get it into Fedora Extras.
wsmancli is a part of openwsman, an Opensource Implementation of WS-Management.
This is the command line client of openwsman providing an easy way to access
WS-Management service, especially for testing and exploration purposes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453520





--- Comment #5 from Jens Petersen [EMAIL PROTECTED]  2008-08-06 03:08:39 EDT 
---
Still rpmlint clean :)

This looks good now.  I can do a full review tomorrow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453465] Review Request: transbot - An irc bot designed to translate between languages on separate channels

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453465


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #12 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 03:16:58 EDT 
---
(In reply to comment #10)
  * What I asked you is to change the _version_ number when you modify the
  tarball
itself (release number is rpm specific). 
  
Changing tarball without changing version may confuse people who are 
  trying
  to 
use this package.
 
 Alright.  I guess I read your comment too quickly and equated version with
 release in my mind.  My apologies.  For future reference, should the version
 number be changed even for one-line corrections?

If you want to modify the tarball itself and wants to publish a new tarball
_formally_, please.
(Note: some upstream developer and also Fedora package submitter doesn't
publish
   new tarball formally until the review request he/she submitted is
passed)

Well,
* Now this package itself is okay
* You are currently in need of sponsor.
  In this case you have to either submit another review request or do
pre-review
  of other person's review request as described in
  http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored
  You have 2 other review requests, which seem good to some extent (they
  need fixing on some points... however it is not sure if I have enough time
  to review your other review requests at the moment)

--
  This package (transbot) is APPROVED by mtasaka
--

Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from Get a Fedora Account.
At a point a mail should be sent to sponsor members which notifies
that you need a sponsor. At the stage, please also write on
this bug for confirmation that you requested for sponsorship and
your FAS (Fedora Account System) name. Then I will sponsor you.

If you want to import this package into Fedora 8/9, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on koji Fedora rebuilding system).

If you have questions, please ask me.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363


Parag AN(पराग) [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #42 from Parag AN(पराग) [EMAIL PROTECTED]  2008-08-06 04:05:38 
EDT ---
Ok. lets start review again with new maintainer.
rpmlint on binary gave me
redhat-lsb.i386: W: dangling-relative-symlink /lib/ld-lsb.so.3 ld-linux.so.2
redhat-lsb.i386: W: dangling-relative-symlink /usr/lib/lsb/install_initd
../../../sbin/chkconfig
redhat-lsb.i386: W: dangling-relative-symlink /usr/lib/lsb/remove_initd
../../../sbin/chkconfig
== Ok 

redhat-lsb.i386: E: zero-length /etc/lsb-release.d/core-3.1-ia32
redhat-lsb.i386: W: non-conffile-in-etc /etc/lsb-release.d/core-3.1-ia32
redhat-lsb.i386: E: zero-length /etc/lsb-release.d/graphics-3.1-noarch
redhat-lsb.i386: W: non-conffile-in-etc /etc/lsb-release.d/graphics-3.1-noarch
redhat-lsb.i386: E: zero-length /etc/lsb-release.d/graphics-3.1-ia32
redhat-lsb.i386: W: non-conffile-in-etc /etc/lsb-release.d/graphics-3.1-ia32
redhat-lsb.i386: E: zero-length /etc/lsb-release.d/core-3.1-noarch
redhat-lsb.i386: W: non-conffile-in-etc /etc/lsb-release.d/core-3.1-noarch
== As explained in SPEC needed these LSB Profile files and as they are not
having any contents no issue of marking as conffile.

redhat-lsb.i386: E: non-readable /usr/sbin/redhat_lsb_trigger.i386 0700
redhat-lsb.i386: E: non-standard-executable-perm
/usr/sbin/redhat_lsb_trigger.i386 0700
redhat-lsb.i386: E: statically-linked-binary /usr/sbin/redhat_lsb_trigger.i386
== Not sure about this for now

redhat-lsb.i386: E: only-non-binary-in-usr-lib
== From LSB documentation page 460 this looks ok.

redhat-lsb.i386: W: incoherent-version-in-changelog 3.1-22 3.1-21.fc10
== Change Release tag in SPEC

rpmlint on SRPM gave

redhat-lsb.src:361: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/lsb
redhat-lsb.src:379: E: hardcoded-library-path in
$RPM_BUILD_ROOT/usr/lib/lsb/install_initd
redhat-lsb.src:380: E: hardcoded-library-path in
$RPM_BUILD_ROOT/usr/lib/lsb/remove_initd
=== From spec comment its ok

As comment in SPEC said # LSB uses /usr/lib rather than /usr/lib64 even for
64bit OS

you should revert your changes
437c436
 /lib/lsb
---
 /%{_lib}/lsb
otherwise there will be problem on 64bit machine as those files will be
installed in /usr/lib64

cp command should be used like cp -p

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453520


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #6 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 04:04:10 EDT 
---
Some notes before Petersen-san do a full review:

* Seemingly redundant version specific (Build)Requires
  - Please explain why you want version specific (Build)Requires such as
glib2-devel  2.4, splite-devel  3.0
Even Fedora Core 3 shipped sqlite-3.1.2 and glib2-2.4.7.

* Cflags
  http://fedoraproject.org/wiki/Packaging/Guidelines#Compiler_flags
  - Fedora specific compilation flags are not correctly honored:
http://koji.fedoraproject.org/koji/taskinfo?taskID=761796
And as a result the debuginfo rpm creation is not correct.

By the way why do you call cmake twice?

* Unihan.h
  - %_includedir/Unihan.h contains:
-
29  #ifndef UNIHAN_H_
30  #define UNIHAN_H_
31  #include config.h
32  #include Unihan_enum.h
33  #include str_functions.h
--
However I can find these 3 headers nowhere.
Also please note that installing autotool-generated config.h as
system-wide
header file must be avoided:
https://bugzilla.redhat.com/show_bug.cgi?id=208034#c43

* Directory ownership issue
  - %_datadir/doc/%name is not owned by any packages.
By the way, do you really want two document directories:
%_datadir/%doc/%name-version
and %_datadir/doc/%name ? Please consider to unify document directories.

* linkage mistakes
  - rpmlint shows:
---
$ rpmlint libUnihan
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4 g_free
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4 g_free
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4
sqlite3_close
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4
g_strsplit_set
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4
sqlite3_value_int
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4
sqlite3_result_text
libUnihan.i386: W: undefined-non-weak-symbol /usr/lib/libUnihan.so.0.4 g_strdup
.. (and many)

   You can check this also by $ ldd -r /usr/lib/libUnihan.so.0.4 /dev/null
   For shipping -devel subpackages this is not allowed because leaving
undefined
   non-weak symbols will cause linkage error.

   This usually means that libUnihan.so is not correctly linked against proper
libraries.
   Please fix this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 456086] Review request: hellanzb - Hands-free nzb downloader and post processor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=456086





--- Comment #2 from Conrad Meyer [EMAIL PROTECTED]  2008-08-06 04:08:06 EDT 
---
Thanks for your suggestion of dumping README.Fedora if no configuration is
found; I believe I've added this and fixed the other issues in these new
spec/srpms:

Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/hellanzb.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/hellanzb-0.13-4.fc9.src.rpm

Additionally it now builds in mock [0].

[0]: http://koji.fedoraproject.org/koji/taskinfo?taskID=761872

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 449135] Review Request: gforge - GForge Collaborative Development Environment

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=449135





--- Comment #17 from Andrea Musuruane [EMAIL PROTECTED]  2008-08-06 04:10:48 
EDT ---
(In reply to comment #16)
 Regarding the way to hardcoding any password in the package, I'm thinking 
 about
 writing a helper script that must be run by superuser and performs that task
 after the package has been installed. The point is the proper way to inform
 about it. Issuing a message on %post might be unnoticed by many users. Mailing
 to root account sounds better, but might not be correctly redirected. And
 adding that into a README/ENABLE file on %doc does not appear as a better
 solution to me.
 Any ideas ?

The Fedora way is a README.Fedora file in %doc. It is used by many packages and
that is what an admin should look for after installing if manual intervention
is needed.

https://fedoraproject.org/wiki/Packaging/Guidelines#summary

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363





--- Comment #43 from Parag AN(पराग) [EMAIL PROTECTED]  2008-08-06 04:12:42 
EDT ---
Ok got statically linked rpmlint messages bug 232918. Because of this bug fix
this package got that rpmlint message.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458030] New: Review Request: avogadro - Avogadro is an advanced Molecular editor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: avogadro - Avogadro is an advanced Molecular editor

https://bugzilla.redhat.com/show_bug.cgi?id=458030

   Summary: Review Request: avogadro - Avogadro is an advanced
Molecular editor
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://sdz.fedorapeople.org/rpmbuilds/avogadro.spec
SRPM URL: http://sdz.fedorapeople.org/rpmbuilds/avogadro-0.8.1-1.fc9.src.rpm
Description: Avogadro is an advanced molecular editor designed for
cross-platform use in computational chemistry, molecular modeling,
bioinformatics, materials science, and related areas. It offers flexible
rendering and a powerful plugin architecture.
By the way: This is my first package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458030] Review Request: avogadro - Avogadro is an advanced Molecular editor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458030


Thorsten Leemhuis [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457261] Review Request: open-cobol - OpenCOBOL - COBOL compiler

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457261





--- Comment #6 from Dan Horák [EMAIL PROTECTED]  2008-08-06 04:41:40 EDT ---
For the test-suite it is possible to do the review with the one test failing
(just comment out the %check section from the spec) and only wait for a
solution before building the package. Does the test #98 fail only on rawhide or
in F-9/8 too?

But there are still unresolved issues from comment #3
- wrong license tag
- -g is filtered out from the CFLAGS during running configure (should be
solvable with appending --enable-debug)
- incomplete debuginfo (depends on the CFLAGS issue), will require chmod a-x
for all *.c and *.h files
- excessive --with-readline as an parameter for configure

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363





--- Comment #44 from Hao Liu [EMAIL PROTECTED]  2008-08-06 04:54:16 EDT ---
Revise the spec according to the comment #42 and #43,
SPEC: [http://hliu.fedorapeople.org/redhat-lsb-3.1-22.spec]
SRPM: [http://hliu.fedorapeople.org/redhat-lsb-3.1-22.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 432205] Review Request: exe - eXe eLearning XHTML editor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=432205


Hans de Goede [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




--- Comment #7 from Hans de Goede [EMAIL PROTECTED]  2008-08-06 05:14:24 EDT 
---
Okay, all (almost all?) needed info for cleaning up the licensing mess that is
eXe is now available here:
http://eduforge.org/forum/message.php?msg_id=5768

But I simply don't have the time / motivation to clean this up, so I'm closing
this review request as wontfix.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 451996] Review Request: prover9 - Thereom Prover and Countermodel Generator

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=451996





--- Comment #20 from Fedora Update System [EMAIL PROTECTED]  2008-08-06 
05:31:12 EDT ---
prover9-200805a-4.fc9 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 445153] Review Request: libacpi - General purpose library for ACPI

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=445153





--- Comment #17 from manuel wolfshant [EMAIL PROTECTED]  2008-08-06 05:32:07 
EDT ---
I have not forgotten about this. I am trying to identify if these libs work on
PPC  but for the moment I do not have the hardware at hand.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 445152] Review Request: yacpi - ncurses based acpi viewer

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=445152





--- Comment #12 from manuel wolfshant [EMAIL PROTECTED]  2008-08-06 05:33:06 
EDT ---
https://bugzilla.redhat.com/show_bug.cgi?id=445153#c17 applies here too, for
the moment. will come back once I have news.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454207] Review Request: Terminator - Multiple terminals in one window

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454207





--- Comment #24 from manuel wolfshant [EMAIL PROTECTED]  2008-08-06 05:42:29 
EDT ---
(In reply to comment #23)
Thanks, Mamoru. I've added myself to the CC: list of those bugs

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454980] Review Request: axel - Download accelerator, wget replacement

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454980


Pavel Alexeev [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #3 from Pavel Alexeev [EMAIL PROTECTED]  2008-08-06 06:16:32 EDT 
---
If there is no objection may be I can continue maintenance this package?

SPEC: http://hubbitus.net.ru/rpm/Fedora9/axel/axel.spec
SRPM: http://hubbitus.net.ru/rpm/Fedora9/axel/axel-1.1-1.fc9.src.rpm

(In reply to comment #1)
All changes made by your comment (see changelog):

- Correct BuildRoot
- License changed to GPLv2+
- Add patch0 - axel-1.1-fedora-build.patch
- Regular file /usr/etc/axelrc changed to %config(noreplace)
%{_sysconfdir}/axelrc
- Disable strip binaries (switch --strip=0 to configure script)
- Add internationalisation support:
 . Switch --i18n=1 to configure script
 . Add BR gettext
 . Add files for de and nl locales.

rpmlint is quiet.
And it is built in koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=761927

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 451996] Review Request: prover9 - Thereom Prover and Countermodel Generator

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=451996


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #21 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 06:18:57 EDT 
---
Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454980] Review Request: axel - Download accelerator, wget replacement

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454980





--- Comment #4 from Pavel Alexeev [EMAIL PROTECTED]  2008-08-06 06:26:08 EDT 
---
This is my 4th (in Fedora package review, not in packaging history at all)
package and I am looking for sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458054] New: Review Request: arm4 - Application Response Measurement (ARM) agent

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: arm4 - Application Response Measurement (ARM) agent

https://bugzilla.redhat.com/show_bug.cgi?id=458054

   Summary: Review Request: arm4 - Application Response
Measurement (ARM) agent
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://arm4.org/Downloads/arm4.spec
SRPM URL: http://arm4.org/Downloads/arm4-0.8-0.1.fc9.beta2.src.rpm
Description: An open source implementation of the Open Group's Application
Response Measurement standard Version 4.0. This agent and daemon is capable of
measuring transaction response times across multiple tiers and correlating them
to determine the true source of application response problems. See
http://www.arm4.org for more information.

As this is my first package submission, I'm in need of a sponsor. I hope to
contribute many related packages in the near future.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458054] Review Request: arm4 - Application Response Measurement (ARM) agent

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458054


David Carter [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 226363] Merge Review: redhat-lsb

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226363





--- Comment #45 from Parag AN(पराग) [EMAIL PROTECTED]  2008-08-06 08:06:35 
EDT ---
with updated SRPM I see that build.log gave me
warning: File listed twice: /lib/lsb
warning: File listed twice: /lib/lsb/init-functions

Your %files should look like this
%files
%defattr(-,root,root)
%doc README
%{_sysconfdir}/redhat-lsb
%dir %{_sysconfdir}/lsb-release.d
# These files are needed because they shows which LSB we're supporting now,
# for example, if core-3.1-noarch exists, it means we are supporting LSB3.1 now
%{_sysconfdir}/lsb-release.d/*
%{_mandir}/*/*
%{_bindir}/*
#/bin/mailx
/bin/redhat_lsb_init
/usr/lib/lsb
/%{_lib}/*so*
/lib/lsb/init-functions
%{_sbindir}/redhat_lsb_trigger.%{_target_cpu}


Otherwise with SRPM you submitted rpm failed to build on x86_64. Please, update
SPEC and SRPM with no release increment.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 456086] Review request: hellanzb - Hands-free nzb downloader and post processor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=456086





--- Comment #3 from Erik van Pienbroek [EMAIL PROTECTED]  2008-08-06 09:49:41 
EDT ---
Some new comments:
* BuildRequires: python-setuptools-devel is actually unncessary,
python-setuptools should be sufficient enough
* Why did you merge the unrar and the configuration patch? They both have
different goals so they should be kept seperate

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 244593] Review Request: postgresql-pgbouncer - Lightweight connection pooler for PostgreSQL

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=244593





--- Comment #20 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:49:47 EDT 
---
Again ping?

I will close this bug if no response from the reporter is received within
ONE WEEK:
http://fedoraproject.org/wiki/PackageMaintainers/Policy/StalledReviews#Submitter_not_responding

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=428798





--- Comment #13 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:52:18 EDT 
---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 426867] Review Request: scala - Hybrid functional/object-oriented language

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426867





--- Comment #47 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:51:27 EDT 
---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 434906] Review Request: xosview - OS resource viewer

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=434906





--- Comment #31 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:54:49 EDT 
---
I will close this bug as NOTABUG if no response is received from the reporter
within ONE 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 456678] Review Request: honeyd-1.5c-2.src.rpm - Honeypot daemon

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=456678


Peter Vrabec [EMAIL PROTECTED] changed:

   What|Removed |Added

Customer Facing||---




--- Comment #2 from Peter Vrabec [EMAIL PROTECTED]  2008-08-06 09:52:49 EDT 
---
Spec URL: http://people.redhat.com/pvrabec/rpms/honeyd.spec
SRPM URL: http://people.redhat.com/pvrabec/rpms/honeyd-1.5c-4.fc9.src.rpm

next turn :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458080] New: Review Request: perl-Email-Find - Find RFC 822 email addresses in plain text

[bugzilla]

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-Email-Find - Find RFC 822 email addresses in 
plain text

https://bugzilla.redhat.com/show_bug.cgi?id=458080

   Summary: Review Request: perl-Email-Find - Find RFC 822 email
addresses in plain text
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://kwizart.fedorapeople.org/SPECS/perl-Email-Find.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/perl-Email-Find-0.10-1.fc8.kwizart.src.rpm
Description: Find RFC 822 email addresses in plain text

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 456723] Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=456723


Lennart Poettering [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 456828] Review Request: libasyncns - Asynchronous Name Service Library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=456828


Lennart Poettering [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 438543] Review Request: Synopsis - Source-code introspection tool.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=438543





--- Comment #38 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:56:56 EDT 
---
I will close this bug as NOTABUG if no response is received from the reporter
within ONE 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 438750] Review Request: qtoctave - fronted for octave written using qt4 widgets

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=438750





--- Comment #29 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:57:43 EDT 
---
I will close this bug as NOTABUG if no response is received from the reporter
within ONE 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=421241





--- Comment #48 from Alexander Kahl [EMAIL PROTECTED]  2008-08-06 09:59:33 
EDT ---
Gianluca:
I've fixed the no-documentation issue by adding the license to all
subpackages. The joliet problem should go away with 1.6 stable; for the rest
I've verified the files are necessary and correct for the unit tests. Right now
there's nothing special required to have Zend Framework run on Fedora, if it
ever will, I'll add a README.
Thanks again!

Jess:
Which way do you recommend for staying in contact, do you want to be set CC for
Zend Framework related bugs in RedHat Bugzilla or should I open a Zend Jira
account and forward bug reports where applicable?


Sorry guys for taking so long but whenever I tried posting Bugzilla's update
seemed to get into my way..

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458085] New: Review Request: nes_ntsc - Provides a NES NTSC video filtering library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: nes_ntsc - Provides a NES NTSC video filtering library

https://bugzilla.redhat.com/show_bug.cgi?id=458085

   Summary: Review Request: nes_ntsc - Provides a NES NTSC video
filtering library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://members.iinet.net.au/~timmsy/nes_ntsc/nes_ntsc.spec
SRPM URL: 
http://members.iinet.net.au/~timmsy/nes_ntsc/nes_ntsc-0.2.2-1.fc9.src.rpm

Description:
NES NTSC video filter library. Pixel artifacts and color mixing play an 
important role in NES games console graphics. Accepts pixels in native 6-bit
NES palette format, or a 9-bit format that includes the three color emphasis
bits in PPU register $2001. Can also output an RGB palette for use in a 
regular blitter

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 450243] Review Request: google-gadgets - Google Gadgets for Linux

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450243





--- Comment #10 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 09:59:44 EDT 
---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453465] Review Request: transbot - An irc bot designed to translate between languages on separate channels

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453465





--- Comment #13 from John McLean [EMAIL PROTECTED]  2008-08-06 10:05:20 EDT 
---
I've requested sponsorship.  My FAS name is mcleanj.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458090] New: Review Request: LuxRender - Lux Renderer, an unbiased rendering system

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: LuxRender - Lux Renderer, an unbiased rendering system

https://bugzilla.redhat.com/show_bug.cgi?id=458090

   Summary: Review Request: LuxRender - Lux Renderer, an unbiased
rendering system
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://kwizart.fedorapeople.org/SPECS/LuxRender.spec
SRPM URL:
http://kwizart.fedorapeople.org/SRPMS/LuxRender-0.5-1.fc9.kwizart.src.rpm
Description: Lux Renderer, an unbiased rendering 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457277] Review Request: pentaho-reporting-flow-engine - Reporting Engine

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457277


Colin Walters [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




--- Comment #1 from Colin Walters [EMAIL PROTECTED]  2008-08-06 10:05:09 EDT 
---
Took a brief look; need to do the build deps first though.

[+] source files match upstream
[+] package meets naming and versioning guidelines.
[+] specfile is properly named, is cleanly written and uses macros consistently
[+] dist tag is present.
[+] build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+] license field matches the actual license.
[+] license is open source-compatible.
[+] license text included in package.
[+] latest version is being packaged.
[+] BuildRequires are proper
[+] %clean is present.

-- TO BE REVIEWED

[?] package builds in koji
[?] package installs properly
[?] rpmlint is silent.
please fix.
[?] owns the directories it creates.
[?] doesn't own any directories it shouldn't.
[?] no duplicates in %files.
[?] file permissions are appropriate.
[?] code, not content.
[?] documentation is small, so no -docs subpackage is necessary.
[?] %docs are not necessary for the proper functioning of the package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=421241





--- Comment #49 from Alexander Kahl [EMAIL PROTECTED]  2008-08-06 10:11:22 
EDT ---
New Package CVS Request
===
Package Name:  php-ZendFramework
Short Description: Leading open-source PHP framework
Owners:akahl
Branches:  F-9
InitialCC: 
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458085] Review Request: nes_ntsc - Provides a NES NTSC video filtering library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458085





--- Comment #2 from David Timms [EMAIL PROTECTED]  2008-08-06 10:12:06 EDT ---
I plan on up/packaging upstreams other *_ntsc libs. The author mentions the
latest release include a source cleanup and refactoring, so that some files are
now identical between the 3x *_ntsc sources. Would it be acceptable/worth the
effort to create a single libntsc package, that includes all three upstream
source packages, with sub packages for -nes, -snes, -sms ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457038] Review Request: primer3 - PCR primer design tool

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457038





--- Comment #4 from Pierre-YvesChibon [EMAIL PROTECTED]  2008-08-06 10:16:05 
EDT ---
There are the new files
http://pingou.fedorapeople.org/RPMs/primer3.spec
http://pingou.fedorapeople.org/RPMs/primer3-1.1.4-2.fc9.src.rpm

%changelog
* Wed Aug 06 2008 pingou [EMAIL PROTECTED] 1.1.4-2
- Keep the timestamp in the README.txt
- Change the CFLAG for the compilation
- Remove BR perl

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453465] Review Request: transbot - An irc bot designed to translate between languages on separate channels

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453465


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #14 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 10:16:54 EDT 
---
Confirmed. Now I am sponsoring you. Please follow Join wiki again.

Removing NEEDSPONSOR.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 438750] Review Request: qtoctave - fronted for octave written using qt4 widgets

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=438750





--- Comment #30 from Chitlesh GOORAH [EMAIL PROTECTED]  2008-08-06 10:20:44 
EDT ---
I'll step in if the reporter is not willing to maintain this package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458085] Review Request: nes_ntsc - Provides a NES NTSC video filtering library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458085





--- Comment #1 from David Timms [EMAIL PROTECTED]  2008-08-06 10:11:39 EDT ---
Created an attachment (id=313566)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=313566)
diff between original dribble spec0.2.0 and this spec.

I request some assistance regarding lib sonames. This library has previously
been packaged and used by out of Fedora repo applications as
libnes_ntsc.so.0.2.0

Have I gone about the soname change in an effective way ?
Should I keep the original 0.2.0 soname instead ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=421241





--- Comment #50 from Jess Portnoy [EMAIL PROTECTED]  2008-08-06 10:22:12 EDT 
---
Hi Alex,

I'd love to be set CC. It can't hurt :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457035] Review Request: libproxy - A library handling all the details of proxy configuration

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457035





--- Comment #5 from Nicolas Chauvet (kwizart) [EMAIL PROTECTED]  2008-08-06 
10:33:17 EDT ---
New Package CVS Request
===
Package Name: libproxy
Short Description: A library handling all the details of proxy configuration
Owners: kwizart
Branches: F-8 F-9 EL-5
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454199] Review Request: pentaho-libxml - Namespace aware SAX-Parser utility library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454199


Colin Walters [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




--- Comment #1 from Colin Walters [EMAIL PROTECTED]  2008-08-06 10:55:15 EDT 
---
[+] source files match upstream
[+] package meets naming and versioning guidelines
 - Can you annotate the patch to say whether it is going to be upstreamed?  
 https://fedoraproject.org/wiki/Packaging/PatchUpstreamStatus 

[+] specfile is properly named, is cleanly written and uses macros consistently
[+] dist tag is present.
[+] build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+] license field matches the actual license.
[+] license is open source-compatible.
[+] license text included in package.
[+] latest version is being packaged (as far as I can tell)
[+] BuildRequires are proper
[+] %clean is present.
[+] package builds in koji
[+] package installs properly
[+] rpmlint is silent.
[+] owns the directories it creates.
[+] doesn't own any directories it shouldn't.
[+] no duplicates in %files.
[+] file permissions are appropriate.
[+] code, not content.
[+] -javadoc package exists and works
[+] %doc exists

Looks good to me.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457546] Review Request: perl-Net-RawIP - A Perl extension for manipulating raw ip packets using libpcap

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457546


Miloslav Trmac [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Miloslav Trmac [EMAIL PROTECTED]  2008-08-06 10:40:46 EDT 
---
Thanks for the 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457545] Review Request: perl-Net-Ping-External - Cross-platform interface to ICMP ping utilities

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457545





--- Comment #4 from Miloslav Trmac [EMAIL PROTECTED]  2008-08-06 10:50:17 EDT 
---
CVS admins: Can you help, please?

$ cvs -d :ext:[EMAIL PROTECTED]:/cvs/pkgs co perl-Net-Ping-External
cvs server: cannot find module `perl-Net-Ping-External' - ignored
cvs [checkout aborted]: cannot expand modules

(Checking out perl-Net-RawIP, which was added roughly at the same time, works
fine.)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 450371] Review Request: pokegen - Strategy/RPG game engine

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450371


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Comment #17 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 10:50:26 EDT 
---
For -12:

* Scriptlets
---
# env LANG=C rpm -Fvh *i386*rpm
Preparing...### [100%]
   1:pokegen### [100%]
/sbin/ldconfig: relative path `2' used to build cache
error: %post(pokegen-0.0.2-0.12.20080804svn236.fc10.i386) scriptlet failed,
exit status 1
---
  - %post -p /sbin/ldconfig can be used when only /sbin/ldconfig is called
on %post. If other commands are executed like this, don't use -p
/sbin/ldconfig
and simply use:
---
%post
/sbin/ldconfig
update-desktop-database  /dev/null || :
.
---

! rpmlint
  - rpmlint shows there are still some unused direct dependencies on
installed libraries.
You can check this also by
---
$ ldd -u /usr/lib/libpokebattle.so.0
Unused direct dependencies:

/usr/lib/libpokemod.so.0
/usr/lib/libphonon.so.4
/usr/lib/libkrosscore.so.4
/usr/lib/libQtGui.so.4
/usr/lib/libQtXml.so.4
/lib/libm.so.6
---
, for example. While this is _not_ a blocker, you may want to fix this.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457768] Review Request: xorg-x11-drv-r128 - r128 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457768


Lubomir Rintel [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458085] Review Request: nes_ntsc - Provides a NES NTSC video filtering library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458085


Andrea Musuruane [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #3 from Andrea Musuruane [EMAIL PROTECTED]  2008-08-06 11:39:12 
EDT ---
(In reply to comment #2)
 I plan on up/packaging upstreams other *_ntsc libs. The author mentions the
 latest release include a source cleanup and refactoring, so that some files 
 are
 now identical between the 3x *_ntsc sources. Would it be acceptable/worth the
 effort to create a single libntsc package, that includes all three upstream
 source packages, with sub packages for -nes, -snes, -sms ?

IMHO it is not worth the trouble and you could have a serious drawback. What if
upstream update just -sms? You should make a new release for all three - and
this is something unneeded by both end users (who would download a release for
-nes and -snes for nothing) and for infrastructure (who would build a release
for -nes and -snes for nothing).

Separate packages is the way to go if upstream have separate independent
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457768] Review Request: xorg-x11-drv-r128 - r128 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457768





--- Comment #1 from Adam Jackson [EMAIL PROTECTED]  2008-08-06 11:40:48 EDT 
---
Sorry, wrong SRPM URL.

http://ajax.fedorapeople.org/r128/xorg-x11-drv-r128-6.8.0-1.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454917


Mamoru Tasaka [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 Blocks|177841  |182235




--- Comment #10 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 11:30:10 EDT 
---
As debian says, currently I think that this is non-free.

The license of crc.c (and others):
http://packages.debian.org/changelogs/pool/non-free/f/figlet/figlet_2.2.2-1/figlet.copyright

The issue
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=274950
does not seem to have disappeared.

Blocking FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457767] Review Request: xorg-x11-drv-mach64 - mach64 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457767





--- Comment #1 from Adam Jackson [EMAIL PROTECTED]  2008-08-06 11:40:08 EDT 
---
Sorry, wrong SRPM URL.

SRPM URL:
http://ajax.fedorapeople.org/mach64/xorg-x11-drv-mach64-6.8.0-1.fc9..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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457767] Review Request: xorg-x11-drv-mach64 - mach64 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457767


Rakesh Pandit [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #2 from Rakesh Pandit [EMAIL PROTECTED]  2008-08-06 11:43:55 EDT 
---
Removed Extra dot ;-)
http://ajax.fedorapeople.org/mach64/xorg-x11-drv-mach64-6.8.0-1.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457926] python-wikimarkup - Python module to format text to Mediawiki syntax

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457926





--- Comment #2 from Soumya Kanti Chakraborty [EMAIL PROTECTED]  2008-08-06 
11:44:04 EDT ---
This is my First package and I am seeking for a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454667] Review Request: truecrypt - Free open-source disk encryption software

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454667


Tom spot Callaway [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




--- Comment #3 from Tom spot Callaway [EMAIL PROTECTED]  2008-08-06 
12:04:04 EDT ---
License as-is is non-free. Going to try to talk to upstream to see if they are
interested in resolving the issues (plural) with the license.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457767] Review Request: xorg-x11-drv-mach64 - mach64 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457767


Rakesh Pandit [EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 426985] Review Request: php-suhosin - Suhosin extension for the php language

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426985


Adam Tkac [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 426985] Review Request: php-suhosin - Suhosin extension for the php language

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426985





--- Comment #11 from Adam Tkac [EMAIL PROTECTED]  2008-08-06 12:21:01 EDT ---
I will take care about this review.

- spec looks fine for me
- mock build OK
- rpmlint doesn't print any error/warning

= Reviewed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454199] Review Request: pentaho-libxml - Namespace aware SAX-Parser utility library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454199





--- Comment #2 from Caolan McNamara [EMAIL PROTECTED]  2008-08-06 11:13:20 
EDT ---
New Package CVS Request
===
Package Name: pentaho-libxml
Short Description:  Namespace aware Java SAX-Parser utility library
Owners: caolanm
Branches:
InitialCC:
Cvsextras Commits: yes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 426985] Review Request: php-suhosin - Suhosin extension for the php language

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426985


Adam Tkac [EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #304019|application/octet-stream|text/plain
  mime type||




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454439] Review Request: R-nws - R functions for NetWorkSpaces and Sleigh

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454439


Pierre-YvesChibon [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454667] Review Request: truecrypt - Free open-source disk encryption software

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454667





--- Comment #4 from Levente Farkas [EMAIL PROTECTED]  2008-08-06 12:38:11 EDT 
---
can you tell me exactly which part of the license should have to change?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454917





--- Comment #11 from John McLean [EMAIL PROTECTED]  2008-08-06 12:41:00 EDT 
---
Figlet has been re-licensed (in 2005) under the Academic Free License, rather
than the Artistic license.  See comment #1.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458140] New: seahorse-plugins - Plugins and utilities for encryption in GNOME

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: seahorse-plugins - Plugins and utilities for encryption in GNOME

https://bugzilla.redhat.com/show_bug.cgi?id=458140

   Summary: seahorse-plugins - Plugins and utilities for
encryption in GNOME
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec: http://people.redhat.com/mclasen/seahorse-plugins.spec
SRPM: http://people.redhat.com/mclasen/seahorse-plugins-2.23.6-1.fc10.src.rpm

Description:
The plugins and utilities in this package integrate seahorse into
the GNOME desktop environment and allow users to perform operations
from applications like nautilus or gedit.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457768] Review Request: xorg-x11-drv-r128 - r128 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457768





--- Comment #2 from Lubomir Rintel [EMAIL PROTECTED]  2008-08-06 12:53:15 EDT 
---
Seems well.

APPROVED

You may want to remove an useless comment (# aclocal ; automake -a ; autoconf),
and clarify the origin of r128.xinf a bit before importing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458139] Review Request: ruby-pam - Ruby bindings for pam

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458139


Bryan Kearney [EMAIL PROTECTED] changed:

   What|Removed |Added

 Blocks||177841
Customer Facing||---




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457547] Review Request: mediascrapper - A script to scrap media files from different sites

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457547





--- Comment #5 from Kushal Das [EMAIL PROTECTED]  2008-08-06 13:03:28 EDT ---
(In reply to comment #4)
 (In reply to comment #3)
  Remove the sitearch macro definition and CFLAGS line. Those are not needed 
  for
  a noarch package. Include a copy of the license under a file called COPYING.
  The license of the source should match the tag in the spec file. 
  
  You might also want to move your project to fedorahosted.org and make the
  script more generic.
 
 Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec
 SRPM URL:
 http://kushal.fedorapeople.org/packages/mediascrapper-0.1-2.fc9.src.rpm
 
 I will move the project under fedorahosted. The upcoming release will have
 feature to download photos from flickr

Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec
SRPM URL:
http://kushal.fedorapeople.org/packages/mediascrapper-0.1-3.fc9.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454917





--- Comment #12 from Mamoru Tasaka [EMAIL PROTECTED]  2008-08-06 12:51:02 EDT 
---
No. 

You are just saying that the license text is changed from ASL to AFL.
Yes, actually with this license change figlet.c is relicensed under
from ASL to AFL

But I am referring to the license used in crc.c, inflate.c, and so on.
The license used in these codes are not ASL and not AFL. They are licensed
under what debian regards as non-free and I agree with this (i.e. these codes
are non-free).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457547] Review Request: mediascrapper - A script to scrap media files from different sites

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457547





--- Comment #4 from Kushal Das [EMAIL PROTECTED]  2008-08-06 12:53:55 EDT ---
(In reply to comment #3)
 Remove the sitearch macro definition and CFLAGS line. Those are not needed for
 a noarch package. Include a copy of the license under a file called COPYING.
 The license of the source should match the tag in the spec file. 
 
 You might also want to move your project to fedorahosted.org and make the
 script more generic.

Spec URL: http://kushal.fedorapeople.org/packages/mediascrapper.spec
SRPM URL:
http://kushal.fedorapeople.org/packages/mediascrapper-0.1-2.fc9.src.rpm

I will move the project under fedorahosted. The upcoming release will have
feature to download photos from flickr

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457340] Review Request: rome - Java tools for parsing, generating and publishing RSS and Atom feeds

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457340


Kevin Fenzi [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
Summary|Review Request: ROME  - |Review Request: rome  -
   |Java tools for parsing, |Java tools for parsing,
   |generating and publishing   |generating and publishing
   |RSS and Atom feeds  |RSS and Atom feeds




--- Comment #5 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:29:30 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454199] Review Request: pentaho-libxml - Namespace aware SAX-Parser utility library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454199





--- Comment #3 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:30:50 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457035] Review Request: libproxy - A library handling all the details of proxy configuration

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457035





--- Comment #6 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:33:21 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 455111] Review Request: python-webob - WSGI request and response object

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=455111





--- Comment #10 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:44:00 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457545] Review Request: perl-Net-Ping-External - Cross-platform interface to ICMP ping utilities

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457545





--- Comment #5 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:41:16 EDT ---
Sorry about that. 

Can you try again in about 20min or so? 
I think it's fixed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 453082] Review Request: python-dtopt - Add options to doctest examples while they are running

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453082





--- Comment #10 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:44:56 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457808] Review Request: gwibber - An open source microblogging client for GNOME developed with Python and GTK

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457808





--- Comment #3 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:42:21 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457263] Review Request: liblayout - positioning library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457263


Matthias Clasen [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
Customer Facing||---




--- Comment #1 from Matthias Clasen [EMAIL PROTECTED]  2008-08-06 13:43:29 
EDT ---
Upon first look (and studying the java packaging guidelines for the first
time...) it looks mostly ok (detailed review to follow). The one thing that I
found so far is:

If the number of provided JAR files exceeds two, you must place them into a
sub-directory.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=421241





--- Comment #51 from Kevin Fenzi [EMAIL PROTECTED]  2008-08-06 13:40:08 EDT 
---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457767] Review Request: xorg-x11-drv-mach64 - mach64 driver

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457767





--- Comment #3 from Rakesh Pandit [EMAIL PROTECTED]  2008-08-06 13:35:03 EDT 
---
APPROVED

Build on koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=762691

rpmlint -i xorg-x11-drv-mach64-6.8.0-1.fc9.src.rpm 
xorg-x11-drv-mach64.src: W: mixed-use-of-spaces-and-tabs (spaces: line 5, tab:
line 3)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance.  Use either spaces or tabs for indentation, not both.

Required:
[x] Name
[x] License
[x] Spec file is in American Eng and legible
[x] Build successfully
[x] BuildRequires 
[x] Duplicate files - nil
[NA] locale
[x] permissions -- okay
[x]  source link correct
[x] packaging guidlines
[x] removing .la 
[x] Buildroot correct
[x] owns every directory it creates
[x] file encoding - checked
[x] package has no dependency on files in %doc
[NA] GUI
[x] No dependencies outside FHS guidelines
[x] md5sum
Source from site: 6081b8fa50c689d51f85c2fbaf93867e
Source from srpm: 6081b8fa50c689d51f85c2fbaf93867e

Optional suggestions:
a. You may like to add some more info to description 
b. clean #{_mandir}/man4/mach64.4*
c. BuildRequires: automake autoconf libtool pkgconfig may be removed

Key NA = N/A, x = Check, ! = Problem, ? = Not evaluated

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457547] Review Request: mediascrapper - A script to scrap media files from different sites

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457547





--- Comment #6 from Rahul Sundaram [EMAIL PROTECTED]  2008-08-06 14:00:59 EDT 
---

Koji scratch build

http://koji.fedoraproject.org/koji/taskinfo?taskID=763547

Remove the note on the specific website.

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454199] Review Request: pentaho-libxml - Namespace aware SAX-Parser utility library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454199


Caolan McNamara [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457263] Review Request: liblayout - positioning library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457263


Bug 457263 depends on bug 454199, which changed state.

Bug 454199 Summary: Review Request: pentaho-libxml - Namespace aware SAX-Parser 
utility library
https://bugzilla.redhat.com/show_bug.cgi?id=454199

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457263] Review Request: liblayout - positioning library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457263





--- Comment #2 from Caolan McNamara [EMAIL PROTECTED]  2008-08-06 14:13:32 
EDT ---
It's just /usr/share/java/liblayout.jar despite the glob, so only one .jar

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458024] Review Request: sblim-sfcc - Small Footprint CIM Client Library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458024


Matt Domsch [EMAIL PROTECTED] changed:

   What|Removed |Added

  Group|dell|
 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458025] Review Request: wsmancli - Opensource Implementation of WS-Management - Command line Utility

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458025


Matt Domsch [EMAIL PROTECTED] changed:

   What|Removed |Added

  Group|dell|
 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458012] Review Request: openwsman - Opensource Implementation of WS-Management

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458012


Matt Domsch [EMAIL PROTECTED] changed:

   What|Removed |Added

  Group|dell|
 CC||[EMAIL PROTECTED]




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458140] seahorse-plugins - Plugins and utilities for encryption in GNOME

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458140


Colin Walters [EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #1 from Colin Walters [EMAIL PROTECTED]  2008-08-06 14:25:48 EDT 
---
Doesn't build for me; seems to install an epiphany extension that's not in
%files?

error: Installed (but unpackaged) file(s) found:
   /usr/lib64/epiphany/2.22/extensions/libseahorseextension.so
   /usr/lib64/epiphany/2.22/extensions/seahorse.ephy-extension

[+] source files match upstream
[+] package meets naming and versioning guidelines
[+] specfile is properly named, is cleanly written and uses macros consistently
[+] dist tag is present.
[+] build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+] license field matches the actual license.
[+] license is open source-compatible.
[+] license text included in package.
[+] latest version is being packaged
[+] BuildRequires are proper
[+] %clean is present.
[] package builds in koji
[] package installs properly
[] rpmlint is silent.
[] owns the directories it creates.
[] doesn't own any directories it shouldn't.
[] no duplicates in %files.
[] file permissions are appropriate.
[] code, not content.
[+] No doc subpackage needed
[+] %doc exists

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 454667] Review Request: truecrypt - Free open-source disk encryption software

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=454667





--- Comment #5 from Tom spot Callaway [EMAIL PROTECTED]  2008-08-06 
12:42:30 EDT ---
It is quite a bit of changes, not a small matter. I emailed this to upstream,
if you'd like a copy, I can email it to you as well.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 458140] seahorse-plugins - Plugins and utilities for encryption in GNOME

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458140





--- Comment #3 from Matthias Clasen [EMAIL PROTECTED]  2008-08-06 14:51:46 
EDT ---
Oh, sorry. I meant to investigate why it wouldn't build the ephy extension on
my system, but forgot.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 230142] Review Request: SBLIM megapackage

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=230142


Matt Domsch [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||[EMAIL PROTECTED]
 Resolution||NOTABUG




--- Comment #5 from Matt Domsch [EMAIL PROTECTED]  2008-08-06 14:56:03 EDT ---
Mark is no longer working on this.  Last review comment indicated it was a bad
idea to merge these.  Closing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Bug 457543] Review Request: perl-IPTables-Parse - A Perl extension for parsing iptables firewall rulesets

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457543


Miloslav Trmac [EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Miloslav Trmac [EMAIL PROTECTED]  2008-08-06 11:12:37 EDT 
---
Thanks for the 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review