[Bug 226277] Merge Review: perl-RPM-Specfile

2008-12-15 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=226277


Marcela Maslanova  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #7 from Marcela Maslanova   2008-12-15 
03:29:41 EDT ---
This package is needed in EPEL-{4,5} for perl-Satcon. I can maintain it for
EPEL, if you don't want it.

-- 
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 476387] Review Request: vhd2vl - VHDL to Verilog translator

2008-12-15 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=476387





--- Comment #3 from manuel wolfshant   2008-12-15 
03:35:25 EDT ---
Are you targeting only F10 ? rawhide build still fails:
http://koji.fedoraproject.org/koji/taskinfo?taskID=998903

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

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


[Bug 226277] Merge Review: perl-RPM-Specfile

2008-12-15 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=226277


Marcela Maslanova  changed:

   What|Removed |Added

   Flag|fedora-cvs? |




--- Comment #8 from Marcela Maslanova   2008-12-15 
03:44:23 EDT ---
Hm false alarm. This package have branches for RHEL, but it wasn't included
into comps.

-- 
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 475082] Review Request: guiloader - GuiXml Loader Library

2008-12-15 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=475082





--- Comment #8 from Fedora Update System   
2008-12-15 03:49:39 EDT ---
guiloader-2.13.0-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/guiloader-2.13.0-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 476414] Review Request: assogiate - Editor for the file types database

2008-12-15 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=476414


Fabian Affolter  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Fabian Affolter   2008-12-15 
03:52:18 EDT ---
New Package CVS Request
===
Package Name: assogiate
Short Description: Editor for the file types database
Owners: fab
Branches: F-9 F-10
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 462982] Review Request: buffer - General purpose buffer program

2008-12-15 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=462982


Bruno Cornec  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 474843] Review Request: pdfbook - Rearrange pages in a PDF file into signatures

2008-12-15 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=474843


Mary Ellen Foster  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 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818





--- Comment #3 from Marcela Maslanova   2008-12-15 
04:06:29 EDT ---
Thank you. BR was added.
Spec URL:
http://mmaslano.fedorapeople.org/perl-Class-XSAccessor/perl-Class-XSAccessor.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-Class-XSAccessor/perl-Class-XSAccessor-0.11-1.fc10.src.rpm


See http://koji.fedoraproject.org/koji/taskinfo?taskID=998951

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

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


[Bug 472027] Review Request: onboard - Simple on-screen Keyboard

2008-12-15 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=472027





--- Comment #11 from Fabian Affolter   2008-12-15 
04:17:35 EDT ---
Many thanks for the review.  The .desktop files are already present in the
trunk of the upstream VCS.

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

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


[Bug 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818





--- Comment #5 from Parag AN(पराग)   2008-12-15 04:18:53 
EDT ---
oops s/now/not

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

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


[Bug 472027] Review Request: onboard - Simple on-screen Keyboard

2008-12-15 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=472027


Fabian Affolter  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #12 from Fabian Affolter   2008-12-15 
04:18:57 EDT ---
New Package CVS Request
===
Package Name: onboard
Short Description: Simple on-screen Keyboard
Owners: fab
Branches: F-9 F-10
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 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818





--- Comment #4 from Parag AN(पराग)   2008-12-15 04:18:25 
EDT ---
Just saw that upstream source is now downloadable and new sources are already
in releases.
Can you resubmit this SRPM with new source
http://www.cpan.org/authors/id/S/SM/SMUELLER/Class-XSAccessor-0.14.tar.gz ?

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

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


[Bug 234831] Review Request: fgfs-base - FlightGear base scenery and data files

2008-12-15 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=234831


Fabrice Bellet  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #6 from Fabrice Bellet   2008-12-15 04:17:43 
EDT ---
New Package CVS Request
===
Package Name: FlightGear-data
Short Description: FlightGear base scenery and data files
Owners: fabr...@bellet.info
Branches:
InitialCC:

I'll rename this package, because upstream name changed, since version 1.99.5:
ftp://ftp.de.flightgear.org/pub/fgfs/Shared/

-- 
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 467179] Review Request: pngcrush - Optimizer for PNG (Portable Network Graphics) files

2008-12-15 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=467179





--- Comment #14 from Gerd Hoffmann   2008-12-15 04:23:47 EDT 
---
Yes, this referes to osmarender / tiles-at-home.

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

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


[Bug 472027] Review Request: onboard - Simple on-screen Keyboard

2008-12-15 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=472027





--- Comment #13 from Parag AN(पराग)   2008-12-15 04:31:46 
EDT ---
(In reply to comment #11)
> Many thanks for the review.  The .desktop files are already present in the
> trunk of the upstream VCS.

you welcome. Here I mean upstream .desktop files are different than Fedora
.desktop files now. So if you drop these .desktop files in next update for new
upstream release in spec file then existing .desktop files in fedora will get
overwritten.

see comment#2 suggestion 1.

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

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


[Bug 226506] Merge Review: ttmkfdir

2008-12-15 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=226506





--- Comment #8 from Pravin Satpute   2008-12-15 06:39:57 
EDT ---
Done
built it for rawhide
http://koji.fedoraproject.org/koji/taskinfo?taskID=999184

Thanks for completing merge review of ttmkfdir

-- 
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 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Parag AN(पराग)   2008-12-15 08:15:03 
EDT ---
Review:
+ package builds in mock (rawhide i386).
koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=999048
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
6cc5e214d9873f9205c2c49a1b7f3008  Class-XSAccessor-0.14.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.
+ make test outputs
All tests successful.
Files=6, Tests=65,  0 wallclock secs ( 0.02 usr  0.00 sys +  0.19 cusr  0.02
csys =  0.23 CPU)
+ Package perl-Class-XSAccessor-0.14-1.fc11 =>
  Provides: XSAccessor.so perl(Class::XSAccessor) = 0.14
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) perl >=
0:5.006 perl(Carp) perl(XSLoader) perl(strict) perl(warnings) rtld(GNU_HASH)

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 449037] Review Request: afio - cpio compatible archiver

2008-12-15 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=449037





--- Comment #22 from Tom "spot" Callaway   2008-12-15 
08:39:43 EDT ---
(In reply to comment #21)
> I'll try to contact someone I know at Sun to see if he can help me with this.
> Will let you know how it progresses. I'd like to wait a bit before closing it
> here.

Sure. Keep in mind that we'd need some statement from Sun that they do hold the
ownership of the copyright of this code.

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

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


[Bug 435276] Review Request: kBuild - A cross-platform build enviroment

2008-12-15 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=435276


Lubomir Rintel  changed:

   What|Removed |Added

  Status Whiteboard|NotReady|




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

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


[Bug 226415] Merge Review: sgml-common

2008-12-15 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=226415


Patrice Dumas  changed:

   What|Removed |Added

 AssignedTo|pertu...@free.fr|nob...@fedoraproject.org
   Flag|needinfo?   |




-- 
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 475107] Review Request: guiloader-c++ - C++ Binding to GuiLoader Library

2008-12-15 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=475107





--- Comment #2 from Mamoru Tasaka   2008-12-15 
09:15:50 EDT ---
Please write CVS request.

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

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


[Bug 476536] New: Review Request: zapplet - Zenoss monitoring tray applet

2008-12-15 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: zapplet - Zenoss monitoring tray applet

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

   Summary: Review Request: zapplet - Zenoss monitoring tray
applet
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: da...@gnsa.us
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ke4qqq.fedorapeople.org/zapplet.spec
SRPM URL: http://ke4qqq.fedorapeople.org/zapplet-0.1-1.src.rpm
Description: Zapplet is a tray applet for monitoring aspects of your Zenoss
installation from the desktop. The intention is to work from all desktops
supported by GTK.

I need a sponsor 


Here is the output from rpmlint:
[ke4...@nalleyt61 SPECS]$ rpmlint ./zapplet.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint ../SRPMS/zapplet-0.1-1.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint ../RPMS/noarch/zapplet-0.1-1.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 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 225693] Merge Review: dialog

2008-12-15 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=225693


Patrice Dumas  changed:

   What|Removed |Added

 AssignedTo|pertu...@free.fr|nob...@fedoraproject.org
   Flag|fedora-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 476475] Review Request: python-zope-filesystem - Python-Zope Libraries Base Filesystem

2008-12-15 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=476475


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476524




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

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


[Bug 226506] Merge Review: ttmkfdir

2008-12-15 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=226506





--- Comment #5 from Pravin Satpute   2008-12-15 05:39:05 
EDT ---
modified as per comments
updated spec file
http://pravins.fedorapeople.org/ttmkfdir/ttmkfdir.spec

updated srpm 
http://pravins.fedorapeople.org/ttmkfdir/ttmkfdir-3.0.9-28.fc10.src.rpm

rpmlint on srpm gives
ttmkfdir.src: W: no-url-tag

-- 
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 473452] Review Request: system-config-services-docs - Documentation for configuring system services

2008-12-15 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=473452





--- Comment #11 from Nils Philippsen   2008-12-15 09:30:56 
EDT ---
Fixed:

Spec: http://nphilipp.fedorapeople.org/review/system-config-services-docs.spec
SRPM:
http://nphilipp.fedorapeople.org/review/system-config-services-docs-1.1.3-1.fc10.src.rpm

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

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


[Bug 469471] Review Request: skinlf - Java look and feel for swing

2008-12-15 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=469471





--- Comment #21 from Mamoru Tasaka   2008-12-15 
09:42:27 EDT ---
Please also rebuild for dist-f11.

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

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


[Bug 475018] Review Request: xtvd - A client java library for easy access to the tv data from schedulesdirect.org

2008-12-15 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=475018


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #3 from Mamoru Tasaka   2008-12-15 
10:01:03 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 475508] Review Request: javassist - The Java Programming Assistant provides simple Java bytecode manipulation

2008-12-15 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=475508


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
09:59:38 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 475019] Review Request: opencsv - A very simple csv (comma-separated values) parser library for Java

2008-12-15 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=475019


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:00:15 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 467643] Review Request: sugar-speak - Speak for Sugar

2008-12-15 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=467643


Bryan Kearney  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #8 from Bryan Kearney   2008-12-15 09:58:49 
EDT ---
APPROVED

Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import
the package. Close this bug as RAWHIDE once it's been successfully
imported
and built.

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

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


[Bug 474981] Review Request: jcalendar - A Java date chooser bean for graphically picking a date

2008-12-15 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=474981


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 Blocks|177841  |
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review+




--- Comment #5 from Mamoru Tasaka   2008-12-15 
09:57:33 EDT ---
Okay, no problem.

-
   This package (jcalendar) is APPROVED by mtasaka
-

Please follow the procedure written on:
http://fedoraproject.org/wiki/PackageMaintainers/Join
from "Install the Client Tools (Koji) ".

Now I am sponsoring you.

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

Note that now you are using red_AT_fedoraproject.org as bugzilla
account, but using sm_AT_XX as FAS account mail address.
To set fedora-cvs flag (this is needed for review process, which
is written in "Join" wiki) or fedora-review flag (if you want to
review other person's review request), you have to create bugzilla
account with sm_AT_X mail address (if not exist) and login
bugzilla as sm_AT_X account.

If you have some question, please ask me.

Removing NEEDSPONSOR


If you have questions, please ask me.

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

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


[Bug 475017] Review Request: l2fprod-common - In JavaSE missing Swing components, inspired from modern user interfaces

2008-12-15 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=475017


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:01:49 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 474999] Review Request: gdata-java - Client libraries to write Google Data API client applications in Java

2008-12-15 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=474999


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:02:29 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 474985] Review Request: jakarta-commons-compress - API for working with tar, zip and bzip2 files

2008-12-15 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=474985


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:03:08 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 474983] Review Request: TVAnytimeAPI - A java API for parsing, manipulating and creating TV-Anytime metadata

2008-12-15 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=474983


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:05:19 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 474982] Review Request: nachocalendar - Provides a flexible Calendar component to the Java Platform

2008-12-15 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=474982


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:06:06 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 475471] Review Request: poi - Java API to Access Microsoft Format Files

2008-12-15 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=475471


Mamoru Tasaka  changed:

   What|Removed |Added

 Blocks|177841  |




--- Comment #2 from Mamoru Tasaka   2008-12-15 
10:07:04 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 472144] Review Request: tvbrowser - Free EPG for over 500 stations.

2008-12-15 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=472144


Mamoru Tasaka  changed:

   What|Removed |Added

 Depends on|177841  |




--- Comment #4 from Mamoru Tasaka   2008-12-15 
10:08:02 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 226506] Merge Review: ttmkfdir

2008-12-15 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=226506





--- Comment #6 from Pravin Satpute   2008-12-15 06:05:31 
EDT ---
updated spec file
http://pravins.fedorapeople.org/ttmkfdir/ttmkfdir.spec

updated srpm 
http://pravins.fedorapeople.org/ttmkfdir/ttmkfdir-3.0.9-28.fc10.src.rpm

-- 
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 475082] Review Request: guiloader - GuiXml Loader Library

2008-12-15 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=475082


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||UPSTREAM




--- Comment #9 from Mamoru Tasaka   2008-12-15 
09:16:12 EDT ---
Okay, thanks.

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

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


[Bug 476526] New: Review Request: python-zope-event - Zope Event Publication

2008-12-15 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-zope-event - Zope Event Publication

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

   Summary: Review Request: python-zope-event - Zope Event
Publication
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-zope-event.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zope-event-3.4.0-1.fc9.src.rpm
Description:
The zope.event package provides a simple event system. It provides
an event publishing system and a very simple event-dispatching system
on which more sophisticated event dispatching systems can be built.
(For example, a type-based event dispatching system that builds on
zope.event can be found in zope.component.)

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

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


[Bug 476526] Review Request: python-zope-event - Zope Event Publication

2008-12-15 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=476526


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476524




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

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


[Bug 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818





--- Comment #7 from Marcela Maslanova   2008-12-15 
06:25:03 EDT ---
http://koji.fedoraproject.org/koji/taskinfo?taskID=999048

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

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


[Bug 226506] Merge Review: ttmkfdir

2008-12-15 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=226506


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #9 from Parag AN(पराग)   2008-12-15 08:14:33 
EDT ---
Thanks for building package.

APPROVED This Merge-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 226417] Merge Review: shared-mime-info

2008-12-15 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=226417


Julian Sikorski  changed:

   What|Removed |Added

 CC|beleg...@gmail.com  |




--- Comment #4 from Julian Sikorski   2008-12-15 08:18:49 
EDT ---
I'm not a real maintainer, just did a drive-by fix.

-- 
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 476474] Review Request: ebook-tools - Tools for accessing and converting various ebook file formats

2008-12-15 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=476474


Marcela Maslanova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-review+




--- Comment #1 from Marcela Maslanova   2008-12-15 
07:24:25 EDT ---
I try to use it on command line, but no luck yet. lit2epub doesn't work,
because clit wasn't and maybe can't be in our repo. 

Anyway it looks like the libepub will be working well with okular
(http://okular.kde.org/download.php). 

OK source files match upstream: 
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field MIT matches the actual license.
OK license is open source-compatible. License text not included upstream.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock (Rawhide/x86_64).
http://koji.fedoraproject.org/koji/taskinfo?taskID=999213
OK debuginfo package looks complete.
OK rpmlint is silent.
ebook-tools.x86_64: W: no-documentation
OK final provides and requires look sane.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.

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

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


[Bug 476528] New: Review Request: python-ZConfig - Structured Configuration Library

2008-12-15 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-ZConfig - Structured Configuration Library

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

   Summary: Review Request: python-ZConfig - Structured
Configuration Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-ZConfig.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-ZConfig-2.6.1-1.fc9.src.rpm
Description:
ZConfig is a configuration library intended for general use. It supports
a hierarchical schema-driven configuration model that allows a schema to
specify data conversion routines written in Python. ZConfig's model is
very different from the model supported by the ConfigParser module found
in Python's standard library, and is more suitable to
configuration-intensive applications.

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

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


[Bug 472027] Review Request: onboard - Simple on-screen Keyboard

2008-12-15 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=472027





--- Comment #14 from Fabian Affolter   2008-12-15 
04:39:21 EDT ---
Now, I get the point. I will get in touch with upstream to see if they are kind
enough to change the .desktop files to our needs.

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

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


[Bug 474193] Review Request: glog - A C++ application logging library

2008-12-15 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=474193





--- Comment #12 from Fedora Update System   
2008-12-15 10:43:26 EDT ---
glog-0.1.2-6.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/glog-0.1.2-6.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 474193] Review Request: glog - A C++ application logging library

2008-12-15 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=474193


John A. Khvatov  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 476474] Review Request: ebook-tools - Tools for accessing and converting various ebook file formats

2008-12-15 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=476474


John5342  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from John5342   2008-12-15 10:56:34 EDT 
---
New Package CVS Request
===
Package Name: ebook-tools
Short Description: Tools for accessing and converting various ebook file
formats
Owners: john5342
Branches: F-9 F-10
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 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476523




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

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


[Bug 473318] Review Request: system-config-nfs-docs - Documentation for configuring an NFS server

2008-12-15 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=473318





--- Comment #2 from Nils Philippsen   2008-12-15 09:32:19 
EDT ---
New files:

Spec: http://nphilipp.fedorapeople.org/review/system-config-nfs-docs.spec
SRPM:
http://nphilipp.fedorapeople.org/review/system-config-nfs-docs-1.0.3-1.fc10.src.rpm

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

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


[Bug 439630] Review Request: jogl - Java bindings for OpenGL

2008-12-15 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=439630


Levente Farkas  changed:

   What|Removed |Added

 CC||lfar...@lfarkas.org




--- Comment #25 from Levente Farkas   2008-12-15 11:04:53 
EDT ---
does it means lwjgl also good for fedora?

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

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


[Bug 476525] Review Request: python-zc-lockfile - Basic Inter-Process Locks

2008-12-15 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=476525


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476524




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

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


[Bug 226417] Merge Review: shared-mime-info

2008-12-15 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=226417





--- Comment #5 from Orcan 'oget' Ogetbil   2008-12-15 
11:03:02 EDT ---
Thanks for the update. Everything looks fine. I just have two more comments.

> > * The BR perl-XML-Parser >= 2.31-16 is not used at all and can be removed. 
> > Am I
> > wrong?
> 
> Needed for intltool, I change it to "perl(XML::Parser)" though.
> 

Ah, I noticed that now. Just a side-note: BR'ing "intltool" will then pull up
"perl(XML::Parser)", so still, explicitly BR'ing "perl(XML::Parser)" is not
required. But if you think this dependency might change in the future, I'm OK
with leaving it as it is.

> > * The files ChangeLog, HACKING and most importantly COPYING need to be 
> > listed
> > under %doc.
> 
> Added HACKING and COPYING, not ChangeLog, as it replicates data from NEWS
> whilst being much bigger.
> 

Does NEWS contain all the relevant information from ChangeLog (both from a
user's and a developer's point of view)?

-- 
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 475593] Review Request: fontpackages - Common directory and macro definitions used by font packages

2008-12-15 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=475593





--- Comment #9 from Nicolas Mailhot   2008-12-15 
04:40:09 EDT ---
This is for new fonts. A lot of packagers will only work on new packages if
they can import in stable release without waiting F11 time.

I will actively discourage anyone who suggests converting existing F9 and  F10
packages to the new templates.

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

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


[Bug 466147] Review Request:fedora-ksplice - Script Collection for Using KSplice on Fedora Linux

2008-12-15 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=466147





--- Comment #6 from Jochen Schmitt   2008-12-15 
11:21:56 EDT ---
New upstream release:

Spec URL: http://www.herr-schmitt.de/pub/fedora-ksplice/fedora-ksplice.spec
SRPM URL:
http://www.herr-schmitt.de/pub/fedora-ksplice/fedora-ksplice-0.5-1.fc10.src.rpm

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

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


[Bug 475410] Review Request: symmetrica - A Collection of Routines for Solving Symmetric Groups

2008-12-15 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=475410


Jerry James  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #4 from Jerry James   2008-12-15 11:25:04 EDT 
---
If this is truly intended to be consumed by SAGE only, then I can see being a
little more lax with it.  Are you certain that nobody else will want to consume
it?  Because if anybody does, I think you really will want to build a shared
library and have a separate -devel subpackage.

I'll do the review in a moment.  I don't know that fixing the warnings is
really all that important if they aren't indicative of runtime issues.

I've had no response from upstream yet.

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

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


[Bug 226506] Merge Review: ttmkfdir

2008-12-15 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=226506





--- Comment #7 from Parag AN(पराग)   2008-12-15 06:16:50 
EDT ---
package built successfully
http://koji.fedoraproject.org/koji/taskinfo?taskID=999153

rpmlint only complained 
ttmkfdir.src: W: no-url-tag
But I see that there is already comment as per
http://fedoraproject.org/wiki/Packaging/SourceURL#We_are_Upstream

Can you build this package in rawhide now? Once its built I will approve
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 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476525




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

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


[Bug 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476527




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

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


[Bug 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818





--- Comment #6 from Marcela Maslanova   2008-12-15 
06:24:41 EDT ---
Updated.
Spec URL:
http://mmaslano.fedorapeople.org/perl-Class-XSAccessor/perl-Class-XSAccessor.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-Class-XSAccessor/perl-Class-XSAccessor-0.14-1.fc10.src.rpm

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

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


[Bug 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476526




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

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


[Bug 476524] New: Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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-zope-testing - Zope Testing Framework

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

   Summary: Review Request: python-zope-testing - Zope Testing
Framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-zope-testing.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zope-testing-3.7.1-1.fc9.src.rpm
Description:
This package provides a number of testing frameworks. It includes a
flexible test runner, and supports both doctest and unittest.

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

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


[Bug 476474] Review Request: ebook-tools - Tools for accessing and converting various ebook file formats

2008-12-15 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=476474


R P Herrold  changed:

   What|Removed |Added

 CC||herr...@owlriver.com




--- Comment #3 from R P Herrold   2008-12-15 11:46:39 EDT 
---
I get a cmake related error:

-- Found LibXml2: /usr/lib64/libxml2.so
CMake Error at cmake/FindPackageHandleStandardArgs.cmake:51 (MESSAGE):
  Could not find REQUIRED package LibZip
Call Stack (most recent call first):
  cmake/FindLibZip.cmake:28 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:10 (find_package)


CMake Warning (dev) in CMakeLists.txt:
  No cmake_minimum_required command is present.  A line of code such as

cmake_minimum_required(VERSION 2.6)

  should be added at the top of the file.  The version specified may be lower
  if you wish to support older CMake versions for this project.  For more
  information run "cmake --help-policy CMP".
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Configuring incomplete, errors occurred!
error: Bad exit status from /var/tmp/rpm-tmp.15551 (%build)

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

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


[Bug 474818] Review Request: perl-Class-XSAccessor - Generate fast XS accessors without runtime compilation

2008-12-15 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=474818


Marcela Maslanova  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Marcela Maslanova   2008-12-15 
08:24:27 EDT ---
New Package CVS Request
===
Package Name: perl-Class-XSAccessor
Short Description: Generate fast XS accessors without runtime compilation
Owners: mmaslano
Branches: F-10
InitialCC: perl-sig

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

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


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

2008-12-15 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=457947





--- Comment #22 from Ankur Sinha   2008-12-15 
06:02:10 EDT ---
(In reply to comment #21)
> > The doc on changelog doesnt tell me which ones dist and which ones 
> > version.. :(
> > so i havent changed that in this one.. (pablo please help me with this)
> It's quite simple to understand, for exemple, your spec:
> Version:1.1
> Release:1%{?dist}
> The number after "Version:" is the version one, it's given by upstream and you
> must not change it. So put "1.0" as it's the version of the font.
> The "Release:" number is a distro/package specific number, so each time you 
> add
> an entry in the changelog, you increment by one this number. So currently, it
> should be "3%{?dist}".
> 
> Also your changelog format is not correct, you have to add your email address
> as said in the guidelines
> (https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs).
> 
> 
> > The warnings for the docs is still present.. How do i correct that?
> You should add the code proposed by Nicolas in the review request of the
> bonveno fonts in the %prep section  .
> 
> > You have many font substitution examples in the dejavu font packages. Do not
> > forget to declare your font is an acceptable substitute for all the 
> > different
> > Old Standard variant out there (TTF, OTF, etc)
> Ok, so you have to duplicate the XML code I gave you, and just change the 
> first
> Old Standard to Old Standard TTF.
> For register this font as a serif one, add the following to the fontconfig
> file:
> 
>   serif
>   
> Old Standard SFD
>   
> 

hi,

sorry for the delay.. Fedora 10 got me stuck up a bit.. Here are packages.. I
used the new source the author has uploaded.. Also the FONTLOG in the src zip
has version 2.0.2 so i used that in the spec. 

http://ankursinha.fedorapeople.org/oldstandard_sfd_fonts/oldstandard-sfd-fonts-2.0.2-1.fc10.src.rpm

http://ankursinha.fedorapeople.org/oldstandard_sfd_fonts/oldstandard-sfd-fonts.spec

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

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


[Bug 476530] New: Review Request: projxp - Agile project management server

2008-12-15 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: projxp - Agile project management server

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

   Summary: Review Request: projxp - Agile project management
server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dpie...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://mcpierce.fedorapeople.org/rpms/projxp.spec
SRPM URL: http://mcpierce.fedorapeople.org/rpms/projxp-0.0.1-1.fc10.src.rpm
Description: ProjXP is an agile project management system. It enables a
development team to maintain a backlog of features, plan and track sprints, and
enable project and product owners to view status reports easily.

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

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


[Bug 474999] Review Request: gdata-java - Client libraries to write Google Data API client applications in Java

2008-12-15 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=474999


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?




--- Comment #3 from Mamoru Tasaka   2008-12-15 
11:51:04 EDT ---
Some comments:

* Release number
  - As same as jcalendar, please fix release number.

* Requires
  - Also fix Requires as same as jcalender.

* About Patch0 and "-classpath /usr/share/java/javamail.jar"
  - I guess rather using sed with "build-classpath" (in 
jpackage-utils) is preferable:
https://fedoraproject.org/wiki/Packaging/Java#build-classpath

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

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


[Bug 476527] Review Request: python-zdaemon - Python Daemon Process Control Library

2008-12-15 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=476527


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476524




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

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


[Bug 476525] New: Review Request: python-zc-lockfile - Basic Inter-Process Locks

2008-12-15 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-zc-lockfile - Basic Inter-Process Locks

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

   Summary: Review Request: python-zc-lockfile - Basic
Inter-Process Locks
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-zc-lockfile.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zc-lockfile-1.0.0-1.fc9.src.rpm
Description:
The zc.lockfile package provides a basic portable implementation of
interprocess locks using lock files. The purpose if not specifically
to lock files, but to simply provide locks with an implementation based
on file-locking primitives. Of course, these locks could be used to
mediate access to other files. For example, the ZODB file storage
implementation uses file locks to mediate access to file-storage
database files. The database files and lock file files are separate 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 226417] Merge Review: shared-mime-info

2008-12-15 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=226417





--- Comment #3 from Bastien Nocera   2008-12-15 08:03:07 
EDT ---
(In reply to comment #1)
> I reviewed this package. It surely needs some work:
> 
> * Can bug #459365 be closed now?

Will be when the latest update is stable.

> * gawk is among the default package set and hence doesn't need to be BR'd.

Fixed.

> * The BR perl-XML-Parser >= 2.31-16 is not used at all and can be removed. Am 
> I
> wrong?

Needed for intltool, I change it to "perl(XML::Parser)" though.

> * rpmlint says:
>shared-mime-info.x86_64: W: devel-file-in-non-devel-package
> /usr/share/pkgconfig/shared-mime-info.pc
> Is there a valid reason why this file is not in a -devel package?
> Also, from the guidelines: "Packages containing pkgconfig(.pc) files
> must 'Requires: pkgconfig' (for directory ownership and usability)."

It's used for applications to detect where the database is installed. It's not
a development file though.

>shared-mime-info.x86_64: E: explicit-lib-dependency glib2
>shared-mime-info.x86_64: E: explicit-lib-dependency libxml2
> I believe that these explicit R's can be dropped since rpmbuild itself picks 
> up
> these dependencies.

Done

> * Group tag is "System Environment/Libraries" but I don't see a library in 
> this
> package.

Changed to SE/Base

> * What is wrong with the locale files in the tarball? (A more detailed
> explanation as a comment within the SPEC file please.)

Done.

> Also, assuming you have a legitimate reason to remove these files, why are you
> BR'ing gettext?

Because it's needed to put the translations in the .xml file.

> * The files ChangeLog, HACKING and most importantly COPYING need to be listed
> under %doc.

Added HACKING and COPYING, not ChangeLog, as it replicates data from NEWS
whilst being much bigger.

> * Macros should be used consistently. If you want to use %{__rm} notation, use
> macros for the other commands as well (%{__cat}, %{__make}, etc.).
> OR do it the other way around. But please stay consistent.

Used the commands directly now.

> * Buildroot should be one of these:
>%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
>%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
>%{_tmppath}/%{name}-%{version}-%{release}-root

Changed it to the second one.

> * According to the COPYING and test-tree-magic.c files, the license tag should
> be GPLv2+

Done.

> * The main source file (Source0) should be given in full URL.

Done.

> * Parallel make must be supported whenever possible. If it is not supported,
> this should be noted in the SPEC file as a comment.

Done.

> * About the defaults.list: Can't we provide a separate list for KDE users? 
> This
> may need some hacking on the source code.

Because KDE doesn't use that file, and they never expressed interest.

> * See
>http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#mimeinfo
> for correct usage of the snippet. It's interesting to see that the very 
> package
> that this guideline is based on doesn't obey the guideline itself (at least,
> partially).

Added a note as to why it shouldn't ignore errors.

-- 
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 476523] New: Review Request: python-zope-proxy - Generic Transparent Proxies

2008-12-15 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-zope-proxy - Generic Transparent Proxies

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

   Summary: Review Request: python-zope-proxy - Generic
Transparent Proxies
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-zope-proxy.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zope-proxy-3.4.2-1.fc9.src.rpm
Description:
Proxies are special objects which serve as mostly-transparent wrappers
around another object, intervening in the apparent behavior of the
wrapped object only when necessary to apply the policy (e.g., access
checking, location brokering, etc.) for which the proxy is responsible.

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

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


[Bug 476474] Review Request: ebook-tools - Tools for accessing and converting various ebook file formats

2008-12-15 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=476474





--- Comment #4 from Marcela Maslanova   2008-12-15 
11:52:04 EDT ---
I'm not familiar with cmake, but I was able to build it in koji (mock). So this
package is probably all right.
Which version of cmake do you have? I think this package will be in F-9, F-10,
where would be cmake-2.6.2.

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

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


[Bug 225622] Merge Review: boost

2008-12-15 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=225622


Patrice Dumas  changed:

   What|Removed |Added

 AssignedTo|pertu...@free.fr|nob...@fedoraproject.org
   Flag|fedora-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 476528] Review Request: python-ZConfig - Structured Configuration Library

2008-12-15 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=476528


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476524




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

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


[Bug 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476475




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

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


[Bug 226550] Merge Review: xcdroast

2008-12-15 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=226550


Fabian Affolter  changed:

   What|Removed |Added

 CC||fab...@bernewireless.net




--- Comment #1 from Fabian Affolter   2008-12-15 
05:11:11 EDT ---
Just some comments...

- X-CD-Roast 0.98alpha16 released
  http://www.xcdroast.org/#download
- Summary ends with a dot
- Use %defattr(-,root,root,-) instead of %defattr(-,root,root)
- Use parallel make (make %{?_smp_mflags})
- Package doesn't consistently use macros
- Please add a note because 'PREFIX=/usr' is used
  https://fedoraproject.org/wiki/Packaging/Guidelines#Relocatable_packages

-- 
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 472027] Review Request: onboard - Simple on-screen Keyboard

2008-12-15 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=472027





--- Comment #15 from Parag AN(पराग)   2008-12-15 04:43:09 
EDT ---
Thanks.

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

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


[Bug 476527] New: Review Request: python-zdaemon - Python Daemon Process Control Library

2008-12-15 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-zdaemon - Python Daemon Process Control Library

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

   Summary: Review Request: python-zdaemon - Python Daemon Process
Control Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: kon...@tylerc.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/python-zdaemon.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zdaemon-2.0.2-1.fc9.src.rpm
Description:
Daemon process control library and tools for Unix-bases systems.

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

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


[Bug 476474] Review Request: ebook-tools - Tools for accessing and converting various ebook file formats

2008-12-15 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=476474





--- Comment #5 from John5342   2008-12-15 12:09:00 EDT 
---
(In reply to comment #3)
> I get a cmake related error:
> 
> -- Found LibXml2: /usr/lib64/libxml2.so
> CMake Error at cmake/FindPackageHandleStandardArgs.cmake:51 (MESSAGE):
>   Could not find REQUIRED package LibZip
> Call Stack (most recent call first):
>   cmake/FindLibZip.cmake:28 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
>   CMakeLists.txt:10 (find_package)

What arch/release are you building for?
Version of cmake and libzip?
If build was on koji have you got a link to the build?


> CMake Warning (dev) in CMakeLists.txt:
>   No cmake_minimum_required command is present.  A line of code such as
> 
> cmake_minimum_required(VERSION 2.6)

Warning is known about and will be reported upstream shortly (doesnt stop
build)


Thanks for the feedback

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

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


[Bug 476524] Review Request: python-zope-testing - Zope Testing Framework

2008-12-15 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=476524


Conrad Meyer  changed:

   What|Removed |Added

 Blocks||476528




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

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


[Bug 226426] Merge Review: spamassassin

2008-12-15 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=226426





--- Comment #7 from Jon Ciesla   2008-12-15 09:49:48 EDT ---
(In reply to comment #6)

> >spamassassin.src:72: W: unversioned-explicit-obsoletes perl-Mail-SpamAssassin
> >The specfile contains an unversioned Obsoletes: token, which will match all
> >older, equal and newer versions of the obsoleted thing.  This may cause 
> >update
> >problems, restrict future package/provides naming, and may match something it
> >was originally not inteded to match -- make the Obsoletes versioned if
> >possible.
> >
> >Fix.
> 
> Well, the problem here is that upsteam uses that package name. 
> So, if someone installs the upstream rpms, then decides to upgrade 
> to the fedora one, without this they will get a confusing mix. ;( 

Then commenting this in the spec should be sufficient.

> 
> >spamassassin.src:101: W: rpm-buildroot-usage %build %{__perl} Makefile.PL
> >DESTDIR=$RPM_BUILD_ROOT/ SYSCONFDIR=%{_sysconfdir} INSTALLDIRS=vendor
> >ENABLE_SSL=yes < /dev/null
> >$RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it 
> >will
> >break short circuiting.
> >
> >There may be a good reason for this.  Is there?
> 
> It's not clear to me where it is using the build root. It's setting DESTDIR to
> it, but it shouldn't be using it. Will dig more, but ideas welcome. 

I'll peek at it.


> >spamassassin.src: W: strange-permission spamassassin-helper.sh 0755
> >A file that you listed to include in your package has strange permissions.
> >Usually, a file should have 0644 permissions.
> >
> >Fix, or document in spec.
> 
> It's a shell script that runs and shows the exit code (spam/notspam). 
> I guess I can add a comment that it's expected to be executable. 

That'd be perfect.

> 
> >rpmlint on RPMS:
> >
> >spamassassin.i386: E: incoherent-logrotate-file /etc/logrotate.d/sa-update
> >Your logrotate file should be named /etc/logrotate.d/.
> >
> >Fix, if it won't be too catastrophic.
> 
> Well, it's not spamassassin itself that logs anything, it's the daily
> sa-update job that pulls updates to rules. I think it makes more sense 
> to leave it as sa-update since thats the command that generates the logs. 

Agreed, might want to comment in spec.


> 
> >spamassassin.i386: E: executable-marked-as-config-file
> >/etc/mail/spamassassin/spamassassin-helper.sh
> >Executables must not be marked as config files because that may prevent
> >upgrades from working correctly. If you need to be able to customize an
> >executable, make it for example read a config file in /etc/sysconfig.
> >
> >
> 
> Humm. Not sure why thats marked as config. No one should ever change it. 
> Sadly, thats generated the file that the make process generates. 
> It might need a patch or getting upstream to fix it. 

Hmm.



> >spamassassin.i386: W: conffile-without-noreplace-flag
> >/etc/rc.d/init.d/spamassassin
> >A configuration file is stored in your package without the noreplace flag. A
> >way to resolve this is to put the following in your SPEC file:
> >%config(noreplace) /etc/your_config_file_here
> 
> Fixed above by making it not a config file. 
> 
> >spamassassin.i386: W: dangerous-command-in-%post cp
> 
> This is so that updates with old config file options that are no longer 
> supported will get updated. I don't see any easy way around it. 

Neither do I.  Comment in spec.

> >spamassassin.i386: W: no-reload-entry /etc/rc.d/init.d/spamassassin
> >In your init script (/etc/rc.d/init.d/your_file), you don't have a 'reload'
> >entry, which is necessary for good functionality.
> 
> spamd doesn't have any functionality to do a reload without just restarting
> as far as I know. 

In this instance, just make reload do what restart does.

> >Otherwise, full review looks good, no other blockers.
> 
> Ok. 
> 
> new spec: http://www.scrye.com/~kevin/fedora/spamassassin.spec
> diff against old: http://www.scrye.com/~kevin/fedora/spamassassin.diff
> scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=998690
> 
> Warren is going to look it over as well.

I'll await his input.

-- 
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 473413] Review Request: system-config-samba-docs - Documentation for configuring a Samba server

2008-12-15 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=473413





--- Comment #2 from Nils Philippsen   2008-12-15 09:33:09 
EDT ---
New files:

Spec: http://nphilipp.fedorapeople.org/review/system-config-samba-docs.spec
SRPM:
http://nphilipp.fedorapeople.org/review/system-config-samba-docs-1.0.3-1.fc10.src.rpm

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

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


[Bug 476440] Review Request: latexdiff - Determine and mark up significant differences between latex files

2008-12-15 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=476440





--- Comment #2 from Dan Kenigsberg   2008-12-15 
05:44:36 EDT ---
Thanks for the review.

As I understand it, "version 0.42" in latexdiff and "version 0.3" in
latexrevise relate to the versions of the separate scripts. Since I did not
find any version number for the whole package, I chose the latest update date
of files within.

Do you have a better idea (maybe 0.0.20071020, to let upstream choose 0.43 one
day in the future ?)

See http://www.cs.technion.ac.il/~danken/latexdiff-0.0.20071020-1.fc10.src.rpm
and http://www.cs.technion.ac.il/~danken/latexdiff.spec for updated version and
license.

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

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


[Bug 475107] Review Request: guiloader-c++ - C++ Binding to GuiLoader Library

2008-12-15 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=475107


Bug 475107 depends on bug 475082, which changed state.

Bug 475082 Summary: Review Request: guiloader - GuiXml Loader Library
https://bugzilla.redhat.com/show_bug.cgi?id=475082

   What|Old Value   |New Value

 Status|ON_QA   |CLOSED
 Resolution||UPSTREAM



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

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


[Bug 464013] Review Request: findbugs-bcel - Byte Code Engineering Library with findbugs extensions

2008-12-15 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=464013


Mary Ellen Foster  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||pche...@redhat.com
 AssignedTo|nob...@fedoraproject.org|mefos...@gmail.com




--- Comment #4 from Mary Ellen Foster   2008-12-15 05:14:22 
EDT ---
I asked on fedora-packaging about this package, and got the (grudging) response
that it would be okay, *if* the bcel maintainer is also okay with it:
   
https://www.redhat.com/archives/fedora-packaging/2008-December/msg00027.html
The bcel maintainer seems to be Permaine Cheung  -- I've
added him to the CC.


That aside, here's my review:
* source files match upstream.  
  3b3d451664855b0c9aae15dd05b69bf1063d8a1d  bcel-5.2-src.tar.gz
  patch is modified, but the differences are documented (but see below)

* package meets versioning guidelines.

X specfile is properly named, is cleanly written and uses macros consistently.
  I'm not sure about this -- should it be findbugs-bcel or bcel-findbugs?

* summary is OK.
X description
  Should probably mention the findbugs aspect in the description somewhere

* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license is included as %doc
* latest version is being packaged.
* BuildRequires are proper
X compiler flags are appropriate.
  If you're using javac directly, maybe you should use the %{_javac} macro
  Similarly, you can use %{_jar} instead of jar

* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly.
* debuginfo package looks good
* rpmlint is silent (except for Group tags which are apparently okay)
* final provides and requires are sane
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* javadoc package looks right
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

Regarding the modified patch -- the original version from findbugs seems to
also create a file
src/java/org/apache/bcel/classfile/AbstractLocalVariableTable.java
but the modified patch in the SRPM doesn't. Is there a reason for that?

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

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


[Bug 225608] Merge Review: basesystem

2008-12-15 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=225608


Patrice Dumas  changed:

   What|Removed |Added

 AssignedTo|pertu...@free.fr|nob...@fedoraproject.org
   Flag|fedora-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 457955] Review Request: bonvenocf-fonts - BonvenoCF font

2008-12-15 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=457955





--- Comment #12 from Ankur Sinha   2008-12-15 
06:45:40 EDT ---
hi,

new packages..sorry for the delay.

http://ankursinha.fedorapeople.org/cf-bonveno-fonts/cf-bonveno-fonts-1.1-1.fc10.src.rpm

http://ankursinha.fedorapeople.org/cf-bonveno-fonts/cf-bonveno-fonts.spec

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

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


[Bug 234835] Review Request: fgfs-Atlas - Flightgear map tools

2008-12-15 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=234835


Fabrice Bellet  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #13 from Fabrice Bellet   2008-12-15 04:40:58 
EDT ---
New Package CVS Request
===
Package Name: FlightGear-Atlas
Short Description: Flightgear map tools
Owners: fabr...@bellet.info
Branches:
InitialCC:

I'd like to rename this package. The data tarball for FlightGear has been
renamed to FlightGear-data recently. So I'll take this opportunity to unify the
name of this other package too. With this renaming, all FlightGear related
packages will have the same prefix.

The upstream name of this package is 'Atlas' (http://atlas.sourceforge.net/),
and another unrelated package with this name already exists in the packagedb
(atlas for http://math-atlas.sourceforge.net/). So this is why this move
doesn't reflect an upstream project modification in this case.

-- 
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 226463] Merge Review: system-config-netboot

2008-12-15 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=226463


Fabian Affolter  changed:

   What|Removed |Added

 CC||fab...@bernewireless.net




--- Comment #1 from Fabian Affolter   2008-12-15 
06:14:05 EDT ---
Just some comments on this spec file

- Use %defattr(-,root,root,-) instead of %defattr(-,root,root)
- Use parallel make (make %{?_smp_mflags})
- Is there a reason that %makeinstall is used? 
 
https://fedoraproject.org/wiki/Packaging/Guidelines#Why_the_.25makeinstall_macro_should_not_be_used

-- 
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 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

2008-12-15 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=453083





--- Comment #25 from Matthew Barnes   2008-12-15 07:24:49 
EDT ---
(In reply to comment #24)
> Otherwise, wouldn't the direction of this package be faulty? If one or more
> "shared things" are in the forecast then, it would seem, disabling it until v.
> 4 is "official" is the way to go.

Bearing in mind the primary reason for packaging Samba4 at this time is to
support OpenChange, and OpenChange doesn't need the winbind stuff, I'm inclined
to just omit the subpackage for now.

Will try to post another round of updates later today.

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

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


[Bug 476523] Review Request: python-zope-proxy - Generic Transparent Proxies

2008-12-15 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=476523


Conrad Meyer  changed:

   What|Removed |Added

 Depends on||476524




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

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


[Bug 473452] Review Request: system-config-services-docs - Documentation for configuring system services

2008-12-15 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=473452





--- Comment #12 from Jason Tibbitts   2008-12-15 12:40:28 
EDT ---
Thanks for that, but something's gone wrong.  This now has no runtime
dependency on rarian-compat, just a build dependency.  It will definitely need
Requires(pre) and Requires(post), conditionalized for the old releases you want
to support.

Also, I can't for the life of me figure out what is supposed to own
/usr/share/gnome/help. This package leaves it unowned, which isn't a good idea.
 I'm thinking that the filesystem package should own it (as it does with
/usr/share/gnome) and then everything would be OK, but that will need to be
acked by the filesystem maintainer first.

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

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


[Bug 474982] Review Request: nachocalendar - Provides a flexible Calendar component to the Java Platform

2008-12-15 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=474982


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   Flag||fedora-review?




--- Comment #3 from Mamoru Tasaka   2008-12-15 
12:39:56 EDT ---
Notes:

* License
  - As far as I checked the whole source codes, the license
tag should be "LGPLv2+".

* EVR (Epoch-Version-Release)
  - Please fix the release number

! %setup
  - "-n %{name}-%{version}" is the default of %setup and
this can be removed.

* Documents
  - Please consider to add the following file(s) to %doc

CHANGELOG.txt


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

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


[Bug 226426] Merge Review: spamassassin

2008-12-15 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=226426





--- Comment #8 from Warren Togami   2008-12-15 12:59:35 EDT 
---
I'm OK with his proposed changes.  I also suggested switching it to pidgin.spec
style automatic boolean setting.

-- 
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 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 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=476315





--- Comment #3 from Matthew Barnes   2008-12-15 13:18:16 
EDT ---
(In reply to comment #1)
> This fails to build for me on rawhide; it needs openchange-devel which is not
> available.

Jason, see http://fedoraproject.org/wiki/Features/OpenChange.

OpenChange and Samba4 are also under review to support 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 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-15 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=476315





--- Comment #4 from Jason Tibbitts   2008-12-15 13:24:35 EDT 
---
Then could you perhaps set the ticket dependencies properly so that something
indicates that this isn't actually reviewable now?

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

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


  1   2   >