[Bug 236521] Review Request: nspluginwrapper - A compatibility layer for Mozilla/Firefox plugins

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

Summary: Review Request: nspluginwrapper - A compatibility layer for 
Mozilla/Firefox plugins
Alias: nspluginwrapper

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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 02:03 EST ---
(In reply to comment #29)
 Re: comment #23
 Also, limiting nspluginwrapper to flash is IMHO a bad idea, there are many 
 other
 plugins.

I don't think so. Have you tried any other plug-ins? They just didn't work in
the wrapped environment. It can be a bug in packaging or something else, but
only flash works fine under nspluginwrapper now (at least for me).

But it's really easy to add any other plug-in to wrap, there's a list of wrapped
plug-ins in a spec file. 



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

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


[Bug 240287] Review Request: muParser - A fast math parser library

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

Summary: Review Request: muParser  - A fast math parser library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 02:43 EST ---
Ok, here the new packages.
SPEC: http://downloads.sourceforge.net/qsmartcard/muParser.spec?use_mirror=osdn
SRPM:
http://downloads.sourceforge.net/qsmartcard/muParser-1.27-4.fc6.src.rpm?use_mirror=osdn

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

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


[Bug 240287] Review Request: muParser - A fast math parser library

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

Summary: Review Request: muParser  - A fast math parser library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 02:59 EST ---
Yes, that's fine.  I already approved this package, so you can make your CVS
request whenever you're ready.

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

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


[Bug 240287] Review Request: muParser - A fast math parser library

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

Summary: Review Request: muParser  - A fast math parser library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 03:10 EST ---
New Package CVS Request
===
Package Name: muParser
Short Description: A fast math parser library
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 F-7
InitialCC: [EMAIL PROTECTED]

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

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


[Bug 243501] New: Review Request: qtiplot-doc - Documentation of qtiplot

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




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

   Summary: Review Request: qtiplot-doc - Documentation of qtiplot
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://downloads.sourceforge.net/qsmartcard/qtiplot-doc.spec?use_mirror=osdn
SRPM URL: 
http://downloads.sourceforge.net/qsmartcard/qtiplot-doc-0.9_rc1-1.fc6.src.rpm?use_mirror=osdn
Description: The documentation of the application.

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

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


[Bug 243501] Review Request: qtiplot-doc - Documentation of qtiplot

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

Summary: Review Request: qtiplot-doc - Documentation of qtiplot


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240373




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

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


[Bug 240373] Review Request: qtiplot - Data Analysis and Scientific Plotting

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

Summary: Review Request: qtiplot -  Data Analysis and Scientific Plotting


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||243501
  nThis||




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

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


[Bug 240373] Review Request: qtiplot - Data Analysis and Scientific Plotting

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

Summary: Review Request: qtiplot -  Data Analysis and Scientific Plotting


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|243501  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 04:01 EST ---
ok will be done.
Here the new files:
SRPM:
http://downloads.sourceforge.net/qsmartcard/qtiplot-0.9_rc1-3FC5.src.rpm?use_mirror=osdn
SPEC: http://downloads.sourceforge.net/qsmartcard/qtiplot.spec?use_mirror=osdn

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

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


[Bug 243501] Review Request: qtiplot-doc - Documentation of qtiplot

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

Summary: Review Request: qtiplot-doc - Documentation of qtiplot


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn|240373  |




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

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


[Bug 220402] Review Request: jscall-sharp-0.0.2 - a JS binding for gecko-sharp

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

Summary: Review Request: jscall-sharp-0.0.2 - a JS binding for gecko-sharp


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||238142
  nThis||




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

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


[Bug 192436] Review Request: xorg-x11-server-Xgl

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

Summary: Review Request: xorg-x11-server-Xgl


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 07:47 EST ---
Thanks!

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

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


[Bug 241486] Review Request: ocaml-curl - OCaml Curl library (ocurl)

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

Summary: Review Request: ocaml-curl - OCaml Curl library (ocurl)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240557




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 09:09 EST ---
This package has a dependency on ocaml-findlib-devel which isn't in the
repository yet. This dependency has been requested for a review which shoulb be
approved and imported before make a full review of this one.

Note: So i added this review request in dependency block.

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241486
  nThis||




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

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


[Bug 239546] Review Request: mdsplib - METAR Decoder Software Package Library

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

Summary: Review Request: mdsplib - METAR Decoder Software Package Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 09:16 EST ---
okay ;-)

 Sorry for the trouble here, but static libs are just to be avoided if 
 possible,
 and it looks very possible here. ;) 

I do agree with that.



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

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


[Bug 230549] Review Request: oxine - Lightweight, purely OSD based xine frontend

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

Summary: Review Request: oxine - Lightweight, purely OSD based xine frontend


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 09:22 EST ---
mdslib is being re-reviewing with included shared lib instead of static one.
So this review will be delayed until the dependency will be approved yet :)

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

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


[Bug 237355] Review Request: kflickr - an easy to use photo uploader for flickr written for KDE.

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

Summary: Review Request: kflickr  - an easy to use photo uploader for flickr 
written for KDE.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|NEXTRELEASE |
   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 10:35 EST ---
New Package CVS Request
===
Package Name: kflickr
Short Description: an easy to use photo uploader for flickr written for KDE.
Owners: [EMAIL PROTECTED]
Branches:  F7
InitialCC: 

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

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


[Bug 210823] Review Request: wxsvg - C++ library to create, manipulate and render SVG files

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

Summary: Review Request: wxsvg - C++ library to create, manipulate and render 
SVG files


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 10:54 EST ---
Please close this bug when rebuilding is done.

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

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


[Bug 227669] Review Request: ppl-0.9 - A modern C++ library providing numerical abstractions

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

Summary: Review Request: ppl-0.9 - A modern C++ library providing numerical 
abstractions


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 11:13 EST ---
Well, for rpath, try to disable system-wide rpath by removing
hardcode_libdir_flag_spec from libtool.
Even after that, ppl_gprolog should have rpath for %_libdir/ppl
because of ppl-0.9-makefiles.patch.

For concrete,
---
%configure
sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=|g' libtool
sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool
---
I suppose this will work for this package (the section Removing rpath in
http://fedoraproject.org/wiki/Packaging/Guidelines). After doing this,
ppl-0.9-makefiles.patch should be needed.

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

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


[Bug 227669] Review Request: ppl - A modern C++ library providing numerical abstractions

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

Summary: Review Request: ppl - A modern C++ library providing numerical 
abstractions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: ppl-0.9 - A |Review Request: ppl - A
   |modern C++ library providing|modern C++ library providing
   |numerical abstractions  |numerical abstractions




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 11:33 EST ---
Okay, undefined non-weak symbols disappeared. I will recheck
again. Please let me know if rpath issue disappears by comment 34.

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

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


[Bug 243538] New: Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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




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

   Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns
account
   Product: Fedora Core
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: spec info here
SRPM URL: srpm info here
Description: Installed Fedora 7, came with ddclient 3.7.1, doesn't update my 
dyndns account, ran it in debug/query/verbose, it shown correct new IP address, 
it just doesn't update my dyndns account of the new IP address.  Bother to 
downgrade to ddclient 3.6.6, that is working.

Read thru:
http://forums.fedoraforum.org/showpost.php?p=807103postcount=1

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

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


[Bug 243539] New: Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update dyndns account

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




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

   Summary: Review Request: ddclient.noarch 0:3.7.1-1.fc7 -
Doesn't update dyndns account
   Product: Fedora Core
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: spec info here
SRPM URL: srpm info here
Description: Installed Fedora 7, came with ddclient 3.7.1, doesn't update my 
dyndns account, ran it in debug/query/verbose, it shown correct new IP address, 
it just doesn't update my dyndns account of the new IP address.  Bother to 
downgrade to ddclient 3.6.6, that is working.

Read thru:
http://forums.fedoraforum.org/showpost.php?p=807103postcount=1

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

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


[Bug 243540] New: Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update dyndns account

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




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

   Summary: Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't
update dyndns account
   Product: Fedora Core
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: spec info here
SRPM URL: srpm info here
Description: Installed Fedora 7, came with ddclient 3.7.1, doesn't update my 
dyndns account, ran it in debug/query/verbose, it shown correct new IP address, 
it just doesn't update my dyndns account of the new IP address.  Bother to 
downgrade to ddclient 3.6.6, that is working.

Read thru:
http://forums.fedoraforum.org/showpost.php?p=807103postcount=1

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

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


[Bug 236162] Review Request: kadischi - Fedora based LiveCD/LiveDVD creation utility

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

Summary: Review Request: kadischi - Fedora based LiveCD/LiveDVD creation utility


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 12:22 EST ---
+ BuildRoot good
+ Uses SMP flags
+ documentation tagged with %doc
+ good Summary and %description
+ files in the right places, %{_sysconfdir} for configs, %{_datadir} for scripts
and data, %{_libexecdir} for the tools


%description says Fedora Core, you might want to remove Core when building on 
F7+ :)

Comment 34 still applies.

APPROVED

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

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


[Bug 243539] Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update dyndns account

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

Summary: Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update 
dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |ddclient.noarch 0:3.7.1-   |ddclient.noarch 0:3.7.1-
   |1.fc7 - Doesn't update |1.fc7 - Doesn't update
   |dyndns account  |dyndns account
 Status|NEW |CLOSED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 12:25 EST ---


*** This bug has been marked as a duplicate of 243538 ***

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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account 


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 12:25 EST ---
*** Bug 243539 has been marked as a duplicate of this bug. ***

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

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


[Bug 243540] Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update dyndns account

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

Summary: Review Request: ddclient.noarch 0:3.7.1-1.fc7 - Doesn't update dyndns 
account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |ddclient.noarch 0:3.7.1-|ddclient.noarch 0:3.7.1-
   |1.fc7 - Doesn't update  |1.fc7 - Doesn't update
   |dyndns account  |dyndns account
 Status|NEW |CLOSED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 12:25 EST ---


*** This bug has been marked as a duplicate of 243538 ***

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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account 


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 12:25 EST ---
*** Bug 243540 has been marked as a duplicate of this bug. ***

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

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


[Bug 227244] Review Request: gfa-0.4.1 - GTK+ fast address book

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

Summary: Review Request: gfa-0.4.1 - GTK+ fast address book


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|CLOSED
 Resolution||NEXTRELEASE
   Flag|needinfo?([EMAIL PROTECTED]|
   |m)  |




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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: ddclient|Review Request: ddclient
   |3.7.1 - Doesn't update  |3.7.1 - Doesn't update
   |dyndns account  |dyndns account
Product|Fedora Core |Fedora Extras
  Component|Package Review  |Package Review




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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Package Review  |ddclient
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |




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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||fedora-package-
   ||[EMAIL PROTECTED]




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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora Core
  Component|ddclient|Package Review




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

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


[Bug 227669] Review Request: ppl - A modern C++ library providing numerical abstractions

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

Summary: Review Request: ppl - A modern C++ library providing numerical 
abstractions


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:25 EST ---
Well, for 0.9-7:

* %check
  - This package contains test/ directory and some
check programs seem to be executed. If make check
makes sense, please add:
--
%check
make check
--
(Well, this is very very long...)

* Documentation
  - It seems that README.configure is for people who want to
rebuild ppl by themselves (i.e. who want to call configure
by themselves) and so IMO this document should be removed
for rpm user.

  - IMO TODO should be added to documents.

* %description
  - Please refrect static archive split. For example,
-devel package says:
---
%description devel
The header files, documentation and static libraries for developing

---

* rpath issue
  - As said above, please let me know if rpath issue is
resolved by hacking libtool.

All other things are okay.

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

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


[Bug 227669] Review Request: ppl - A modern C++ library providing numerical abstractions

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

Summary: Review Request: ppl - A modern C++ library providing numerical 
abstractions


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:27 EST ---
Well, as this is sponsor needed ticket:

-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to show that you have an understanding 
of the process and of the packaging guidelines as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a pre-review of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
http://fedoraproject.org/PackageReviewStatus/NEW.html
(NOTE: please don't choose Merge Review)


Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets


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

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


[Bug 236488] Review Request: aimject - AIM packet injection tool

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

Summary: Review Request: aimject - AIM packet injection tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:30 EST ---
Removing needsponsor

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

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


[Bug 238248] Review Request: ddccontrol - TFT monitor parameters control

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

Summary: Review Request: ddccontrol - TFT monitor parameters control


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:33 EST ---
Again ping?

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

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


[Bug 234721] Review Request: sakura - A terminal emulator based on GTK+ and VTE

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

Summary: Review Request: sakura - A terminal emulator based on GTK+ and VTE


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|CLOSED
 Resolution||NOTABUG
OtherBugsDependingO||201449
  nThis||
   Flag|fedora-review?, |
   |needinfo?([EMAIL PROTECTED]|
   |m)  |




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:38 EST ---
CLOSING.

If someone want to maintain this package, please sumbit a new
review request, Thank you.

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

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


[Bug 222618] Review Request: llk_linux - A very popular game from internet

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

Summary: Review Request: llk_linux - A very popular game from internet
Alias: llk

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|CLOSED
 Resolution||NOTABUG
OtherBugsDependingO|163776, 177841  |201449
  nThis||
   Flag|needinfo?([EMAIL PROTECTED]|
   |om) |




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 13:46 EST ---
CLOSING.

If someone want to maintain this package, please file 
a new review request, thank you.

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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Core |Fedora Extras
  Component|Package Review  |Package Review
   Flag|fedora-review?  |




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:02 EST ---
This is simply not a package review.  Let me try to fix up the mess.

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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Package Review  |ddclient




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

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


[Bug 239546] Review Request: mdsplib - METAR Decoder Software Package Library

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

Summary: Review Request: mdsplib - METAR Decoder Software Package Library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:03 EST ---
Well,

Taking in account first part of full review above which doesn't change,

OK - Removed Static lib from package
OK - Sahred lib correctly generated/included in package.
OK - Sub-packages are proper
Ok - No sub-package -lib are required
Ok - Scriptlets for shared lib are properly applied.

--
rpmlint ouputs
--

Silent from both rpm (main pakcage) and srpm.

From -devel package:
W: mdsplib-devel no-documentation

Harmless, this can be ignored.



==
** APPROVED **
==

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

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


[Bug 243538] Review Request: ddclient 3.7.1 - Doesn't update dyndns account

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

Summary: Review Request: ddclient 3.7.1 - Doesn't update dyndns account


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:07 EST ---
OK, that should be done.  The maintainer is assigned, and co-maintainer and
initialCC are CC'd.  Sorry for all the bugspam.

And to the initial submitter, if there's some piece of documentation or wiki
page somewhere that pointed you to the Fedora Extras Package Review Request form
as the proper way to submit a but report for a package, could you let us know
where it is so that it can be fixed?  Occasionally someone tries to report bugs
that way but we've never been able to figure out where we've given the
impression that it's the proper way to do that.

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

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


[Bug 239527] Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI

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

Summary: Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:12 EST ---
New Package CVS Request
===
Package Name: wammu
Short Description: Mobile Phone Manager Frontend - Gammu GUI
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 F-7 EL-4 EL-5 Devel
InitialCC: 



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

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


[Bug 234079] Review Request: kicker-compiz - Pager applet for kicker to make it work with compiz

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

Summary: Review Request: kicker-compiz - Pager applet for kicker to make it 
work with compiz


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:38 EST ---
It will fix this week-end.


ps: Sorry for the late of this bug, its mail notification had drowned in my 
mailbox.

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

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


[Bug 243109] Review Request: loki-lib - Loki C++ Library

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

Summary: Review Request: loki-lib - Loki C++ Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:53 EST ---
I took a quick look at this.  It builds ok in mock on rawhide; here's what
rpmlint says:

W: loki-lib unused-direct-shlib-dependency /usr/lib64/libloki.so.0.1.6
/lib64/libpthread.so.0
W: loki-lib unused-direct-shlib-dependency /usr/lib64/libloki.so.0.1.6
/lib64/libm.so.6

Now, the makefile explicitly adds -lpthread on linux; I can only assume they do
this because it's required in some way.  This library does seem to wrap pthread
functions but perhaps they aren't directly called.  In any case I don't really
understand enough to decide whether this is a problem.

I don't know where the link against libm comes from; I don't see it specified
anywhere.

There's a test suite in the package, but it requires that you do a static
compile and even then it doesn't actually seem to work.  You might want to make
a comment about that in the spec (or perhaps investigate and see if you can make
it work).

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

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


[Bug 220979] Review Request: tesseract - Raw OCR Engine

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

Summary: Review Request: tesseract - Raw OCR Engine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:54 EST ---
REVIEW:
 * dist tag present
 * valid Apache license
 * rpmlint:
W: tesseract-devel no-documentation
 * good md5 (c39bd7b465c37a3863140e88d51cd839)
 * the newest version packaged
 * package owns all directories well
 * proper %clean section
 * proper buildroot
 * nothing wrong with %files
 * no scriplets needed
 * devel subpackage present and looks good
 * no .la files
 * no GUI

I wasn't able to test it in mock, but I hope it's working.
Approved.

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

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


[Bug 235672] Review Request: zope3 - Zope 3 web application server framework

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

Summary: Review Request: zope3 - Zope 3 web application server framework


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:55 EST ---
None of these links seem to be valid.

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

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


[Bug 228255] Review Request: grub2 - grub next generation tools

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

Summary: Review Request: grub2 - grub next generation tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 14:57 EST ---
Any progress here?

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

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


[Bug 220979] Review Request: tesseract - Raw OCR Engine

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

Summary: Review Request: tesseract - Raw OCR Engine


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 15:01 EST ---
It does build fine in mock.  The only rpmlint warning I saw from that besides
the one you noted is
  W: tesseract mixed-use-of-spaces-and-tabs (spaces: line 16, tab: line 1)
which is not really an issue.


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

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


[Bug 241076] Review Request: redet-doc - Documentation for redet

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

Summary: Review Request: redet-doc - Documentation for redet


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 15:20 EST ---
redet-doc-8.21-1.fc7 has been pushed to the Fedora 7 stable repository.  If 
problems still persist, please make note of it in this bug report.

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

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


[Bug 241076] Review Request: redet-doc - Documentation for redet

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

Summary: Review Request: redet-doc - Documentation for redet


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||ERRATA
   Fixed In Version||8.21-1.fc7




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

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


[Bug 241075] Review Request: redet - Regular expression development and execution tool

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

Summary: Review Request: redet - Regular expression development and execution 
tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 15:20 EST ---
redet-8.22-4.fc7 has been pushed to the Fedora 7 stable repository.  If 
problems still persist, please make note of it in this bug report.

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

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


[Bug 241075] Review Request: redet - Regular expression development and execution tool

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

Summary: Review Request: redet - Regular expression development and execution 
tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||ERRATA
   Fixed In Version||8.22-4.fc7




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

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


[Bug 225667] Merge Review: cryptsetup-luks

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

Summary: Merge Review: cryptsetup-luks


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 15:48 EST ---
Now we have 1.0.5 release, where cryptsetup-luks becomes cryptsetup.

If you upgrade to this release bug #215199 and bug #217983 can be resolved.

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

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


[Bug 192438] Review Request: fedora-xgl-settings

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

Summary: Review Request: fedora-xgl-settings


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 16:00 EST ---
We should really close this bug as fedora-xgl-settings is no longer being worked
on in preference to a better configuration solution. Please comment Alphonse.

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

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


[Bug 192436] Review Request: xorg-x11-server-Xgl

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

Summary: Review Request: xorg-x11-server-Xgl


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 16:08 EST ---
Alphonse,

After some discussion about Xgl and why we are working to make it usable, it
seems the best place for Xgl is going to be a 3rd party repo. Really, the best
solution for providing the features that Xgl provides is aiglx. I understand the
reasoning behind why Xgl is still currently needed. Please comment on moving
things to Livna (my choice would be at least) and closing this bug.

Advantages to Livna:

1) No resistance to this package from within the Fedora dev world
2) No bugs on the fedora bugzilla, so no additional overhead for the Fedora devs
3) Ability to use livna-config-display vs yet another package
4) Existing trusted 3rd party repo that is enabled on a lot of systems
5) Existing infrastructure to support collaborative maintenance of the package
6) Livna already packages fglrx

Disadvantages:

1) It's Xgl, we want a good solution... such as aiglx
2) It does not get the Fedora stamp, which might end up being more good then 
bad
3) Less of a mirror infrastructure, but really... not that big of a deal

I've maybe missed some +- stuff, but I do think we should move this to livna.

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

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


[Bug 221776] Review Request: digitemp - Dallas Semiconductor 1-wire device reading console application

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

Summary: Review Request: digitemp - Dallas Semiconductor 1-wire device reading 
console application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 16:11 EST ---
New Package CVS Request
===
Package Name: digitemp
Short Description: Dallas Semiconductor 1-wire device reading console 
application
Owners: [EMAIL PROTECTED]
Branches: FC-6 F-7 EL-4 EL-5
InitialCC: 

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

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


[Bug 212984] Review Request: hunspell-ar - Arabic word list/dictionaries for OpenOffice

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

Summary: Review Request: hunspell-ar  - Arabic word list/dictionaries for 
OpenOffice


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 16:32 EST ---
Thanks for the great work.
I would not mind at all if you completely take over hunspell-ar. In fact, I even
prefer it that way (though it would be nice if you mention me in the spec).
However, I'm willing to be the one checking hunspell-ar in if this is required.

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

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


[Bug 242901] Review Request: perl-Geo-METAR - Perl module for accessing aviation weather information

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

Summary: Review Request: perl-Geo-METAR - Perl module for accessing aviation 
weather information


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 16:41 EST ---
Spec URL:
http://kwizart.free.fr/fedora/7/testing/perl-Geo-METAR/perl-Geo-METAR.spec
SRPM URL:
http://kwizart.free.fr/fedora/7/testing/perl-Geo-METAR/perl-Geo-METAR-1.14-2.kwizart.fc7.src.rpm
Description: Perl module for accessing aviation weather information

So only have updated the  BR perl(ExtUtils::MakeMaker) instead of BR perl (or
perl-devel)
mock build log are in the same dir... (fedora 7 x86_64)
I don't knwo if i miss something for the test suite but build.log says:
All tests successful.



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

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


[Bug 217735] Review Request: tcldict - Tcl dictionary extensiuon

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

Summary: Review Request: tcldict - Tcl dictionary extensiuon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 17:18 EST ---
It took me a bit to figure out why this refused to build due to the requirement
for tcl-devel  8.5.  Turns out that tcl grew an epoch.  Which is kind of
unfortunate, since tcl-devel-0:8.5  tcl-devel-1:8.5, but there's no way around
it.  In any case, that build dependency needs to be on tcp-devel  1:8.5.

Also, there's a static library.  Could you add a note as to why it's needed? 
Note that I know zilch about tcl extensions.  Normally I'd say it needs to be in
-static, but I don't know what this library would be used for.

I can't seem to get to upstream at the moment, so I'll have to save the source
comparison for later.

rpmlint has a couple of complaints.  This one is OK:
   W: tcldict-devel no-documentation

Generally static libraries aren't executable.  Why our dynamic libs are
executable and our static libs aren't, I don't know.
   W: tcldict-devel spurious-executable-perm 
/usr/lib64/dict8.5.1/libdictstub8.5.1.a

Review:
? can't check upstream source.
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
? can't check upstream for latest version.
X BuildRequires needs epoch on tcl-devel
* compiler flags are appropriate.
* %clean is present.
* package builds in mock after fixing deps (development, x86_64)
* package installs properly
* debuginfo package looks complete.
X rpmlint is silent.
* final provides and requires are sane.
* %check is present and all tests pass:
   Tests ended at Sat Jun 09 15:53:32 EDT 2007
   all.tcl:Total   189 Passed  188 Skipped 1   Failed  0
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
X executable .a file probably shouldn't be.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* headers are in -devel subpackage.
* no pkgconfig files.
X static libraries present, in -devel package.
* no libtool .la files.

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

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


[Bug 226377] Merge Review: rpm

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

Summary: Merge Review: rpm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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


[Bug 242566] Review Request: cegui (0.5.x branch) - Free library providing windowing and widgets for graphics APIs / engines

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

Summary: Review Request: cegui (0.5.x branch) - Free library providing 
windowing and widgets for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 17:31 EST ---
I'll take a quick look.  I won't do a full review because I trust things like
the upstream source and licensing and such are OK.

The versioning seems OK; the b is a post-release update, not beta, or at
least it seems that way from the upstream web pages.

The spec looks OK; if the dependencies of the -devel packages you need are OK,
you shouldn't need to actually list a build dependency on pkgconfig but it
doesn't hurt.

rpmlint spews a massive boatload of warnings.  Well over 1300 of them.  Most are
undefined-non-weak-symbol warnings; some are unused-direct-shlib-dependency.  I
haven't a clue about how you'd go about fixing them.

I'd give my ack to this if I understood the rpmlint warnings or was convinced
they're unfixable.  They're the only thing I see of issue with this package.

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

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


[Bug 192436] Review Request: xorg-x11-server-Xgl

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

Summary: Review Request: xorg-x11-server-Xgl


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 18:32 EST ---
Livna contain non free software that cannot be include in Fedora because of
license so IMO Xgl is not a good pace for a free software. The only free
software on livna is livna-config-display (not entirely sure about that).

Fedora provide many soft that do the same stuff, personally I like that and
don't see it like a problem.

Like we have discus in private, I would prefer make a dedicate repo for Xgl and
like you know, your are welcome to help and we are already working in that way
at the moment.

http://fedoraxgl.tuxfamily.org (web site -- must be updated)
http://download.tuxfamily.org/fedoraxgl (the repo)





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

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


[Bug 241903] Review Request: etherbat - Ethernet topology discovery

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

Summary: Review Request: etherbat - Ethernet topology discovery


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 18:51 EST ---
I always build in rawhide.  However, I also always build on x86_64 which is the
problem here.  The library lives in /usr/lib64/libnet.a on that platform.  In
additionm, the compiler isn't called with the proper set of flags.  This breaks
the debuginfo package.

To fix these, you just need to change the make line to
   CFLAGS=$RPM_OPT_FLAGS make LIBNET_STATIC=%{_libdir}/libnet.a
and everything builds fine.

Unfortunately this does a static link, and according to the current rules this
needs to go through FESCo.  Ugh.  And it's possible that they won't approve it,
because it's possible to have a dynamic libnet.  (Debian ships it as a dynamic
lib, for example.)

Review:
* source files match upstream:
   2245dab0a9db77e4a742e3bf07862161de6623f46ae063312c38e32c8fb344f9  
   etherbat-1.0.1.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
* BuildRequires are proper.
X compiler flags not correct.
* %clean is present.
X package fails to build in mock (development, x86_64)
* package installs properly (once build is fixed, at least)
X debuginfo package is incomplete.
* rpmlint is silent (once build is fixed)
* final provides and requires are sane:
   etherbat = 1.0.1-1.fc8
  =
   /usr/bin/ruby
   libglib-2.0.so.0()(64bit)
   libpcap.so.0.9()(64bit)
   ruby
* %check is not present; no test suite upstream.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

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

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


[Bug 239165] Review Request: tcptraceroute - A traceroute implementation using TCP packets

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

Summary: Review Request: tcptraceroute - A traceroute implementation using TCP 
packets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 243571] New: Review Request: python-mechanoid - Python Programmatic Web Browser

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




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

   Summary: Review Request: python-mechanoid - Python Programmatic
Web Browser
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-mechanoid.spec
SRPM URL: 
ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-mechanoid-0.6.9-1.fc7.src.rpm
Description: mechanoid is a programmatic browser written in Python.  It is not a
web-bot and does not parse /robots.txt. It is intended to function as
the backend of special browsers, such as voice-text/text-voice browsers,
or to perform directed tasks, such as login to SourceForge to do a Quick
Release or to send and receive Yahoo! web mail.

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

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


[Bug 243573] New: Review Request: python-memcached - A Python memcached client library.

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




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

   Summary: Review Request: python-memcached - A Python memcached
client library.
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-memcached.spec
SRPM URL: 
ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/python-memcached-1.36-1.fc7.src.rpm
Description: This software is a 100% Python interface to the memcached memory 
cache
daemon.  It is the client side software which allows storing values in one
or more, possibly remote, memcached servers.  Search google for memcached
for more information.

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

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


[Bug 242566] Review Request: cegui (0.5.x branch) - Free library providing windowing and widgets for graphics APIs / engines

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

Summary: Review Request: cegui (0.5.x branch) - Free library providing 
windowing and widgets for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:24 EST ---
(In reply to comment #1)
 I'll take a quick look.  I won't do a full review because I trust things like
 the upstream source and licensing and such are OK.

No probs :-)
 
 The versioning seems OK; the b is a post-release update, not beta, or at
 least it seems that way from the upstream web pages.

Yep, that's pretty much the case. the 'b' release was a quick bug fixed version
of 0.5.0.

 rpmlint spews a massive boatload of warnings.  Well over 1300 of them.  Most 
 are
 undefined-non-weak-symbol warnings; some are unused-direct-shlib-dependency.  
 I
 haven't a clue about how you'd go about fixing them.

Hmm, I was confused by this because for me rpmlint only reports two messages,
the first one is the usual no documentation found in the devel package, and the
second is that the devel-doc package requires the -devel package. Both of wish
are not real errors and are ingorable. Can I ask which release/arch you tested
it on? I was using FC6/i386. 

 I'd give my ack to this if I understood the rpmlint warnings or was convinced
 they're unfixable.  They're the only thing I see of issue with this package.

No problem Jason. Thanks for the headsup, I'll look into those message and see
if I can figure out the cause and if anything needs doing.


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

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


[Bug 242566] Review Request: cegui (0.5.x branch) - Free library providing windowing and widgets for graphics APIs / engines

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

Summary: Review Request: cegui (0.5.x branch) - Free library providing 
windowing and widgets for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:27 EST ---
Ahhh, just realised I do indeed get those errors if I check in the installed
package rather than the RPMs themselves.

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241473
  nThis||




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

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


[Bug 241472] Review Request: ocaml-ocamlnet - Network protocols for OCaml

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

Summary: Review Request: ocaml-ocamlnet - Network protocols for OCaml


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review-  |fedora-review?




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

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


[Bug 241473] Review Request: ocaml-ssl - SSL bindings for OCaml

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

Summary: Review Request: ocaml-ssl - SSL bindings for OCaml


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:45 EST ---
Added bug dependency #240557

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

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


[Bug 241475] Review Request: ocaml-ulex - OCaml lexer generator for Unicode

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

Summary: Review Request: ocaml-ulex - OCaml lexer generator for Unicode


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240557




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:47 EST ---
Added bug dependency #240557

Review is suspend

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241475
  nThis||




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

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


[Bug 241476] Review Request: ocaml-pxp - Validating XML parser

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

Summary: Review Request: ocaml-pxp - Validating XML parser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240557, 241472, 241475




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:54 EST ---
Added bug dependencies blocks which must be review first.


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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241476
  nThis||




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

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


[Bug 241472] Review Request: ocaml-ocamlnet - Network protocols for OCaml

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

Summary: Review Request: ocaml-ocamlnet - Network protocols for OCaml


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241476
  nThis||




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

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


[Bug 241475] Review Request: ocaml-ulex - OCaml lexer generator for Unicode

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

Summary: Review Request: ocaml-ulex - OCaml lexer generator for Unicode


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241476
  nThis||




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

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


[Bug 241483] Review Request: ocaml-expat - OCaml wrapper for the Expat XML parsing library

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

Summary: Review Request: ocaml-expat - OCaml wrapper for the Expat XML parsing 
library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240557




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 19:55 EST ---
Added bug dependency which must be review first.

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241483
  nThis||




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

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


[Bug 241487] Review Request: cduce - An XML-oriented functional language.

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

Summary: Review Request: cduce - An XML-oriented functional language.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||240557, 241475, 241476,
   ||241483, 241486




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:02 EST ---
Added other bug dependencies blocks

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

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


[Bug 241476] Review Request: ocaml-pxp - Validating XML parser

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

Summary: Review Request: ocaml-pxp - Validating XML parser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241487
  nThis||




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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241487
  nThis||




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

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


[Bug 241486] Review Request: ocaml-curl - OCaml Curl library (ocurl)

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

Summary: Review Request: ocaml-curl - OCaml Curl library (ocurl)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241487
  nThis||




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

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


[Bug 241483] Review Request: ocaml-expat - OCaml wrapper for the Expat XML parsing library

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

Summary: Review Request: ocaml-expat - OCaml wrapper for the Expat XML parsing 
library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241487
  nThis||




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

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


[Bug 241475] Review Request: ocaml-ulex - OCaml lexer generator for Unicode

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

Summary: Review Request: ocaml-ulex - OCaml lexer generator for Unicode


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||241487
  nThis||




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

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


[Bug 242566] Review Request: cegui (0.5.x branch) - Free library providing windowing and widgets for graphics APIs / engines

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

Summary: Review Request: cegui (0.5.x branch) - Free library providing 
windowing and widgets for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:03 EST ---
Yes, you have to run rpmlint against the installed packages.  I install all of
the packages I test (into the mock chroot, a convenient hack) and run rpmlint on
them but I don't think many other reviewers do.

BTW, I build on x86_64 and almost always using rawhide.

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:10 EST ---
Nigel, do you still want to pre-review this request?
I'd like to start review other bug which depend on this one.

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

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


[Bug 239165] Review Request: tcptraceroute - A traceroute implementation using TCP packets

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

Summary: Review Request: tcptraceroute - A traceroute implementation using TCP 
packets


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:10 EST ---
This builds fine and rpmlint is quiet.

According to
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#PreReleasePackages
release should be 0.1.beta7%{?dist}.

This is another package that links statically with libnet.  Not that it has any
choice, but this requires a trip through FESCo.  Either that or the libnet
maintainer needs to get whatever patches debian uses to build libnet 
dynamically.

Review:
* source files match upstream:
   aed5b163ed4886f04242b46005a6cb4876ef38ad72001a94facb62a99dc99c57  
   tcptraceroute-1.5beta7.tar.gz
X package doesn't meet versioning guidelines for prerelease packages.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   tcptraceroute = 1.5-0.beta7.1.fc8
  =
   libpcap.so.0.9()(64bit)
* %check is not present; no test suite upstream.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.  (Actually the 
  documentation is nearly twice as large as the lone executable, but the 
package 
  is only 142K.)
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

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

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


[Bug 214751] Review Request: xview - XView widget libraries

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

Summary: Review Request: xview - XView widget libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:28 EST ---
Christian ping ?

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

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


[Bug 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper

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

Summary: Review Request: ocaml-findlib - Objective CAML package manager and 
build helper


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:46 EST ---
(In reply to comment #7)
 Nigel, do you still want to pre-review this request?
 I'd like to start review other bug which depend on this one.

Richard needs to be sponsored, so you'll just have to wait until the Ocaml
Packaging guidelines are approved, and changes to the ocaml rpm (if needed are
made), until then the review of all the ocaml packages (and this one) are pretty
much useless, it's actually the reason why sponsors aren't currently interested
in the packages, they want to wait for the packaging committee to approve it.

But no, if you want to review them all, fine by me.

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

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


[Bug 233602] Review Request: pykdeextentions - A collection of python packages to support KDE applications

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

Summary: Review Request: pykdeextentions - A collection of python packages to 
support KDE applications


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:52 EST ---
Rex, The SRPM url seem down, could you fix this ?



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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

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

Summary: Review Request: glump - A small web application to glue files from 
multiple sources


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:54 EST ---
1) I noticed and it's making rpmlint sad too, that you are not using correct tab
spaces in your spec file.

2) Your %install must have a cleaning buildroot section, I recommend you start
with:

%install
rm -rf %{buildroot}

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

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


[Bug 241549] Review Request: pixie - 3D renderer Renderman compliant

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

Summary: Review Request: pixie - 3D renderer Renderman compliant


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 20:58 EST ---
Hi Mamoru,

i'll check this out and launch the test on my main machine which has 2go of
memory (dual channel) 

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

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


[Bug 233602] Review Request: pykdeextentions - A collection of python packages to support KDE applications

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

Summary: Review Request: pykdeextentions - A collection of python packages to 
support KDE applications


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 21:10 EST ---
(In reply to comment #4)
 Rex, The SRPM url seem down, could you fix this ?
 
 

It's an apt repository, so of course it appears down, it got moved from testing
to stable, no big deal.

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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

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

Summary: Review Request: glump - A small web application to glue files from 
multiple sources


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 21:59 EST ---
Spec URL: http://repo.ocjtech.us/misc/fedora/6/SRPMS/glump-0.9.11-3.fc6.spec
SRPM URL: http://repo.ocjtech.us/misc/fedora/6/SRPMS/glump-0.9.11-3.fc6.src.rpm

I've updated the package to reflect Marek's comments.

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

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


[Bug 241549] Review Request: pixie - 3D renderer Renderman compliant

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

Summary: Review Request: pixie - 3D renderer Renderman compliant


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 22:03 EST ---
Okay. Please go again. Actually rebuilding this package
is too heavy for my machine...

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

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


[Bug 243576] New: Review Request: gst-inspector - An introspection data viewer for the GStreamer multimedia framework

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




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

   Summary: Review Request: gst-inspector - An introspection data
viewer for the GStreamer multimedia framework
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/gst-inspector-0.1-1.fc6.spec
SRPM URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/gst-inspector-0.1-1.fc6.src.rpm
Description:
GStreamer Inspector is an introspection data viewer for the GStreamer
multimedia framework. Developers of GStreamer applications can use it
to find out about the various elements available in the system.

It can be thought of as a graphical variant of the gst-inspect
program, which currently serves as the canonical source of
introspection information for many developers.

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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

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

Summary: Review Request: glump - A small web application to glue files from 
multiple sources


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





--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 22:14 EST ---
Crap, I was typing this up but had to break for dinner before I could submit 
it.  Here's what I wrote:

rpmlint output is OK:

W: glump mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 13)
  Not a blocker; fix it if you wish.

E: glump explicit-lib-dependency libxml2-python
  This is a false positive.  The package is necessary, and the dependency 
  generator doesn't find it automatically.

Marek is right; you should clean the buildroot at the beginning of install.  
Usually rpmlint complains about that; I'm not sure why it didn't.

* source files match upstream:
   634325e4efb057c5b849bf831a51ce8e7e11cec572b6769e9645f06443ae51ab  
   glump-0.9.11.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* rpmlint has only acceptable complaints.
* final provides and requires are sane:
   config(glump) = 0.9.11-2.fc8
   glump = 0.9.11-2.fc8
  =
   /usr/bin/python
   config(glump) = 0.9.11-2.fc8
   httpd
   libxml2-python
   mod_python
* %check is not present; no test suite upstream.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

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

Summary: Review Request: glump - A small web application to glue files from 
multiple sources


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-06-09 22:17 EST ---
And now looking at the -3 package, I see you fixed everything I had an issue
with, so we're ready to go.

APPROVED

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

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


  1   2   >