[Bug 615575] Review Request: perl-Parse-Method-Signatures - Perl6 like method signature parser

[bugzilla]

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2010-07-19 02:57:04 EDT 
---
Review:
+ package builds in mock (rawhide i686).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2325443
+ rpmlint output for SRPM and for RPM.
perl-Parse-Method-Signatures.src: W: no-cleaning-of-buildroot %clean
perl-Parse-Method-Signatures.src: W: no-buildroot-tag
perl-Parse-Method-Signatures.src: W: no-%clean-section
==> this is due to new guidelines. 
+ source files match upstream url (sha1sum)
1df162eca8fd1b99497f7677b714116ebfb8305c 
Parse-Method-Signatures-1.003013.tar.gz
1df162eca8fd1b99497f7677b714116ebfb8305c 
Parse-Method-Signatures-1.003013.tar.gz.srpm

+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present which is not needed now.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=3, Tests=344,  2 wallclock secs ( 0.07 usr  0.00 sys +  2.32 cusr  0.08
csys =  2.47 CPU)

+ Package  perl-Parse-Method-Signatures-1.003013-1.fc14.noarch =>
Provides: perl(Parse::Method::Signatures) = 1.003013
perl(Parse::Method::Signatures::Param)
perl(Parse::Method::Signatures::Param::Bindable)
perl(Parse::Method::Signatures::ParamCollection)
perl(Parse::Method::Signatures::Param::Named)
perl(Parse::Method::Signatures::Param::Placeholder)
perl(Parse::Method::Signatures::Param::Positional)
perl(Parse::Method::Signatures::Param::Unpacked)
perl(Parse::Method::Signatures::Param::Unpacked::Array)
perl(Parse::Method::Signatures::Param::Unpacked::Hash)
perl(Parse::Method::Signatures::Sig)
perl(Parse::Method::Signatures::TypeConstraint)
perl(Parse::Method::Signatures::Types)

Requires: perl(base) perl(Carp) perl(List::MoreUtils) perl(Moose)
perl(Moose::Role) perl(Moose::Util::TypeConstraints) perl(MooseX::Types)
perl(MooseX::Types::Moose) perl(MooseX::Types::Structured)
perl(MooseX::Types::Util) perl(namespace::clean)
perl(Parse::Method::Signatures::ParamCollection)
perl(Parse::Method::Signatures::Types) perl(PPI) perl(strict) perl(warnings)
rpmlib(VersionedDependencies) <= 3.0.3-1

+ Not a GUI application

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla]

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


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

--- Comment #22 from Hans de Goede  2010-07-19 02:55:51 
EDT ---
Hi Michael,

Almost there :)

Needs work:
-missing BuildRequires gengetopt
-you are not applying Patch3 anywhere
-Now the %{_datadir}/uvcdynctrl/data and the %{_datadir}/uvcdynctrl/data/046d
 directories are unowned. If you want to list each file under the dir
 explicitly, you need to also list all dirs, iow you need to use:
%dir %{_datadir}/uvcdynctrl
%dir %{_datadir}/uvcdynctrl/data
%dir %{_datadir}/uvcdynctrl/data/046d
%{_datadir}/uvcdynctrl/data/046d/logitech.xml
 but as said before instead you can also put just the following in %files:
%{_datadir}/uvcdynctrl
 note that this time there is no %dir in front of it! When you list a dir this
 way rpmbuild will make the rpm own the dir, and put all files and dirs
 below it in the rpm.

Regards,

Hans

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 614075] Review Request: libqttracker - Qt Bindings for the Tracker media indexing infrastructure

[bugzilla]

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


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

Chen Lei  changed:

   What|Removed |Added

 Depends on|613881  |

--- Comment #1 from Chen Lei  2010-07-19 02:44:33 EDT ---
New upstream release:
SPEC:http://dl.dropbox.com/u/1338197/1/libqttracker.spec
SRPM:http://dl.dropbox.com/u/1338197/1/libqttracker-6.9.8-1.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 613881] Review Request: contextkit - Contextual information collection framework

[bugzilla]

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


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

Chen Lei  changed:

   What|Removed |Added

 Blocks|614075  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225617] Merge Review: bitmap-fonts

[bugzilla]

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


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

Pravin Satpute  changed:

   What|Removed |Added

   Flag||needinfo?(nicolas.mail...@l
   ||aposte.net)

--- Comment #39 from Pravin Satpute  2010-07-19 02:41:34 
EDT ---
all blocking things are fixed
Nicholas can you check this once? may be we can close this now :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615849] Review Request: oflb-icelandic-fonts - Icelandic Magical Staves

[bugzilla]

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


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

Parag  changed:

   What|Removed |Added

 Blocks||477398

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615848] Review Request: oflb-brettfont-fonts - A handwriting font

[bugzilla]

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


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

Parag  changed:

   What|Removed |Added

 Blocks||477333

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615851] Review Request: oflb-sportrop-fonts - A multiline decorative font

[bugzilla]

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


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

Parag  changed:

   What|Removed |Added

 Blocks||477459

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615850] Review Request: oflb-roadstencil-fonts - Roadstencil Fonts

[bugzilla]

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


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

Parag  changed:

   What|Removed |Added

 Blocks||477449

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615847] Review Request: oflb-asana-math-fonts - An OpenType font with a MATH table

[bugzilla]

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


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

Parag  changed:

   What|Removed |Added

 Blocks||477331

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615851] New: Review Request: oflb-sportrop-fonts - A multiline decorative font

[bugzilla]

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

Summary: Review Request: oflb-sportrop-fonts  - A multiline decorative font

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

   Summary: Review Request: oflb-sportrop-fonts  - A multiline
decorative font
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pnem...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://paragn.fedorapeople.org/fedora-work/SPECS/oflb-sportrop-fonts.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/oflb-sportrop-fonts-0.9-8.fc13.src.rpm
Description:
A neat multiline font

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615849] New: Review Request: oflb-icelandic-fonts - Icelandic Magical Staves

[bugzilla]

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

Summary: Review Request: oflb-icelandic-fonts  - Icelandic Magical Staves

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

   Summary: Review Request: oflb-icelandic-fonts  - Icelandic
Magical Staves
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pnem...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://paragn.fedorapeople.org/fedora-work/SPECS/oflb-icelandic-fonts.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/oflb-icelandic-fonts-1.001-9.fc13.src.rpm
Description:
A font that includes most "magical" staves that have been "used" in Iceland.
Original drawings from the Museum of Sorcery & Witchcraft web site at
www.galdrasyning.is/.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615848] New: Review Request: oflb-brettfont-fonts - A handwriting font

[bugzilla]

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

Summary: Review Request: oflb-brettfont-fonts  - A handwriting font

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

   Summary: Review Request: oflb-brettfont-fonts  - A handwriting
font
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pnem...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://paragn.fedorapeople.org/fedora-work/SPECS/oflb-brettfont-fonts.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/oflb-brettfont-fonts-20080506-7.fc13.src.rpm
Description:
A handwriting font made by Brett Alton.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615847] New: Review Request: oflb-asana-math-fonts - An OpenType font with a MATH table

[bugzilla]

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

Summary: Review Request: oflb-asana-math-fonts - An OpenType font with a MATH 
table

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

   Summary: Review Request: oflb-asana-math-fonts - An OpenType
font with a MATH table
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pnem...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://paragn.fedorapeople.org/fedora-work/SPECS/oflb-asana-math-fonts.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/oflb-asana-math-fonts-0.914-8.fc13.src.rpm
Description:
An OpenType font with a MATH table that can be used with XeTeX to typeset math
content

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615850] New: Review Request: oflb-roadstencil-fonts - Roadstencil Fonts

[bugzilla]

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

Summary: Review Request: oflb-roadstencil-fonts  - Roadstencil Fonts

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

   Summary: Review Request: oflb-roadstencil-fonts  - Roadstencil
Fonts
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pnem...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://paragn.fedorapeople.org/fedora-work/SPECS/oflb-roadstencil-fonts.spec
SRPM URL:
http://paragn.fedorapeople.org/fedora-work/SRPMS/oflb-roadstencil-fonts-1.0-9.fc13.src.rpm
 
Description:
A rough font influenced by roadwork stencils

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 608421] Review Request: perl-Test-Without-Module - Test fallback behaviour in absence of modules

[bugzilla]

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


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

--- Comment #5 from Kevin Fenzi  2010-07-19 00:45:28 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 614967] Review Request: perl-Git-PurePerl - Pure Perl interface to Git repositories

[bugzilla]

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


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

--- Comment #3 from Kevin Fenzi  2010-07-19 00:45:53 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes

[bugzilla]

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


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

--- Comment #9 from Kevin Fenzi  2010-07-19 00:45:03 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 502686] Review Request: wsdlpull - C++ Web Services client library

[bugzilla]

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


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

--- Comment #23 from Kevin Fenzi  2010-07-19 00:44:07 EDT ---
CVS done (by process-cvs-requests.py).

We no longer do F11 branches (it's end of life)
We aren't yet doing F14 branches. 

Otherwise, all done. ;)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226094] Merge Review: libXxf86dga

[bugzilla]

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


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

--- Comment #2 from Peter Hutterer  2010-07-19 
00:46:28 EDT ---
(In reply to comment #1)
> Needswork:
> - https://fedoraproject.org/wiki/Packaging/Guidelines#Parallel_make
done
> - please use INSTALL="install -p", when installing
done

> - libXxf86dga.x86_64: W: name-repeated-in-summary C libXxf86dga
>   A slightly bigger description wouldn't hurt.

no, people may be tempted to use it then ;)

> - libXxf86dga.src:51: W: macro-in-comment %doc -> should be %%doc
removed

> - %doc: There is also a README

added.

> Not much to do, but not approving this now, because this is a merge review and
> not a normal package review...

out of interest: the package has been around for ages, what would the formal
approval really do?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 593851] Review Request: clide - Color and style highlighting program for text

[bugzilla]

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


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

--- Comment #16 from Kevin Fenzi  2010-07-19 00:44:23 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 488910] Review Request: bio2jack - A library for porting blocked io(OSS/ALSA) applications to jack

[bugzilla]

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


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

--- Comment #17 from Kevin Fenzi  2010-07-19 00:41:00 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 483451] Review Request: k3guitune - Musical instrument tuner

[bugzilla]

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


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

--- Comment #17 from Kevin Fenzi  2010-07-19 00:40:41 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 499087] Review Request: perl-Locale-PO - Perl module for manipulating .po entries from GNU gettext

[bugzilla]

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


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

--- Comment #10 from Kevin Fenzi  2010-07-19 00:41:35 EDT ---
CVS done (by process-cvs-requests.py).

Make sure and build the exact same version thats in RHEL.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 474980] Review Request: ovm - Open Verification Methodology : IEEE 1800 SystemVerilog standard

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

 CC||dw...@infradead.org
  Component|Package Review  |0x
   Flag|fedora-cvs? |fedora-cvs+

--- Comment #27 from Kevin Fenzi  2010-07-19 00:40:03 EDT ---
This is a Package Add request, not a change request. :) 

cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 247402] Review Request: GspiceUI - A GUI to freely available Spice Electronic circuit similators

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #50 from Kevin Fenzi  2010-07-19 00:35:51 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 243248] Review Request: avr-gdb - GDB for (remote) debugging avr binaries

[bugzilla]

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


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

--- Comment #10 from Kevin Fenzi  2010-07-19 00:34:34 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 223598] Review Request: liborigin - Library for reading OriginLab OPJ project files

[bugzilla]

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


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

--- Comment #8 from Kevin Fenzi  2010-07-19 00:34:10 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 451681] Review Request: sk2py - Migrates Cadence Skill based PCells to Python PyCells

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #10 from Kevin Fenzi  2010-07-19 00:30:13 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 246117] Review Request: drawtiming - A command line tool for generating timing diagrams

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #8 from Kevin Fenzi  2010-07-19 00:30:53 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 506174] Review Request: qtdmm - a digital multimeter readout software

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |

--- Comment #20 from Kevin Fenzi  2010-07-19 00:32:45 EDT ---
Have you asked 'zap' if he would like to maintain this for EL5/EL6?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 251190] Review Request: gds2pov - GDS2 layout file to POV-Ray conversion

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #9 from Kevin Fenzi  2010-07-19 00:29:38 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 219539] Review Request: qt-qsa - QT Script for Applications

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #16 from Kevin Fenzi  2010-07-19 00:29:04 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 208254] Review Request: pikloops - Code generator for PIC delays

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #24 from Kevin Fenzi  2010-07-19 00:26:09 EDT ---
cvs done

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 263121] Review Request: pharosc - VLSI and ASIC Technology Standard Cell Libraries

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #8 from Kevin Fenzi  2010-07-19 00:28:22 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 196120] Review Request: gresistor

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+

--- Comment #21 from Kevin Fenzi  2010-07-19 00:24:15 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 192475] Review Request: perl-PadWalker - Play with other peoples' lexical variables

[bugzilla]

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


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

--- Comment #4 from Kevin Fenzi  2010-07-19 00:23:19 EDT ---
CVS done (by process-cvs-requests.py).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 562585] Review Request: ccd2iso - CloneCD image to ISO image file converter

[bugzilla]

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


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

Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

   Flag||needinfo?(Mohammed_ElAfifi@
   ||yahoo.com)

--- Comment #11 from Orcan 'oget' Ogetbil  2010-07-19 
00:15:59 EDT ---
Mohammed, any progress? Are you still interested in this package?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla]

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


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

--- Comment #21 from Michael Cronenworth  2010-07-19 00:17:40 
EDT ---
Created an attachment (id=432759)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=432759)
disable udev log by default

I have a Logitech Quickcam 9000 Pro (first gen) and I also see the problem with
missing controls (Focus, etc) after first boot. I disabled logging and that
resolved it. The attached patch disables logging by default, unless the log
should truely be written to /dev/.udev? Anyone needing logging can enable it
post-install.

I've applied your patches and changed ownership of the uvcdynctrl data
directory.
spec: http://michael.cronenworth.com/RPMS/libwebcam.spec
srpm:
http://michael.cronenworth.com/RPMS/libwebcam-0.2.0-0.5.20100322svn.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 608419] Review Request: perl-CGI-Application-FastCGI - For using CGI::Application under FastCGI

[bugzilla]

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


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

Rakesh Pandit  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|iarn...@gmail.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 611010] Review Request: webkitgtk3 - GTK+ 3 port of webkit

[bugzilla]

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


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

Rakesh Pandit  changed:

   What|Removed |Added

Summary|webkitgtk3 - GTK+ 3 port of |Review Request: webkitgtk3
   |webkit  |- GTK+ 3 port of webkit

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 613879] Review Request: fillmore-lombard - audio and video editors for GNOME

[bugzilla]

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


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

Rakesh Pandit  changed:

   What|Removed |Added

Summary|fillmore-lombard - audio|Review Request:
   |and video editors for GNOME |fillmore-lombard - audio
   ||and video editors for GNOME

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 516539] Review Request: globus-gram-job-manager-setup-lsf - Globus Toolkit - LSF Job Manager Setup

[bugzilla]

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


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

Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||oget.fed...@gmail.com
 AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
   Flag||fedora-review+

--- Comment #4 from Orcan 'oget' Ogetbil  2010-07-18 
22:51:59 EDT ---
I made the full review. We have the same small issue as with the condor and
fork.
Otherwise looks good.

- rpmlint says:
   globus-gram-job-manager-setup-lsf.src: W: invalid-url Source0:
globus_gram_job_manager_setup_lsf-2.5.tar.gz
this can be ignored

*  %{perl_vendorlib}/Globus is already owned by globus-gram-job-manager-scripts
which is required by this package. You need to own
%{perl_vendorlib}/Globus/GRAM instead.



This package (globus-gram-job-manager-setup-lsf) is APPROVED by oget


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

[bugzilla]

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


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

--- Comment #41 from Mark Rader  2010-07-18 22:23:23 EDT ---
David

Per your comment, "This version of Frama-C is not the current version.  If it'd
be trivial to
upgrade it to the current version, I think the *current* version would be best.
 But if that's non-trivial, this version is a great place to start.  Would that
upgrade be trivial, or not?"  It is definitely non trivial.  Let me explain so
that we can document.  

I tried to perform a substitution for the package just as a sanity check.  It
error'ed on the strip of frama-c and frama-c-gui.  On checking these had not
built.  I thne tried to do a manual build and no jou.  They still did not
build.  I next tried to force a build and got the following confiuration error.

 checking for /usr/lib64/ocaml/dynlink.cmxa... yes
checking for lablgtk2's custom tree model... yes
configure: *
configure: * CHECKING FOR PLUG-IN DEPENDENCIES *
configure: *
configure: WARNING: ltl2ba not found.
configure: WARNING: aorai partially enabled because ltl2ba missing.
configure: WARNING: lablgtksourceview2.cmxa not found
configure: error: gui requested but
/usr/lib64/ocaml/lablgtk2/lablgtksourceview2.cmxa missing.


The missing file is part of OCAML.  Particularly it is part of LABLGTK, however
it is part of lablgtk2 which has not been packaged so we will need to request
it be packaged, package it ourselves or find some solution.  I dont know what
else may be missing but the newer version is looking at some effort.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 587087] Review Request: Spawning - A HTTP server for hosting WSGI python web applications

[bugzilla]

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


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

--- Comment #14 from Fedora Update System  
2010-07-18 21:33:25 EDT ---
Spawning-0.9.3-0.rc2.5.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/Spawning-0.9.3-0.rc2.5.el5

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 516538] Review Request: globus-gram-job-manager-setup-condor - Globus Toolkit - Condor Job Manager Setup

[bugzilla]

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


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

Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||oget.fed...@gmail.com
 AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
   Flag||fedora-review+

--- Comment #3 from Orcan 'oget' Ogetbil  2010-07-18 
21:01:18 EDT ---
I made a full review on the package. There are two minor issues that can be
fixed before committing. Otherwise, the package is good to go.

- rpmlint says
   globus-gram-job-manager-setup-condor.src: W: invalid-url Source0:
globus_gram_job_manager_setup_condor-4.4.tar.gz
which can be ignored.

? Can you explain in the spec file where Source1 comes from, or its upstream
status? Why do we need it instead of what comes with the default installation?

*  %{perl_vendorlib}/Globus is already owned by globus-gram-job-manager-scripts
which is required by this package. You need to own
%{perl_vendorlib}/Globus/GRAM instead.


---
This package (globus-gram-job-manager-setup-condor) is APPROVED by oget
---

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup

[bugzilla]

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


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

--- Comment #7 from Orcan 'oget' Ogetbil  2010-07-18 
20:54:11 EDT ---
Oh I found a minor issue. Please correct this before you commit:
! You don't need to own the directory %{perl_vendorlib}/Globus by this package.
That is already owned by globus-gram-job-manager-scripts which is required by
this package. Instead, you should own %{perl_vendorlib}/Globus/GRAM

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 554243] Review Request: moovida - Media Center

[bugzilla]

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


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

--- Comment #37 from Alex Lancaster  2010-07-18 
20:30:29 EDT ---
(In reply to comment #35)

> > Can you briefly outline the issues with Moovida 1.0?  Did they change the
> > license?  Can we cherry-pick stuff from their SVN/Git repo or whatever they 
> > use
> > even if they don't release proper tarballs?
> 
> Sorry Should have clarified. Moovida 1.0 is the same license and still has
> proper tarballs, nothing has changed, and thus there is nothing wrong with
> putting moovida 1.0 into Fedora.


Sorry, actually I misspoke (mistyped?) I meant what were the issues with
Moovida 2.0.  Since then I checked out the blogposts:

http://www.moovida.com/blog/2010/05/03/moovida-2-0s-stance-on-open-source-and-licensing/

and:

http://www.moovida.com/blog/2010/04/30/moovida-2-0-a-new-beginning/

and it does seem that 2.0 will (at least at first) have some proprietary
components and that they're moving away from Linux...  Can you clarify any
further about their long-range plans?

> > 
> > > On top of
> > > this, moovida 1.0 is no longer formerly supported by fluendo, so getting 
> > > futher
> > > updates or bugs fixed in moovida may be tricky.
> > 
> > If that's the case, perhaps we could consider moving moovida to RPM Fusion
> > (assuming licensing would be OK for RPM Fusion)?  It seems like a lot of 
> > work
> > to get 1.x into Fedora if it isn't going to be maintained upstream.
> 
> More than happy to move this into RPM Fusion if that's what people feel is the
> most appropriate place.

RPM Fusion would only be appropriate for Moovida 2.x, and even then it would
require source code if it went in to the free RPM Fusion repository and would
need to be at least distributable as a binary for the non-free RPM Fusion repo.
 It isn't clear how that would work at the moment.  So I think that getting 1.x
into Fedora seems the best course at the moment until it's clear how 2.x could
be distributed.

> > Aside: It was annoying that they changed the name from elisa for no good
> > reason, creating churn (like the need for this re-review in the first 
> > place),
> > now they are retooling again so soon?  What's fluendo's strategy here? (feel
> > free to e-mail me offline since this is a bit off-topic for the review 
> > request)
> 
> I don't profess to know things 100%, buy understanding is the name change and
> the complete rewrite for 2.0 is part of a strategy that was developed when
> Fluendo was acquired by another company (not 100% sure of this). Although
> www.fluendo.com maintains their linux focus, representatives at Moovida have
> come out and said that they are primarily focussing on the windows platform 
> for
> Moovida 2.0, and at the moment there is no source code available for either
> windows or linux platforms (just packages for Ubuntu).
> 
> On top of this, the rather confusing post at [1] (see comments) implies that
> their work on Moovida 2.0 won't have source code made available and it will be
> under a more closed license, meaning there is little chance of it getting into
> Fedora.
> 
> There seems to be enough general interest in Moovida 1.0 to get it packaged 
> for
> Fedora, and I am constantly on the lookout for a project looking to fork
> Moovida 1.0 and continue, but ultimately if the lack of a real solid upstream
> for fixes etc. is an issue we may have to abandon this package review or move
> it into RPM Fusion
> 
> [1] http://www.moovida.com/blog/2010/04/30/moovida-2-0-a-new-beginning/

Yep, I think go ahead with getting 1.x into Fedora.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225985] Merge Review: libaio

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org

--- Comment #6 from Thomas Spura  2010-07-18 
20:10:59 EDT ---
Ping, any progress here?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226813] Merge Review: zsh

[bugzilla]

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


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

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #17 from Kevin Fenzi  2010-07-18 20:07:06 EDT ---
Wow... here we are 3 years later. :) 

I went ahead and re-ran my review checklist against the current version: 

OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
74c5b275544400082a1cde806c98682a  zsh-4.3.10.tar.bz2
74c5b275544400082a1cde806c98682a  zsh-4.3.10.tar.bz2.orig
OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good. 
OK - Package has a correct %clean section. 
OK - Package has correct buildroot
OK - Package is code or permissible content. 
OK - Doc subpackage needed/used. 
OK - Packages %doc files don't affect runtime. 
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - Package compiles and builds on at least one arch. 
OK - Package has no duplicate files in %files. 
OK - Package doesn't own any directories other packages own. 
OK - Package owns all the directories it creates. 
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output. 
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock. 
OK - Should build on all supported archs
OK - Should function as described. 
OK - Should have sane scriptlets. 
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or
/usr/sbin
OK - check for outstanding bugs on package (merge reviews/rename/re-reviews).  

Issues: 

1. Non blocker, but would '--enable-cap' and '--enable-multibyte' be worth
enabling?

2. the tests do indeed hang in local mock here. How does this work in koji?

3. Non blocker, I tried to nuke the %makeinstall with no luck. 
Perhaps just add a note that it's required? 

4. rpmlint says: 

Can't this just be removed now: 

zsh.src:26: E: prereq-use fileutils grep /sbin/install-info
zsh.src: E: specfile-error warning: line 26: prereq is deprecated: Prereq:
fileutils grep /sbin/install-info

Misc ignorable issues, fix if you like: 

zsh.src:136: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 136)
zsh.x86_64: E: non-executable-script
/usr/share/zsh/4.3.10/functions/sticky-note 0644L /bin/zsh
zsh.x86_64: E: script-without-shebang /usr/share/zsh/4.3.10/functions/mere
zsh.x86_64: E: non-executable-script
/usr/share/zsh/4.3.10/functions/calendar_add 0644L /bin/zsh
zsh.x86_64: W: file-not-utf8 /usr/share/doc/zsh-4.3.10/LICENCE
zsh.x86_64: W: dangerous-command-in-%postun cp
zsh.x86_64: E: use-tmp-in-%postun

I don't see any blockers here, just suggestions, so this package is APPROVED. 

Please do consider the above suggestions for the package moving forward. ;) 

You can close this at your leasure...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225949] Merge Review: joystick

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org

--- Comment #1 from Thomas Spura  2010-07-18 
20:07:40 EDT ---
Hmm, is this package still needed?

According to the upstream website:
"The input drivers already support all the joysticks and gamepads the 1.2.15
joystick driver does. [snip]"

That sounds to me, that this is obsolete - isn't it?

I'll review it, when it's still needed - just give me a ping.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225898] Merge Review: imake

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org

--- Comment #3 from Thomas Spura  2010-07-18 
20:03:10 EDT ---
(In reply to comment #2)
> I agree.

Me too.

> No progress on this?

Ping.

- %{_datadir}/X11 is owned by filesystem and shouldn't be owned by this
package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615812] Review Request: geronimo-interceptor-3.0-api - Java EE: Interceptor API v3.0

[bugzilla]

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


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

--- Comment #1 from chris.sp...@arcor.de 2010-07-18 20:01:29 EDT ---
Actually, the last koji scratch build was successful. Seems
maven-remote-resources-plugin 1.1-6 is now available there.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615812] New: Review Request: geronimo-interceptor-3.0-api - Java EE: Interceptor API v3.0

[bugzilla]

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

Summary: Review Request: geronimo-interceptor-3.0-api - Java EE: Interceptor 
API v3.0

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

   Summary: Review Request: geronimo-interceptor-3.0-api - Java
EE: Interceptor API v3.0
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: chris.sp...@arcor.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://spike.fedorapeople.org/geronimo-interceptor-3.0-api/geronimo-interceptor-3.0-api.spec
SRPM URL:
http://spike.fedorapeople.org/geronimo-interceptor-3.0-api/geronimo-interceptor-3.0-api-1.0.1-1.fc14.src.rpm
Description: Contains annotations and interfaces for defining interceptor
methods, interceptor classes and for binding interceptor classes to target
classes.

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

Note: koji scratch build fails due to an old version of
maven-remote-resources-plugin. This should be fixed in rawhide (->
https://bugzilla.redhat.com/show_bug.cgi?id=613582). A mock build against
rawhide works flawlessly.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615809] New: Review Request: geronimo-annotation-1.0-api - Java EE: Annotation API v1.0

[bugzilla]

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

Summary: Review Request: geronimo-annotation-1.0-api - Java EE: Annotation API 
v1.0

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

   Summary: Review Request: geronimo-annotation-1.0-api - Java EE:
Annotation API v1.0
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: chris.sp...@arcor.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://spike.fedorapeople.org/geronimo-annotation-1.0-api/geronimo-annotation-1.0-api.spec
SRPM URL:
http://spike.fedorapeople.org/geronimo-annotation-1.0-api/geronimo-annotation-1.0-api-1.1.1-1.fc14.src.rpm
Description: This package defines the common annotations.

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226094] Merge Review: libXxf86dga

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||a...@redhat.com,
   ||peter.hutte...@redhat.com,
   ||toms...@fedoraproject.org
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org
   Flag||fedora-review?

--- Comment #1 from Thomas Spura  2010-07-18 
19:30:51 EDT ---
Review:
(CC'ing the people, who modified this recently.)

Good:
- name ok
- BR ok (pkgconfig is unneeded, but that doesn't matter here)
- %prep ok
- rpmlint:
libXxf86dga.src: W: invalid-url Source0:
ftp://ftp.x.org/pub/individual/lib/libXxf86dga-1.1.1.tar.bz2 
libXxf86dga.x86_64: W: spelling-error Summary(en_US) runtime -> run time,
run-time, runtish
- %files ok
- license ok
- latest version packaged
- libs correctly packaged
- no locale files
- no *.la


Needswork:
- https://fedoraproject.org/wiki/Packaging/Guidelines#Parallel_make
- please use INSTALL="install -p", when installing
- libXxf86dga.x86_64: W: name-repeated-in-summary C libXxf86dga
  A slightly bigger description wouldn't hurt.
- libXxf86dga.src:51: W: macro-in-comment %doc -> should be %%doc
- %doc: There is also a README

___


Not much to do, but not approving this now, because this is a merge review and
not a normal package review...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226040] Merge Review: librsvg2

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||at...@redhat.com,
   ||bnoc...@redhat.com,
   ||mcla...@redhat.com,
   ||toms...@fedoraproject.org

--- Comment #2 from Thomas Spura  2010-07-18 
19:16:02 EDT ---
This package is orphaned in all branches, but still maintained  by the
co-maintainers.

So who stepps in to maintain this?
(CC'ing all people, who recently modified this package.)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 554243] Review Request: moovida - Media Center

[bugzilla]

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


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

--- Comment #36 from Alex G.  2010-07-18 19:11:11 EDT ---
Sorry for just jumping in, but IMHO, Moovida 1.0 should be put into Fedora, at
least until a proper media center program finds its way into the repositories.

At the moment, the only media center program that I'm aware of is XBMC (MythTV
is rather difficult to set up), and that's in RPM Fusion.

I wouldn't mind seeing moovida in F13, and even F14. Until then, there will be
time to get a replacement (maybe Enna) in.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225645] Merge Review: cleanfeed

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||odvor...@redhat.com,
   ||rra...@redhat.com,
   ||toms...@fedoraproject.org

--- Comment #2 from Thomas Spura  2010-07-18 
19:01:55 EDT ---
(Adding all current maintainers)

Any progress here?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225946] Merge Review: jgroups

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||fnas...@redhat.com,
   ||toms...@fedoraproject.org
   Flag||needinfo?(fnas...@redhat.co
   ||m)

--- Comment #1 from Thomas Spura  2010-07-18 
18:47:41 EDT ---
This package is highly outdated (5 years old) and there are many versions in
between now...

Is this software still in use?

'repoquery --whatrequires jgroups' is empty...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 554243] Review Request: moovida - Media Center

[bugzilla]

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


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

--- Comment #35 from Graeme Gillies  2010-07-18 18:39:16 
EDT ---
(In reply to comment #33)
> (In reply to comment #32)
> > Hi,
> > 
> > I apologise for not replying to this ticket sooner. 
> 
> Hi Graeme,
> 
> Glad to hear you're still interested, sometimes closing a bug will elicit a
> response... :)
> 
> > I was keeping an eye on it,
> > but I was holding off doing any more work on packaging moovida as moovida 
> > 2.0
> > had been anounced and the linux port released to limited testing, and I 
> > wanted
> > to see the feasibility of packaging that for fedora instead. Unfortunately 
> > due
> > to licensing and fluendo's resistance to releasing pure tarballs, it looks 
> > like
> > moovida 2.0 probably won't be making it into fedora anytime soon. 
> 
> Can you briefly outline the issues with Moovida 1.0?  Did they change the
> license?  Can we cherry-pick stuff from their SVN/Git repo or whatever they 
> use
> even if they don't release proper tarballs?

Sorry Should have clarified. Moovida 1.0 is the same license and still has
proper tarballs, nothing has changed, and thus there is nothing wrong with
putting moovida 1.0 into Fedora.

> 
> > On top of
> > this, moovida 1.0 is no longer formerly supported by fluendo, so getting 
> > futher
> > updates or bugs fixed in moovida may be tricky.
> 
> If that's the case, perhaps we could consider moving moovida to RPM Fusion
> (assuming licensing would be OK for RPM Fusion)?  It seems like a lot of work
> to get 1.x into Fedora if it isn't going to be maintained upstream.

More than happy to move this into RPM Fusion if that's what people feel is the
most appropriate place.

> 
> > I was also away for the last 3 weeks on holiday which is why I didn't reply
> > straight away.
> > 
> > Anyway I have had a look over the spec files and the comments above and have
> > fixed the outstanding problems raised. I have created new rpms for both
> > moovida, and the good and bad plugins packages (see the other bugs for the
> > other packages).
> > 
> > moovida.spec
> > http://ggillies.fedorapeople.org/moovida.spec
> > 
> > moovida-1.0.9-3.fc13.src.rpm
> > http://ggillies.fedorapeople.org/moovida-1.0.9-3.fc13.src.rpm
> > 
> > I'm still keen to get 1.0 into fedora, and I am re-opening this review in 
> > the
> > hopes of taking up Hans de Goede's offer of review and sponsorship
> 
> I guess there's no harm in getting into Fedora, however we may have to think
> about the long-term if we end up moving it to RPM Fusion later.
> 
> Aside: It was annoying that they changed the name from elisa for no good
> reason, creating churn (like the need for this re-review in the first place),
> now they are retooling again so soon?  What's fluendo's strategy here? (feel
> free to e-mail me offline since this is a bit off-topic for the review 
> request)

I don't profess to know things 100%, buy understanding is the name change and
the complete rewrite for 2.0 is part of a strategy that was developed when
Fluendo was acquired by another company (not 100% sure of this). Although
www.fluendo.com maintains their linux focus, representatives at Moovida have
come out and said that they are primarily focussing on the windows platform for
Moovida 2.0, and at the moment there is no source code available for either
windows or linux platforms (just packages for Ubuntu).

On top of this, the rather confusing post at [1] (see comments) implies that
their work on Moovida 2.0 won't have source code made available and it will be
under a more closed license, meaning there is little chance of it getting into
Fedora.

There seems to be enough general interest in Moovida 1.0 to get it packaged for
Fedora, and I am constantly on the lookout for a project looking to fork
Moovida 1.0 and continue, but ultimately if the lack of a real solid upstream
for fixes etc. is an issue we may have to abandon this package review or move
it into RPM Fusion

[1] http://www.moovida.com/blog/2010/04/30/moovida-2-0-a-new-beginning/

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 456353] Review Request: libffado - Free firewire audio driver library

[bugzilla]

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


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

--- Comment #48 from Orcan 'oget' Ogetbil  2010-07-18 
18:38:07 EDT ---
Hi Jeff,
No worries, bodhi will close the ticket automatically when it is pushed to
stable. Please read the line numbered with "10" in the link you gave.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 593851] Review Request: clide - Color and style highlighting program for text

[bugzilla]

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


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

Richard Fearn  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #15 from Richard Fearn  2010-07-18 18:30:31 
EDT ---
Package Change Request
==
Package Name: clide
New Branches: EL-5 EL-6
Owners: richardfearn

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 456353] Review Request: libffado - Free firewire audio driver library

[bugzilla]

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


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

--- Comment #47 from Jeff Garzik  2010-07-18 18:33:37 EDT 
---
Per review process, now that packages are built, I think you (as Contributor)
can set this bug to CLOSED - NEXTRELEASE:
https://fedoraproject.org/wiki/PackageReviewProcess#Contributor

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes

[bugzilla]

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


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

Richard Fearn  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Richard Fearn  2010-07-18 18:28:55 
EDT ---
Thank you Thomas!

>From http://en.wikipedia.org/wiki/Regular_expression:

"In computing, regular expressions, also referred to as regex or regexp..."

So I think regex/regexes is OK :-)



New Package CVS Request
===
Package Name: txt2regex
Short Description: Regular expression wizard that converts human sentences to
regexes
Owners: richardfearn
Branches: F-12 F-13 EL-5 EL-6
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225666] Merge Review: crypto-utils

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org
   Flag||needinfo?

--- Comment #4 from Thomas Spura  2010-07-18 
18:32:35 EDT ---
There is no upstream url, so where is this package located?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225663] Merge Review: cryptix-asn1

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||toms...@fedoraproject.org
 Resolution||WONTFIX
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org

--- Comment #1 from Thomas Spura  2010-07-18 
18:27:33 EDT ---
This package was retired on:
Tue Aug 11 20:11:58 2009 UTC (11 months ago) by jkeating 

See:
http://cvs.fedoraproject.org/viewvc/devel/cryptix-asn1/dead.package?view=log

-> WONTFIX

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 502686] Review Request: wsdlpull - C++ Web Services client library

[bugzilla]

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


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

--- Comment #22 from Denis Arnaud  2010-07-18 
18:24:42 EDT ---
(In reply to comment #16)
> The only thing remaining is to submit the patches upstream (to Vivek).

All the patches have been committed to the WSDLPull official repository, and
integrated in the new release, namely 1.24. That latter has to be approved by
the project owner, namely Vivek Krishna. Once it will be done, I will of course
update the Fedora packaging CVS repository with that last version.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 537325] Review Request: lv2-fil-plugins - Four-band parametric equalisers

[bugzilla]

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


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

--- Comment #19 from Orcan 'oget' Ogetbil  2010-07-18 
18:20:26 EDT ---
Please read Joerg Schilling's response in the above mailing list thread.

According to his explanation, lv2fil can be classified as a "collective work"
since the codes of various author are brought together into a single product.
The source files are not written by a single author.

Now we know that, 
* The source files are:
  filter.*: GPLv2+ by Nedko Arnaudov. But the DSP code is based on 
ladspa:1970 by Fons Adriaensen.
  log.*   : GPLv2 by Nedko Arnaudov.
  lv2_external_ui.h : Public Domain
  lv2_ui.h: LGPLv2+ by various authors
  lv2_ui.c: GPLv2+ by Nedko Arnaudov.
  lv2filter.* : GPLv2 by Nedko Arnaudov
  lv2plugin   : GPLv2 by Nedko Arnaudov

* In an email, Nedko Arnaudov states that he wanted his code to be GPLv2 only.

So from here, we can say that Nedko Arnaudov's part of the code is GPLv2. We
have 2 possibilities:

1- If Fons Adriaensen's part of the code in filter.* is GPLv2 too, we can say
that the collective work of Nedko Arnaudov, Fons Adriaensen and all the authors
of lv2_external_ui.h is GPLv2 and LGPLv2+.

2- However, if Fons Adriaensen's part of the filter.* code is GPLv2+, then the
collective work has license LGPLv2+ and GPLv2 and GPLv2+.

We find Fons' code in our ladspa-fil-plugins package in file filters.cc. This
file clearly states that the license is GPLv2+.

Hence the collective work is  LGPLv2+ and GPLv2 and GPLv2+.

I think I interpreted it correctly this time.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 225664] Merge Review: cryptix

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||toms...@fedoraproject.org
 Resolution||WONTFIX
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org

--- Comment #1 from Thomas Spura  2010-07-18 
18:17:04 EDT ---
This package was retired on:
Tue Aug 11 20:11:46 2009 UTC (11 months ago) by jkeating 

See:
http://cvs.fedoraproject.org/viewvc/devel/cryptix/dead.package?view=log

-> WONTFIX

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

[bugzilla]

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


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

David A. Wheeler  changed:

   What|Removed |Added

   Flag||fedora-review+

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes

[bugzilla]

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


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

Thomas Spura  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Thomas Spura  2010-07-18 
18:01:16 EDT ---
Alright then:

$ rpmlint ./txt2regex-0.8-3.fc13.src.rpm noarch/txt2regex-0.8-3.fc13.noarch.rpm 
txt2regex.src: W: spelling-error Summary(en_US) regexes -> regexps, reg exes,
reg-exes
txt2regex.src: W: spelling-error %description -l en_US regexes -> regexps, reg
exes, reg-exes
txt2regex.src: W: spelling-error %description -l en_US regex -> regexp, remex,
reg ex
txt2regex.noarch: W: spelling-error Summary(en_US) regexes -> regexps, reg
exes, reg-exes
txt2regex.noarch: W: spelling-error %description -l en_US regexes -> regexps,
reg exes, reg-exes
txt2regex.noarch: W: spelling-error %description -l en_US regex -> regexp,
remex, reg ex
2 packages and 0 specfiles checked; 0 errors, 6 warnings.

I'm no naitive speaker, so don't know if it's regexes or regexps. I guess both
words don't exist in standard English^^

-> ignorable

__

APPROVED, great to have it in fedora now

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

[bugzilla]

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


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

--- Comment #40 from David A. Wheeler  2010-07-18 
17:57:49 EDT ---
Okay, I understand that switching to the current version of Frama-C would be a
big deal, due to dependencies on other packages.  So, let's go with the version
of Frama-C that's packaged here.

All of the issues I identified earlier have been resolved, and I don't see any
new issues to deal with.  So, thanks for your hard work!

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 456353] Review Request: libffado - Free firewire audio driver library

[bugzilla]

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


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

--- Comment #45 from Fedora Update System  
2010-07-18 17:31:07 EDT ---
libffado-2.0.1-3.20100706.svn1864.fc13 has been submitted as an update for
Fedora 13.
http://admin.fedoraproject.org/updates/libffado-2.0.1-3.20100706.svn1864.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 456353] Review Request: libffado - Free firewire audio driver library

[bugzilla]

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


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

--- Comment #46 from Fedora Update System  
2010-07-18 17:31:54 EDT ---
libffado-2.0.1-3.20100706.svn1864.fc12 has been submitted as an update for
Fedora 12.
http://admin.fedoraproject.org/updates/libffado-2.0.1-3.20100706.svn1864.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 600914] Review Request: txt2regex - Regular expression wizard that converts human sentences to regexes

[bugzilla]

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


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

--- Comment #6 from Richard Fearn  2010-07-18 16:41:35 
EDT ---
I heard back from Aurélio, the author of txt2regex. He said:

"Please include the email, I don't intend to release a new version of txt2regex
soon."

So I have created 0.8-3, which includes the email I received from him on
2010-06-08 confirming that txt2regex is GPLv2 only. New spec/SRPM are here:

http://richardfearn.fedorapeople.org/txt2regex/0.8-3/txt2regex.spec
http://richardfearn.fedorapeople.org/txt2regex/0.8-3/txt2regex-0.8-3.fc13.src.rpm

The only rpmlint warnings are spelling errors, and they are all false
positives.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 593800] Review Request: python-keyring - keyring module for python

[bugzilla]

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


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

Mamoru Tasaka  changed:

   What|Removed |Added

 CC||mtas...@ioa.s.u-tokyo.ac.jp

--- Comment #18 from Mamoru Tasaka  2010-07-18 
15:37:21 EDT ---
Some notes:

* Macro definition
  - For defining %python_sitearch macro and so on, please follow
below:
https://fedoraproject.org/wiki/Packaging/Python#Macros

* Filtering Provides
  - If you want to filter some Provides generated by rpmbuild,
you can now use "%filter_provides_in" macro as defined in
/usr/lib/rpm/redhat/macro (in redhat-rpm-config rpm):
   
https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering#Preventing_files.2Fdirectories_from_being_scanned_for_provides_.28pre-scan_filtering.29

! Note that current way of yours does not work as you expected, see:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=2327705
  http://koji.fedoraproject.org/koji/getfile?taskID=2327705&name=build.log

* Release number
  - Please use %{?dist} for Release number (expect for some
exceptions)
(2%{?dist}, for example).

* BuildRoot
  - BuildRoot tag is now no longer used on Fedora so please remove
this.
! Note:
  - If you want to import this package also into EPEL, please follow:
   
https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#Distribution_specific_guidelines

* python-devel BuildRequires:
  - Now please explicitly specify python2-devel or python3-devel for
(Build)Requires:
https://fedoraproject.org/wiki/Packaging/Python#BuildRequires

* KDE BuildRequires
  - This packge uses Qt4, so only KDE4 should be needed.
Please write "BR: kdelibs4-devel" only and remove "BR: kdelibs3-devel".
! Note
  - Unfortunately, it seems that the needed "libkdeui.so" (for -lkdeui) is
under %_libdir/kde4/devel (in kdelibs-devel), so you have to
add "export LDFLAGS=-L%{_libdir}/kde/devel" or something else
before "python setup.py build".

* %defattr
  - We usually use %defattr(-,root,root,-) for default %defattr

* Directory ownership issue
  - The directory %{python_sitelib}/%{upstream_name}/ itself is not owned
by any packages.

* python_sitelib v.s. python_sitearch
-
24  # Actually the main package is noarch but -gnome and -kwallet contain
-
  - Well, however on x86_64 files in even main package are also installed
under %python_sitearch:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2327573

* rpmlint issue
-
python-keyring.src: W: strange-permission keyring-0.2.tar.gz 0777L
python-keyring.src: W: strange-permission python-keyring.spec 0777L
-
  - Please change the permission of the files in srpm to 0644.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 583327] Review Request: clementine - A music player and library organiser

[bugzilla]

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


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

--- Comment #7 from Orcan 'oget' Ogetbil  2010-07-18 
15:18:11 EDT ---
SPEC: http://oget.fedorapeople.org/review/clementine.spec
SRPM: http://oget.fedorapeople.org/review/clementine-0.4.2-3.fc13.src.rpm

Better, more portable patch for splitting qxt. Patch sent upstream.

Since all the dependencies are built and are in testing now, this package is
ready to review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615783] New: Review Request: liblouisxml - Translate XML files into proper Braille

[bugzilla]

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

Summary: Review Request: liblouisxml - Translate XML files into proper Braille

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

   Summary: Review Request: liblouisxml - Translate XML files into
proper Braille
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lbj...@broadpark.no
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: 
SRPM URL: 

Description: liblouisxml is a software component which can be
incorporated into software packages to provide the capability of
translating any file in the computer lingua franca XML format into
properly transcribed braille. This includes translation into grade
two, if desired, mathematical codes, etc. It also includes formatting
according to a built-in style sheet which can be modified by the user.
The first program into which liblouisxml has been incorporated is
xml2brl. This program will translate an XML or text file into an
embosser-ready braille file. It is not necessary to know XML, because
MSWord and other word processors can export files in this format. If
the word processor has been used correctly xml2brl will produce an
excellent braille file.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 350341] Review Request: postfix-logwatch - A postfix log analyzer for logwatch

[bugzilla]

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


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

--- Comment #23 from Richard Fearn  2010-07-18 14:20:55 
EDT ---
I will get to it. Just haven't had much time lately to work on it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 350341] Review Request: postfix-logwatch - A postfix log analyzer for logwatch

[bugzilla]

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


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

--- Comment #22 from Kevin Fenzi  2010-07-18 14:15:08 EDT ---
Richard: were you going to submit a new updated package here for review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 474980] Review Request: ovm - Open Verification Methodology : IEEE 1800 SystemVerilog standard

[bugzilla]

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


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

--- Comment #26 from Kevin Fenzi  2010-07-18 14:11:55 EDT ---
Great news Chitlish. :) 

I don't see any reason why we can't include this now.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 588941] Review Request: pypy - Implementation of the Python language, using Python itself

[bugzilla]

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


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

--- Comment #8 from Kevin Fenzi  2010-07-18 14:14:03 EDT ---
Dave: Any news here? (No hurry, just wondering... ;)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

[bugzilla]

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


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

--- Comment #39 from David A. Wheeler  2010-07-18 
13:34:02 EDT ---
EXCELLENT.   This is looking awesome.

The SELinux fix works perfectly; at least for trivial uses, SELinux is now
happy.  The new comment seems clear enough, too.

I should go back and see if the changes have messed up anything, or if there
are big problems I didn't notice before, but so far it's looking great.

This version of Frama-C is not the current version.  If it'd be trivial to
upgrade it to the current version, I think the *current* version would be best.
 But if that's non-trivial, this version is a great place to start.  Would that
upgrade be trivial, or not?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup

[bugzilla]

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


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

Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Orcan 'oget' Ogetbil  2010-07-18 
12:44:23 EDT ---
Oh, nevermind, 
   /usr/share/globus/packages/setup/globus_gram_job_manager_service_setup/
is owned by globus-gram-job-manager-10.59-2, and 
   /usr/share/globus/packages/setup/
is owned by globus-common.

All good. The package also built fine in koji:
   http://koji.fedoraproject.org/koji/taskinfo?taskID=2327547


-
This package (globus-gram-job-manager-setup-fork) is APPROVED by oget
-

Is this the longest package name in Fedora?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup

[bugzilla]

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


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

--- Comment #5 from Orcan 'oget' Ogetbil  2010-07-18 
12:37:05 EDT ---
There are a few issues:
First, globus-gram-job-manager-10.59-2 doesn't seem to own the directory
   /usr/share/globus/packages/setup/
See Files section in
   http://koji.fedoraproject.org/koji/rpminfo?rpmID=2075534
Did you mean some other package?

Second, this package must own the directory
   /usr/share/globus/packages/setup/globus_gram_job_manager_service_setup/
but it doesn't.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615319] Review Request: cgnslib - Computational Fluid Dynamics General Notation System

[bugzilla]

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


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

Shakthi Kannan  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Shakthi Kannan  2010-07-18 12:05:31 
EDT ---
New Package CVS Request
===
Package Name: cgnslib
Short Description: Computational Fluid Dynamics General Notation System
Owners: shakthimaan chitlesh
Branches: F-13 F-14 EL-6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 612384] Review Request: teal - Verification utility and connection library

[bugzilla]

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


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

--- Comment #6 from Shakthi Kannan  2010-07-18 11:54:50 
EDT ---
I have removed the -devel package, and moved all contents to the base package,
as this package itself is a development tool.

SPEC: http://shakthimaan.fedorapeople.org/SPECS/teal.spec
SRPM: http://shakthimaan.fedorapeople.org/SRPMS/teal-1_40b-3.fc14.src.rpm

Koji builds are fine.

$ rpmlint teal.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint teal-1_40b-3.fc14.i686.rpm 
teal.i686: W: spelling-error %description -l en_US multithreaded ->
multitudinous, multitude, multicolored
teal.i686: W: spelling-error %description -l en_US verilog -> verily, Verizon,
veritably
teal.i686: W: devel-file-in-non-devel-package /usr/lib/libteal.ivl.so
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_vmemory.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_vout.h
teal.i686: W: devel-file-in-non-devel-package
/usr/include/teal/teal_dictionary.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_reg.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_vthreads.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_vrandom.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal.h
teal.i686: W: devel-file-in-non-devel-package /usr/include/teal/teal_vevents.h
1 packages and 0 specfiles checked; 0 errors, 11 warnings.

$ rpmlint teal-1_40b-3.fc14.src.rpm 
teal.src: W: spelling-error %description -l en_US multithreaded ->
multitudinous, multitude, multicolored
teal.src: W: spelling-error %description -l en_US verilog -> verily, Verizon,
veritably
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 555655] Review Request: sslstrip - tool that provides a demonstration of HTTPS stripping attacks

[bugzilla]

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


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

Arun SAG  changed:

   What|Removed |Added

 Blocks||563471(FE-SECLAB)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 563471] Tracker: Review Requests for Fedora Security Lab related packages

[bugzilla]

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


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

Arun SAG  changed:

   What|Removed |Added

 Depends on||555655

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592579] Review Request: Frama-c - Framework for source code analysis of C software

[bugzilla]

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


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

--- Comment #38 from Mark Rader  2010-07-18 11:47:28 EDT ---
Spec URL: http://tpath3.dnsalias.net/openproofs/frama-c.spec
SRPM URL: http://tpath3.dnsalias.net/openproofs/frama-c-1.4-5.fc12.src.rpm
Description:Frama-C is a suite of tools dedicated to the analysis of the
source code of software written in C.

I think I have corrected the SELinux error.  I have tested it on two systems
multiple times deleting the SELinux entry each time.


[ra...@tpath3 SPECS]$ rpmlint frama-c.spec ../SRPMS/frama-c-1.4-4.fc12.src.rpm
../RPMS/x86_64/frama-c-1.4-5.fc12.x86_64.rpm
../RPMS/x86_64/frama-c-devel-1.4-5.fc12.x86_64.rpm 
frama-c.src: W: invalid-license QPL with modifications
frama-c.x86_64: W: invalid-license QPL with modifications
frama-c-devel.x86_64: W: invalid-license QPL with modifications
frama-c-devel.x86_64: W: no-documentation
3 packages and 1 specfiles checked; 0 errors, 4 warnings.

[ra...@tpath3 SPECS]$ koji build --scratch dist-f14
../SRPMS/frama-c-1.4-5.fc12.src.rpm 
Uploading srpm: ../SRPMS/frama-c-1.4-5.fc12.src.rpm
[] 100% 00:01:32  18.84 MiB 207.62 KiB/sec
Created task: 2327526
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=2327526
None
Watching tasks (this may be safely interrupted)...
2327526 build (dist-f14, frama-c-1.4-5.fc12.src.rpm): free
2327526 build (dist-f14, frama-c-1.4-5.fc12.src.rpm): free -> open
(x86-12.phx2.fedoraproject.org)
  2327528 buildArch (frama-c-1.4-5.fc12.src.rpm, i686): free
  2327527 buildArch (frama-c-1.4-5.fc12.src.rpm, x86_64): free
  2327527 buildArch (frama-c-1.4-5.fc12.src.rpm, x86_64): free -> open
(x86-20.phx2.fedoraproject.org)
  2327528 buildArch (frama-c-1.4-5.fc12.src.rpm, i686): free -> open
(x86-18.phx2.fedoraproject.org)
  2327528 buildArch (frama-c-1.4-5.fc12.src.rpm, i686): open
(x86-18.phx2.fedoraproject.org) -> closed
  0 free  2 open  1 done  0 failed
  2327527 buildArch (frama-c-1.4-5.fc12.src.rpm, x86_64): open
(x86-20.phx2.fedoraproject.org) -> closed
  0 free  1 open  2 done  0 failed
2327526 build (dist-f14, frama-c-1.4-5.fc12.src.rpm): open
(x86-12.phx2.fedoraproject.org) -> closed
  0 free  0 open  3 done  0 failed

2327526 build (dist-f14, frama-c-1.4-5.fc12.src.rpm) completed successfully

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 592670] Review Request: mongoose - An easy-to-use self-sufficient web server

[bugzilla]

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


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

--- Comment #24 from Rafael Aquini  2010-07-18 11:13:56 EDT 
---
PING...

I'm still looking forward sponsorship. Besides this submission, in order to
show my understanding of Package Review Guidelines, I've also done some
informal reviews as follows:
https://bugzilla.redhat.com/show_bug.cgi?id=592770
https://bugzilla.redhat.com/show_bug.cgi?id=595011

Thanks in advance for your attention!

Best regards.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla]

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


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

--- Comment #19 from Hans de Goede  2010-07-18 09:49:39 
EDT ---
Created an attachment (id=432702)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=432702)
Patch adding support for dynamic menu controls

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla]

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


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

--- Comment #18 from Hans de Goede  2010-07-18 09:46:07 
EDT ---
Hi Michael,

I've added the upstream add menu support for dynamic control patches to the
Fedora kernel tree in pkg cvs for F-13 and F-14, so the next build for either
will pick these up.

I'll attach a patch for libwebcam (and an updated uvcvideo.h) which add support
for these. I've tested this both with an unpatched and with a patched kernel
and it works fine in either case.

I would very much like to see the patch be added to the Fedora packages,

Thanks,

Hans

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver

[bugzilla]

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


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

--- Comment #20 from Hans de Goede  2010-07-18 09:50:00 
EDT ---
Created an attachment (id=432703)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=432703)
updated uvcvideo.h

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615577] Review Request: -

[bugzilla]

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


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

Randall Berry  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #6 from Randall Berry  2010-07-18 09:28:53 
EDT ---
I am not a sponsor but you'll find all the steps to find one you need here:

http://fedoraproject.org/wiki/PackageMaintainers/Join
http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group
http://fedoraproject.org/wiki/Join_the_package_collection_maintainers

My review was incomplete and contained only a few items I noticed right away.
Your sponsor will have to do a full review in order to approve your package.

It would also be a good idea to read the documentation on the Fedora review
process. That documentation can be found in several places:

http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Package_Review_Process

Although you cannot review your own packages the review guidelines can help you
prepare your package for review. It will show you all the steps a reviewer will
take to review your package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615319] Review Request: cgnslib - Computational Fluid Dynamics General Notation System

[bugzilla]

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


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

--- Comment #5 from Ankur Sinha  2010-07-18 09:01:01 
EDT ---
hey!

I was reviewing this one, why take it like this? It would be nice for you to
ask/inform the current reviewer once. It's not like I'd say "no".

Ankur

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 615577] Review Request: -

[bugzilla]

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


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

--- Comment #5 from Chia-Pao Kuo  2010-07-18 08:58:44 EDT 
---
Thank you for reviewing my package. I've just fixed all the issues above and
updated srpm and spec file.

I think it is necessary for opencc-libs to cantain the locale files because the
API function "opencc_perror()" may be called to display the errors.

Actually it is my first package. But how can I find a so-called "sponser"?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 453422] Review Request: songbird - Mozilla based multimedia player

[bugzilla]

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


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

Hicham HAOUARI  changed:

   What|Removed |Added

 CC||hicham.haou...@gmail.com

--- Comment #141 from Hicham HAOUARI  2010-07-18 
08:07:09 EDT ---
Bundling its own runtime libs instead of system ones isn't a problem, since
other packages do it. And btw, songbird uses a heavily patched xulrunner.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 601577] Review Request: liblockfile - This library implements a number of functions found in -lmail on SysV systems

[bugzilla]

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


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

--- Comment #7 from Matthias Runge  2010-07-18 
07:58:04 EDT ---
Thank you for your review and sorry for the delay; I have been on vacation and
am currently scanning through my emails. I will correct those issues within the
next few days.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review