[Bug 503202] Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer

2009-06-01 Thread 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=503202





--- Comment #2 from Parag AN(पराग) panem...@gmail.com  2009-06-01 02:05:06 
EDT ---
also, I think according to
https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F
license for this package is GPL+

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

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


[Bug 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189





--- Comment #3 from Peter Lemenkov lemen...@gmail.com  2009-06-01 02:36:07 
EDT ---
Thanks for the comments - I fixed this issue:

http://peter.fedorapeople.org/python-gnutls.spec
http://peter.fedorapeople.org/python-gnutls-1.1.8-2.fc10.src.rpm

koji scratchbuild:

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

-- 
Configure 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 503174] Review Request: python-application - Basic building blocks for python applications

2009-06-01 Thread 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=503174





--- Comment #1 from Peter Lemenkov lemen...@gmail.com  2009-06-01 02:37:50 
EDT ---
Added missing BR - python-setuptools-devel

http://peter.fedorapeople.org/python-application.spec
http://peter.fedorapeople.org/python-application-1.1.1-2.fc10.src.rpm

koji scratchbuild

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

-- 
Configure 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 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #4 from Tomas Smetana tsmet...@redhat.com  2009-06-01 02:48:51 
EDT ---
I've committed some changes to the devel branch: rpmlint seems to be silent
now.  I used only part of the diff you posted here.

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

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


[Bug 478925] Review Request: globus-gass-transfer - Globus Toolkit - Globus Gass Transfer

2009-06-01 Thread 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=478925





--- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-06-01 
03:06:22 EDT ---
Package available in devel.

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

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


[Bug 478922] Review Request: globus-ftp-control - Globus Toolkit - GridFTP Client Control Library

2009-06-01 Thread 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=478922





--- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-06-01 
03:06:19 EDT ---
Package available in devel.

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

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


[Bug 502101] Review Request: agedu - An utility for tracking down wasted disk space

2009-06-01 Thread 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=502101





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 03:19:12 EDT ---
agedu-0-1.r8442.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/agedu-0-1.r8442.fc10

-- 
Configure 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 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm

2009-06-01 Thread 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=503256





--- Comment #22 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-01 03:18:23 
EDT ---
Yes, you cannot ship binary executables in %doc. Now it's just a question of

a) finding a place to put them (say, %{_libdir}/%{name})
b) not shipping them at all

If you decide to ship them, they should be branched out to a separate package,
-examples.

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

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


[Bug 502101] Review Request: agedu - An utility for tracking down wasted disk space

2009-06-01 Thread 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=502101





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 03:19:47 EDT ---
agedu-0-1.r8442.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/agedu-0-1.r8442.fc11

-- 
Configure 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 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #5 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-01 03:30:04 
EDT ---
Okay.

- I wouldn't ship man as %doc.

- Change
 BuildRequires: /usr/bin/desktop-file-install
to
 BuildRequires: desktop-file-utils

- You need to conserve time stamps in character set conversion, use 
 for i in AUTHORS docs/man/{de,es}/*.?; do
  iconv -f iso-8859-1 -t utf-8 $i  ${i}_  \
  touch -r $i ${i}_  \
  mv ${i}_ $i
 done
This is also safe, since it fails if a command is not completed OK.

- I'm wondering if autoreconfing is really needed; you could just use the trick
at
 http://fedoraproject.org/wiki/Packaging/Guidelines#Removing_Rpath

- Why is clean commented out?

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

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


[Bug 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #6 from Tomas Smetana tsmet...@redhat.com  2009-06-01 03:48:49 
EDT ---
(In reply to comment #5)
 Okay.
 
 - I wouldn't ship man as %doc.

This is something I couldn't find in the guidelines so I didn't change what I
inherited.  Both make sense to me.

 - Change
  BuildRequires: /usr/bin/desktop-file-install
 to
  BuildRequires: desktop-file-utils
 
 - You need to conserve time stamps in character set conversion, use 
  for i in AUTHORS docs/man/{de,es}/*.?; do
   iconv -f iso-8859-1 -t utf-8 $i  ${i}_  \
   touch -r $i ${i}_  \
   mv ${i}_ $i
  done
 This is also safe, since it fails if a command is not completed OK.

Right. Will do.

 - I'm wondering if autoreconfing is really needed; you could just use the 
 trick
 at
  http://fedoraproject.org/wiki/Packaging/Guidelines#Removing_Rpath

This is due to the new ALSA mixer patch and libtoolize.  I think it's simpler
like this.

 - Why is clean commented out?  

Because I'm an idiot and don't check what I commit carefully. (It's a remain of
debugging the new libtool issue.)

Thanks for your help.  I will incorporate the changes to the spec file, rebuild
and update this bugzilla.

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

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


[Bug 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #7 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-01 04:19:01 
EDT ---
Okay, once you have built the new revision in rawhide I will do the full
review.

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

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


[Bug 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #8 from Tomas Smetana tsmet...@redhat.com  2009-06-01 04:29:50 
EDT ---
Done.  Hope it's better this time.  Maybe even good.

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

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


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

2009-06-01 Thread 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 #46 from Nicolas Chauvet (kwizart) kwiz...@gmail.com  2009-06-01 
04:50:55 EDT ---
(In reply to comment #44)
 (In reply to comment #43)
..
 I am also using x86_64. Here is the picture:
 http://orion.lcg.ufrj.br/temp/blender-yafaray.png
I'm using a modified version of blender-2.48a-22 and that's the first time I
see it, I will retry with an unmodified version from F-11 x86_64.

 That is how upstream is doing. They moved from 1.0 - 305 to 1.0 - 315:
Okay, that was my bad, the version is indeed 0.1.0.315.

 I have posted a message in yafaray Building forum and did not receive a
 single reply.
Please reminds that the docs is the source code itself at the end. If you
cannot have a

  The current package provided above doesn't work at all.  
 
 What do you mean by that? It is working for me just fine. Have you tried
 it? Does it segfault?  
Funny that you talk about segfault, because there is indeed one. Quoting:

creating new QApplication
/usr/bin/blender: line 88:  7113 Segmentation fault  /usr/bin/${blend}.bin
$@


Now what remains weird, is that even before the segfault the yafaray plugin
doesn't seems to remind the previous configuration. (example: for using the xml
parser for rendering).

-- 
Configure 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 474787] Review Requrest: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787





--- Comment #14 from Michael Schwendt mschwe...@gmail.com  2009-06-01 
05:01:18 EDT ---
All members of the packager group may do official package-reviews and
package-approvals.

-- 
Configure 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 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review?




--- Comment #3 from Parag AN(पराग) panem...@gmail.com  2009-06-01 05:15:18 
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1386455
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
4b196b07524f55422c64113aee1a5bcbbf249ee9  easystroke-0.4.4.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ gtk-update-icon-cache scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Desktop file installed correctly.
+ GUI application

Suggestions:
1) I don't see mimetype written in desktop file so according to
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database,
drop update-desktop-database from SPEC.

2) I think you should own only svg file and add
Requires: hicolor-icon-theme
and change in %files as
%{_datadir}/icons/hicolor/scalable/apps/easystroke.svg

-- 
Configure 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.

2009-06-01 Thread 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 #47 from Paulo Roma Cavalcanti pro...@gmail.com  2009-06-01 
05:38:06 EDT ---
(In reply to comment #46)
 (In reply to comment #44)
  (In reply to comment #43)
 ..
  I am also using x86_64. Here is the picture:
  http://orion.lcg.ufrj.br/temp/blender-yafaray.png
 I'm using a modified version of blender-2.48a-22 and that's the first time I
 see it, I will retry with an unmodified version from F-11 x86_64.

I am using F10 x86_64.

 
  That is how upstream is doing. They moved from 1.0 - 305 to 1.0 - 315:
 Okay, that was my bad, the version is indeed 0.1.0.315.
 
  I have posted a message in yafaray Building forum and did not receive a
  single reply.
 Please reminds that the docs is the source code itself at the end. If you
 cannot have a
 
   The current package provided above doesn't work at all.  
  
  What do you mean by that? It is working for me just fine. Have you tried
  it? Does it segfault?  
 Funny that you talk about segfault, because there is indeed one. Quoting:
 
 creating new QApplication
 /usr/bin/blender: line 88:  7113 Segmentation fault  /usr/bin/${blend}.bin
 $@
 

I used to have segfaults before the 1.0.305 version. It worked with F8 but not
with F10. I think it was because of different xorg versions. 
Since the official 1.0 release, I did not have any problem at all with F10,
though.

What video card are you using?  



 
 Now what remains weird, is that even before the segfault the yafaray plugin
 doesn't seems to remind the previous configuration. (example: for using the 
 xml
 parser for rendering).  

I would suggest that you try on F10 first, if possible. Then, you go to F11
so we can compare the results.

There are packages for F9 to F11. Hopefully, we can get some people willing
to test them.

-- 
Configure 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 503482] New: Review Request: sblim-cmpi-nfsv4 - SBLIM nfsv4 instrumentation

2009-06-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sblim-cmpi-nfsv4 - SBLIM nfsv4 instrumentation

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

   Summary: Review Request: sblim-cmpi-nfsv4 - SBLIM nfsv4
instrumentation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vcrho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-nfsv4/sblim-cmpi-nfsv4.spec
SRPM URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-nfsv4/sblim-cmpi-nfsv4-1.0.12-1.fc10.src.rpm
Description: Standards Based Linux Instrumentation Nfsv4 Providers

-- 
Configure 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 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |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 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189





--- Comment #4 from Parag AN(पराग) panem...@gmail.com  2009-06-01 06:08:06 
EDT ---
Koji scratch build link is for python-application package.



Review:
+ package builds in mock (rawhide i586).
koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1386637
+ rpmlint is NOT silent for RPM.
== See suggestion2.
+ source files match upstream url
599bba969380fadc719fb49a94e5efd95084637d  python-gnutls-1.1.8.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package python-gnutls-1.1.8-2.fc12.i586 =
Provides: _gnutls_init.so
Requires: /usr/bin/env libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3)
libgnutls-extra.so.26 libgnutls-extra.so.26(GNUTLS_1_4) libgnutls.so.26
libgnutls.so.26(GNUTLS_1_4) libpthread.so.0 libpthread.so.0(GLIBC_2.0)
libpython2.6.so.1.0 python(abi) = 2.6 rtld(GNU_HASH)
+ Not a GUI application

Suggestions:
1) use %global instead %define
2) Add to %prep
chmod 644 examples/*.py
to silent rpmlint messages
python-gnutls.i586: W: spurious-executable-perm
/usr/share/doc/python-gnutls-1.1.8/examples/twisted-server.py
python-gnutls.i586: W: spurious-executable-perm
/usr/share/doc/python-gnutls-1.1.8/examples/client.py
python-gnutls.i586: W: spurious-executable-perm
/usr/share/doc/python-gnutls-1.1.8/examples/server.py
python-gnutls.i586: W: spurious-executable-perm
/usr/share/doc/python-gnutls-1.1.8/examples/twisted-client.py
python-gnutls.i586: W: spurious-executable-perm
/usr/share/doc/python-gnutls-1.1.8/examples/crypto.py
python-gnutls.i586: W: doc-file-dependency
/usr/share/doc/python-gnutls-1.1.8/examples/twisted-server.py /usr/bin/env
python-gnutls.i586: W: doc-file-dependency
/usr/share/doc/python-gnutls-1.1.8/examples/client.py /usr/bin/env
python-gnutls.i586: W: doc-file-dependency
/usr/share/doc/python-gnutls-1.1.8/examples/server.py /usr/bin/env
python-gnutls.i586: W: doc-file-dependency
/usr/share/doc/python-gnutls-1.1.8/examples/twisted-client.py /usr/bin/env
python-gnutls.i586: W: doc-file-dependency
/usr/share/doc/python-gnutls-1.1.8/examples/crypto.py /usr/bin/env

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 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031





--- Comment #4 from Zarko (grof) zarko.pin...@gmail.com  2009-06-01 06:18:15 
EDT ---
done.

Spec URL: http://wiki.open.hr/~zpintar/fedora-11/SPECS/easystroke.spec
SRPM URL:
http://wiki.open.hr/~zpintar/fedora-11/SRPMS/easystroke-0.4.4-3.fc11.src.rpm


We do not need this:
Requires: hicolor-icon-theme

I built some packages witohut that and nobody compliant (include my 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 502477] Review Request: arista - Easy to use multimedia transcoder for the GNOME Desktop

2009-06-01 Thread 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=502477


Michal Nowak mno...@redhat.com changed:

   What|Removed |Added

 CC||mno...@redhat.com




--- Comment #4 from Michal Nowak mno...@redhat.com  2009-06-01 06:19:32 EDT 
---
Dunno the code logic but ripping out:

else:
raise IOError(_(Can't find %(path)s in any known prefix!) % {
path: path,
})

from /usr/lib/python2.6/site-packages/arista/utils.py makes you start the app
at least.

-- 
Configure 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 474787] Review Requrest: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787





--- Comment #15 from D Haley my...@yahoo.com  2009-06-01 06:50:13 EDT ---
I agree with all of the changes made in your SRPM, and support its promotion
pending review. I suppose this makes you co-maintainer? Do we need another
reviewer?

Here are koji builds for the above SRPM, F9 failed, but F10 and F11 are OK. F9
is relatively close to EOL, so I suppose its not a big problem.

F9 : http://koji.fedoraproject.org/koji/taskinfo?taskID=1386757
F10: http://koji.fedoraproject.org/koji/taskinfo?taskID=1386748
F11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1386762

If you consider to bring a GNU Autotools environment (and submit the 
corresponding patch upstream), and if I can help you, do not hesitate.  

I think it is not necessarily a good idea to bring a different build system to
STXXL, but instead to help upstream to improve their chosen method. I would
hesitate to recommend a change of build sys to upstream, unless they expressed
an interest in doing so. For this reason I do not have any plans to do such a
thing, unless upstream indicate they would be interested.

In any case, I don't see this as important to this package for this version.
Maybe in a newer upstream version after some petitioning...

-- 
Configure 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 474787] Review Requrest: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787





--- Comment #16 from D Haley my...@yahoo.com  2009-06-01 06:57:43 EDT ---
Hmm I spoke a bit too soon. The use of noarch is causing problems with rpmlint,
as noarch is being applied to the main package. If we remove noarch from the
-doc section, all is well.

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

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


[Bug 474787] Review Requrest: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787





--- Comment #17 from Peter Lemenkov lemen...@gmail.com  2009-06-01 07:05:31 
EDT ---
(In reply to comment #12)

 Note that, as I have not the sponsor status, that is not an official review.
 However, there are a number of issues (some important, some others less) that 
 I

Denis, feel free to reassign this ticket to yourself (and, therefore, proceed
with reviewing further). As you're a packager, then you may do the formal
review (actually, you're almost finished reviewing 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 502986] Review Request: ytree- A filemanager similar to XTree

2009-06-01 Thread 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=502986


Minto Joseph mvali...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189





--- Comment #5 from Peter Lemenkov lemen...@gmail.com  2009-06-01 07:16:10 
EDT ---
Done.

http://peter.fedorapeople.org/python-gnutls.spec
http://peter.fedorapeople.org/python-gnutls-1.1.8-2.fc10.src.rpm

New Package CVS Request
===
Package Name: python-gnutls
Short Description: Python wrapper for the GNUTLS library
Owners: peter
Branches: F-10 F-11
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 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #5 from Parag AN(पराग) panem...@gmail.com  2009-06-01 07:17:32 
EDT ---
you can find packages requiring hicolor-icon-theme using
repoquery -q --whatrequires hicolor-icon-theme

But, anyway hicolor-icon-theme is in the dependency chain, don't add it.
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 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 226209] Merge Review: nut

2009-06-01 Thread 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=226209


Tomas Smetana tsmet...@redhat.com changed:

   What|Removed |Added

 CC|tsmet...@redhat.com |mhlav...@redhat.com




--- Comment #3 from Tomas Smetana tsmet...@redhat.com  2009-06-01 07:18:46 
EDT ---
Adding the current maintainer to cc:

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

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


[Bug 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031


Zarko (grof) zarko.pin...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031





--- Comment #6 from Zarko (grof) zarko.pin...@gmail.com  2009-06-01 07:21:41 
EDT ---
Thank you for review.

New Package CVS Request
===
Package Name: easystroke
Short Description: Gesture-recognition application for X11
Owners: grof
Branches: F9 F10 F11
InitialCC: grof

-- 
Configure 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 503490] New: Review Request: ayttm Universal Instant Messaging Client

2009-06-01 Thread 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: ayttm Universal Instant Messaging Client

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

   Summary: Review Request: ayttm Universal Instant Messaging
Client
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mvali...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mintojoseph.fedorapeople.org/packages/newguide/ayttm.spec
SRPM URL:
http://mintojoseph.fedorapeople.org/packages/newguide/ayttm-0.5.0-89.fc11.src.rpm
Description: 
Ayttm is designed to become a Universal Instant Messaging client, seamlessly
integrating all existing Instant Messaging clients and providing a single
consistant user interface. Currently, Ayttm supports sending and receiving
messages through AOL, ICQ, Yahoo, MSN, IRC and Jabber.

-- 
Configure 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 503490] Review Request: ayttm - Universal Instant Messaging Client

2009-06-01 Thread 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=503490


Minto Joseph mvali...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: ayttm   |Review Request: ayttm -
   |Universal Instant Messaging |Universal Instant Messaging
   |Client  |Client




-- 
Configure 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 484934] Review Request: vidalia - QT-GUI for tor

2009-06-01 Thread 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=484934


Simon Wesp cassmod...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #18 from Simon Wesp cassmod...@fedoraproject.org  2009-06-01 
08:16:29 EDT ---
Package Change Request
==
Package Name: vidalia
New Branches: EL-5
Owners: cassmodiah

-- 
Configure 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 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #2 from Robert Scheck redhat-bugzi...@linuxnetz.de  2009-06-01 
08:08:23 EDT ---
Parag, can you please point out, where my spec file does not match with the
https://fedoraproject.org/wiki/Packaging/Perl guidelines? The following list
compares https://fedoraproject.org/wiki/Packaging/Perl with my spec file where
NA means that it doesn't applicate here and -- that I'm missing it. The
only thing, I'm really missing is %check, that's right. And that's solved in
-2 now.

[OK] License tag
[OK] Directory Ownership
[OK] Perl Requires and Provides
[OK] Core modules as buildrequires
[OK] Versioned MODULE_COMPAT_ Requires
[NA] Filtering Requires: and Provides
[OK] Manual Requires and Provides
[OK] URL tag
[--] Testing and Test Suites
[NA] When to *not* test
[NA] Conditionally enabling/disabling tests
[OK] Makefile.PL vs Build.PL
[NA] .h files in module packages

Why should I use cpanspec if my spec file is valid and follows the policies?


Spec URL: http://labs.linuxnetz.de/bugzilla/perl-Crypt-GPG.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/perl-Crypt-GPG-1.63-2.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 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175


Paul Howarth p...@city-fan.org changed:

   What|Removed |Added

 CC||p...@city-fan.org




--- Comment #3 from Paul Howarth p...@city-fan.org  2009-06-01 08:28:51 EDT 
---
Looks reasonable to me except that you shouldn't be generating and shipping the
COPYING and Artistic files since they're not included in the upstream
distribution.

-- 
Configure 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 226508] Merge Review: tvtime

2009-06-01 Thread 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=226508





--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-06-01 08:40:10 
EDT ---
rpmlint output is clean.


MUST: The spec file for the package is legible and macros are used
consistently. NEEDSWORK
- You are using %{buildroot} and $RPM_OPT_FLAGS, please change the latter to
%{optflags}.

MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK

MUST: The License field in the package spec file must match the actual license.
OK
- Some files are  MIT licensed (e.g. src/xfullscreen.{c,h}), but MIT license is
compatible with GPL/LGPL.
- Also LGPL is compatible with GPL, so license tag could in principle be just
GPLv2+.
- Current GPLv2+ and LGPLv2+ is most informative, so it's best to keep it as it
is.

MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. OK

MUST: Optflags are used and time stamps preserved. NEEDSWORK
- Time stamps are not conserved in install, add INSTALL=install -p as
argument to make install.

MUST: Packages containing shared library files must call ldconfig. N/A

MUST: A package must own all directories that it creates or require the package
that owns the directory. NEEDSWORK
- Package must Requires: hicolor-icon-theme for dir ownership. (no
Requires(post): and Requires(postun): hicolor-icon-theme, should be just
Requires: hicolor-icon-theme).

MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK

MUST: Permissions on files must be set properly. NEEDSWORK
- I don't think directories can be listed as config. I'd change
 %config(noreplace) %{_sysconfdir}/tvtime/
to
 %dir %{_sysconfdir}/tvtime/
 %config(noreplace) %{_sysconfdir}/tvtime/tvtime.xml

MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. NEEDSWORK
- Include COPYING.LGPL.

MUST: Header files must be in a -devel package. N/A
MUST: Static libraries must be in a -static package. N/A
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. N/A
MUST: Packages does not contain any .la libtool archives. N/A
MUST: Desktop files are installed properly. OK
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK

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

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


[Bug 503495] New: Review Request: sblim-cim-client - Java CIM Client library

2009-06-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sblim-cim-client - Java CIM Client library

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

   Summary: Review Request: sblim-cim-client - Java CIM Client
library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vcrho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://vcrhonek.fedorapeople.org/sblim-cim-client/sblim-cim-client.spec
SRPM URL:
http://vcrhonek.fedorapeople.org/sblim-cim-client/sblim-cim-client-1.3.3-1.fc10.src.rpm
Description: 
The purpose of this package is to provide a CIM Client Class Library for Java
applications. It complies to the DMTF standard CIM Operations over HTTP and
intends to be compatible with JCP JSR48 once it becomes available. To learn
more about DMTF visit http://www.dmtf.org.
More infos about the Java Community Process and JSR48 can be found at
http://www.jcp.org and http://www.jcp.org/en/jsr/detail?id=48.

-- 
Configure 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 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #4 from Robert Scheck redhat-bugzi...@linuxnetz.de  2009-06-01 
08:44:44 EDT ---
According to bug #225735 comment #9 from Fedora Legal it doesn't really matter 
which license file is shipped within the RPM package. There upstream shipped a
GPLv3 license file for a GPLv2-only software. And at bug #193960 comment #6 I 
even got asked to generate the missing license file to avoid confusion, which
is the same situation here IMHO. I already sent an upstream request to add this
file for newer releases.

-- 
Configure 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 503496] New: Review Request: sblim-cmpi-params - SBLIM params instrumentation

2009-06-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sblim-cmpi-params - SBLIM params instrumentation

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

   Summary: Review Request: sblim-cmpi-params - SBLIM params
instrumentation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vcrho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-params/sblim-cmpi-params.spec
SRPM URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-params/sblim-cmpi-params-1.2.6-1.fc10.src.rpm
Description: Standards Based Linux Instrumentation Params Providers

-- 
Configure 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 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #5 from Paul Howarth p...@city-fan.org  2009-06-01 08:59:12 EDT 
---
The request in which you were asked to generate the missing file is somewhat
dated - in fact it was common practice amongst the perl SIG at that time to add
the files. However, since then the packaging guidelines have changed and now
quite clearly state that license text should be included if and only if
upstream includes them:

https://fedoraproject.org/wiki/Packaging:ReviewGuidelines

MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc

-- 
Configure 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 503510] New: Review Request: sblim-cmpi-sysfs - SBLIM sysfs instrumentation

2009-06-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: sblim-cmpi-sysfs - SBLIM sysfs instrumentation

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

   Summary: Review Request: sblim-cmpi-sysfs - SBLIM sysfs
instrumentation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: vcrho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-sysfs/sblim-cmpi-sysfs.spec
SRPM URL:
http://vcrhonek.fedorapeople.org/sblim-cmpi-sysfs/sblim-cmpi-sysfs-1.1.9-1.fc10.src.rpm
Description: Standards Based Linux Instrumentation Sysfs Providers

-- 
Configure 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 503519] New: Review Request: bastet - An evil Tetris like game

2009-06-01 Thread 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: bastet - An evil Tetris like game

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

   Summary: Review Request: bastet - An evil Tetris like game
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cheekybo...@foresightlinux.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:

http://cheekyboinc.fedorapeople.org/bastet.spec

SRPM URL:

http://cheekyboinc.fedorapeople.org/bastet-0.43-1.fc11.src.rpm

Description:

Bastet stands for bastard tetris, and is a simple ncurses-based 
Tetris(R) clone for Linux. Unlike normal Tetris(R), however, Bastet 
does not choose your next brick at random. Instead, it uses a special 
algorithm designed to choose the worst brick possible. As you can 
imagine, playing Bastet can be a very frustrating experience!

rpmlint: no output
scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1387035
md5sum: 

local b47090daa7b6d89b98b5b477cf155733  bastet-0.43.tgz
upstream b47090daa7b6d89b98b5b477cf155733  bastet-0.43.tgz

-- 
Configure 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 503174] Review Request: python-application - Basic building blocks for python applications

2009-06-01 Thread 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=503174


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review?




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

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


[Bug 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #6 from Parag AN(पराग) panem...@gmail.com  2009-06-01 11:05:49 
EDT ---
(In reply to comment #5)
 The request in which you were asked to generate the missing file is somewhat
 dated - in fact it was common practice amongst the perl SIG at that time to 
 add
 the files. However, since then the packaging guidelines have changed and now
 quite clearly state that license text should be included if and only if
 upstream includes them:
 
 https://fedoraproject.org/wiki/Packaging:ReviewGuidelines
 
 MUST: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s) for the
 package must be included in %doc  

I agree here. This is one point that I see capnspec does not needed to create
manually COPYING and Artistic files.

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

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


[Bug 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #7 from Robert Scheck redhat-bugzi...@linuxnetz.de  2009-06-01 
11:11:56 EDT ---
I'll drop COPYING and Artistic files generation, if that makes you happy.
What else has to be changed or work needs to happen?

-- 
Configure 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 492224] Review Request: gnome-mud - MUD client for GNOME desktop

2009-06-01 Thread 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=492224


Sean Middleditch s...@middleditch.us changed:

   What|Removed |Added

   Flag|needinfo?(s...@middleditch. |
   |us) |




--- Comment #8 from Sean Middleditch s...@middleditch.us  2009-06-01 11:21:16 
EDT ---
Still on it, I've been a bit swamped getting ready for a cross-country move.

-- 
Configure 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 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm

2009-06-01 Thread 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=503256





--- Comment #23 from Michael Schwendt mschwe...@gmail.com  2009-06-01 
11:40:28 EDT ---
 (actually gtkmm24-devel pulls in also glibmm24-devel, so basically
 you need only BR: gtkmm24-devel)

Trimming down BR is a waste of time. Please don't suggest it. There have been
several cases where it has lead to problems (when files were moved, packages
renamed, inter-package dependencies changed). Remember that the spec file's
BuildRequires become the src.rpm's Requires, which you may want to query with
tools. And gtkmm-utils looks for the glibmm-2.4.pc and gtkmm-2.4.pc pkgconfig
files, so your previous suggestion...

 - The package should only
  BuildRequires: gtkmm24-devel glibmm24-devel

... is fine.

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

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


[Bug 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #8 from Parag AN(पराग) panem...@gmail.com  2009-06-01 11:59:55 
EDT ---
(In reply to comment #2)
 
 Why should I use cpanspec if my spec file is valid and follows the policies?
 
1)Can you please tell me why its needed to use PREFIX=%{_prefix} in %build.
2) according to standard perl spec template, I see that %install should use
make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2/dev/null ';'
chmod -R u+w $RPM_BUILD_ROOT/*

I will leave this to you. If you are unhappy with this do tell 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 478930] Review Request: globus-rls-client - Globus Toolkit - Replica Location Service Client

2009-06-01 Thread 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=478930


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #5 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-06-01 
12:11:25 EDT ---
Thanks!


This package (globus-rls-client) is APPROVED by oget


-- 
Configure 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 491980] Review Request: bauble - Biodiversity collection manager

2009-06-01 Thread 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=491980


Simon Wesp cassmod...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CANTFIX




--- Comment #8 from Simon Wesp cassmod...@fedoraproject.org  2009-06-01 
12:10:13 EDT ---
thx for your review Pierre, 

I tried to contact upstream more than twice. no reaction..
I haven't the elan to solve for this issue. Let's close 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 503519] Review Request: bastet - An evil Tetris like game

2009-06-01 Thread 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=503519


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 CC||tcall...@redhat.com
 Blocks||182235(FE-Legal)




--- Comment #1 from Tom spot Callaway tcall...@redhat.com  2009-06-01 
12:11:17 EDT ---
Yeah... so you're going to have to remove the visible references to the
Tetris trademark here. The rule of thumb is:

If it is visible to the end user, it needs to go away.

This includes your spec Summary and Description, any places in the code where
Tetris is displayed to the user, and any references in the documentation.

Once you've done that, please post an updated SRPM and I'll audit 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 503175] Review Request: perl-Crypt-GPG - Perl Object Oriented Interface to GnuPG

2009-06-01 Thread 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=503175





--- Comment #9 from Robert Scheck redhat-bugzi...@linuxnetz.de  2009-06-01 
12:45:58 EDT ---
Usually using pure_install fails on RHEL 4, looks like this package is an
exception compared to others, tested it explicitly on RHEL 4 few minutes ago.


Spec URL: http://labs.linuxnetz.de/bugzilla/perl-Crypt-GPG.spec
SRPM URL: http://labs.linuxnetz.de/bugzilla/perl-Crypt-GPG-1.63-3.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 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm

2009-06-01 Thread 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=503256





--- Comment #24 from Gareth John gareth.l.j...@googlemail.com  2009-06-01 
14:06:41 EDT ---
---
* Mon Jun 01 2009 Gareth John gareth.l.j...@googlemail.com 0.4.0-6
- Added doc pack/description/files
- Enabled doc build in configure
- Added doxygen buildreq for -doc
- Added glibmm24-devel to BR
- Added examples pkg
- Added move commands under install
- Added mkdir command under install

Rpmlint says:

$ rpmlint -i RPMS/i386/gtkmm-utils-example-0.4.0-6.fc10.i386.rpm 
gtkmm-utils-example.i386: W: only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.

gtkmm-utils-example.i386: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.

1 packages and 0 specfiles checked; 0 errors, 2 warnings.

$ rpmlint -i RPMS/i386/gtkmm-utils-doc-0.4.0-6.fc10.i386.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint RPMS/i386/gtkmm-utils-devel-0.4.0-6.fc10.i386.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint RPMS/i386/gtkmm-utils-0.4.0-6.fc10.i386.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint SRPMS/gtkmm-utils-0.4.0-6.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
-
Files:
http://garethsrpms.googlecode.com/files/gtkmm-utils.spec
http://garethsrpms.googlecode.com/files/gtkmm-utils-0.4.0-6.fc10.src.rpm
-
I followed Michaels comment on the BRs and edited them. Also added the doc
subpackage which includes the docs for developers, this also made me add
doxygen as a BR. Following on from previous comments i assume that the one
warning in examples is justified as it would just be replicated doc from main
package. Also the dir i put it in following comments threw up a warning again i
assume this is 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 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

2009-06-01 Thread 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=492895


Antti Andreimann antti.andreim...@mail.ee changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Antti Andreimann antti.andreim...@mail.ee  2009-06-01 
14:15:16 EDT ---
Thank You!

New Package CVS Request
===
Package Name: xml-security-c
Short Description: C++ Implementation of W3C security standards for XML
Owners: anttix
Branches: F-10 F-11 EL-5
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 491980] Review Request: bauble - Biodiversity collection manager

2009-06-01 Thread 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=491980





--- Comment #9 from Pierre-YvesChibon pin...@pingoured.fr  2009-06-01 
14:15:32 EDT ---
It's a pity but I understand.

If you ever manage to get upstream in touch and able to make Bauble working on
the latest version of Fedora I will be happy to re-do the review :)

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

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


[Bug 503256] Review Request: gtkmm-utils - C++ utility and widget library based on glibmm and gtkmm

2009-06-01 Thread 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=503256





--- Comment #25 from Gareth John gareth.l.j...@googlemail.com  2009-06-01 
14:18:22 EDT ---
Also just added brackets to email address in changelog, file has been updated.
Further to this checked spec file in rpmlint and got no errors or warnings.

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

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


[Bug 484934] Review Request: vidalia - QT-GUI for tor

2009-06-01 Thread 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=484934


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #19 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 14:21:53 
EDT ---
CVS done.

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

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


[Bug 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

2009-06-01 Thread 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=492895


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #12 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 14:23:05 
EDT ---
CVS done.

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

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


[Bug 503031] Review Request: easystroke - Gesture-recognition application for X11

2009-06-01 Thread 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=503031


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 14:24:43 EDT 
---
Please note branch names are F-9, F-10, F-11.

CVS done.

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

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


[Bug 502986] Review Request: ytree- A filemanager similar to XTree

2009-06-01 Thread 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=502986


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #12 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 14:26:08 
EDT ---
CVS done.

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

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


[Bug 503189] Review Request: python-gnutls - Python wrapper for the GNUTLS library

2009-06-01 Thread 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=503189


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 14:25:29 EDT 
---
CVS done.

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

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


[Bug 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop

2009-06-01 Thread 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=496635





--- Comment #18 from Paul Lange pala...@gmx.de  2009-06-01 14:34:17 EDT ---
What's the state of this? Anyone working on 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 478922] Review Request: globus-ftp-control - Globus Toolkit - GridFTP Client Control Library

2009-06-01 Thread 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=478922





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:02 EDT ---
globus-ftp-control-2.10-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/globus-ftp-control-2.10-1.fc10

-- 
Configure 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 478925] Review Request: globus-gass-transfer - Globus Toolkit - Globus Gass Transfer

2009-06-01 Thread 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=478925





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:06 EDT ---
globus-gass-transfer-3.4-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/globus-gass-transfer-3.4-1.fc10

-- 
Configure 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 478922] Review Request: globus-ftp-control - Globus Toolkit - GridFTP Client Control Library

2009-06-01 Thread 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=478922





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:08 EDT ---
globus-ftp-control-2.10-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/globus-ftp-control-2.10-1.fc9

-- 
Configure 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 478925] Review Request: globus-gass-transfer - Globus Toolkit - Globus Gass Transfer

2009-06-01 Thread 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=478925





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:16 EDT ---
globus-gass-transfer-3.4-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/globus-gass-transfer-3.4-1.fc9

-- 
Configure 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 478925] Review Request: globus-gass-transfer - Globus Toolkit - Globus Gass Transfer

2009-06-01 Thread 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=478925





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:12 EDT ---
globus-gass-transfer-3.4-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/globus-gass-transfer-3.4-1.fc11

-- 
Configure 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 478922] Review Request: globus-ftp-control - Globus Toolkit - GridFTP Client Control Library

2009-06-01 Thread 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=478922





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 14:33:13 EDT ---
globus-ftp-control-2.10-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/globus-ftp-control-2.10-1.fc11

-- 
Configure 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 490355] Review Request: emacs-jde - Emacs module for java develop

2009-06-01 Thread 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=490355


Jerry James loganje...@gmail.com changed:

   What|Removed |Added

 CC||loganje...@gmail.com




--- Comment #3 from Jerry James loganje...@gmail.com  2009-06-01 14:51:55 EDT 
---
The ChangeLog for the xemacs-packages-extra RPM has this entry:

* Fri Mar 03 2006 Ville Skyttä ville.skytta at iki.fi - 20051208-2
- Drop JDE; the full source for jde.jar doesn't seem to be available
  and even the included parts won't build with gcj eg. due to use of
  com.sun.* things.  Additionally it requires third party jars which are
  not currently available in Fedora Core or Extras.  So let's not inflict
  a crippled version of JDE on anyone; JDE users can install the upstream
  jde package using XEmacs package tools. (#180941)

How much of that is true for 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 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop

2009-06-01 Thread 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=496635





--- Comment #19 from Toshio Ernie Kuratomi a.bad...@gmail.com  2009-06-01 
14:57:55 EDT ---
Paul, if you want it, feel free to take over the review.  I've gotten suddenly
busy with the release looming.  (I can still sponsor if you are satisfied with
Mauricio'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 503586] New: Review Request: python-tgext-crud - Crud Controller Extension for TG2

2009-06-01 Thread 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:  python-tgext-crud - Crud Controller Extension for TG2

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

   Summary: Review Request:  python-tgext-crud - Crud Controller
Extension for TG2
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lmac...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://lmacken.fedorapeople.org/rpms/python-tgext-crud.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-tgext-crud-0.2.4-1.fc10.src.rpm
Description:  Crud Controller Extension for TG2

-- 
Configure 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 501393] Review Request: 389-dsgw - 389 Directory Server Gateway

2009-06-01 Thread 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=501393





--- Comment #3 from Jochen Schmitt joc...@herr-schmitt.de  2009-06-01 
15:34:03 EDT ---
Good:
+ License tag says GPLv2 now.

Bad:
- Header files has a reverence of special exceptions on the copyright notes,
which are not clarified.
- package doesn't contains a verbatin copy of the license text. The LICENSE
files only told anything about the GPL and down't match with the copyright note
of the source files, because the LICENSE files allow the use of a later version
of the GPL, but the copyright notes on the sources doesn't.

Please calrified the licensing situation of your 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 503590] New: Review Request: python-tgext-admin - Admin Controller add-on for basic TG identity model

2009-06-01 Thread 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: python-tgext-admin - Admin Controller add-on for basic 
TG identity model

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

   Summary: Review Request: python-tgext-admin - Admin Controller
add-on for basic TG identity model
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lmac...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://lmacken.fedorapeople.org/rpms/python-tgext-crud.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-tgext-crud-0.2.4-1.fc10.src.rpm
Description: Admin Controller add-on for basic TG identity model

-- 
Configure 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 503590] Review Request: python-tgext-admin - Admin Controller add-on for basic TG identity model

2009-06-01 Thread 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=503590





--- Comment #1 from Luke Macken lmac...@redhat.com  2009-06-01 15:39:45 EDT 
---
Sorry, wrong links:

Spec URL: http://lmacken.fedorapeople.org/rpms/python-tgext-admin.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-tgext-admin-0.2.4-1.fc10.src.rpm
Description: Admin Controller add-on for basic TG identity model

-- 
Configure 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 503586] Review Request: python-tgext-crud - Crud Controller Extension for TG2

2009-06-01 Thread 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=503586


John (J5) Palmieri jo...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jo...@redhat.com




--- Comment #1 from John (J5) Palmieri jo...@redhat.com  2009-06-01 15:40:02 
EDT ---
rpmlint python-tgext-crud-0.2.4-1.fc9.noarch.rpm
python-tgext-crud.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Should probably have a readme and a copying file at the least

-- 
Configure 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 503591] New: Review Request: python-catwalk - A way to view your models using TurboGears

2009-06-01 Thread 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: python-catwalk - A way to view your models using 
TurboGears

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

   Summary: Review Request: python-catwalk - A way to view your
models using TurboGears
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lmac...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://lmacken.fedorapeople.org/rpms/python-catwalk.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-catwalk-2.0.2-1.fc10.src.rpm

Description:
Catwalk is a database manipulation application built on top of TurboGears.
With it you can easily interact with your application data model and manage
your data.

-- 
Configure 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 503586] Review Request: python-tgext-crud - Crud Controller Extension for TG2

2009-06-01 Thread 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=503586





--- Comment #2 from John (J5) Palmieri jo...@redhat.com  2009-06-01 15:47:50 
EDT ---
No need to add the docs but upstream should be alerted

-- 
Configure 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 503592] Review Request: python-sprox - A package for creation of web widgets directly from database schema

2009-06-01 Thread 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=503592


Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |python-sproxy - A package   |python-sprox - A package
   |for creation of web widgets |for creation of web widgets
   |directly from database  |directly from database
   |schema  |schema




-- 
Configure 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 503592] New: Review Request: python-sproxy - A package for creation of web widgets directly from database schema

2009-06-01 Thread 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: python-sproxy - A package for creation of web widgets 
directly from database schema

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

   Summary: Review Request: python-sproxy - A package for creation
of web widgets directly from database schema
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lmac...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://lmacken.fedorapeople.org/rpms/python-sprox.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-sprox-0.5.5-1.fc10.src.rpm
Description: 
Sprox is a widget generation library that has a slightly different take on the
problem of creating custom web content directly from database schemas. Sprox
provides an easy way to create forms for web content which are: automatically
generated, easy to customize, and validated. Sprox also has powerful tools to
help you display your content the way you want to with table and record
viewers. Sprox provides a way to fill your widgets, whether they are forms or
other content with customizable data.

-- 
Configure 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 501393] Review Request: 389-dsgw - 389 Directory Server Gateway

2009-06-01 Thread 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=501393





--- Comment #4 from Rich Megginson rmegg...@redhat.com  2009-06-01 15:56:01 
EDT ---
Can you please verify the md5sum of the 389-dsgw-1.1.2.tar.bz2 you are looking
at? All of those licensing issues have been resolved.  The LICENSE file now
contains the full text of the GPLv2.  You should have this file:
md5sum  389-dsgw-1.1.2.tar.bz2 
0eed1a7a66ce7d77895067e31e064445  389-dsgw-1.1.2.tar.bz2

-- 
Configure 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 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

2009-06-01 Thread 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=492895





--- Comment #13 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 15:56:10 EDT ---
xml-security-c-1.4.0-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/xml-security-c-1.4.0-2.fc10

-- 
Configure 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 492895] Review Request: xml-security-c - C++ Implementation of W3C security standards for XML

2009-06-01 Thread 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=492895





--- Comment #14 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 15:56:16 EDT ---
xml-security-c-1.4.0-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/xml-security-c-1.4.0-2.fc11

-- 
Configure 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 503301] Review Request: vim-nerdtree - Tree File Browser for VIM

2009-06-01 Thread 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=503301


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)




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

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


[Bug 503586] Review Request: python-tgext-crud - Crud Controller Extension for TG2

2009-06-01 Thread 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=503586





--- Comment #3 from John (J5) Palmieri jo...@redhat.com  2009-06-01 16:12:06 
EDT ---
*  MUST: rpmlint must be run on every package. The output should be posted
in the review.[1]

Warning waved - Passes with one warning about docs - waved

* MUST: The package must be named according to the Package Naming
Guidelines .

Pass

* MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. [2] .

Pass

* MUST: The package must meet the Packaging Guidelines .

Pass  

* MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines .

Pass - MIT

* MUST: The License field in the package spec file must match the actual
license. [3]

Pass

* MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.[4]

Pass - No license file included in tarball

* MUST: The spec file must be written in American English. [5]

Pass

* MUST: The spec file for the package MUST be legible. [6]

Pass

* MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.

Pass - md5sum 7dda7f32440bf74c2a9694bebdcf8a55

* MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture. [7]

Pass

* MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]

Pass - Noarch package

* MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.

Pass

* MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. [12]

Pass 

* MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. [13]

Pass

* MUST: Permissions on files must be set properly. Executables should be
set with executable permissions, for example. Every %files section must include
a %defattr(...) line. [14]

Pass

* MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT). [15]

Pass

* MUST: Each package must consistently use macros. [16]

Pass

* MUST: The package must contain code, or permissable content. [17]

Pass

* MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here is that the first package to be installed
should own the files or directories that other packages may rely upon. This
means, for example, that no package in Fedora should ever share ownership with
any of the files or directories owned by the filesystem or man package. If you
feel that you have a good reason to own a file or directory that another
package owns, then please present that at package review time. [24]

Pass

* MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} (or $RPM_BUILD_ROOT). [25]

Pass 

 * MUST: All filenames in rpm packages must be valid UTF-8. [26]

Pass

SHOULD Items:
Items marked as SHOULD are things that the package (or reviewer) SHOULD do, but
is not required to do.

* SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
[27]

Please query upstream if you have the chance

-- 
Configure 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 503300] Review Request: vim-c-support - C/C++ IDE for VIM

2009-06-01 Thread 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=503300


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)




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

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


[Bug 503586] Review Request: python-tgext-crud - Crud Controller Extension for TG2

2009-06-01 Thread 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=503586


John (J5) Palmieri jo...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+




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

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


[Bug 503598] New: Review Request: sphinxbase - Common library for CMU Sphinx voice recognition products

2009-06-01 Thread 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: sphinxbase - Common library for CMU Sphinx voice 
recognition products

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

   Summary: Review Request: sphinxbase - Common library for CMU
Sphinx voice recognition products
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: loganje...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://jjames.fedorapeople.org/sphinxbase/sphinxbase.spec
SRPM URL:
http://jjames.fedorapeople.org/sphinxbase/sphinxbase-0.4.1-1.fc10.src.rpm
Description: Sphinxbase is a common library for CMU Sphinx voice recognition
products.  This package does not provide voice recognition capabilities by
itself.

-- 
Configure 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 478930] Review Request: globus-rls-client - Globus Toolkit - Replica Location Service Client

2009-06-01 Thread 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=478930


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-06-01 
16:39:06 EDT ---
Thank you for the review.

New Package CVS Request
===
Package Name: globus-rls-client
Short Description: Globus Toolkit - Replica Location Service Client
Owners: ellert
Branches: F-9 F-10 F-11 EL-4 EL-5
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 503197] Review Request: moserial - Serial terminal for the gnome desktop

2009-06-01 Thread 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=503197


Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #9 from Terje Røsten terje...@phys.ntnu.no  2009-06-01 16:47:04 
EDT ---
Import, built and pushed to bodhi for rawhide and Fedora 11.

Turns out moserial needs gtk = 2.14.0 leaving Fedora  11 out.
If anyone needs it elsewhere, please open a new ticket.

-- 
Configure 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.

2009-06-01 Thread 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 #48 from Jochen Schmitt joc...@herr-schmitt.de  2009-06-01 
16:54:39 EDT ---
I want to inform you, that blender-2.49 will available in rawhide in the next
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 474787] Review Request: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787


Denis Arnaud denis.arnaud_fed...@m4x.org changed:

   What|Removed |Added

Summary|Review Requrest: stxxl -|Review Request: stxxl - C++
   |C++ STL drop-in replacement |STL drop-in replacement for
   |for extremely large |extremely large datasets
   |datasets|




-- 
Configure 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 490355] Review Request: emacs-jde - Emacs module for java develop

2009-06-01 Thread 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=490355





--- Comment #4 from Dan Atmakin beol...@gmail.com  2009-06-01 17:04:53 EDT ---
This is a package for emacs and in emacs pattern, this package is absolutely
all-sufficient and does not requires any special jars for correct work. Only
preinstalled jre (sun or another, it does not matter). I don't know how JDE
work in xemacs.

sorry but this srpm is rather ugly. I will recreate it according with
/Packaging/Emacs#GNU_Emacs guidelines. I can't do it immediately, but i'll try
to fix all problems in the near future.

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

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


[Bug 474787] Review Request: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787


Denis Arnaud denis.arnaud_fed...@m4x.org changed:

   What|Removed |Added

 AssignedTo|lemen...@gmail.com  |denis.arnaud_fed...@m4x.org




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

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


[Bug 478930] Review Request: globus-rls-client - Globus Toolkit - Replica Location Service Client

2009-06-01 Thread 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=478930


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-06-01 17:33:42 EDT 
---
CVS done.

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

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


[Bug 503519] Review Request: bastet - An evil falling bricks game

2009-06-01 Thread 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=503519


Stefan Posdzich cheekybo...@foresightlinux.org changed:

   What|Removed |Added

Summary|Review Request: bastet - An |Review Request: bastet - An
   |evil Tetris like game   |evil falling bricks game




--- Comment #2 from Stefan Posdzich cheekybo...@foresightlinux.org  
2009-06-01 17:35:08 EDT ---
Spec URL:

http://cheekyboinc.fedorapeople.org/bastet.spec

SRPM URL:

http://cheekyboinc.fedorapeople.org/bastet-0.43-2.fc11.src.rpm

- Removed reference to Tetris to match our guidelines

-- 
Configure 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 492397] Review Request: websvn - Online subversion repository browser

2009-06-01 Thread 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=492397





--- Comment #11 from Jan Klepek jan.kle...@hp.com  2009-06-01 17:48:34 EDT ---
i will do review today/tommorow, but so far it looks good, i will test if it
works correctly on my system.

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

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


[Bug 474787] Review Request: stxxl - C++ STL drop-in replacement for extremely large datasets

2009-06-01 Thread 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=474787





--- Comment #18 from Denis Arnaud denis.arnaud_fed...@m4x.org  2009-06-01 
18:14:00 EDT ---
That's fine, if you like, I shall be honoured to go on with the review (thanks
Peter and Michael!).

As for the failing build on Fedora 9, I just forgot to enclose the 'BuildArch:
noarch' directive within the '%if 0%{?fedora} = 10 [...] %endif' condition...

It now works well, as Koji shows:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1388361

Spec URL: http://denisarnaud.fedorapeople.org/stxxl/121/7/stxxl.spec
(as well as the corresponding Source RPM:
http://denisarnaud.fedorapeople.org/stxxl/121/7/stxxl-1.2.1-7.fc10.src.rpm)

As I have changed only the BuildArch directive, it does not imply any change on
Fedora 10, 11 and 12. So, if you agree with that new version, I shall approve
the package.



As for the build system, I proposed to ADD a new one, not to replace the
existing one. Hence, (part of) the upstream team may still work with their
current building system, if they are happy with it (though I do not understand
how they install that library, since there is apparently no such possibility in
the current build system!). And an additional GNU Autotools build system would
allow (plenty of) other developers to build, install and package in a standard
way, without too much maintenance. That is simply what I suggested, and see no
reason why the upstream team should be against.

Without such a standard build system, it means that, as the main maintainer of
the package, you will have more work every time a new upstream will be
released. But, that is my $0.02, and as far as I understand, it should not
interfere with Fedora packaging :)
---

-- 
Configure 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 478930] Review Request: globus-rls-client - Globus Toolkit - Replica Location Service Client

2009-06-01 Thread 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=478930





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-06-01 18:52:15 EDT ---
globus-rls-client-5.1-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/globus-rls-client-5.1-2.fc11

-- 
Configure 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


  1   2   >