[Bug 466997] Review Request: sl - Joke command for when you type 'sl' instead of 'ls'

[bugzilla]

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


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





--- Comment #11 from Marc Bradshaw <[EMAIL PROTECTED]>  2008-10-24 23:26:54 EDT 
---
Sorry for the delay, been a busy week.  The changes have been made to new SRPM
located at...

http://marcbradshaw.co.uk/packages/review/sl/sl-3.03-5.fc9.src.rpm

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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


Permaine Cheung <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-review+




--- Comment #8 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 22:51:36 
EDT ---
Links for the spec file and SRPM that builds successfully:
http://zarb.org/~david/hamcrest.spec
http://zarb.org/~david/hamcrest-1.1-5.1.jpp5.src.rpm

APPROVED

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

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

[Bug 435015] Review Request: libGPP4 - LGPL CCP4 library

[bugzilla]

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


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





--- Comment #14 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 22:48:45 EDT 
---
(In reply to comment #13)
> (In reply to comment #10)
> > Umm.. I cannot guess why you want to name binary rpm names as
> > libgpp4 and libgpp4-devel. Simply gpp4 and gpp4-devel is better.
> > 
> 
> I'd be happy to rename it gpp4 and gpp4-devel (as I originally packaged it),
> but see comments 4-8 above - I'm a bit confused as to what is best here.

Both Ralf and Jason were saying that you seemed to have some reason
you want to name the binary rpm as libgpp4.

However as I said on Fedora it is preferable to use tarball name for
srpm/binary rpms as much as possible. And as you say you are happy with
naming binary rpms as gpp4/gpp4-devel please just use these names.

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

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

[Bug 468329] Review Request: sblim-indication_helper - Toolkit for CMPI Indication Providers

[bugzilla]

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


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


Christopher D. Stover <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED],
   ||[EMAIL PROTECTED]




--- Comment #2 from Christopher D. Stover <[EMAIL PROTECTED]>  2008-10-24 
22:44:47 EDT ---
***Unofficial Review***

* rpmlint looks good:
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
3 packages and 0 specfiles checked; 0 errors, 0 warnings.

* Why is your build number 134?  I believe it should start at 1.

* You don't need gcc-c++ in your BuildRequires.

* I'd run rpmdev-newspec to create a sample spec file and format yours like
that.  I think it helps to have everything standardized for when someone looks
at it.  For instance, you have the 'BuildRequires' tag coming after 'Name' and
it should be 'Version'.

* I'm sure you'll do this if you look at the sample spec file but you should
use %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) as your
BuildRoot.

* Your spec file license shows IBM but the upstream license shows "Common
Public License" so this should probably be changed from IBM to CPL.

* You should use Source0:  to match the form of SourceX.

* Requires -- you can leave out the following, RPM can figure them out.
Requires:   sblim-cmpi-devel glibc-devel

* You should use %{__rm} under %install to be consistent within the spec file.

* You must run %{__rm} -rf %{buildroot} at the beginning of %install.

That's about all I have.

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

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

[Bug 468298] Review Request: python-pygle - A cross-platform windowing and multimedia 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.


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


Paulo Roma Cavalcanti <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED]) |




--- Comment #2 from Paulo Roma Cavalcanti <[EMAIL PROTECTED]>  2008-10-24 
21:59:51 EDT ---
Try these ones then:

Spec URL: http://dl.lcg.ufrj.br/RPMS/SPECS/python-pyglet.spec

SRPM URL: http://dl.lcg.ufrj.br/RPMS/src/python-pyglet-1.1.2-1.fc8.src.rpm

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

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

[Bug 454412] Review Request: mingw32-runtime - MinGW Windows cross-compiler runtime and root filesystem

[bugzilla]

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


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





--- Comment #7 from Harry Waye <[EMAIL PROTECTED]>  2008-10-24 20:59:19 EDT ---
I have spoken to certain people in the #boost irc chat room and have compiled
boost against your packages.  It works with some additions to the boost.build
scripts.  I'll try publish such things in a bit.

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

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

[Bug 468298] Review Request: python-pygle - A cross-platform windowing and multimedia 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.


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


Christopher D. Stover <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag||needinfo?([EMAIL PROTECTED])




--- Comment #1 from Christopher D. Stover <[EMAIL PROTECTED]>  2008-10-24 
20:51:52 EDT ---
Your links for the Spec & SRPM aren't working for me.

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

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

[Bug 466789] Review Request: jmol - an open-source Java viewer for chemical structures in 3D

[bugzilla]

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


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


Fedora Update System <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 458346] Review Request: gflags - Library for commandline flag processing

[bugzilla]

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


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





--- Comment #21 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:54:02 EDT ---
gflags-0.9-6.fc9 has been pushed to the Fedora 9 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.

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

[Bug 451772] Review Request: ume-launcher - a full screen application launcher for gnome

[bugzilla]

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


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


Fedora Update System <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 466789] Review Request: jmol - an open-source Java viewer for chemical structures in 3D

[bugzilla]

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


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





--- Comment #11 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:53:07 EDT ---
jmol-11.6-5.10137svn.fc9 has been pushed to the Fedora 9 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.

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

[Bug 451772] Review Request: ume-launcher - a full screen application launcher for gnome

[bugzilla]

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


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





--- Comment #16 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:53:20 EDT ---
ume-launcher-0.6.3-1.fc9 has been pushed to the Fedora 9 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.

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

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

[bugzilla]

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


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





--- Comment #10 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:51:31 EDT ---
R-nws-1.7.0.0-1.fc8 has been pushed to the Fedora 8 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.

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

[Bug 458346] Review Request: gflags - Library for commandline flag processing

[bugzilla]

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


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





--- Comment #20 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:52:23 EDT ---
gflags-0.9-6.fc8 has been pushed to the Fedora 8 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.

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

[Bug 455211] Review Request: php-laconica - PHP tool for microblogging

[bugzilla]

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


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





--- Comment #22 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:51:43 EDT ---
php-laconica-0.5.0-3.fc9 has been pushed to the Fedora 9 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.

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

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

[bugzilla]

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


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





--- Comment #9 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:50:28 EDT ---
R-nws-1.7.0.0-1.fc9 has been pushed to the Fedora 9 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.

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

[Bug 460000] Review Request: rxtx - Parallel communication for the Java Development Toolkit

[bugzilla]

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


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





--- Comment #22 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:50:47 EDT ---
rxtx-2.1-0.2.7r2.fc9 has been pushed to the Fedora 9 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.

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

[Bug 466829] Review Request: openalchemist - reflexion game

[bugzilla]

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


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





--- Comment #16 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:51:26 EDT ---
openalchemist-0.3-4.fc9 has been pushed to the Fedora 9 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.

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

[Bug 466829] Review Request: openalchemist - reflexion game

[bugzilla]

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


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





--- Comment #15 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:50:14 EDT ---
openalchemist-0.3-4.fc8 has been pushed to the Fedora 8 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.

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

[Bug 466829] Review Request: openalchemist - reflexion game

[bugzilla]

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


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


Fedora Update System <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 455211] Review Request: php-laconica - PHP tool for microblogging

[bugzilla]

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


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





--- Comment #21 from Fedora Update System <[EMAIL PROTECTED]>  2008-10-24 
19:49:44 EDT ---
php-laconica-0.5.0-3.fc8 has been pushed to the Fedora 8 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.

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #7 from David Walluck <[EMAIL PROTECTED]>  2008-10-24 18:44:56 EDT 
---
hamcrest.i386: W: non-conffile-in-etc

This is a bug in maven, as this file is not really a config file.

hamcrest-demo.i386: W: no-documentation

The demo depends on the main package, which contains the license. Otherwise,
there is no other documentation.

The all-1.1.jar.so was added automatically. It seems kind of weird.

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #6 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 18:23:34 
EDT ---
Running rpmlint against the mock built rpms:
rpmlint /var/lib/mock/fedora-development-i386/result/*rpm
hamcrest.i386: W: non-conffile-in-etc /etc/maven/fragments/hamcrest
hamcrest-demo.i386: W: no-documentation

rpm -qp --provides hamcrest-1.1-5.1.fc10.i386.rpm
all-1.1.jar.so  
hamcrest = 0:1.1-5.1.fc10
hamcrest(x86-32) = 0:1.1-5.1.fc10

Is the Provides for all-1.1.jar.so correct?

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #5 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 17:48:48 
EDT ---
Naming: OK
Licensing: OK
No inclusion of pre-built binaries or libraries: OK
Filesystem layout: OK
No Packager, Vendor tag: OK
Summary tag not ending with a period: OK
Source tag: OK (md5sum matches)
Buildroot tag: OK
%install begins with removing buildroot: OK
Changelog: OK

Jar file naming: OK
Directory structure: OK
BuildRequires and Requires:  OK

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

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

[Bug 468466] New: Review Request: libgksu - Simple API for su and sudo

[bugzilla]

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

Summary: Review Request: libgksu - Simple API for su and sudo

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

   Summary: Review Request: libgksu - Simple API for su and sudo
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
http://cassmodiah.fedorapeople.org/libgksu-2.0.7/libgksu.spec

SRPM URL: 
http://cassmodiah.fedorapeople.org/libgksu-2.0.7/libgksu-2.0.7-1.fc10.src.rpm

Description:
Libgksu is a library from the gksu program that provides a simple API for
using su and sudo in programs that need to execute tasks as other users.
It provides X authentication facilities for running programs in a X session

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

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

[Bug 468467] New: Review Request: gksu - Graphical frontend to su

[bugzilla]

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

Summary: Review Request: gksu - Graphical frontend to su

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

   Summary: Review Request: gksu - Graphical frontend to su
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
http://cassmodiah.fedorapeople.org/gksu-2.0.0/gksu.spec

SRPM URL: 
http://cassmodiah.fedorapeople.org/gksu-2.0.0/gksu-2.0.0-1.fc10.src.rpm

Description: 
Gtk+ frontend to su. It supports login shells and preserving environment
when acting as a su frontend. It is useful to menu items or other graphical
programs that need to ask a user's password to run another program as another
user.

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

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

[Bug 468466] Review Request: libgksu - Simple API for su and sudo

[bugzilla]

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


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


Simon Wesp <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468467




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

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

[Bug 468467] Review Request: gksu - Graphical frontend to su

[bugzilla]

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


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


Simon Wesp <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Depends on||468466




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

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

[Bug 457947] Review Request: 69oldstandard-fonts - Old Standard Fonts

[bugzilla]

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


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


Martin-Gomez Pablo <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   ||net




--- Comment #9 from Martin-Gomez Pablo <[EMAIL PROTECTED]>  2008-10-24 17:08:27 
EDT ---
As I'm building this package ( I haven't seen this review request), I will help
Ankur to pushed it to the official repos.

For the name, I approve the "old-standard-sfd-fonts" name (as the author accept
that we just put a suffix).
For the fontconfig.conf, it's not my cup of tea, so better you follow Nicolas'
advices ^^

I propose this spec (http://home.scarlet.be/~tsi30161/oldstandard.spec) to show
how to build the font (without changelog nor fontconfig.conf). I choose to
build to .ttf as it seems that the font author prefers it.

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

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

[Bug 454410] Review Request: mingw32-gcc - MinGW Windows cross-compiler (GCC) for C and C++

[bugzilla]

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


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





--- Comment #2 from Richard W.M. Jones <[EMAIL PROTECTED]>  2008-10-24 16:37:05 
EDT ---
I looked at Jakub's branch and decided to stay with GCC 4.3.2
for now.  We may move later, but this version of GCC has been
used to build everything and it works.

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

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

[Bug 468465] New: Review Request: posterazor - Make your own poster

[bugzilla]

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

Summary: Review Request: posterazor - Make your own poster

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

   Summary: Review Request: posterazor - Make your own poster
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://nicoleau.fabien.free.fr/rpms/SPECS/posterazor.spec
SRPM URL:
http://nicoleau.fabien.free.fr/rpms/srpms.fc9/posterazor-1.5-1.fc9.src.rpm
Description:
The PosteRazor cuts a raster image into pieces which can afterwards be
printed out and assembled to a poster. As input, the PosteRazor takes a
raster image. The resulting poster is saved as a multipage PDF document.
An easy to use, wizard like user interface guides through 5 steps.

rpmlint is silent :
[EMAIL PROTECTED] tmp]$ rpmlint posterazor-1.5-1.fc9.i386.rpm
posterazor-1.5-1.fc9.src.rpm posterazor-debuginfo-1.5-1.fc9.i386.rpm
3 packages and 0 specfiles checked; 0 errors, 0 warnings.
[EMAIL PROTECTED] tmp]$ 

Rebuild under mock fedora-9-i386 and fedora-8-i386 is OK
Rebuild under koji is OK :
http://koji.fedoraproject.org/koji/taskinfo?taskID=901793

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

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

[Bug 435015] Review Request: libGPP4 - LGPL CCP4 library

[bugzilla]

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


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





--- Comment #13 from Tim Fenn <[EMAIL PROTECTED]>  2008-10-24 16:28:18 EDT ---
(In reply to comment #10)
> Umm.. I cannot guess why you want to name binary rpm names as
> libgpp4 and libgpp4-devel. Simply gpp4 and gpp4-devel is better.
> 

I'd be happy to rename it gpp4 and gpp4-devel (as I originally packaged it),
but see comments 4-8 above - I'm a bit confused as to what is best here.

> 
> By the way rebuild fails on ppc64:
> http://koji.fedoraproject.org/koji/taskinfo?taskID=901509
> Now that I am sponsoring you, you can try to rebuild arbitrary srpms
> on koji by
> 
> $ koji build --scratch  
> where target can be dist-f11, dist-f10, dist-f9-updates-candidate
> or dist-f8-updates-candidate.
> If the build is successful, the rebuilt rpms and some logs are
> put for one week or so under:
> http://koji.fedoraproject.org/scratch//task_

Many thanks for the tip, Mamoru.

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

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

[Bug 467854] Review Request: parprouted - Proxy ARP IP bridging daemon

[bugzilla]

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


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





--- Comment #7 from Paulo Roma Cavalcanti <[EMAIL PROTECTED]>  2008-10-24 
16:25:38 EDT ---
(In reply to comment #6)
> (In reply to comment #5)
> > Thanks for the review.
> > 
> > I added a missing "Requires: iproute" to the spec file and .src.rpm
> 
> The same URLs?  And what about the permissions?

I recreated the .src.rpm with mock and now the files are owned by mockbuild:

[cascavel:~/RPMS8/lcgrpms/src] als parprouted-0.70-2.fc8.src.rpm
-rw-rw-r--1 mockbuilmockbuil  686 Aug 10 10:18 makefile.patch
-rw-r--r--1 mockbuilmockbuil18844 Jan 27  2008
parprouted-0.7.tar.gz
-rw-rw-r--1 mockbuilmockbuil 1542 Oct 24 14:13 parprouted.spec


SRPM: http://people.atrpms.net/~pcavalcanti/srpms/parprouted-0.70-2.fc8.src.rpm

SPEC: http://people.atrpms.net/~pcavalcanti/specs/parprouted.spec

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

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

[Bug 468462] New: Review Request: sbackup - Simple Backup Suite for desktop use

[bugzilla]

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

Summary: Review Request: sbackup - Simple Backup Suite for desktop use

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

   Summary: Review Request: sbackup - Simple Backup Suite for
desktop use
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
http://cassmodiah.fedorapeople.org/sbackup-0.10.5/sbackup.spec

SRPM URL: 
http://cassmodiah.fedorapeople.org/sbackup-0.10.5/sbackup-0.10.5-1.fc10.src.rpm

Description: 
Simple Backup Suite is a set of backend backup daemon and GNOME
GUI frontends that provide a simple yet powerful backup
solution for common desktop users.

Backups can be written to local directory or remote servers using
GNOME VFS technology. A fine control is possible regarding what
folders and files to backup. Files can be excluded even with a set
of regular expressions. Regular backups can be scheduled.

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

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

[Bug 468299] Review Request: jabberpy - Python xmlstream and jabber IM protocol libs

[bugzilla]

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


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


Michael Stahnke <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Michael Stahnke <[EMAIL PROTECTED]>  2008-10-24 15:49:22 
EDT ---
New Package CVS Request
===
Package Name: jabbperpy 
Short Description: Python xmlstream and jabber IM protocol libs
Owners: stahnma
Branches: F-8, F-9, EL4, EL5
InitialCC:

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

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

[Bug 467854] Review Request: parprouted - Proxy ARP IP bridging daemon

[bugzilla]

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


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





--- Comment #6 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 15:29:09 EDT ---
(In reply to comment #5)
> Thanks for the review.
> 
> I added a missing "Requires: iproute" to the spec file and .src.rpm

The same URLs?  And what about the permissions?

> parprouted has no Initscripts because it needs a fixed IP
> and a pre-created Interface, typically tap0.
> 
> Generally, parprouted is called before starting Virtualbox and killed
> afterwards.
> 
> The version number follows upstream (the previous was 0.65):

I see. That works.

> http://hazardous-area.org/parprouted/

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

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

[Bug 468453] New: Review Request: shed - simple ncurses based hex editor

[bugzilla]

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

Summary: Review Request: shed - simple ncurses based hex editor

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

   Summary: Review Request: shed - simple ncurses based hex editor
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://maxamillion.fedorapeople.org/shed.spec
SRPM URL: http://maxamillion.fedorapeople.org/shed-1.13-1.src.rpm
Description: shed is an easy to use hex editor written for unix/linux using
ncurses, with a friendly pico-style interface

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

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

[Bug 467854] Review Request: parprouted - Proxy ARP IP bridging daemon

[bugzilla]

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


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





--- Comment #5 from Paulo Roma Cavalcanti <[EMAIL PROTECTED]>  2008-10-24 
15:22:40 EDT ---
Thanks for the review.

I added a missing "Requires: iproute" to the spec file and .src.rpm

parprouted has no Initscripts because it needs a fixed IP
and a pre-created Interface, typically tap0.

Generally, parprouted is called before starting Virtualbox and killed
afterwards.

The version number follows upstream (the previous was 0.65):

http://hazardous-area.org/parprouted/

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #4 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 14:49:01 
EDT ---
I can't build this in mock (fedora-devel-i386), please see attached build.log.

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #3 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 14:48:06 
EDT ---
Created an attachment (id=321455)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=321455)
build log

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

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

[Bug 454412] Review Request: mingw32-runtime - MinGW Windows cross-compiler runtime and root filesystem

[bugzilla]

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


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





--- Comment #6 from Richard W.M. Jones <[EMAIL PROTECTED]>  2008-10-24 14:34:17 
EDT ---
(In reply to comment #1)
> You SRPM link should be fc8.  Also, have you any plans to add some C++
> libraries?  ie boost or gtkmm?

I did gtkmm - it's in our development repository.

Boost is a fscking nightmare.  People who write their own build
systems should be shot.  Or maybe since they have to program in
C++ they've been punished enough already.

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

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

[Bug 435015] Review Request: libGPP4 - LGPL CCP4 library

[bugzilla]

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


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





--- Comment #11 from Jason Tibbitts <[EMAIL PROTECTED]>  2008-10-24 14:04:07 
EDT ---
(In reply to comment #10)
> Umm.. I cannot guess why you want to name binary rpm names as
> libgpp4 and libgpp4-devel. Simply gpp4 and gpp4-devel is better.

He's done that for mmdb as well; I had thought that eventually I would
understand the reason for it but I haven't figured it out yet.

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

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

[Bug 468299] Review Request: jabberpy - Python xmlstream and jabber IM protocol libs

[bugzilla]

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


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


Dennis Gilmore <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Dennis Gilmore <[EMAIL PROTECTED]>  2008-10-24 14:21:07 EDT 
---
10bbc72c280e6c21a1101ccf3a227331b19bc3b6 
/home/dgilmore/Download/jabberpy-0.5-0.tar.gz
10bbc72c280e6c21a1101ccf3a227331b19bc3b6  SOURCES/jabberpy-0.5-0.tar.gz


builds in mock

rpmlint gives
rpmlint /var/lib/mock//fedora-rawhide-x86_64/result/*rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

remove --record=INSTALLED_FILES from "%{__python} setup.py install
--root=$RPM_BUILD_ROOT --record=INSTALLED_FILES" when importing

APPROVED

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

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

[Bug 435015] Review Request: libGPP4 - LGPL CCP4 library

[bugzilla]

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


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





--- Comment #12 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 14:15:24 EDT 
---
(In reply to comment #11)
> (In reply to comment #10)
> > Umm.. I cannot guess why you want to name binary rpm names as
> > libgpp4 and libgpp4-devel. Simply gpp4 and gpp4-devel is better.
> 
> He's done that for mmdb as well; I had thought that eventually I would
> understand the reason for it but I haven't figured it out yet.

I guess Tim will reply in a few days.

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

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

[Bug 426167] Review Request: PyYAML - YAML parser and emitter for Python

[bugzilla]

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


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





--- Comment #14 from John Eckersberg <[EMAIL PROTECTED]>  2008-10-24 14:08:37 
EDT ---
Ignore F-9 branch on previous request, I was having a CVS idiot moment.

Please create EL-4 and EL-5 still:


Package Change Request
==
Package Name: PyYAML
New Branches: EL-4 EL-5
Owners: jeckersb

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

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

[Bug 435015] Review Request: libGPP4 - LGPL CCP4 library

[bugzilla]

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


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #10 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 13:58:35 EDT 
---
Umm.. I cannot guess why you want to name binary rpm names as
libgpp4 and libgpp4-devel. Simply gpp4 and gpp4-devel is better.

Debian seems to name binary debs providing system wide libraries
as libX (like libpango1.0-0) while Fedora simply names binary
rpms using source tarball name as much as possible (like pango)
(For me I maintain oniguruma while debian uses libonig as binary
 deb names...)
https://fedoraproject.org/wiki/Packaging/NamingGuidelines#General_Naming

By the way rebuild fails on ppc64:
http://koji.fedoraproject.org/koji/taskinfo?taskID=901509
Now that I am sponsoring you, you can try to rebuild arbitrary srpms
on koji by

$ koji build --scratch  
where target can be dist-f11, dist-f10, dist-f9-updates-candidate
or dist-f8-updates-candidate.
If the build is successful, the rebuilt rpms and some logs are
put for one week or so under:
http://koji.fedoraproject.org/scratch//task_

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

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

[Bug 468299] Review Request: jabberpy - Python xmlstream and jabber IM protocol libs

[bugzilla]

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


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





--- Comment #2 from Michael Stahnke <[EMAIL PROTECTED]>  2008-10-24 13:59:10 
EDT ---
I accidentally used the wrong version of the SRPM.  The makefile in git doesn't
version unless you have commited, and apparently that bit me.  

http://stahnma.fedorapeople.org/reviews/jabberpy-0.5-0.16.fc9.src.rpm
http://stahnma.fedorapeople.org/reviews/jabberpy.spec

Sorry about that.  This should be a whole lot better.

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

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

[Bug 454019] Review Request: tcl-tktreectrl - Multi-column hierarchical listbox widget for Tk

[bugzilla]

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


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





--- Comment #2 from Tom "spot" Callaway <[EMAIL PROTECTED]>  2008-10-24 
14:00:29 EDT ---
No problem.

New SPEC:
http://auroralinux.org/people/spot/review/coccinella/tcl-tktreectrl.spec
New SRPM:
http://auroralinux.org/people/spot/review/coccinella/tcl-tktreectrl-2.2.8-1.fc10.src.rpm

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

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

[Bug 466496] Review Request: python-suds - A lightweight python soap web services client

[bugzilla]

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


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





--- Comment #2 from Jeff Ortel <[EMAIL PROTECTED]>  2008-10-24 13:53:19 EDT ---
Thanks for the great comments!
Implemented all suggestions with the exception of changing %Source0 to be a URL
to the fedorahosted wiki.  I'm not sure I understand this.  If I make this
change, rpmbuild fails.  Also, I looked at (2) other fedora hosted projects
that I'm pretty sure are fedora packages and their %Source0 looked like mine. 
Can you help me understand this better?

Also, seems like the spec should be named suds.spec.  Can I change this?

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

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

[Bug 458746] Review Request: python-netaddr - Network address manipulation, done Pythonically

[bugzilla]

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


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


John Eckersberg <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #8 from John Eckersberg <[EMAIL PROTECTED]>  2008-10-24 13:50:15 
EDT ---
Package Change Request
==
Package Name: python-netaddr
New Branches: EL-4 EL-5
Owners: jeckersb

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

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

[Bug 426167] Review Request: PyYAML - YAML parser and emitter for Python

[bugzilla]

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


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





--- Comment #13 from John Eckersberg <[EMAIL PROTECTED]>  2008-10-24 13:47:42 
EDT ---
Package Change Request
==
Package Name: PyYAML
New Branches: F-9 EL-4 EL-5
Owners: jeckersb

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

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

[Bug 426167] Review Request: PyYAML - YAML parser and emitter for Python

[bugzilla]

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


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


John Eckersberg <[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 on the CC list for the bug.

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

[Bug 459675] Review Request: python-sybase - new package request

[bugzilla]

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


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





--- Comment #15 from Andy Theuninck <[EMAIL PROTECTED]>  2008-10-24 13:44:43 
EDT ---
easy-install.pth can go. I'll repackage it as soon as I'm at a Fedora machine.

I'm working on packaging webmin (off the wishlist), but I don't have it quite
put together yet. The author has it set up to put pretty much EVERYTHING into
/usr/libexec, and rpmlint isn't particularly happy about that.

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #21 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 13:35:05 EDT 
---
Sadly I get function problems, but these seem outside of homestead

[EMAIL PROTECTED] homestead]$ sudo rpm -Uvh
/home/herrold/rpmbuild/RPMS/x86_64/homestead-0.86-2orc.x86_64.rpm
[sudo] password for herrold:
Preparing...### [100%]
   1:homestead  ### [100%]
[EMAIL PROTECTED] homestead]$ homestead
libGL warning: 3D driver claims to not support visual 0x5a
Segmentation fault
[EMAIL PROTECTED] homestead]$ homestead
libGL warning: 3D driver claims to not support visual 0x5a
Segmentation fault
[EMAIL PROTECTED] homestead]$ homestead
libGL warning: 3D driver claims to not support visual 0x5a
Segmentation fault
[EMAIL PROTECTED] homestead]$ homestead
libGL warning: 3D driver claims to not support visual 0x5a
Segmentation fault
[EMAIL PROTECTED] homestead]$ homestead
libGL warning: 3D driver claims to not support visual 0x5a
Segmentation fault
[EMAIL PROTECTED] homestead]$ rpm -qa \*GL
mesa-libGL-6.5.1-7.5.el5
mesa-libGL-6.5.1-7.5.el5
[EMAIL PROTECTED] homestead]$

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

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

[Bug 459675] Review Request: python-sybase - new package request

[bugzilla]

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


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





--- Comment #14 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 13:19:08 EDT 
---
Okay, one question for this package:

* easy-install.pth
  - Is this file needed? It seems that this file is created when using
setuptools.command.easy_install so if other package also creates this file
by using setuptools.command.easy_install it will easily makes conflict 
with the package.

And I will wait for your pre-review or another review request
(if you have no other packages currently you want to submit you can
 do a pre-review of other person's review request).

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

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

[Bug 467530] Review Request: ladvd - CDP/LLDP sender for unix

[bugzilla]

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


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


Andreas Thienemann <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Andreas Thienemann <[EMAIL PROTECTED]>  2008-10-24 13:08:47 
EDT ---
Thank you Jon.

New Package CVS Request
===
Package Name: ladvd
Short Description: CDP/LLDP sender for unix
Owners: ixs
Branches: F-9

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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


Permaine Cheung <[EMAIL PROTECTED]> changed:

   What|Removed |Added

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




--- Comment #2 from Permaine Cheung <[EMAIL PROTECTED]>  2008-10-24 13:03:21 
EDT ---
I'll take this one.

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #20 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 13:01:25 EDT 
---
This patch does the split to meet that goal.  As I have mand a substantive
change, and add a Changelog entry I bump the Release here

[EMAIL PROTECTED] homestead]$ rpm -qp --requires
/home/herrold/rpmbuild/RPMS/x86_64/homestead-sampler-0.86-2orc.x86_64.rpm
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libgcc_s.so.1()(64bit)
libm.so.6()(64bit)
libpcap.so.0.9.4()(64bit)
libstdc++.so.6()(64bit)
libstdc++.so.6(CXXABI_1.3)(64bit)
libstdc++.so.6(GLIBCXX_3.4)(64bit)
rpmlib(CompressedFileNames) <= 3.0.4-1
rpmlib(PayloadFilesHavePrefix) <= 4.0-1
rtld(GNU_HASH)
[EMAIL PROTECTED] homestead]$

[EMAIL PROTECTED] homestead]$ diff -u homestead.spec-ORIG homestead.spec
--- homestead.spec-ORIG 2008-10-24 11:49:24.0 -0400
+++ homestead.spec  2008-10-24 13:00:32.0 -0400
@@ -1,9 +1,9 @@
 %define srcversion 086
-%definevendor  '%vendor%nil'
+%definelocalvendor %vendor%nil

 Name:   homestead
 Version:0.86
-Release:1%{?dist}
+Release:2%{?dist}
 Summary:3D real-time network visualiser

 Group:  Applications/Internet
@@ -18,6 +18,10 @@
 BuildRequires:  libpcap-devel
 BuildRequires:  desktop-file-utils

+%package sampler
+Summary:sampling client for %{name}
+Group:  Applications/Internet
+License:GPLv2+

 %description
 Homestead is a 3D real-time network visualizer, displaying
@@ -36,6 +40,12 @@
 same subnet, receiving packet header information
 from hsen via broadcast).

+%description sampler
+hsen (Homestead Sensor) is a (possibly remote) packet capture daemon which
+reads and sends packet header information to Homestead, locally or remotely.
+hsen also equates hostname to IP by reading DNS packets (UDP type A class IN
+standard query response).
+
 %prep
 %setup -q -n %{name}%{srcversion}

@@ -49,7 +59,10 @@
 rm -rf %{buildroot}
 make install DESTDIR=%{buildroot}

-desktop-file-install --dir=%{buildroot}%{_datadir}/applications \
+#  seemingly 'desktop-file-install' is needed as its requirements
+#  could somehow not be handled by 'install' at freedesktop.org for
+#  managing .desktop files, whereas 'install' seems fine for pixmaps
+desktop-file-install --vendor=%localvendor
--dir=%{buildroot}%{_datadir}/applications \
 %{SOURCE1}

 install -p -D -m 0644 %{SOURCE2} %{buildroot}%{_datadir}/pixmaps/%{name}.png
@@ -62,14 +75,25 @@
 %defattr(-,root,root,-)
 %doc README COPYING ChangeLog
 %{_bindir}/%{name}
-%{_sbindir}/hsen
 %{_mandir}/man1/%{name}.*
-%{_mandir}/man8/hsen.*
 %{_datadir}/pixmaps/%{name}.png
-%{_datadir}/applications/%{name}.desktop
+#
+#  two possible approaches here --specific or wildcarded
+%{_datadir}/applications/%{localvendor}-%{name}.desktop
+%{_datadir}/applications/*%{name}.desktop
+
+
+%files sampler
+%defattr(-,root,root,-)
+%{_sbindir}/hsen
+%{_mandir}/man8/hsen.*


 %changelog
+* Fri Oct 24 2008 R P Herrold <[EMAIL PROTECTED]> - 0.86-2
+- split out a -sampler sub-package to avoid dragging in the unneeded
+  GUI on remote TUI only sample heads
+
 * Wed Oct 22 2008 Simon Wesp <[EMAIL PROTECTED]> - 0.86-1
 - Update to 0.86

[EMAIL PROTECTED] homestead]$

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

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

[Bug 460886] Review Request: libnetdude - a libpcap trace file manipulation library

[bugzilla]

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


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





--- Comment #13 from Andreas Thienemann <[EMAIL PROTECTED]>  2008-10-24 
12:57:15 EDT ---
A note about the libnetdude.spec from me, that one is a package I created for
some project at Red Hat and posted here for reference.

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

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

[Bug 460886] Review Request: libnetdude - a libpcap trace file manipulation library

[bugzilla]

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


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


Andreas Thienemann <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|needinfo?([EMAIL PROTECTED] |
   |rg) |




--- Comment #12 from Andreas Thienemann <[EMAIL PROTECTED]>  2008-10-24 
12:54:51 EDT ---
Jon got that right.

As discussed with Christian, he's happy for me to do the review, so I'll just
set the flags then, and assign it.

Everybody happy with that?

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

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

[Bug 463922] Review Request: ifstat - Interface statistics

[bugzilla]

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


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||needinfo?([EMAIL PROTECTED]
   ||com.br)




--- Comment #13 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 12:52:36 EDT 
---
Reis, would you update your srpm?

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

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

[Bug 467530] Review Request: ladvd - CDP/LLDP sender for unix

[bugzilla]

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


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


Jon Ciesla <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 12:54:24 EDT ---
APPROVED.  I essentially re-reviewed it to check Paulo's 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.

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

[Bug 467530] Review Request: ladvd - CDP/LLDP sender for unix

[bugzilla]

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


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


Jon Ciesla <[EMAIL PROTECTED]> changed:

   What|Removed |Added

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




--- Comment #7 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 12:53:11 EDT ---
Taking assignment. .

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

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

[Bug 460886] Review Request: libnetdude - a libpcap trace file manipulation library

[bugzilla]

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


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


Andreas Thienemann <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 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.

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #19 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 12:50:56 EDT 
---
Preparing to install homestead on a border router to place 'hsen' on a well
placed sampling location, I find that it pulls in WAYY too much, and
particularly GUI parts that are not welcome, and will never be used in such a
context.  This causes problems with the security policy model for such units.


[EMAIL PROTECTED] ~]$ rpm -q --requires homestead
libGL.so.1()(64bit)
libGLU.so.1()(64bit)
 ...
libglut.so.3()(64bit)
 ...
[EMAIL PROTECTED] ~]$

The 'hsen' sampler needs to be split out to a sub package, such as '-sampler'

-- Russ herrold

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

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

[Bug 460886] Review Request: libnetdude - a libpcap trace file manipulation library

[bugzilla]

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


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





--- Comment #11 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 12:48:28 EDT ---
Christian is the submitter, and Andreas is the reviewer, AFAIK.  Andreas can
now sponsor.

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

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

[Bug 468299] Review Request: jabberpy - Python xmlstream and jabber IM protocol libs

[bugzilla]

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


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


Dennis Gilmore <[EMAIL PROTECTED]> changed:

   What|Removed |Added

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




--- Comment #1 from Dennis Gilmore <[EMAIL PROTECTED]>  2008-10-24 12:48:10 EDT 
---
fails to build in mock 
+ /usr/bin/python setup.py build
/var/tmp/rpm-tmp.j4MeWz: line 29: /usr/bin/python: No such file or directory


need to BR python

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

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

[Bug 467530] Review Request: ladvd - CDP/LLDP sender for unix

[bugzilla]

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


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





--- Comment #6 from Andreas Thienemann <[EMAIL PROTECTED]>  2008-10-24 12:49:07 
EDT ---
Jon, I'd appreciate that. Thx.

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

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

[Bug 460886] Review Request: libnetdude - a libpcap trace file manipulation library

[bugzilla]

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


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||needinfo?([EMAIL PROTECTED]
   ||rg)




--- Comment #10 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 12:44:44 EDT 
---
Currently who is the submitter and who is the reviewer?

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

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

[Bug 468414] Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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


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





--- Comment #1 from David Walluck <[EMAIL PROTECTED]>  2008-10-24 12:42:17 EDT 
---
This package is required by junit4 4.4 and later.

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

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

[Bug 455426] Review Request: gstreamer-java - Java interface to the gstreamer framework

[bugzilla]

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


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


Mamoru Tasaka <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #24 from Mamoru Tasaka <[EMAIL PROTECTED]>  2008-10-24 12:42:36 EDT 
---
Okay.

2 Notes:
* Documents
  - Please add the following files to %doc
-
src/org/freedesktop/tango/COPYING
-
Note that this file also has "COPYING" name so this
has to be renamed, for example by:
-
%prep
%setup -q -n %{name}
cp -p src/org/freedesktop/tango/COPYING COPYING.CC-BY-SA
# remove prebuild binaries
find . -name '*.jar' -exec rm {} \;
find . -name '.hg*' -exec rm -rf {} \; || :
-

* %changelog
  - Please update %changelog (the last entry does not
coincide with current rpm EVR).

Other things are okay

-
   This package (gstreamer-java) is APPROVED by mtasaka
-

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

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

[Bug 468414] New: Review Request: hamcrest - Hamcrest matcher object framework

[bugzilla]

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

Summary: Review Request: hamcrest - Hamcrest matcher object framework

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

   Summary: Review Request: hamcrest - Hamcrest matcher object
framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED], fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
SRPM URL: 
Description :
Provides a library of matcher objects (also known as
constraints or predicates) allowing 'match' rules to
be defined declaratively, to be used in other frameworks.
Typical scenarios include testing frameworks, mocking
libraries and UI validation rules.

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #18 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 12:38:51 EDT 
---
I have upstreamed the needed enhancement to 
desktop-file-install
to permit SEEING and FIXING suspect .desktop files at:
   https://bugs.freedesktop.org/show_bug.cgi?id=18206

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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


R P Herrold <[EMAIL PROTECTED]> changed:

   What|Removed |Added

External Bug ID||FreeDesktop.org 18206




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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #17 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 12:36:21 EDT 
---
An Implementation of comment 15 
   https://bugzilla.redhat.com/show_bug.cgi?id=459924#c15
showing a vendor and dist tag neutral packaging approach, and a 'worked case'
diff are as follows:

[EMAIL PROTECTED] homestead]$ ls
homestead-0.86-1.fc10.src.rpm  homestead.desktop-RPH~  homestead.spec-RPH
homestead.desktop  homestead.spec  homestead.spec-RPH~
homestead.desktop-ORIG homestead.spec~
homestead.desktop-RPH  homestead.spec-ORIG
[EMAIL PROTECTED] homestead]$ rpm -qlp
/home/herrold/rpmbuild/SRPMS/homestead-0.86-1orc.src.rpm
homestead.desktop
homestead.png
homestead.spec
homestead086.tar.gz
[EMAIL PROTECTED] homestead]$ rpm -qlp
/home/herrold/rpmbuild/RPMS/x86_64/homestead-0.86-1orc.x86_64.rpm
/usr/bin/homestead
/usr/sbin/hsen
/usr/share/applications/orc-homestead.desktop
/usr/share/doc/homestead-0.86
/usr/share/doc/homestead-0.86/COPYING
/usr/share/doc/homestead-0.86/ChangeLog
/usr/share/doc/homestead-0.86/README
/usr/share/man/man1/homestead.1.gz
/usr/share/man/man8/hsen.8.gz
/usr/share/pixmaps/homestead.png
[EMAIL PROTECTED] homestead]$ diff -u homestead.spec-ORIG homestead.spec
--- homestead.spec-ORIG 2008-10-24 11:49:24.0 -0400
+++ homestead.spec  2008-10-24 12:32:47.0 -0400
@@ -1,5 +1,5 @@
 %define srcversion 086
-%definevendor  '%vendor%nil'
+%definelocalvendor %vendor%nil

 Name:   homestead
 Version:0.86
@@ -49,7 +49,10 @@
 rm -rf %{buildroot}
 make install DESTDIR=%{buildroot}

-desktop-file-install --dir=%{buildroot}%{_datadir}/applications \
+#  seemingly 'desktop-file-install' is needed as its requirements
+#  could somehow not be handled by 'install' at freedesktop.org for
+#  managing .desktop files, whereas 'install' seems fine for pixmaps
+desktop-file-install --vendor=%localvendor
--dir=%{buildroot}%{_datadir}/applications \
 %{SOURCE1}

 install -p -D -m 0644 %{SOURCE2} %{buildroot}%{_datadir}/pixmaps/%{name}.png
@@ -66,7 +69,10 @@
 %{_mandir}/man1/%{name}.*
 %{_mandir}/man8/hsen.*
 %{_datadir}/pixmaps/%{name}.png
-%{_datadir}/applications/%{name}.desktop
+#
+#  two possible approaches here --specific or wildcarded
+%{_datadir}/applications/%{localvendor}-%{name}.desktop
+%{_datadir}/applications/*%{name}.desktop


 %changelog
[EMAIL PROTECTED] homestead]$

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

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

[Bug 468287] Review Request: cim-schema - Common Information Model (CIM) Schema

[bugzilla]

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


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


Christopher D. Stover <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED],
   ||[EMAIL PROTECTED]




--- Comment #2 from Christopher D. Stover <[EMAIL PROTECTED]>  2008-10-24 
12:20:08 EDT ---
***Unofficial Review***
This is a pretty easy review considering "- now meets Fedora packaging
guidelines too" is a change in 2.19.1-1. :D  However, here are my comments.

* Output from rpmlint:
cim-schema.src: W: invalid-license Any permissive
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Hopefully Spot can provide some insight into that for you.

* In your %install macro, you have MOFDIR=/usr/share/mof.  I don't think it's
required but you can substitute %{_datadir} for /usr/share.  The list of these
macros is here: https://fedoraproject.org/wiki/Packaging/RPMMacros.

* Builds without any problems on i386 32-bit.

Looks pretty good to me.

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #16 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 12:12:53 EDT 
---
actually digging futher, it seems that there is are a couple of warnings
(although not properly flagged as such by the freedesktop.org tool)

[EMAIL PROTECTED] homestead]$ desktop-file-install --vendor=orc
--dir=/var/tmp/homestead-0.86-1orc-root-herrold/usr/share/applications
/home/herrold/rpmbuild/SOURCES/homestead.desktop

/home/herrold/rpmbuild/SOURCES/homestead.desktop: missing encoding  (guessed
UTF-8)

/home/herrold/rpmbuild/SOURCES/homestead.desktop: key "Categories" string list
not semicolon-terminated, fixing

==

This patch to SOURCE1 fixes the matter

[EMAIL PROTECTED] homestead]$ diff -u homestead.desktop-ORIG homestead.desktop
--- homestead.desktop-ORIG  2008-10-24 12:11:12.0 -0400
+++ homestead.desktop   2008-10-24 12:11:34.0 -0400
@@ -5,4 +5,7 @@
 Comment=An 3D real-time network visualiser
 Exec=homestead
 Icon=homestead
-Categories=Network
+Categories=Network;
+#  and a deprecated item needed to quell a warning
+Encoding=UTF-8
+
[EMAIL PROTECTED] homestead]$

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

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

[Bug 468299] Review Request: jabberpy - Python xmlstream and jabber IM protocol libs

[bugzilla]

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


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


Michael Stahnke <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||452450




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

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

[Bug 452107] Review Request: cfdg - Context Free Design Grammar

[bugzilla]

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


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


Jon Ciesla <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #22 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 12:06:30 EDT ---
Thanks!

New Package CVS Request
===
Package Name: cfdg
Short Description: Context Free Design Grammar
Owners: limb
Branches: F-9
InitialCC:

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

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

[Bug 452107] Review Request: cfdg - Context Free Design Grammar

[bugzilla]

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


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


Terje Røsten <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #21 from Terje Røsten <[EMAIL PROTECTED]>  2008-10-24 12:00:18 EDT 
---
Great, I can't find any issues with the update 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.

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

[Bug 468329] Review Request: sblim-indication_helper - Toolkit for CMPI Indication Providers

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468400




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

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

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

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468400




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

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

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

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468400




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

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

[Bug 468287] Review Request: cim-schema - Common Information Model (CIM) Schema

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468400




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

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

[Bug 466183] Review Request: sblim-sfcb - Small Footprint CIM Broker

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Blocks||468400




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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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


R P Herrold <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #15 from R P Herrold <[EMAIL PROTECTED]>  2008-10-24 11:44:04 EDT 
---
The law of unintended packaging consequesces may be in play here.  On a scratch
build without a set --vendor tag set I get:

+ desktop-file-install
--dir=/var/tmp/homestead-0.86-1orc-root-herrold/usr/share/applications
/home/herrold/rpmbuild/SOURCES/homestead.desktop
Must specify the vendor namespace for these files with --vendor
error: Bad exit status from /var/tmp/rpm-tmp.49715 (%install)

so a portibility patch of having a 

%define localvendor %vendor%nil  

and then using localvendor construct is needed, here and in the the broader
policy?

No criticism of this fine effort within the four walls of fedora and rhel
%vendor, but ...

Just talking out loud here on the broader issue of NOT fighting with other
efforts in how spec files are written.

-- Russ herrold

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

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

[Bug 459878] Review Request: genome - Package for the Genome Project

[bugzilla]

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


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


Jeroen van Meeuwen <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Jeroen van Meeuwen <[EMAIL PROTECTED]>  2008-10-24 
11:46:38 EDT ---
New Package CVS Request
===
Package Name: genome
Short Description: Package for the Genome Project
Owners: kanarip
Branches: EL-4 EL-5 F-8 F-9 devel
InitialCC: kanarip

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

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

[Bug 459924] Review Request: Homestead - 3D real-time network visualiser

[bugzilla]

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


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





--- Comment #14 from Randall Berry <[EMAIL PROTECTED]>  2008-10-24 11:36:46 EDT 
---
Approved, everything looks good. Thanks for your patience.

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

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

[Bug 467655] Review Request: yafaray - a raytracer for Blender.

[bugzilla]

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


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





--- Comment #13 from Paulo Roma Cavalcanti <[EMAIL PROTECTED]>  2008-10-24 
11:36:19 EDT ---
I think I got it:


Just do:

cd ~/.blender/scripts

ln -s /usr/share/blender/scripts/_yafrayinterface.so .

ln -s /usr/share/blender/scripts/_yafqt.so .

and it should work with the official Fedora version.


These two shared libraries are in the yafaray-blender package.

My blender version creates these two symbolic links in the user home directory,
but the Fedora version does not.

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

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

[Bug 441378] Review Request: smokeping - Latency Logging and Graphing System

[bugzilla]

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


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





--- Comment #34 from Terje Røsten <[EMAIL PROTECTED]>  2008-10-24 11:35:08 EDT 
---
Ok, adding F-8.

New Package CVS Request
===
Package Name: smokeping
Short Description: Latency Logging and Graphing System
Owners: terjeros
Branches: F-8 F-9
InitialCC:

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

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

[Bug 452107] Review Request: cfdg - Context Free Design Grammar

[bugzilla]

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


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





--- Comment #20 from Jon Ciesla <[EMAIL PROTECTED]>  2008-10-24 11:35:31 EDT ---
I'm happy with a modified tarball:

SPEC: http://zanoni.jcomserv.net/fedora/cfdg/cfdg.spec
SRPM: http://zanoni.jcomserv.net/fedora/cfdg/cfdg-2.1-5.fc9.src.rpm

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

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

[Bug 441378] Review Request: smokeping - Latency Logging and Graphing System

[bugzilla]

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


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





--- Comment #33 from manuel wolfshant <[EMAIL PROTECTED]>  2008-10-24 11:24:24 
EDT ---
For the record: I am using it for several months on F7 without any issues, so I
guess (provided the deps are met), F-8 is also a branch which could make use of
this package.

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

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

[Bug 452107] Review Request: cfdg - Context Free Design Grammar

[bugzilla]

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


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





--- Comment #19 from Terje Røsten <[EMAIL PROTECTED]>  2008-10-24 11:17:59 EDT 
---
Sorry for delay. It's up to you, you decide.

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

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

[Bug 441378] Review Request: smokeping - Latency Logging and Graphing System

[bugzilla]

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


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


Terje Røsten <[EMAIL PROTECTED]> changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #32 from Terje Røsten <[EMAIL PROTECTED]>  2008-10-24 11:13:10 EDT 
---
Thanks!

Will add SElinux info to README.



New Package CVS Request
===
Package Name: smokeping
Short Description: Latency Logging and Graphing System
Owners: terjeros
Branches: F-9
InitialCC:

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

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

[Bug 452413] Review Request: BkChem - Chemical drawing program

[bugzilla]

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


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


Terje Røsten <[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.

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

[Bug 462381] Review Request: beediff - Visual tool for comparing and merging files

[bugzilla]

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


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


Terje Røsten <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 453109] Review Request: nocpulse-common - Add NOCpulse users and includes common files for NOCpulse.

[bugzilla]

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


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


Dennis Gilmore <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]




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

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

[Bug 462381] Review Request: beediff - Visual tool for comparing and merging files

[bugzilla]

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


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





--- Comment #10 from Terje Røsten <[EMAIL PROTECTED]>  2008-10-24 11:09:26 EDT 
---
Imported and built.

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

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

[Bug 453109] Review Request: nocpulse-common - Add NOCpulse users and includes common files for NOCpulse.

[bugzilla]

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


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


Dennis Gilmore <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag|fedora-review?  |fedora-review+




--- Comment #16 from Dennis Gilmore <[EMAIL PROTECTED]>  2008-10-24 11:02:39 
EDT ---
52289f16f8504f954369c6a3f065eec7892d6649  nocpulse-common-2.0.8.tar.gz
52289f16f8504f954369c6a3f065eec7892d6649 
/home/dgilmore/rpmbuild/SOURCES/nocpulse-common-2.0.8.tar.gz

sha1sum of source matches 
rpmlint gives
nocpulse-common.noarch: W: non-standard-uid /var/log/nocpulse nocpulse  
nocpulse-common.noarch: W: non-standard-gid /var/log/nocpulse nocpulse
nocpulse-common.noarch: W: non-standard-uid /var/lib/nocpulse nocpulse
nocpulse-common.noarch: W: non-standard-gid /var/lib/nocpulse nocpulse
nocpulse-common.noarch: W: obsolete-not-provided NPusers
nocpulse-common.noarch: W: obsolete-not-provided np-config


i disagree with not putting provides in  but ill accept it without.

builds in mock on rawhide.

# This src.rpm is cannonical upstream
# You can obtain it using this set of commands
# git clone git://git.fedorahosted.org/git/spacewalk.git/
# cd monitoring/nocpulse-common
# make srpm

needs to be removed from the spec.  but can be done at import time.

Approved

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

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

[Bug 468329] Review Request: sblim-indication_helper - Toolkit for CMPI Indication Providers

[bugzilla]

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


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


Matt Domsch <[EMAIL PROTECTED]> changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Comment #1 from Matt Domsch <[EMAIL PROTECTED]>  2008-10-24 11:06:15 EDT ---
sblim-sfcb-base will need a minor update after this package is approved, to BR:
sblim-indication_helper-devel, to automatically pick up indication support.

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

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