[Bug 476404] New: Review Request: bullet - 3D Game Multiphysics Library provides collision detection, soft body and rigid body dynamics

2008-12-14 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: bullet - 3D Game Multiphysics Library provides 
collision detection, soft body and rigid body dynamics

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

   Summary: Review Request: bullet - 3D Game Multiphysics Library
provides collision detection, soft body and rigid body
dynamics
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: bruno.m...@laposte.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Hi,

This is my first package and I am looking for a sponsor.


Spec URL: http://www.gnoll.org/download/bullet/bullet.spec
SRPM URL: http://www.gnoll.org/download/bullet/bullet-2.73-1.fc10.src.rpm
Description: Bullet 3D Game Multiphysics Library provides state of the art
collision detection, soft body and rigid body dynamics.

Thanks,
Bruno

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

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


[Bug 470325] Review Request: qd - Double-Double and Quad-Double Arithmetic

2008-12-14 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=470325





--- Comment #7 from Conrad Meyer   2008-12-14 03:30:05 EDT 
---
Er, the spec/%{name} should be qd.spec, not qd-devel.spec. You should however
omit the %files section for the main package and only have one for the devel
subpackage.

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

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


[Bug 476404] Review Request: bullet - 3D Game Multiphysics Library provides collision detection, soft body and rigid body dynamics

2008-12-14 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=476404


Bruno Mahe  changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 438608] Review Request: elisa-plugins-good - Good Plugins for the Elisa Media Center

2008-12-14 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=438608





--- Comment #20 from Alex Lancaster   2008-12-14 
03:40:02 EDT ---
Has this review stalled?  elisa 0.5.20 is now in rawhide but has broken deps
because it requires elisa-plugins-good, see bug #429590 comment #43.

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

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


[Bug 458643] Review Request: dansguardian - Content filtering web proxy

2008-12-14 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=458643





--- Comment #12 from Felix Kaechele   2008-12-14 04:38:49 EDT 
---
Funny that the reviewer itself has never commented on this bug. I'd be willing
to take over the review if the original reviewer doesn't want to do so.

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

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


[Bug 475110] Review Request: monkeystudio - Free crossplatform Qt 4 IDE

2008-12-14 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=475110





--- Comment #7 from Nicoleau Fabien   2008-12-14 
05:01:02 EDT ---
Must I open a specific bug for that ? It's quite difficult as I don't know the
real reason for that (trick with -spec was given by upstream), and I don't have
a x86_64 to ealily reproduce it :(

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

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


[Bug 476406] New: Review Request: sympow - Special Values of Symmetric Power Elliptic Curve L-Functions

2008-12-14 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: sympow - Special Values of Symmetric Power Elliptic 
Curve L-Functions

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

   Summary: Review Request: sympow - Special Values of Symmetric
Power Elliptic Curve L-Functions
   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/sympow.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/sympow-1.019-1.fc9.src.rpm
Description:
SYMPOW is a program for computing special values of symmetric power
elliptic curve L-functions.

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

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


[Bug 469843] Review Request: unhide - Tool to find hidden processes and TCP/UDP ports from rootkits

2008-12-14 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=469843





--- Comment #10 from Rakesh Pandit   2008-12-14 
05:41:52 EDT ---
Yes, I have already dropped a mail. Which check before importing or building.

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

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


[Bug 458643] Review Request: dansguardian - Content filtering web proxy

2008-12-14 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=458643





--- Comment #13 from Sayamindu Dasgupta   2008-12-14 
05:43:12 EDT ---
(In reply to comment #12)
> Funny that the reviewer itself has never commented on this bug. I'd be willing
> to take over the review if the original reviewer doesn't want to do so.

Sorry for the delay, I have been swamped with a lot of work for the past few
months. I did raise the issue regarding this package in the fedora-legal
mailing list, and it looks like it is legal to redistribute the software.

If you want to review the package, feel free to assign it to yourself.

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

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


[Bug 473591] Review Request: iFuse - Mount Apple iPhone and iPod touch devices

2008-12-14 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=473591





--- Comment #3 from Peter Robinson   2008-12-14 06:17:21 
EDT ---
Updated package:
SPEC: http://pbrobinson.fedorapeople.org/ifuse.spec
SRPM:
http://pbrobinson.fedorapeople.org/ifuse-0.1.0-4.20081214gitb0412bf.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 474984] Review Request: bandwidthd - Tracks network usage and builds html and graphs

2008-12-14 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=474984





--- Comment #17 from Fedora Update System   
2008-12-14 06:43:59 EDT ---
bandwidthd-2.0.1-9.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/bandwidthd-2.0.1-9.fc9

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

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


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

2008-12-14 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 #9 from Fabian Affolter   2008-12-14 
07:56:48 EDT ---
(In reply to comment #2)
> Suggestion:-
>1) Good to modify onboard.desktop file line from 
> Categories=GNOME;GTK;Accessibility;X-GNOME-PersonalSettings;
> to
> Categories=GNOME;GTK;Utility;Accessibility;
> 
>This will make sure we have all onboard/onscreen virtual keyboards in same
> category

fixed.  For now I'm OK with the usage of your modified files.  In the trunk of
onBoard are already .desktop files which will be shipped with the next release,
I guess.

>2) This package built successfully by removing
>BuildRequires:  python-setuptools-devel

This is needed for the generation of the egg.  It's mentioned in the guidelines

>BuildRequires:  gettext

I will leave it in place because as soon I'm able to integrate the translations
it's needed.

>3) I think you don't need Requires: gnome-python2 not sure about that
> though.

removed

Here are the new files:
Spec URL: http://fab.fedorapeople.org/packages/SRPMS/onboard.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/onboard-0.91.2-3.fc9.src.rpm

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

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


[Bug 476414] New: Review Request: assogiate - Editor for the file types database

2008-12-14 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: assogiate - Editor for the file types database

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

   Summary: Review Request: assogiate - Editor for the file types
database
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fab...@bernewireless.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fab.fedorapeople.org/packages/SRPMS/assogiate.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/assogiate-0.2.1-1.fc9.src.rpm

Project URL: http://www.kdau.com/projects/assogiate/

Description:
assoGiate allows users to modify the detection and display of file
types. It is compliant with the freedesktop.org Shared MIME Info and
Icon Theme specifications. The standard file types can be modified, 
and additional file types can be added and removed. The file type
detection based on filenames, file contents, and XML root elements
can be specified.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=997883 

rpmlint output:
[...@laptop024 i386]$ rpmlint assogiate*
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

[...@laptop024 SRPMS]$ rpmlint assogiate-0.2.1-1.fc9.src.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 469527] Review Request: tcping - Check of TCP connection to a given IP/Port

2008-12-14 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=469527





--- Comment #6 from Fabian Affolter   2008-12-14 
08:11:02 EDT ---
The license is LGPLv3+. Upstream updated the license.

Here are the new files.

Spec URL: http://fab.fedorapeople.org/packages/SRPMS/tcping.spec
SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/tcping-1.3.4-3.fc9.src.rpm

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=997919

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

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


[Bug 476386] Review Request: perl-verilog - Verilog parsing routines

2008-12-14 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=476386





--- Comment #2 from Chitlesh GOORAH   2008-12-14 07:08:30 
EDT ---
Updated:
Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-verilog.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-verilog-3.044-2.fc10.src.rpm

Scratch build F-10:
https://koji.fedoraproject.org/koji/taskinfo?taskID=997864

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

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


[Bug 471335] Review Request: libgdbus - Library for simple D-Bus integration with GLib

2008-12-14 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=471335





--- Comment #8 from Fedora Update System   
2008-12-14 07:19:15 EDT ---
libgdbus-0.2-2.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/libgdbus-0.2-2.fc9

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

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


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

2008-12-14 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 #20 from Fedora Update System   
2008-12-14 05:36:13 EDT ---
skinlf-6.7-6.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/skinlf-6.7-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 469471] Review Request: skinlf - Java look and feel for swing

2008-12-14 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 #19 from Fedora Update System   
2008-12-14 05:35:29 EDT ---
skinlf-6.7-6.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/skinlf-6.7-6.fc9

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

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


[Bug 474984] Review Request: bandwidthd - Tracks network usage and builds html and graphs

2008-12-14 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=474984





--- Comment #16 from Fedora Update System   
2008-12-14 06:34:10 EDT ---
bandwidthd-2.0.1-9.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/bandwidthd-2.0.1-9.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 476406] Review Request: sympow - Special Values of Symmetric Power Elliptic Curve L-Functions

2008-12-14 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=476406





--- Comment #1 from Conrad Meyer   2008-12-14 05:02:39 EDT 
---
Koji scratch build:

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

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

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


[Bug 475972] Review Request: perl-Apache-DBI-Cache - Perl DBI connection cache

2008-12-14 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=475972


Lubomir Rintel  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




--- Comment #5 from Lubomir Rintel   2008-12-14 06:28:29 EDT ---
Thanks.

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

2008-12-14 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 #2 from Chitlesh GOORAH   2008-12-14 07:08:04 
EDT ---
Updated:
Spec URL: http://chitlesh.fedorapeople.org/RPMS/vhd2vl.spec
SRPM URL: http://chitlesh.fedorapeople.org/RPMS/vhd2vl-2.0-2.fc10.src.rpm


Scratch Build F-10
https://koji.fedoraproject.org/koji/taskinfo?taskID=997814

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

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


[Bug 471335] Review Request: libgdbus - Library for simple D-Bus integration with GLib

2008-12-14 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=471335





--- Comment #7 from Fedora Update System   
2008-12-14 07:19:12 EDT ---
libgdbus-0.2-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/libgdbus-0.2-2.fc10

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

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


[Bug 192164] Review Request: perl-Socket6 - IPv6 related part of the C socket.h defines and structure manipulators

2008-12-14 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=192164


Robert Scheck  changed:

   What|Removed |Added

 CC||redhat-bugzi...@linuxnetz.d
   ||e
   Flag||fedora-cvs?




--- Comment #3 from Robert Scheck   2008-12-14 07:57:22 
EDT ---
Package Change Request
==
Package Name: perl-Socket6
Owners: robert wtogami jpo
Branches: EL-4

-- 
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 467175] Review Request: perl-Set-Object - Set of objects and strings

2008-12-14 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=467175


Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks|182235  |




--- Comment #25 from Tom "spot" Callaway   2008-12-14 
09:32:19 EDT ---
Lifted FE-Legal, sorry. :)

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

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


[Bug 471041] Review Request: diffpdf - PDF files comparator

2008-12-14 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=471041


Rafał Psota  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #12 from Rafał Psota   2008-12-14 09:49:39 EDT 
---
Imported and built.

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

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


[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

2008-12-14 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=468189


Schlomo Schapiro  changed:

   What|Removed |Added

 CC||red...@schlomo.schapiro.org




--- Comment #14 from Schlomo Schapiro   2008-12-14 
10:08:36 EDT ---
Hi,

I am - together with Gratien - the main developer of Relax and Recover.

I am currently trying to find a sponsor, is there anything else missing so far?

A few notes about the issues mentioned above from a ReaR design point of view:

1) ReaR brings along the empty filesystem of a mini linux distribution (no
binaries) which is the reason for the empty files and symlinks pointing
nowhere. As part of the ReaR process to create the rescue media this empty
filesystem is filled in with components from the current linux system to create
a functioning mini linux rescue system.

Is there a way to get along with the empty files and dead symlinks?

2) ReaR does not have a -devel package and - by design - always contains the
entire distribution. That is why you will also find .deb and gentoo files as
part of every ReaR installation so that any user can package his/her (modified)
version of ReaR without installing any additional tools. Since ReaR is written
entirely in Bash there is no platform or distribution dependency beyond what
ReaR is already doing internally.

Of course we could leave these under /usr/share/rear and not copy them to %doc.

Kind regards,
Schlomo

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

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


[Bug 476025] Review Request: mythes-ga - Irish thesaurus

2008-12-14 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=476025


Caolan McNamara  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 476414] Review Request: assogiate - Editor for the file types database

2008-12-14 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


Parag AN(पराग)  changed:

   What|Removed |Added

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




--- Comment #1 from Parag AN(पराग)   2008-12-14 11:51:26 
EDT ---
I think this package better should not own following directories
/usr/share/icons/hicolor
/usr/share/icons/hicolor/*/apps/
instead you can add
Requires: hicolor-icon-theme

so yours %files will then list icon files as
%{_datadir}/icons/hicolor/*/*/*

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

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


[Bug 454482] Review Request: vbindiff - Visual binary diff

2008-12-14 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=454482





--- Comment #9 from Fedora Update System   
2008-12-14 11:51:52 EDT ---
vbindiff-3.0-0.2.beta4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/vbindiff-3.0-0.2.beta4.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 454482] Review Request: vbindiff - Visual binary diff

2008-12-14 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=454482





--- Comment #10 from Fedora Update System   
2008-12-14 11:53:24 EDT ---
vbindiff-3.0-0.2.beta4.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/vbindiff-3.0-0.2.beta4.fc9

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

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


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

2008-12-14 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





--- Comment #7 from Fabian Affolter   2008-12-14 
11:58:39 EDT ---
Thanks Bryan

Here are the new files:

Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-speak.spec
SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-speak-9-3.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 467645] Review Request: sugar-distance - Distance measurement for Sugar

2008-12-14 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=467645





--- Comment #3 from Fabian Affolter   2008-12-14 
12:29:22 EDT ---
Sill no action for upstream about the open issues.  I still hope that the
upstream of this sugar activity is not as dead as 80% of the all activities. 

Last action in the repo is two months ago.  Bug report is open for 5 weeks.

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

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


[Bug 462813] Review Request: ldtp - Desktop testing framework

2008-12-14 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=462813


Debarshi Ray  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from Debarshi Ray   2008-12-14 12:29:55 
EDT ---
New Package CVS Request
===
Package Name: ldtp
Short Description: Desktop testing framework
Owners: rishi
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 476435] New: Review Request: sugar-record - Recording tool for Sugar

2008-12-14 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: sugar-record - Recording tool for Sugar

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

   Summary: Review Request: sugar-record - Recording tool for
Sugar
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fab...@bernewireless.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-record.spec
SRPM URL:
http://fab.fedorapeople.org/packages/XO/sugar-record-60-1.fc10.src.rpm

Project website: http://wiki.laptop.org/go/Record

Description: 
Record is the basic rich-media capture activity for the laptop. It 
lets you capture still images, video, and/or audio. It has a simple 
interface and works in both laptop and ebook mode. An interface for 
sharing pictures among multi XOs during a picture-taking session is
a hallmark of the Record activity.

Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=998124

[...@laptop24 i386]$ rpmlint sugar-record-60*
sugar-record.i386: W: devel-file-in-non-devel-package
/usr/share/sugar/activities/Record.activity/_camera.c
sugar-record.i386: W: non-standard-group Sugar/Activities
sugar-record.i386: E: arch-dependent-file-in-usr-share
/usr/share/sugar/activities/Record.activity/_camera.o
sugar-record.i386: E: arch-dependent-file-in-usr-share
/usr/share/sugar/activities/Record.activity/_camera.so
sugar-record.i386: E: library-not-linked-against-libc
/usr/share/sugar/activities/Record.activity/_camera.so
1 packages and 0 specfiles checked; 3 errors, 2 warnings.

[...@laptop24 SRPMS]$ rpmlint sugar-record-60-1.fc10.src.rpm 
sugar-record.src: W: non-standard-group Sugar/Activities
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

I guess that there will be more to do on this package because it is not noarch
and the camera stuff will not work on 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 476440] New: Review Request: latexdiff - Determine and mark up significant differences between latex files

2008-12-14 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: latexdiff - Determine and mark up significant 
differences between latex files

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

   Summary: Review Request: latexdiff - Determine and mark up
significant differences between latex files
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dan...@cs.technion.ac.il
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.cs.technion.ac.il/~danken/latexdiff.spec
SRPM URL:
http://www.cs.technion.ac.il/~danken/latexdiff-0.20071020-1.fc10.src.rpm
Description: Latexdiff is a Perl script for visual mark up and revision of
significant differences between two latex files.  Various options are available
for visual markup using standard latex packages such as color.  Changes not
directly affecting visible text, for example in formatting commands, are still
marked in the latex source.  A rudimentary revision facility is provided by
another Perl script, latexrevise, which accepts or rejects all changes.  Manual
editing of the difference file can be used to override this default behaviour
and accept or reject selected changes only.

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

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


[Bug 476324] Review Request: perl-URI-Find - Find URIs in plain text

2008-12-14 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=476324


Chris Weyl  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 476160] Review Request: perl-Directory-Scratch - Self-cleaning scratch space for tests

2008-12-14 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=476160


Chris Weyl  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 476443] New: Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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: perl-Hardware-Vhdl-Parser - Complete grammar for 
parsing VHDL code using perl

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

   Summary: Review Request: perl-Hardware-Vhdl-Parser - Complete
grammar for parsing VHDL code using perl
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cgoo...@yahoo.com.au
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-Hardware-Vhdl-Parser.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-Hardware-Vhdl-Parser-0.12-1.fc10.src.rpm
Description:
This module defines the complete grammar needed to parse any VHDL code. By
overloading this grammar, it is possible to easily create perl scripts
which run through VHDL code and perform specific functions.

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

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


[Bug 475861] Review Request: gnustep-gui - The GNUstep gui library

2008-12-14 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=475861





--- Comment #3 from Jochen Schmitt   2008-12-14 
14:12:36 EDT ---
I have uploaded the SRPM again. I hope, it's works fine 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


[Bug 187317] Review Request: mindi

2008-12-14 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=187317


Bruno Cornec  changed:

   What|Removed |Added

 Depends on||476234




-- 
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 462982] Review Request: buffer - General purpose buffer program

2008-12-14 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





--- Comment #21 from Bruno Cornec   2008-12-14 14:16:52 
EDT ---
When trying to edit flags, I now see the fedora-cvs one. However, the combobox
s grey, and I can't put the question mark in it.

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

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


[Bug 476234] Review Request: mindi-busybox - Busybox version suited for Mindi

2008-12-14 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=476234


Bruno Cornec  changed:

   What|Removed |Added

 Blocks||187317




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

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


[Bug 187317] Review Request: mindi

2008-12-14 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=187317





--- Comment #39 from Bruno Cornec   2008-12-14 14:23:01 
EDT ---
I have made available a mindi-busybox package (Cf:
https://bugzilla.redhat.com/show_bug.cgi?id=476234)

-- 
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 461912] Review Request: puzzles - A collection of one-player puzzle games

2008-12-14 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=461912


Victor Bogado  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 461912] Review Request: puzzles - A collection of one-player puzzle games

2008-12-14 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=461912





--- Comment #14 from Victor Bogado   2008-12-14 14:39:52 EDT 
---
New Package CVS Request
===
Package Name: puzzles
Short Description:  A collection of one-player puzzle games
Owners: bogado
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 435276] Review Request: kBuild - A cross-platform build enviroment

2008-12-14 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





--- Comment #25 from Lubomir Rintel   2008-12-14 15:14:37 EDT 
---
Boo!

SPEC: http://netbsd.sk/~lkundrak/SPECS/kBuild.spec
SRPM: http://netbsd.sk/~lkundrak/SRPMS/kBuild-0.1.4-1.el5.src.rpm

* Builds in fedora-devel-i386
* RPMlint silent
* If you type "google" into google, you'll break the internets!

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

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


[Bug 468631] Review Request: libgarmin - C library to parse and use Garmin image files

2008-12-14 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=468631





--- Comment #5 from Fabian Affolter   2008-12-14 
15:22:57 EDT ---
* Sun Dec 14 2008 Fabian Affolter  -
0-0.4.20081214svn
- Updated from revision 307 to revision 315

SPEC: http://fab.fedorapeople.org/packages/SRPMS/libgarmin.spec
SRPM:
http://fab.fedorapeople.org/packages/SRPMS/libgarmin-0-0.4.20081214svn.fc9.src.rpm

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

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


[Bug 474036] Review Request: terminator - Store and run multiple GNOME terminals in one window

2008-12-14 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=474036


Ian Weller  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #9 from Ian Weller   2008-12-14 15:28:17 EDT 
---
New Package CVS Request
===
Package Name: terminator
Short Description: Store and run multiple GNOME terminals in one window
Owners: ianweller
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 476443] Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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=476443


Brennan Ashton  changed:

   What|Removed |Added

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




--- Comment #1 from Brennan Ashton   2008-12-14 
15:38:41 EDT ---
[x]source files match upstream:
95aab98d316ce5289e80840301da7dea8f7fffc4931506afaf092aec1c063fe3 
Hardware-Vhdl-Parser-0.12.tar.gz
[x]package meets naming and versioning guidelines.
[x]specfile is properly named, is cleanly written and uses macros consistently.
[x]dist tag is present.
[x]build root is correct.
[x]license field matches the actual license.
[x]license is open source-compatible.
[x]latest version is being packaged.
[x]BuildRequires are proper.
[x]compiler flags are appropriate.
[x]%clean is present.
[x]package builds in mock.

package installs properly.
NO, you need to add
find . -type f | xargs %{__perl} -pi -e 's|#! /bin/perl|#! /usr/bin/perl|'
to your spec file, the script are looking for perl in /bin/perl and it will not
install because rpm want a package that provides /bin/perl.  When I added this
to the spec file after %setup it is happy.

[x]rpmlint is silent.
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

[x]%check is present and all tests pass:
Executing(%check): /bin/sh -e /var/tmp/rpm-tmp.nopDDm
+ umask 022
+ cd /home/makerpm/rpmbuild/BUILD
+ cd Hardware-Vhdl-Parser-0.12
+ unset DISPLAY
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl "-Iblib/lib" "-Iblib/arch" test.pl
1..2
ok 1
Test 2
OK

[x]no shared libraries are added to the regular linker search paths.
[x]no duplicates in %files.
[x]file permissions are appropriate.
[x]no scriptlets present.
[x]documentation is small, so no -docs subpackage is necessary.
[x]%docs are not necessary for the proper functioning of the package.

Once you have made that change I will approve the package. I made it locally
and it passes tests on a vhd file I made.

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

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


[Bug 476443] Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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=476443





--- Comment #2 from Chitlesh GOORAH   2008-12-14 15:45:36 
EDT ---
Updated
Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-Hardware-Vhdl-Parser.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-Hardware-Vhdl-Parser-0.12-2.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 476443] Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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=476443


Brennan Ashton  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Brennan Ashton   2008-12-14 
15:49:08 EDT ---
Ok it installs clean now I,
APPROVE.
Feel free to submit your 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 476443] Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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=476443





--- Comment #5 from Chitlesh GOORAH   2008-12-14 15:57:09 
EDT ---
Owner: chitlesh

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

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


[Bug 476443] Review Request: perl-Hardware-Vhdl-Parser - Complete grammar for parsing VHDL code using perl

2008-12-14 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=476443


Chitlesh GOORAH  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Chitlesh GOORAH   2008-12-14 15:56:28 
EDT ---
New Package CVS Request
===
Package Name: perl-Hardware-Vhdl-Parser
Short Description: Complete grammar for parsing VHDL code using perl
Branches: F-9 F-10 EL-5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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





--- Comment #2 from Fabian Affolter   2008-12-14 
16:01:07 EDT ---
(In reply to comment #1)
> I think this package better should not own following directories
> /usr/share/icons/hicolor
> /usr/share/icons/hicolor/*/apps/
> instead you can add
> Requires: hicolor-icon-theme
> 
> so yours %files will then list icon files as
> %{_datadir}/icons/hicolor/*/*/*

Thanks, changed and 'Requires: hicolor-icon-theme' added

Updated files are here:

Spec URL: http://fab.fedorapeople.org/packages/SRPMS/assogiate.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/assogiate-0.2.1-2.fc9.src.rpm

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

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


[Bug 473727] Review Request: perl-Data-TreeDumper-Renderer-GTK - Gtk2::TreeView renderer for Data::TreeDumper

2008-12-14 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=473727


Chris Weyl  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 476449] Review Request: perl-SystemC-Vregs - Utility routines used by vregs

2008-12-14 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=476449


Chitlesh GOORAH  changed:

   What|Removed |Added

 Depends on||476386




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

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


[Bug 476386] Review Request: perl-verilog - Verilog parsing routines

2008-12-14 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=476386


Chitlesh GOORAH  changed:

   What|Removed |Added

 Blocks||476449




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

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


[Bug 476449] New: Review Request: perl-SystemC-Vregs - Utility routines used by vregs

2008-12-14 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: perl-SystemC-Vregs - Utility routines used by vregs

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

   Summary: Review Request: perl-SystemC-Vregs - Utility routines
used by vregs
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cgoo...@yahoo.com.au
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://chitlesh.fedorapeople.org/RPMS/perl-SystemC-Vregs.spec
SRPM URL:
http://chitlesh.fedorapeople.org/RPMS/perl-SystemC-Vregs-1.460-1.fc10.src.rpm
Description:
A Vregs object contains a documentation "package" containing enumerations,
definitions, classes, and registers.

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

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


[Bug 476449] Review Request: perl-SystemC-Vregs - Utility routines used by vregs

2008-12-14 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=476449


Brennan Ashton  changed:

   What|Removed |Added

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




--- Comment #1 from Brennan Ashton   2008-12-14 
17:59:40 EDT ---
[x]source files match upstream:
ea8145aa33a10874007187e628cc90805b4c29d8cd57bbed024abb1dc4a0a308 
SystemC-Vregs-1.460.tar.gz

[x]package meets naming and versioning guidelines.
[x]specfile is properly named, is cleanly written and uses macros consistently.
[x]dist tag is present.
build root is correct.
 (%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 is the recommended value, but not the only one)
[FAIL]license field matches the actual license.

It is GPL+ not LGPL according to the webpage and README this also results in
rpmlint errors

[x]license is open source-compatible.
[x]latest version is being packaged.
[x]BuildRequires are proper.
[x]compiler flags are appropriate.
[x]%clean is present.
package builds in mock.
I would have to set my own repo up to do this because of a require still being
under review.
[FAIL]package installs properly.
You will need to include the SystemC-Vregs-1.460/t/ folder for the examples
that you include to run.
[x]debuginfo package looks complete.
[FAIL]rpmlint is silent.

perl-SystemC-Vregs.src: W: invalid-license LGPL+
perl-SystemC-Vregs.src: W: invalid-license Artistic
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/03_spaces.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/25_vregshdrs.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/30_cbitfields.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/20_vregsparse.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/10_language.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/50_gcc_hdr.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/55_gcc_info.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/60_gcc_vderegs.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/00_pod.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/05_number.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/01_manifest.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/02_help.t
perl-SystemC-Vregs.noarch: W: spurious-executable-perm
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/51_gcc_str.t
perl-SystemC-Vregs.noarch: W: invalid-license LGPL+
perl-SystemC-Vregs.noarch: W: invalid-license Artistic
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/25_vregshdrs.t perl(Config)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/00_pod.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/01_manifest.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/02_help.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/03_spaces.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/05_number.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/10_language.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/20_vregsparse.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/25_vregshdrs.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/30_cbitfields.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/50_gcc_hdr.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-SystemC-Vregs-1.460/examples/51_gcc_str.t perl(Test)
perl-SystemC-Vregs.noarch: W: doc-file-dependency
/usr/share/doc/perl-Sys

[Bug 476406] Review Request: sympow - Special Values of Symmetric Power Elliptic Curve L-Functions

2008-12-14 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=476406





--- Comment #2 from Conrad Meyer   2008-12-14 18:45:45 EDT 
---
New URLs:

http://konradm.fedorapeople.org/fedora/SPECS/sympow.spec
http://konradm.fedorapeople.org/fedora/SRPMS/sympow-1.019-2.fc9.src.rpm

I just fixed the launcher script to be multilibs-safe.

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

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


[Bug 472842] Review Request: pytagger - ID3 Tag Reader and Writer Library for Python

2008-12-14 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=472842





--- Comment #10 from Fedora Update System   
2008-12-14 18:58:47 EDT ---
pytagger-0.5-1.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/pytagger-0.5-1.fc9

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

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


[Bug 472842] Review Request: pytagger - ID3 Tag Reader and Writer Library for Python

2008-12-14 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=472842





--- Comment #11 from Fedora Update System   
2008-12-14 19:00:29 EDT ---
pytagger-0.5-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/pytagger-0.5-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 472060] Review Request: mumbles - growl like notification system for gnome

2008-12-14 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=472060





--- Comment #7 from Fabian Affolter   2008-12-14 
19:09:43 EDT ---
Koji scratch build failed

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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 #21 from Bruno Cornec   2008-12-14 19:12:19 
EDT ---
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.

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

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


[Bug 472060] Review Request: mumbles - growl like notification system for gnome

2008-12-14 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=472060





--- Comment #8 from Christoph Wickert   2008-12-14 
19:22:04 EDT ---
This is due to

Patch0:  mumbles-sitedir.patch
...
%prep
%setup -q -n mumbles-%{version}
%patch -p1

needs to be
%patch0 -p1
  ^

or even better:
%patch0 -p1 -b .sitedir

Backing up patched files is useful for debugging patch problems.

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

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


[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

2008-12-14 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 #20 from David Robinson   2008-12-14 19:30:24 
EDT ---
Doesn't this need a "BuildRequires: python-devel"?

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

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


[Bug 453395] Review Request: libmapi - OpenChange: Microsoft Exchange access with native protocols

2008-12-14 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=453395





--- Comment #8 from David Robinson   2008-12-14 19:32:49 
EDT ---
Doesn't this also need a "BuildRequires: autoconf sysconftool automake libtool
flex zlib-devel bison"?

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

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


[Bug 476460] New: Review Request: pymilter - Python interface to sendmail milter API

2008-12-14 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: pymilter - Python interface to sendmail milter API

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

   Summary: Review Request: pymilter - Python interface to
sendmail milter API
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: stu...@gathman.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://customdesigned.users.sourceforge.net/pymilter.spec
SRPM URL:
http://customdesigned.users.sourceforge.net/pymilter-0.9.0-1.fc9.src.rpm
Description: This is a python extension module to enable python scripts to
attach to sendmail's libmilter functionality.  Additional python
modules provide for navigating and modifying MIME parts (adds to email
package), and sending DSNs or doing CBVs.

This is the first Fedora package I am submitting myself.  Other packages of
mine have been submitted by j...@tummy.com (Sean Reifschneider).

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

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


[Bug 476460] Review Request: pymilter - Python interface to sendmail milter API

2008-12-14 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=476460


Stuart D Gathman  changed:

   What|Removed |Added

 Blocks||177841




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

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


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

2008-12-14 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


Kevin Kofler  changed:

   What|Removed |Added

 CC||ke...@tigcc.ticalc.org




--- Comment #3 from Kevin Kofler   2008-12-14 20:10:43 
EDT ---
Right, SAGE just puts the headers into /usr/include/symmetrica and builds the
sources into a libsymmetrica.a. We want to do the same so we can link SAGE
against it (and that's what the specfile does).

The lack of a Makefile sucks, but I don't see it as a blocker, as the
instructions to build it are fairly simple.

The license mentioned in the specfile is actually MIT, not BSD. That's pretty
close to public domain. Either way, the license is definitely acceptable for
Fedora, it matters only for the text to write into the License tag. I'd suggest
just going with License: MIT. Making something truely public domain isn't
allowed under German copyright law (Urheberrechtsgesetz), which I assume is why
the MIT X11 license was used instead. But of course clarification can't hurt if
we can get it.

As for the warnings, I can try to fix them.

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

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


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

2008-12-14 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





--- Comment #4 from Sandro Mathys   2008-12-14 21:07:39 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/jcalendar.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/jcalendar-1.3.2-1.fc10.src.rpm

Thanks for the comments! Everything has been addressed in the new version.

The given script for preserving the timestamp while remove all \r didn't work
but I got the idea and was able to come up with this:

%{__sed} -i.tmp 's/\r//' jcalendar-license.txt
touch -r jcalendar-license.txt.tmp jcalendar-license.txt

This does what was intended I think...and is even a little bit shorter.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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 #6 from Kevin Fenzi   2008-12-14 21:33:20 EDT ---
>rpmlint on SRPM:
>
>spamassassin.src:37: W: redundant-prefix-tag
>The Prefix tag is uselessly defined as %{_prefix} in your spec file. It should
>be removed, as it is redundant with rpm defaults.

Removed. 

>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. ;( 


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


>spamassassin.src:542: W: macro-in-%changelog postun
>Macros are expanded in %changelog too, which can in unfortunate cases lead to
>the package not building at all, or other subtle unexpected conditions that
>affect the build.  Even when that doesn't happen, the expansion results in
>possibly "rewriting history" on subsequent package revisions and generally odd
>entries eg. in source rpms, which is rarely wanted.  Avoid use of macros in
>%changelog altogether, or use two '%'s to escape them, like '%%foo'.

Fixed. 

>spamassassin.src:580: W: macro-in-%changelog post
>Macros are expanded in %changelog too, which can in unfortunate cases lead to
>the package not building at all, or other subtle unexpected conditions that
>affect the build.  Even when that doesn't happen, the expansion results in
>possibly "rewriting history" on subsequent package revisions and generally odd
>entries eg. in source rpms, which is rarely wanted.  Avoid use of macros in
>%changelog altogether, or use two '%'s to escape them, like '%%foo'.

Fixed. 

>spamassassin.src: W: mixed-use-of-spaces-and-tabs (spaces: line 135, tab: line
>108)
>The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
>annoyance.  Use either spaces or tabs for indentation, not both.

At least that instance fixed up. ;) 

>spamassassin.src: W: summary-ended-with-dot Spam filter for email which can be
>invoked from mail delivery agents.
>Summary ends with a dot.
>
>Fix.

Fixed. 

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


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

>spamassassin.i386: W: non-conffile-in-etc /etc/logrotate.d/sa-update
>A non-executable file in your package is being installed in /etc, but is not a
>configuration file. All non-executable files in /etc should be configuration
>files. Mark the file as %config in the spec file.
>
>Fix.

Good catch. Fixed. 

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

>spamassassin.i386: E: non-readable /etc/cron.d/sa-update 0600
>The file can't be read by everybody. If this is expected (for security
>reasons), contact your rpmlint distributor to get it added to the list of
>exceptions for your distro (or add it to your local configuration if you
>installed rpmlint from the source tarball).
>
>Probably OK.

This makes little sense to me. I would e

[Bug 476315] Review Request: evolution-mapi - Exchange 2007 support for Evolution

2008-12-14 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


Sandro Mathys  changed:

   What|Removed |Added

 CC||r...@fedoraproject.org




--- Comment #2 from Sandro Mathys   2008-12-14 21:41:22 
EDT ---
rpmlint output when run on the resulting RPMs:

evolution-mapi.i386: E: library-without-ldconfig-postin
/usr/lib/libexchangemapi-1.0.so.0.0.0
evolution-mapi.i386: E: library-without-ldconfig-postun
/usr/lib/libexchangemapi-1.0.so.0.0.0
evolution-mapi.i386: E: zero-length /usr/share/doc/evolution-mapi-0.1/NEWS
evolution-mapi.i386: E: no-changelogname-tag
evolution-mapi-debuginfo.i386: E: no-changelogname-tag
evolution-mapi-devel.i386: W: no-documentation
evolution-mapi-devel.i386: E: no-changelogname-tag
3 packages and 0 specfiles checked; 6 errors, 1 warnings.

the no-changelogname-tag errors seem to be fixed in the spec file linked above,
but not in the spec file that is included in the SRPM.

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

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


[Bug 453395] Review Request: libmapi - OpenChange: Microsoft Exchange access with native protocols

2008-12-14 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=453395


Sandro Mathys  changed:

   What|Removed |Added

 CC||r...@fedoraproject.org




--- Comment #9 from Sandro Mathys   2008-12-14 21:54:34 
EDT ---
rpmlint output for the SRPM:
openchange.src:132: W: macro-in-%changelog post
openchange.src: W: mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 8)
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

rpmlint output for the resulting RPMs:
openchange.i386: W: no-documentation
openchange.i386: E: library-without-ldconfig-postin /usr/lib/libmapi.so.0.8
openchange.i386: E: library-without-ldconfig-postun /usr/lib/libmapi.so.0.8
openchange.i386: E: library-without-ldconfig-postin /usr/lib/libocpf.so.0.8
openchange.i386: E: library-without-ldconfig-postun /usr/lib/libocpf.so.0.8
openchange.i386: E: library-without-ldconfig-postin
/usr/lib/libmapiadmin.so.0.8
openchange.i386: E: library-without-ldconfig-postun
/usr/lib/libmapiadmin.so.0.8
openchange.i386: W: incoherent-version-in-changelog 0:0.8-0.3.svn960.fc9
['0.8-0.3.svn909.fc10', '0.8-0.3.svn909']
openchange.i386: W: shared-lib-calls-exit /usr/lib/libmapi.so.0.8
e...@glibc_2.0
openchange.i386: W: shared-lib-calls-exit /usr/lib/libocpf.so.0.8
e...@glibc_2.0
4 packages and 0 specfiles checked; 6 errors, 4 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 476471] New: Review Request: fedora-Linux_Security_Guide - A security guide for linux

2008-12-14 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: fedora-Linux_Security_Guide - A security guide for 
linux

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

   Summary: Review Request: fedora-Linux_Security_Guide - A
security guide for linux
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: e...@christensenplace.us
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
SRPM URL:
http://sparks.fedorapeople.org/fedora-Linux_Security_Guide-10-en-US-1.0-1.src.rpm
Description: This is a guide, written in Publican, describing Linux Security
implementation.  This is my first package and I will need a sponsor.

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

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


[Bug 476471] Review Request: fedora-Linux_Security_Guide - A security guide for linux

2008-12-14 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=476471


Eric Christensen  changed:

   What|Removed |Added

 Blocks||177841




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

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


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

2008-12-14 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: ebook-tools - Tools for accessing and converting 
various ebook file formats

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

   Summary: Review Request: ebook-tools - Tools for accessing and
converting various ebook file formats
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: john5...@googlemail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://john5342.fedorapeople.org/rpms/ebook-tools.spec
SRPM URL:
http://john5342.fedorapeople.org/rpms/ebook-tools-0.1.1-1.fc10.src.rpm

Description: Tools for accessing and converting various ebook file formats.

scratch-build: http://koji.fedoraproject.org/koji/taskinfo?taskID=998726

This package besides a couple of utilities includes libepub which provides
support for the epub document format for okular.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 CC||bnoc...@redhat.com,
   ||orcanba...@yahoo.com




--- Comment #1 from Orcan 'oget' Ogetbil   2008-12-14 
23:12:45 EDT ---
I reviewed this package. It surely needs some work:

* Can bug #459365 be closed now?

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

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

* 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)."
   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.

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

* What is wrong with the locale files in the tarball? (A more detailed
explanation as a comment within the SPEC file please.)
Also, assuming you have a legitimate reason to remove these files, why are you
BR'ing gettext?

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

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

* 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

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

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

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

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

* 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).

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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

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




--- Comment #2 from Orcan 'oget' Ogetbil   2008-12-14 
23:15:07 EDT ---
Added Bastien and Julian to CC since they are the last known maintainers. Sorry
if this was not desired.

-- 
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] New: Review Request: python-zope-filesystem - Python-Zope Libraries Base Filesystem

2008-12-14 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-filesystem - Python-Zope Libraries Base 
Filesystem

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

   Summary: Review Request: python-zope-filesystem - Python-Zope
Libraries Base Filesystem
   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-filesystem.spec
SRPM URL:
http://konradm.fedorapeople.org/fedora/SRPMS/python-zope-filesystem-1-1.fc9.src.rpm
Description:
This package contains the base filesystem layout for all Fedora
python-zope-* packages.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list 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-14 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

 CC||felix.schw...@oss.schwarz.e
   ||u




--- Comment #1 from Conrad Meyer   2008-12-14 23:41:58 EDT 
---
Note: this must be non-noarch because python_sitearch's location varies
depending on the arch.

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

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


[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

2008-12-14 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=468189





--- Comment #15 from manuel wolfshant   2008-12-15 
00:10:05 EDT ---
The problems issued by rpmbuild
  /var/tmp/rear-1.7.9-1.fc9-root/usr/share/doc/rear-1.7.9
  + exit 0
  warning: File listed twice: /etc/rear/Debian/ia64.conf
  warning: File listed twice: /etc/rear/GNU/Linux.conf

means that in your %files section, the above mentioned two files are specified
twice. I did not look over the spec, but may times this warning is caused by
including once the directory and a second time a specific file.

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

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


[Bug 453395] Review Request: libmapi - OpenChange: Microsoft Exchange access with native protocols

2008-12-14 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=453395





--- Comment #10 from Jerry Amundson   2008-12-15 00:11:50 
EDT ---
On rawhide, I see this:
$ rpmlint Download/openchange-0.8-0.3.svn909.fc10.src.rpm
/usr/share/rpmlint/Pkg.py:16: DeprecationWarning: The popen2 module is
deprecated.  Use the subprocess module.
  import popen2
openchange.src:132: W: macro-in-%changelog post
openchange.src: W: mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 8)
1 packages and 0 specfiles checked; 0 errors, 2 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 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

2008-12-14 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 #21 from Andrew Bartlett   2008-12-15 00:19:56 
EDT ---
I will note that while it might not help with packaging, it is the intention of
the Samba Team to only have one wbinfo and one nss_winbind, which will talk to
either Samba3 or Samba4's winbindd (ie, share a protocol).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Parag AN(पराग)   2008-12-15 00:25:46 
EDT ---
+ is OK
- Needs work

Review:
+ package builds in mock.
 Koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=998757
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
30cc5c89f9bfeffaf50e366ee309c7fc  assogiate-0.2.1.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ scrollkeeper and gtk-update-icon-cache scriptlets are used.
+ GUI app.

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

2008-12-14 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


Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #10 from Parag AN(पराग)   2008-12-15 00:25:52 
EDT ---
+ is OK
- Needs work

Review:
+ package builds in mock.
 Koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=998755
- rpmlint is silent for RPM. But Not for SRPM.
onboard.src: W: strange-permission onboard.spec 0666
==> easy to fix.

+ source files match upstream.
6e0dde122c50dcdfc27fe32b714edf3f  onboard-0.91.2.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 files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Desktop files installed correctly.
+ Followed python guidelines.
+ GUI app.

Suggestions:-
  You can ask upstream to include/modify .desktop files as used in this
package.

APPROVED.

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

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


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

2008-12-14 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 #1 from manuel wolfshant   2008-12-15 
00:29:13 EDT ---
I'd say that the version number is not correct. Nowhere in the scripts or on
the website do I see any reference to 20071020. OTOH there are references as 
"Version 0.42 November 06  Bug fixes only" (in the script) and "Version  
0.5" (on the main web page of the project)

License is not correct. The website claims GPLv2+ but the scripts themselves
include:
#This program is free software; you can redistribute it and/or modify
#it under the terms of the GNU General Public License Version 2 as
published by
#the Free Software Foundation.
This clearly makes them GPLv2 (not GPLv2+)

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

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


[Bug 476460] Review Request: pymilter - Python interface to sendmail milter API

2008-12-14 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=476460





--- Comment #1 from manuel wolfshant   2008-12-15 
00:41:06 EDT ---
Source0 should follow the guidelines described at
https://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC 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-14 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 #4 from Parag AN(पराग)   2008-12-15 00:38:31 
EDT ---
1) Buildroot should be from
http://fedoraproject.org/wiki/PackagingGuidelines#BuildRoot_tag

2) defattr usage should be %defattr(-,root,root, -)

3) according to 
http://fedoraproject.org/wiki/PackagingGuidelines#Why_the_.25makeinstall_macro_should_not_be_used
use make DESTDIR=$RPM_BUILD_ROOT install

4) Also, in Changelog use %%post instead of %post

-- 
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-14 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 #7 from Mamoru Tasaka   2008-12-15 
00:47:19 EDT ---
Please also submit updates request for F-10.

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

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


[Bug 469527] Review Request: tcping - Check of TCP connection to a given IP/Port

2008-12-14 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=469527


manuel wolfshant  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review?




--- Comment #7 from manuel wolfshant   2008-12-15 
00:48:23 EDT ---
before I start a review, would you mind updating to the latest release (1.3.5)
?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list 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-14 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 #22 from Matthew Barnes   2008-12-15 01:01:21 
EDT ---
What about libnss_winbind.so, is that also compatible with Samba3?

If so, perhaps we should just disable the samba4-winbind subpackage until we
move to Samba4 officially?

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

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


[Bug 473718] Review Request: perl-Directory-Scratch-Structured - Creates temporary files and directories from a structured description

2008-12-14 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=473718


Parag AN(पराग)  changed:

   What|Removed |Added

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

Bug 473718 depends on bug 476160, which changed state.

Bug 476160 Summary: Review Request: perl-Directory-Scratch - Self-cleaning 
scratch space for tests
https://bugzilla.redhat.com/show_bug.cgi?id=476160

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



--- Comment #3 from Parag AN(पराग)   2008-12-15 01:12:01 
EDT ---
Result: FAIL
Failed 1/10 test programs. 1/32 subtests failed.

koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=998817

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

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


[Bug 476483] Review Request: ghc-paths - library for information about ghc paths

2008-12-14 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=476483


Jens Petersen  changed:

   What|Removed |Added

 CC||fedora-haskell-l...@redhat.
   ||com




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

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


[Bug 476483] New: Review Request: ghc-paths - library for information about ghc paths

2008-12-14 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: ghc-paths - library for information about ghc paths

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

   Summary: Review Request: ghc-paths - library for information
about ghc paths
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: peter...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://petersen.fedorapeople.org/ghc-paths/ghc-paths.spec
SRPM URL:
http://petersen.fedorapeople.org/ghc-paths/ghc-paths-0.1.0.5-1.fc10.src.rpm
Description:
Knowledge of GHC's installation directories.

Needed by recent haddock to 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 476483] Review Request: ghc-paths - library for information about ghc paths

2008-12-14 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=476483


Jens Petersen  changed:

   What|Removed |Added

URL||http://hackage.haskell.org/
   ||cgi-bin/hackage-scripts/pac
   ||kage/ghc-paths




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list 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-14 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 #23 from Andrew Bartlett   2008-12-15 02:19:53 
EDT ---
So, I should clarify:

Samba4 and Samba 3.4 are being developed in the samba git repository - under a
branch of 'master'.  

In that branch, there will be soon (I hope) a common libnss_winbind and a
common wbinfo, along with many other shared things.  

But because this is not a stable protocol, the protocol used will be different
to what Samba3 in current Fedora uses.

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

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


[Bug 465897] Review Request: Judy - General purpose dynamic array library

2008-12-14 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=465897


Mamoru Tasaka  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #15 from Mamoru Tasaka   2008-12-15 
02:19:15 EDT ---
Okay.
-
This package (Judy) is APPROVED by mtasaka
-

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

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


[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client

2008-12-14 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 #24 from Jerry Amundson   2008-12-15 02:40:48 
EDT ---
(In reply to comment #23)
> So, I should clarify:
> 
> Samba4 and Samba 3.4 are being developed in the samba git repository - under a
> branch of 'master'.  
> 
> In that branch, there will be soon (I hope) a common libnss_winbind and a
> common wbinfo, along with many other shared things.  
> 
> But because this is not a stable protocol, the protocol used will be different
> to what Samba3 in current Fedora uses.

Then, logically, libnss_winbind and wbinfo, et. al, will not be "common", ergo
they need to be packaged separately, or not at all, right?

Put another way, do the Samba developers consider the 4.x and 3.4 as
environments that can exist in parallel, or not? Either it is or it isn't.

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.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the 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   >