[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

[bugzilla]

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:49 EST ---
* Well, still installing libpythonplugin.so is not good.

make[1]: Entering directory `/builddir/build/BUILD/PyQt-x11-gpl-4.2/designer'
install -m 755 -p "libpythonplugin.so"
"/usr/lib64/qt4/plugins/designer/libpythonplugin.so"
install: cannot create regular file
`/usr/lib64/qt4/plugins/designer/libpythonplugin.so': Permission denied
make[1]: [install_target] Error 1 (ignored)
cp -f -r /builddir/build/BUILD/PyQt-x11-gpl-4.2/designer/python
/usr/lib64/qt4/plugins/designer/

Here libpythonplugin.so is surely rebuilt, however make tries
to install the plugin directly under %_libdir/qt4/plugins/designer, 
which must be %buildroot%_libdir/qt4/plugins/designer/

* Still fails on ppc64
  http://koji.fedoraproject.org/koji/taskinfo?taskID=181782
  Please mark ppc64 as ExcludeArch.

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

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

[Bug 252133] Review Request: quicksynergy - GTK frontend for synergy

[bugzilla]

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

Summary: Review Request: quicksynergy - GTK frontend for synergy


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:53 EST ---
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, or are watching someone who is.

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

[Bug 317851] New: Review Request: lklug-fonts - Font package for Sinhala(si_LK) language

[bugzilla]

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




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

   Summary: Review Request: lklug-fonts - Font package for
Sinhala(si_LK) language
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
SRPM URL: 

Description: This is a font package for lklug fonts which support sinhala 
language of Srilanka. This package is being split up from fonts-sinhala to 
reflect the upstream package name.

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

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

[Bug 253781] Review Request: outerspace - client for 4x on-line strategy game

[bugzilla]

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

Summary: Review Request: outerspace - client for 4x on-line strategy game


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:52 EST ---
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, or are watching someone who is.

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

[Bug 280541] Review Request: setools - SELinux policy analysis tools

[bugzilla]

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

Summary: Review Request: setools - SELinux policy analysis tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:17 EST ---
Well,

* This package is okay
* Your pre-review seems good for initial comments.

---
This package (setools) is APPROVED by me
---

Please follow the procedure according to:
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)
Then I will sponsor you.

If you want to import this package into Fedora 7, you also have
to look at
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT
(after once you rebuilt this package on 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, or are watching someone who is.

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

[Bug 253549] Review Request: TzClock - GTK+ Time Zone Clock

[bugzilla]

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

Summary: Review Request: TzClock - GTK+ Time Zone Clock


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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

[Bug 313221] Review Request: qstars - A screensaver simulating planets and asteroids in space

[bugzilla]

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

Summary: Review Request: qstars - A screensaver simulating planets and 
asteroids in space


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:01 EST ---
(Resetting cvs flag)

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

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

[Bug 313221] Review Request: qstars - A screensaver simulating planets and asteroids in space

[bugzilla]

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

Summary: Review Request: qstars - A screensaver simulating planets and 
asteroids in space


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 02:00 EST ---
(In reply to comment #11)
> Spec URL: http://dribble.org.uk/reviews/qstars.spec
> SRPM URL: http://dribble.org.uk/reviews/qstars-0.4-3.src.rpm
> 
> Gnome support has been removed. I haven't bumped the EVR but can you give it a
> quick look to make sure its still OK. I guess the cvs flag got cleared because
> of additional comments.

Okay.


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

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

[Bug 253549] Review Request: TzClock - GTK+ Time Zone Clock

[bugzilla]

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

Summary: Review Request: TzClock - GTK+ Time Zone Clock


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 01:58 EST ---
Chris, thanks for your patience.

Please see .

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

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

[Bug 313221] Review Request: qstars - A screensaver simulating planets and asteroids in space

[bugzilla]

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

Summary: Review Request: qstars - A screensaver simulating planets and 
asteroids in space


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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

[Bug 226466] Merge Review: system-config-printer

[bugzilla]

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

Summary: Merge Review: system-config-printer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 01:37 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM but Not for RPM.
system-config-printer.i386: W: non-conffile-in-etc
/etc/xdg/autostart/redhat-print-applet.desktop
system-config-printer.i386: E: no-binary
These messages can be ignored for this package.
system-config-printer.i386: W: no-version-in-last-changelog
system-config-printer-libs.i386: W: no-version-in-last-changelog
==> These can be corrected at time of cvs import.
+ source files match upstream.
20b18338b46531b1b28602deb44aa07d  pycups-1.9.27.tar.bz2
a332f8598c0a4afcc72123856733e273  system-config-printer-0.7.74.4.tar.bz2
b265be98565333de000309e14deea408  system-config-printer.console
f5be82be51135cf46de5955695f67c0d  system-config-printer.pam
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ Compiler flags are honoured correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists but --libs exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Desktop files are handled correctly in Makefile only so no need to have
dekstop-file-install called in SPEC.
+ package system-config-printer-0.7.74.4-1.0.0.1.fc8 ->
  Provides: config(system-config-printer) = 0.7.74.4-1.0.0.1.fc8
desktop-printing = 0.20-7.fc7 system-config-printer-gui = 0.6.152
  Requires: /bin/sh /usr/bin/env /usr/bin/python config(system-config-printer) =
0.7.74.4-1.0.0.1.fc8 dbus-x11 desktop-file-utils >= 0.2.92 pirut pygobject2
pygtk2 >= 2.4.0 pygtk2-libglade system-config-printer-libs =
0.7.74.4-1.0.0.1.fc8 usermode >= 1.37
+ package  system-config-printer-libs-0.7.74.4-1.0.0.1.fc8 ->
  Provides: config(system-config-printer-libs) = 0.7.74.4-1.0.0.1.fc8 cups.so
pycups = 1.9.27
  Requires: /usr/bin/env config(system-config-printer-libs) =
0.7.74.4-1.0.0.1.fc8 foomatic libc.so.6 libc.so.6(GLIBC_2.0)
libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.2)
libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libcups.so.2
libpthread.so.0 libpthread.so.0(GLIBC_2.0) libpython2.5.so.1.0 python
python(abi) = 2.5 rtld(GNU_HASH)
+ GUI app.
APPROVED.


Remember->
rpmlint complained some more extra messages now
system-config-printer.i386: W: no-version-in-last-changelog
system-config-printer-libs.i386: W: no-version-in-last-changelog
==> These can be corrected at time of cvs import.


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

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

[Bug 226360] Merge Review: readahead

[bugzilla]

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

Summary: Merge Review: readahead


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 01:10 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM but NOT for RPM.
  readahead.i386: W: non-conffile-in-etc /etc/readahead.d/default.early
  readahead.i386: W: non-conffile-in-etc /etc/readahead.d/default.later
  readahead.i386: W: service-default-enabled /etc/rc.d/init.d/readahead_later
  readahead.i386: W: service-default-enabled /etc/rc.d/init.d/readahead_later
  readahead.i386: W: service-default-enabled /etc/rc.d/init.d/readahead_early
  readahead.i386: W: service-default-enabled /etc/rc.d/init.d/readahead_early
  These can be ignored for this package.
+ source files match upstream.
b1b087681c06b66a4e4c7e944336db  readahead-1.4.2.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ Compiler flags are honoured correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ package  readahead-1.4.2-2.fc8 ->
  Provides: config(readahead) = 1:1.4.2-2.fc8
  Requires: /bin/bash /bin/gawk /bin/sh config(readahead) = 1:1.4.2-2.fc8
libaudit.so.0 libauparse.so.0 libblkid.so.1 libc.so.6 libc.so.6(GLIBC_2.0)
libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.2)
libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libext2fs.so.2
procps rtld(GNU_HASH)
+ Not a GUI app.

APPROVED.


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

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

[Bug 226360] Merge Review: readahead

[bugzilla]

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

Summary: Merge Review: readahead


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 01:13 EST ---
oops forgot to edit review template.
* initscripts scriptlets are used in this package.

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

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

[Bug 257061] Review Request: osslsigncode - Tool for Authenticode signing of EXE/CAB files

[bugzilla]

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

Summary: Review Request: osslsigncode - Tool for Authenticode signing of 
EXE/CAB files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 00:53 EST ---
I don't think this is really the proper forum for doing a patent review; all we
can do is review the packaging.  If you want to pass this to Red Hat's lawyers
before importing, that's up to you.  In the meantime, this is a clean package;
it builds fine and rpmlint is silent.  There's not much to it, really.

I guess you can handle not using the dist tag; I don't expect that this package
will need to change often anyway.

* source files match upstream:
   5cd55fa974b06bf89ee128137a969e58a8c6ea1df20b100ddb6b23a58682bec8  
   osslsigncode-1.2.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   osslsigncode = 1.2-2
  =
   libcrypto.so.6()(64bit)
   libcurl.so.4()(64bit)
   libssl.so.6()(64bit)
   libz.so.1()(64bit)
* %check is not present; no test suite upstream.  I haven't a clue how to test 
   this package.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

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, or are watching someone who is.

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

[Bug 257381] Review Request: python-sqlalchemy0.3 - Modular and flexible ORM library for python

[bugzilla]

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

Summary: Review Request: python-sqlalchemy0.3 - Modular and flexible ORM 
library for python


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-04 00:38 EST ---
Looks good; no more spew from rpmlint and the tests run fine:
  Ran 700 tests in 32.227s

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, or are watching someone who is.

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

[Bug 315351] Review Request: superiotool - Simple program for detecting Super I/O on your mainboard

[bugzilla]

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

Summary: Review Request: superiotool - Simple program for detecting Super I/O 
on your mainboard


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:57 EST ---
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, or are watching someone who is.

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

[Bug 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities

[bugzilla]

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

Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:51 EST ---
Yeah, unless you want a F-8 branch now so you can keep a stable version for F-8
and move forward with a test version in devel/rawhide, there is no need to
request an F-8 branch now. 

All current devel packages will be branched for F-8 before release. 

Please reset the fedora-cvs flag if you want such a branch now. 

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

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

[Bug 215865] Review Request: Zim - Desktop wiki & notekeeper

[bugzilla]

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

Summary: Review Request: Zim - Desktop wiki & notekeeper
Alias: Zim

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:53 EST ---
cvs done.

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

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

[Bug 241553] Review Request: safekeep - simple, centralized configuration for rdiff-backup

[bugzilla]

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

Summary: Review Request: safekeep -  simple, centralized configuration for 
rdiff-backup


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:50 EST ---
Folks, any news on this item? It's getting a bit long in the tooth...

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

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

[Bug 302361] Review Request: freecol - The FreeCol multi-player strategy game

[bugzilla]

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

Summary: Review Request: freecol - The FreeCol multi-player strategy game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:49 EST ---
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, or are watching someone who is.

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

[Bug 313481] Review Request: aget - Console download accelerator

[bugzilla]

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

Summary: Review Request: aget - Console download accelerator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:45 EST ---
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, or are watching someone who is.

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

[Bug 313271] Review Request: perl-Net-IRC - Perl interface to the Internet Relay Chat protocol

[bugzilla]

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

Summary: Review Request: perl-Net-IRC - Perl interface to the Internet Relay 
Chat protocol
Alias: perl-Net-IRC

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:42 EST ---
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, or are watching someone who is.

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

[Bug 290431] Review Request: TnL - Thunder & Lightning - A futuristic action flight simulator game

[bugzilla]

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

Summary: Review Request: TnL - Thunder & Lightning - A futuristic action flight 
simulator game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:40 EST ---
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, or are watching someone who is.

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

[Bug 285561] Review Request: wqy-unibit-fonts - a dual-width bitmap font for maximum unicode coverage

[bugzilla]

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

Summary: Review Request: wqy-unibit-fonts - a dual-width bitmap font for 
maximum unicode coverage


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:39 EST ---
Thanks!

You don't need to worry about CVE for this update -
it is for security updates.

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

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

[Bug 291661] Review Request: asylum - SDL port of the game Asylum, originally for the Archimedes

[bugzilla]

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

Summary: Review Request: asylum - SDL port of the game Asylum, originally for 
the Archimedes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:35 EST ---
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, or are watching someone who is.

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

[Bug 285561] Review Request: wqy-unibit-fonts - a dual-width bitmap font for maximum unicode coverage

[bugzilla]

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

Summary: Review Request: wqy-unibit-fonts - a dual-width bitmap font for 
maximum unicode coverage


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:36 EST ---
sorry for the delay. 

Package was imported to devel/F-7/F-6 and built with no problems. I also pushed
the package for testing on bodhi system; if it turns out no other problem, I
will push it to release.

one side question: where can I find the "CVE" string for this package? it was
asked on Bodhi new update page, I can not find it on
http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAF

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

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

[Bug 313371] Review Request: tempest - An OpenGL screensaver for KDE

[bugzilla]

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

Summary: Review Request: tempest - An OpenGL screensaver for KDE


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:37 EST ---
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, or are watching someone who is.

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

[Bug 314911] Review Request: tuncfg - Userspace TUN/TAP configuration utility

[bugzilla]

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

Summary: Review Request: tuncfg - Userspace TUN/TAP configuration utility


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:29 EST ---
The tun package contains Linux, FreeBSD and Solaris kernel modules, and a
Linux userland configuration utility called 'tuncfg'.  I am not packaging
any of the kernel modules (d'oh), so I have named the package after the
only piece of the upstream tun package that I _am_ packaging, namely 'tuncfg'.

The Release field doesn't contain %{?dist} because tuncfg doesn't depend on
anything, so we can just use the very same binary package for every distro
release.

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

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

[Bug 299271] Review Request: nafees-web-naskh-fonts - Nafees Web font for writing Urdu in the Naskh script

[bugzilla]

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

Summary: Review Request: nafees-web-naskh-fonts - Nafees Web font for writing 
Urdu in the Naskh script


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 22:32 EST ---
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, or are watching someone who is.

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

[Bug 317321] Review Request: ladspa-tap-plugins - Tom's Audio Processing plugin

[bugzilla]

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

Summary: Review Request: ladspa-tap-plugins - Tom's Audio Processing plugin


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 21:32 EST ---
Like other ladspa plugins did you miss in SPEC to use system ladspa.h 
==
# we want to use the system ladspa.h
rm ladspa.h
===

Change defattr to %defattr(-,root,root,-)



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

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

[Bug 285561] Review Request: wqy-unibit-fonts - a dual-width bitmap font for maximum unicode coverage

[bugzilla]

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

Summary: Review Request: wqy-unibit-fonts - a dual-width bitmap font for 
maximum unicode coverage


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 21:28 EST ---
Qianqian, please import and build when you're ready.

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

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

[Bug 215865] Review Request: Zim - Desktop wiki & notekeeper

[bugzilla]

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

Summary: Review Request: Zim - Desktop wiki & notekeeper
Alias: Zim

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




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

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

[Bug 215865] Review Request: Zim - Desktop wiki & notekeeper

[bugzilla]

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

Summary: Review Request: Zim - Desktop wiki & notekeeper
Alias: Zim

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 20:45 EST ---
Please branch for F-8 :)

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

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

[Bug 293971] Review Request: gurlchecker - A Gnome link validity checker

[bugzilla]

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

Summary: Review Request: gurlchecker - A Gnome link validity checker
Alias: gurlchecker

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 20:40 EST ---
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, or are watching someone who is.

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

[Bug 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities

[bugzilla]

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

Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:52 EST ---
Mike, as I understand it, ecryptfs-utils is already in F8; the request above
will get you a branch specifically for F8 rather than leaving it in devel (which
eventually will become F9 after everything gets cloned off to F8)

[EMAIL PROTECTED] v4l-dvb]# cat /etc/redhat-release 
Fedora release 7.91 (Rawhide)
[EMAIL PROTECTED] v4l-dvb]# rpm -q ecryptfs-utils
ecryptfs-utils-18-1.fc8

>From jkeating's fedora-devel note today, "If a package isn't
branched yet, builds from devel/ continue to go to dist-f8."

-Eric

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

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

[Bug 226639] Merge Review: xorg-x11-fonts

[bugzilla]

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

Summary: Merge Review: xorg-x11-fonts


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||317641




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

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

[Bug 226639] Merge Review: xorg-x11-fonts

[bugzilla]

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

Summary: Merge Review: xorg-x11-fonts


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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

[Bug 291661] Review Request: asylum - SDL port of the game Asylum, originally for the Archimedes

[bugzilla]

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

Summary: Review Request: asylum - SDL port of the game Asylum, originally for 
the Archimedes


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:40 EST ---
Thanks very much 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, or are watching someone who is.

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

[Bug 291661] Review Request: asylum - SDL port of the game Asylum, originally for the Archimedes

[bugzilla]

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

Summary: Review Request: asylum - SDL port of the game Asylum, originally for 
the Archimedes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:42 EST ---
New Package CVS Request
===
Package Name: asylum
Short Description: SDL port of the game Asylum, originally for the Archimedes
Owners: oddsocks
Branches: FC6 F7
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, or are watching someone who is.

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

[Bug 257381] Review Request: python-sqlalchemy0.3 - Modular and flexible ORM library for python

[bugzilla]

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

Summary: Review Request: python-sqlalchemy0.3 - Modular and flexible ORM 
library for python


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:25 EST ---
Spec URL: http://toshio.fedorapeople.org/packages/python-sqlalchemy0.3.spec
SRPM URL:
http://toshio.fedorapeople.org/packages/python-sqlalchemy0.3-0.3.10-4.fc7.src.rpm

> rpmlint spews a bunch of complaints because all of the .py* files are
> executable.  Is there some reason they need to be?
> 
Nope.  I seem to be running across a setuptools bug. Patch sent to the
setuptools mailing list and workaround added to this package.

> There's a test suite in the package and it seems to be runnable:
> 
> %check
> export PYTHONPATH=./test
> python test/alltests.py
> 
> Any reason why it isn't called?
> 
Now calling the testsuite.  Thanks!

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

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

[Bug 316971] Review Request: centerim - Text mode menu- and window-driven IM

[bugzilla]

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

Summary: Review Request: centerim - Text mode menu- and window-driven IM
Alias: centerim-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:16 EST ---
There's no need to use the old FE-NEW/FE-REVIEW/FE-ACCEPT blockers any longer;
we have the fedora-review flag for that now.

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

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

[Bug 307891] libvpd - C++ library for system vpd access

[bugzilla]

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

Summary: libvpd - C++ library for system vpd access


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 19:05 EST ---
Not sure if you were waiting for an answer to that.  If so, yes I need 
sponsorship.

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

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

[Bug 317581] New: Review Request: ruby-rpm - Ruby bindings for RPM

[bugzilla]

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




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

   Summary: Review Request: ruby-rpm - Ruby bindings for RPM
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.redhat.com/dlutter/yum/spec/ruby-rpm.spec
SRPM URL: http://people.redhat.com/dlutter/yum/SRPMS/ruby-rpm-1.2.3-1.src.rpm
Description: Bindings for accessing RPM packages and databases from Ruby

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

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

[Bug 291661] Review Request: asylum - SDL port of the game Asylum, originally for the Archimedes

[bugzilla]

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

Summary: Review Request: asylum - SDL port of the game Asylum, originally for 
the Archimedes


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:50 EST ---
well as I said, both are style choices.  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, or are watching someone who is.

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

[Bug 256481] Review Request: gwave - Analog Waveform Viewing Environment

[bugzilla]

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

Summary: Review Request: gwave - Analog Waveform Viewing Environment


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:45 EST ---
Starting...

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

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

[Bug 256481] Review Request: gwave - Analog Waveform Viewing Environment

[bugzilla]

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

Summary: Review Request: gwave - Analog Waveform Viewing Environment


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


Bug 256481 depends on bug 254211, which changed state.

Bug 254211 Summary: Review Request: guile-gnome-platform - Guile wrapper 
collection for the GNOME library stack
https://bugzilla.redhat.com/show_bug.cgi?id=254211

   What|Old Value   |New Value

 Status|ASSIGNED|NEEDINFO
 Status|NEEDINFO|ASSIGNED
 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, or are watching someone who is.

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

[Bug 254211] Review Request: guile-gnome-platform - Guile wrapper collection for the GNOME library stack

[bugzilla]

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

Summary: Review Request: guile-gnome-platform - Guile wrapper collection for 
the GNOME library stack


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


[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, or are watching someone who is.

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

[Bug 254211] Review Request: guile-gnome-platform - Guile wrapper collection for the GNOME library stack

[bugzilla]

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

Summary: Review Request: guile-gnome-platform - Guile wrapper collection for 
the GNOME library stack


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:39 EST ---
imported and built.
thx all

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

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

[Bug 225679] Merge Review: dejagnu

[bugzilla]

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

Summary: Merge Review: dejagnu


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:35 EST ---
I committed version cleaned up per your comments.

The testsuite problem is twofold: first, dejagnu self-testing module was
handling C++ streams erroneously, which was resolved with a simple patch;
second, mock doesn't give terminal to testsuite when it needs one.  This was
resolved with an ugly convoluted setup involving screen and temporary file.  I'd
like you to re-review the package again if possible, and either recommend better
alternative, or ack current approach.  Thanks.

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

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

[Bug 290431] Review Request: TnL - Thunder & Lightning - A futuristic action flight simulator game

[bugzilla]

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

Summary: Review Request: TnL - Thunder & Lightning - A futuristic action flight 
simulator game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:30 EST ---
Thanks!

New Package CVS Request
===
Package Name:  TnL
Short Description: Thunder & Lightning - A futuristic action flight simulator 
game
Owners:jwrdegoede
Branches:  F-7 devel
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, or are watching someone who is.

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

[Bug 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities

[bugzilla]

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

Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:20 EST ---
Package Change Request
==
Package Name: ecryptfs-utils
New Branches: F-8

I request that ecryptfs-utils be included in Fedora 8. This supports the
eCryptfs filesystem in kernel versions 2.6.19 and higher.

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

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

[Bug 290431] Review Request: TnL - Thunder & Lightning - A futuristic action flight simulator game

[bugzilla]

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

Summary: Review Request: TnL - Thunder & Lightning - A futuristic action flight 
simulator game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:20 EST ---
(In reply to comment #3)

> SRPM URL: http://people.atrpms.net/~hdegoede/TnL-070909-1.fc8.src.rpm

The URL is incorrect but I pulled down what I assume to be the right RPM. The
buildroot hasn't been changed, please fix that before import.

Package APPROVED!


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

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

[Bug 190040] Review Request: hydrogen - Advanced drum machine

[bugzilla]

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

Summary: Review Request: hydrogen - Advanced drum machine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 18:06 EST ---
I noticed that this seems kinda stuck and I would like to offer to help.

The patch from comment #21 looks good, I haven't tried it, but it should work,
If it doesn't then the used define probably isn't getting set correctly.


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

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

[Bug 285331] Review Request: jack-rack - Stereo LADSPA effects rack for the JACK audio API

[bugzilla]

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

Summary: Review Request: jack-rack - Stereo LADSPA effects rack for the JACK 
audio API


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:58 EST ---
okay, I've done a full review:

Must Fix:
-

* Doesn't build on devel (as already discussed by mail) I fixed thi by adding 
  this line to %setup:
  sed -i 's/-DGTK_DISABLE_DEPRECATED=1//g' src/Makefile.*
* Set vendor to Fedora when installing .desktop file
* jack-rack.x86_64: W: file-not-utf8 /usr/share/doc/jack-rack-1.4.6/NEWS
* License: should be GPLv2+
* .desktop file Categories, drop the Utility, what did CCRMA have here? If they
  had some special X-Foo entries, please preserve these.

Should Fix:

* Drop the "Encoding=.." from the ,desktop, this is deprecated


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

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

[Bug 313221] Review Request: qstars - A screensaver simulating planets and asteroids in space

[bugzilla]

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

Summary: Review Request: qstars - A screensaver simulating planets and 
asteroids in space


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:51 EST ---
Spec URL: http://dribble.org.uk/reviews/qstars.spec
SRPM URL: http://dribble.org.uk/reviews/qstars-0.4-3.src.rpm

Gnome support has been removed. I haven't bumped the EVR but can you give it a
quick look to make sure its still OK. I guess the cvs flag got cleared because
of additional comments.

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

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

[Bug 291661] Review Request: asylum - SDL port of the game Asylum, originally for the Archimedes

[bugzilla]

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

Summary: Review Request: asylum - SDL port of the game Asylum, originally for 
the Archimedes


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:45 EST ---
(In reply to comment #3)

> Move %doc to top of %files to aid readability

I prefer %doc near the bottom, for me thats more readable. It suits the logical
order that I write specs. Installed files, doc files, ghosted files. Unless
there's a requirement to change it, I'd rather leave as-is.


> Not a blocker but definitely a suggestion.
> Some path details are "hardcoded" into the Makefile and makefile patch.
> While the location of /usr/bin /usr/share/asylum and /var/games/asylum are not
> likely to change anytime soon it would be more visible to make these changes
> with sed in the specfile making use of the available macros.


Well as I need to patch the Makefile anyway, I don't see a great deal of
advantage, otherwise it would make sense. It just turns it from a 1 stage
process to a 2 stage process. 

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

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

[Bug 229391] Review Request: system-config-kdump - graphical tool for configuring kernel crash dumps

[bugzilla]

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

Summary: Review Request: system-config-kdump - graphical tool for configuring 
kernel crash dumps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |MODIFIED




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

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

[Bug 223490] Review Request: alex - The lexer generator for Haskell

[bugzilla]

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

Summary: Review Request: alex - The lexer generator for Haskell
Alias: alex

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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:17 EST ---
alex-2.1.0-5.fc7 has been pushed to the Fedora 7 testing repository.  If 
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update alex'

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

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

[Bug 253134] Review Request: jlint - Java program checker

[bugzilla]

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

Summary: Review Request: jlint - Java program checker


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:11 EST ---
jlint-1.23-1.fc7 has been pushed to the Fedora 7 stable repository.  If 
problems still persist, please make note of it in this bug report.

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

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

[Bug 253134] Review Request: jlint - Java program checker

[bugzilla]

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

Summary: Review Request: jlint - Java program checker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |ERRATA
   Fixed In Version||1.23-1.fc7




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

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

[Bug 229391] Review Request: system-config-kdump - graphical tool for configuring kernel crash dumps

[bugzilla]

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

Summary: Review Request: system-config-kdump - graphical tool for configuring 
kernel crash dumps


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:12 EST ---
License: GPL
This is no longer valid. For this package the correct one is:
License: GPL2+

Everything else seems fine to me. I'll approve this one you change the license
tag. Thanks for the package!

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

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

[Bug 316971] Review Request: centerim - Text mode menu- and window-driven IM

[bugzilla]

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

Summary: Review Request: centerim - Text mode menu- and window-driven IM
Alias: centerim-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:08 EST ---
The BuildRoot tag is not right.
I'll change it to
  BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
in the next spin


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

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

[Bug 231861] Review Request: cyrus-imapd - high-performance mail server (IMAP, POP3, ...)

[bugzilla]

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

Summary: Review Request: cyrus-imapd - high-performance mail server (IMAP, 
POP3, ...)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 17:03 EST ---
Ping on this.

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

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

[Bug 287801] Review Request: Sirius - Othello game for GNOME

[bugzilla]

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

Summary: Review Request: Sirius - Othello game for GNOME
Alias: sirius

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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 16:57 EST ---
(In reply to comment #20)

Issues resolved:

> For -9:
> 
> * Tarball itself

The original tarball sirius-0.8.0.tar.gz is restored (md5sum:
e01d079ca62aa3cc85f542bb5e9389a5). Also I have added a sirius-0.8.0-docs.patch
to patch the required changes.

> * Fedora specific compilation flags
>   - Fedora specific compilation flags are not honored.
> You can check the fedora compilation flags by
> `rpm --eval %opflags
> 
> For this package, configure.in contains unneeded line
> -
> CFLAGS="-g -O3"
> -
> This overrides CFLAGS value. To pass correct cflags for
> this package, 
> -
> make %{?_smp_mflags} CFLAGS="%{optflags}"
> -
> is needed (i.e. again add CFLAGS option).

Added the CFLAGS option again to make tag for that useless line in configure.in.

The updated links are:
Spec URL: http://makghosh.fedorapeople.org/SPECS/sirius.spec
SRPM URL: http://makghosh.fedorapeople.org/SRPMS/sirius-0.8.0-10.fc7.src.rpm

The Koji scratch build is also successful:
http://koji.fedoraproject.org/koji/taskinfo?taskID=181839

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

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

[Bug 285331] Review Request: jack-rack - Stereo LADSPA effects rack for the JACK audio API

[bugzilla]

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

Summary: Review Request: jack-rack - Stereo LADSPA effects rack for the JACK 
audio API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 16:48 EST ---
Ok, bug 232358 isn't going anywhere fast, so I've closed it, and I'll review 
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, or are watching someone who is.

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

[Bug 232358] Review Request: jack-rack - Stereo LADSPA effects rack for the JACK audio API

[bugzilla]

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

Summary: Review Request: jack-rack - Stereo LADSPA effects rack for the JACK 
audio API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||201449
  nThis||
 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 16:47 EST ---
Anthony, I understand / guess that you are awefully busy with other work. Since
this bug isn't going anywhere fast, I'm closing it as a dead review and moving
on with the new review request posted in bug 285331. I'll add you to the CC
there, please drop a note there if you want to co-maintain this, then you can be
added to the owners list when filing the CVS admin request for the CVS branch
creation.


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

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

[Bug 317321] New: Review Request: ladspa-tap-plugins - Tom's Audio Processing plugin

[bugzilla]

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




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

   Summary: Review Request: ladspa-tap-plugins - Tom's Audio
Processing plugin
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.atrpms.net/~hdegoede/ladspa-tap-plugins.spec
SRPM URL: 
http://people.atrpms.net/~hdegoede/ladspa-tap-plugins-0.7.0-3.fc8.src.rpm
Description:
TAP-plugins is short for Tom's Audio Processing plugins. It is a bunch
of LADSPA plugins for digital audio processing, intended for use in a
professional DAW environment such as Ardour.

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

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

[Bug 316141] Review Request: gridengine - Sun Grid Engine - Distributed Computing Management software

[bugzilla]

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

Summary: Review Request: gridengine - Sun Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 16:24 EST ---
Current qmaster install intructions:

cd /usr/share/gridengine
edit my_configuration.conf and and modify the ADMIN_HOST_LIST, SUBMIT_HOST_LIST,
and EXEC_HOST_LIST as needed.
./install_qmaster -auto my_configuration.conf
chkconfig sgemaster on
service sgemaster start

On exec hosts, a simple:

chkconfig sgeexecd on
service sgeexecd start

should do the trick.  Probably need to add this to /etc/services:

sge_qmaster 6444/tcp   Grid Engine Qmaster Service
sge_qmaster 6444/udp   Grid Engine Qmaster Service
sge_execd   6445/tcp   Grid Engine Execution Service
sge_execd   6445/udp   Grid Engine Execution Service


You'll need to source /etc/profile.d/sge.sh or /etc/profile.d/sge.csh or get a
new shell after installing the packages to set SGE_ROOT so the commands work.

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

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

[Bug 245826] Review Request: pida - A Python IDE written in 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 report.

Summary: Review Request: pida - A Python IDE written in Python and GTK


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 15:59 EST ---
Spec: http://rishi.fedorapeople.org/pida.spec
SRPM: http://rishi.fedorapeople.org/pida-0.5.1-3.fc7.src.rpm

I have added a comment regarding the rpmlint error, and also mentioned emacs and
vim-X11 as Requires. These are needed since the first time PIDA is run, it asks
the user to select one of these two editors. Although it runs fine with GVim, it
does not work well with the Emacs shipping with Fedora.

Please advise.

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

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

[Bug 231809] Review Request: xmlrpc-epi - An implementation of the xmlrpc protocol in C

[bugzilla]

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

Summary: Review Request: xmlrpc-epi - An implementation of the xmlrpc protocol 
in C
Alias: xmlrpc-epi

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED])   |




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 15:09 EST ---
It's needed for Second Life. I'm currently concentrating on OpenJPEG, but I've
been busy with various RL issues. I'll get back to it... eventually. :P

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

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

[Bug 257381] Review Request: python-sqlalchemy0.3 - Modular and flexible ORM library for python

[bugzilla]

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

Summary: Review Request: python-sqlalchemy0.3 - Modular and flexible ORM 
library for python


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 14:58 EST ---
rpmlint spews a bunch of complaints because all of the .py* files are
executable.  Is there some reason they need to be?

There's a test suite in the package and it seems to be runnable:

%check
export PYTHONPATH=./test
python test/alltests.py

Any reason why it isn't called?

* source files match upstream:
   49b75b85e3691e6e6a38d6372d0b7153efa929cf5df31abf1b612574c1b062f6  
   SQLAlchemy-0.3.10.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
X rpmlint has complaints
* final provides and requires are sane:
  python-sqlalchemy0.3-0.3.10-3.fc8.noarch.rpm
   python-sqlalchemy0.3 = 0.3.10-3.fc8
  =
   python(abi) = 2.5
   python-setuptools >= 0.6c3
* %check is not present; no test suite upstream.

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
X file permissions are all executable?
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

Extra Eggs Guideline checks:
* egg is built from source
* egg doesn't download any dependencies during the build
* EGG-INFO directory is included in package
* compat package; easy_install -m is called in %install.

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

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

[Bug 305711] Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM theme

[bugzilla]

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

Summary: Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM 
theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 14:26 EST ---
I meant access to upstream too (the "themes" hosted projects)

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

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

[Bug 257381] Review Request: python-sqlalchemy0.3 - Modular and flexible ORM library for python

[bugzilla]

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

Summary: Review Request: python-sqlalchemy0.3 - Modular and flexible ORM 
library for python


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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

[Bug 305711] Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM theme

[bugzilla]

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

Summary: Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM 
theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 14:27 EST ---
> via bodhi
Via pkgdb, you mean, right?

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

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

[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

[bugzilla]

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 14:14 EST ---
Assigning to me.

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

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

[Bug 305711] Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM theme

[bugzilla]

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

Summary: Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM 
theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 14:13 EST ---
probabably, sure.  once pkg is imported, we (Kevin, Than, me) can/will request
it via bodhi.

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

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

[Bug 241082] Review Request: R-tkWidgets-1.14.0 - Widgets to provide user interfaces

[bugzilla]

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

Summary: Review Request: R-tkWidgets-1.14.0 - Widgets to provide user interfaces


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:57 EST ---
> You have them as BuildRequires but not Requires

My mistakes, I forgot to add them at their place...

There is the new SPEC and SRPM, however I still have no answer from either the
maintainer of this package or the one of maanova...
I have kept the license as LGPL for now..

SPEC:
http://www.pingoured.fr/public/RPM/R-tkWidgets/R-tkWidgets.spec
SRPM:
http://www.pingoured.fr/public/RPM/R-tkWidgets/R-tkWidgets-1.14.0-6.fc6.src.rpm

Thanks
Regards

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

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

[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

[bugzilla]

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:24 EST ---
OK, patched that (similar prob as with the sip bug, dev's *assuming* python
installed into prefix/lib):

Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SOURCES/PyQt4/PyQt4.spec
SRPM URL:
http://kde-redhat.unl.edu/apt/kde-redhat/SOURCES/PyQt4/PyQt4-4.2-6.src.rpm

%changelog
* Wed Oct 03 2007 Rex Dieter  4.2-6
- 64bit QtDesigner patch


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

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

[Bug 302361] Review Request: freecol - The FreeCol multi-player strategy game

[bugzilla]

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

Summary: Review Request: freecol - The FreeCol multi-player strategy game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:20 EST ---
Thanks!

New Package CVS Request
===
Package Name:  freecol
Short Description: The FreeCol multi-player strategy game
Owners:jwrdegoede
Branches:  devel
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, or are watching someone who is.

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

[Bug 316971] Review Request: centerim - Text mode menu- and window-driven IM

[bugzilla]

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

Summary: Review Request: centerim - Text mode menu- and window-driven IM
Alias: centerim-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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

[Bug 314911] Review Request: tuncfg - Userspace TUN/TAP configuration utility

[bugzilla]

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

Summary: Review Request: tuncfg - Userspace TUN/TAP configuration utility


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:15 EST ---
I noticed that the %dist macro isn't being used in the revision, and the package
name doesn't match the upstream (tuncfg vs. tun), please see
http://fedoraproject.org/wiki/Packaging/NamingGuidelines which has info on both
of these points.

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

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

[Bug 313361] Review Request: python-zsi - Libraries for developing web services in python

[bugzilla]

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

Summary: Review Request: python-zsi - Libraries for developing web services in 
python


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:17 EST ---
also I don't think this follows the naming conventions in
http://fedoraproject.org/wiki/Packaging/NamingGuidelines

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

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

[Bug 273301] Review Request: perl-Class-CSV - Class based CSV parser/writer

[bugzilla]

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

Summary: Review Request: perl-Class-CSV - Class based CSV parser/writer


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:14 EST ---
Package builds fine, rpmlint is clean.

Actually, I'm a bit surprised that rpmlint doesn't complain about the duplicated
dependency for perl(Text::CSV_XS).  RPM finds one unversioned dependency itself,
while you specify a versioned one.  Frankly I'm not really sure how much of a
problem this is.  I know the same situation with Provides: is rather bad.

Honestly I'd just drop the versioned dependency  The oldest supported Fedora
release has always had the required version of Test::CSV_XS so there shouldn't
be any need for the versioned dependency.

Oddly, RPM doesn't automatically find any dependency on Class:Accessor, but it's
dependency detection is far from perfect.

Review:
* source files match upstream:
   bce148284947a2cf7375eac13260fbfe23cb6b8c215970e3af7049b36de5e354   
   Class-CSV-1.03.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint is silent.
X final provides and requires include a duplicate dependency:
   per(Class::CSV)
   perl(Class::CSV::Base) = 1.3
   perl(Class::CSV::CSV_XS_Options)
   perl(Class::CSV::Line)
   perl-Class-CSV = 1.03-1.fc8
  =
   perl(:MODULE_COMPAT_5.8.8)
   perl(Carp)
   perl(Class::Accessor) >= 0.18
X  perl(Text::CSV_XS)
X  perl(Text::CSV_XS) >= 0.23
   perl(base)
   perl(constant)
   perl(strict)
   perl(vars)
   perl(warnings)

* %check is present and the single included test passes:
   All tests successful.
   Files=1, Tests=1,  0 wallclock secs ( 0.02 cusr +  0.01 csys =  0.03 CPU)
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* 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, or are watching someone who is.

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

[Bug 280541] Review Request: setools - SELinux policy analysis tools

[bugzilla]

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

Summary: Review Request: setools - SELinux policy analysis tools


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:17 EST ---
done(In reply to comment #23)
> (In reply to comment #21)
> > I looked at #314911, which ended up being tiny, and the only thing I saw was
> > that it didn't use %dist in the Release.  The package also doesn't match the
> > upstream package (tuncfg vs. tun).
> > 
> > Then I looked at #313361, which gets the errors:
> > 
> ..
> 
> Please post your pre-review comments on those review requests,
> not on this review requests so that the submitter of those
> requests can check your comments and modify their srpms.
> 
> After that, I will check your pre-review commentts.

Comments added to the respective 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, or are watching someone who is.

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

[Bug 313361] Review Request: python-zsi - Libraries for developing web services in python

[bugzilla]

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

Summary: Review Request: python-zsi - Libraries for developing web services in 
python


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:10 EST ---
I get build errors:

RPM build errors:
user jbowes does not exist - using root
group jbowes does not exist - using root
user jbowes does not exist - using root
group jbowes does not exist - using root
File not found by glob:
/var/tmp/python-zsi-2.0-1.fc8-root-root/usr/lib/python2.5/site-packages/*egg-info

so the files in the srpm need to have their ownership fixed, and the files
sections can be rewritten with wildcard like:

%{python_sitelib}/*
%exclude %{python_sitelib}/ZSI/twisted

so you can get all the files in the site lib, except for the part that goes in
the twisted subpackage.  

Also you need to run rpmlint; I get permissions on scripts that need to be 
fixed:
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/dispatch.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/schema.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/writer.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/wstools/c14n.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/TCnumbers.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/parse.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/auth.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/resolvers.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/wstools/__init__.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/TCtimes.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/ServiceContainer.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/client.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/TC.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/fault.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/TCapache.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/generate/wsdl2dispatch.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/__init__.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/digest_auth.py 0644
python-zsi.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/ZSI/TCcompound.py 0644


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

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

[Bug 317101] Review Request: rats - Rough Auditing Tool for Security

[bugzilla]

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

Summary: Review Request: rats - Rough Auditing Tool for Security


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||163778
  nThis||
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 13:10 EST ---
Taking this for review.

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

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

[Bug 317101] Review Request: rats - Rough Auditing Tool for Security

[bugzilla]

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

Summary: Review Request: rats - Rough Auditing Tool for Security


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




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

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

[Bug 317101] New: Review Request: rats - Rough Auditing Tool for Security

[bugzilla]

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




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

   Summary: Review Request: rats - Rough Auditing Tool for Security
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://sjhserv.net/~sjh/rats.spec
SRPM URL: http://sjhserv.net/~sjh/rats-2.1-1.fc8.src.rpm
Description: 
RATS scans through code, finding potentially dangerous function calls.
The goal of this tool is not to definitively find bugs (yet). The 
current goal is to provide a reasonable starting point for performing 
manual security audits.

The initial vulnerability database is taken directly from things that
could be easily found when starting with the forthcoming book, 
"Building Secure Software" by Viega and McGraw.  

RATS is released under version 2 of the GNU Public License (GPL).

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

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

[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

[bugzilla]

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 12:57 EST ---
(sorry, Bryan, my bad for the mispelling)

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

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

[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

[bugzilla]

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
 Status|NEEDINFO|ASSIGNED
   Flag|fedora-review?, |
   |needinfo?([EMAIL PROTECTED]|
   |)   |




--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 12:47 EST ---
Ugh, still broken, must be something else now.

Brian hasn't responded for 2+ mos, removing his review assignment so that
someone else can step in.

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

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

[Bug 280541] Review Request: setools - SELinux policy analysis tools

[bugzilla]

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

Summary: Review Request: setools - SELinux policy analysis tools


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 12:24 EST ---
(In reply to comment #21)
> I looked at #314911, which ended up being tiny, and the only thing I saw was
> that it didn't use %dist in the Release.  The package also doesn't match the
> upstream package (tuncfg vs. tun).
> 
> Then I looked at #313361, which gets the errors:
> 
..

Please post your pre-review comments on those review requests,
not on this review requests so that the submitter of those
requests can check your comments and modify their srpms.

After that, I will check your pre-review commentts.

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

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

[Bug 316141] Review Request: gridengine - Sun Grid Engine - Distributed Computing Management software

[bugzilla]

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

Summary: Review Request: gridengine - Sun Grid Engine - Distributed Computing 
Management software


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|225196  |




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

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

[Bug 210007] Review Request: libtune - standard API to access the kernel tunables

[bugzilla]

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

Summary: Review Request: libtune - standard API to access the kernel tunables


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 12:19 EST ---
Hi Konrad,

Nadia will have a look at this ASAP to try make it fit for FC8 GA.


jean-pierre


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

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

[Bug 316141] Review Request: gridengine - Sun Grid Engine - Distributed Computing Management software

[bugzilla]

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

Summary: Review Request: gridengine - Sun Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2007-10-03 12:14 EST ---
Whoops, sorry about that.  I've put updated versions there that build the drmaa
java stuff too.  Same release # at the moment, I'll start incrementing from 
here.

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

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

[Bug 226662] Merge Review: yelp (not started)

[bugzilla]

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

Summary: Merge Review: yelp (not started)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Merge Review: yelp  |Merge Review: yelp (not
   ||started)




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

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

[Bug 225834] Merge Review: gnome-python2

[bugzilla]

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

Summary: Merge Review: gnome-python2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




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

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

[Bug 225832] Merge Review: gnome-python2-desktop

[bugzilla]

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

Summary: Merge Review: gnome-python2-desktop


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




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

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

[Bug 225833] Merge Review: gnome-python2-extras

[bugzilla]

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

Summary: Merge Review: gnome-python2-extras


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




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

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

[Bug 226333] Merge Review: pygtk2

[bugzilla]

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

Summary: Merge Review: pygtk2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




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

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

[Bug 226662] Merge Review: yelp

[bugzilla]

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

Summary: Merge Review: yelp


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|medium  |low




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

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