[Bug 436239] Review Request: joda-time - Java date and time API

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

Summary: Review Request: joda-time - Java date and time API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 02:11 EST ---
New URLs:
Spec URL: http://konradm.fedorapeople.org/fedora/SPECS/joda-time.spec
SRPM URL: 
http://konradm.fedorapeople.org/fedora/SRPMS/joda-time-1.5.2-3.tzdata2008a.fc8.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, or are watching someone who is.

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 02:29 EST ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=520520


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

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


[Bug 437923] New: Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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




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

   Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst -
Test::WWW::Mechanize for Catalyst
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Test-WWW-Mechanize-Catalyst/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SRPM URL: 
http://fedora.biggerontheinside.net/review/perl-Test-WWW-Mechanize-Catalyst-0.41-1.fc8.src.rpm
SPEC URL: 
http://fedora.biggerontheinside.net/review/perl-Test-WWW-Mechanize-Catalyst.spec

Description:
Catalyst is an elegant MVC Web Application Framework. Test::WWW::Mechanize
is a subclass of WWW::Mechanize that incorporates features for web
application testing. The Test::WWW::Mechanize::Catalyst module meshes the
two to allow easy testing of Catalyst applications without starting up a
web server.

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

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


[Bug 437918] Review Request: perl-Devel-LexAlias - Alias lexical variables

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

Summary: Review Request: perl-Devel-LexAlias - Alias lexical variables
Alias: perl-Devel-LexAlias

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 02:58 EST ---
Review:
+ package builds in mock.
Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=520466
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
7fe986f50b467fa8575a67f0729fbb1d  Devel-LexAlias-0.04.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=1, Tests=11,  0 wallclock secs ( 0.03 cusr +  0.00 csys =  0.03 CPU)
+ Package perl-Devel-LexAlias-0.04-1.fc9 =
  Provides: LexAlias.so perl(Devel::LexAlias) = 0.04
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) perl =
0:5.005003 perl(Devel::Caller) perl(DynaLoader) rtld(GNU_HASH)

APPROVED.

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

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


[Bug 437917] Review Request: perl-Array-RefElem - Set up array elements as aliases

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

Summary: Review Request: perl-Array-RefElem - Set up array elements as aliases
Alias: perl-Array-RefElem

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 02:57 EST ---
Review:
+ package builds in mock.
Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=520459
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
43ff2dd2049258634cb00697198572d1  Array-RefElem-1.00.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=1, Tests=5,  0 wallclock secs ( 0.00 cusr +  0.00 csys =  0.00 CPU)
+ Package perl-Array-RefElem-1.00-1.fc9 =
  Provides: RefElem.so perl(Array::RefElem) = 1.00
  Requires: libc.so.6 libc.so.6(GLIBC_2.1.3) perl(DynaLoader) perl(Exporter)
perl(strict) perl(vars) rtld(GNU_HASH)

APPROVED.

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

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


[Bug 437919] Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical goodness

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

Summary: Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical 
goodness


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   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, or are watching someone who is.

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


[Bug 437918] Review Request: perl-Devel-LexAlias - Alias lexical variables

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

Summary: Review Request: perl-Devel-LexAlias - Alias lexical variables
Alias: perl-Devel-LexAlias

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 03:05 EST ---
New Package CVS Request
===
Package Name: perl-Devel-LexAlias
Short Description: Alias lexical variables
Owners: cweyl
Branches: F-7, F-8, devel
InitialCC: perl-sig

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

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


[Bug 437917] Review Request: perl-Array-RefElem - Set up array elements as aliases

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

Summary: Review Request: perl-Array-RefElem - Set up array elements as aliases
Alias: perl-Array-RefElem

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 437917] Review Request: perl-Array-RefElem - Set up array elements as aliases

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

Summary: Review Request: perl-Array-RefElem - Set up array elements as aliases
Alias: perl-Array-RefElem

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 03:04 EST ---
New Package CVS Request
===
Package Name: perl-Array-RefElem
Short Description: Set up array elements as aliases
Owners: cweyl
Branches: F-7, F-8, devel
InitialCC: perl-sig

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

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


[Bug 437918] Review Request: perl-Devel-LexAlias - Alias lexical variables

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

Summary: Review Request: perl-Devel-LexAlias - Alias lexical variables
Alias: perl-Devel-LexAlias

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 03:09 EST ---
scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=520534

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

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


[Bug 437924] New: Review Request: perl-Template-Timer - Template::Timer Perl module

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




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

   Summary: Review Request: perl-Template-Timer - Template::Timer
Perl module
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Template-Timer/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SRPM URL: 
http://fedora.biggerontheinside.net/review/perl-Template-Timer-0.04-1.fc8.src.rpm
SPEC URL: http://fedora.biggerontheinside.net/review/perl-Template-Timer.spec

Description:
Template::Timer provides inline timings of the template processing
througout your code.  It's an overridden version of Template::Context that
wraps the process() and include() methods.

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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 04:29 EST ---
Review:
+ package builds in mock.
Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=520534
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
063b7d84c5cd501a2820f9c2e601eadc  Template-Timer-0.04.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=3, Tests=4,  0 wallclock secs ( 0.37 cusr +  0.02 csys =  0.39 CPU)
+ Package perl-Template-Timer-0.04-1.fc9 =
  Provides: perl(Template::Timer) = 0.04
  Requires: perl(Time::HiRes) perl(base) perl(strict) perl(warnings)

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, or are watching someone who is.

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 04:32 EST ---
Review:
+ package builds in mock.
Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=520520
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
efffcede181df9658c99472e51cade3f  Test-WWW-Mechanize-Catalyst-0.41.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=6, Tests=79,  2 wallclock secs ( 2.06 cusr +  0.29 csys =  2.35 CPU)
+ Package perl-Test-WWW-Mechanize-Catalyst-0.41-1.fc9 =
  Provides: perl(Test::WWW::Mechanize::Catalyst) = 0.41
perl(Test::WWW::Mechanize::Catalyst::Aux)
  Requires: perl(Catalyst) perl(Encode) perl(HTML::Entities)
perl(Test::WWW::Mechanize) perl(base) perl(strict) perl(warnings)

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, or are watching someone who is.

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


[Bug 433547] Review Request: nagios-nsca - nagios passive check daemon

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

Summary: Review Request: nagios-nsca - nagios passive check daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 04:53 EST ---
(In reply to comment #10)
 You depending on the nagios package for both the client and server. Only
 server needs nagios itself installed. 

Only nsca (the daemon) Requires: nagios. nsca-client doesn't.
 
 What about the nagios user? Since your compiling NSCA requiring the nagios
 using being present you'll need the user. This package should attempt to
 create the user and otherwise silently ignore if it exists already.

The nagios user is not required at compile time, the --with-nsca-user and
--with-nsca-grp option are only used to generate the default config. The nagios
user will only be needed at run time and then nsca Requires: nagios so it will
be already 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, or are watching someone who is.

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


[Bug 430653] Review Request: baracuda - VNC system

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

Summary: Review Request: baracuda - VNC system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 05:08 EST ---
Baracuda died because in the end I joined to TightVNC group and development done
in baracuda is going to be merged to TightVNC tree. Closing

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

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


[Bug 437321] Review Request: libarchive - A library for handling streaming archive formats

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

Summary: Review Request: libarchive - A library for handling streaming archive 
formats


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 06:35 EST ---
Uploaded new spec and srpm:
Spec URL: http://tbzatek.fedorapeople.org/libarchive/libarchive.spec
SRPM URL: 
http://tbzatek.fedorapeople.org/libarchive/libarchive-2.4.14-1.fc9.src.rpm

- docs are now included
- rebased to 2.4.14

rpmlint now reports no error, please check

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

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


[Bug 429694] Review Request: openfire -- Instant messaging and groupchat server based on XMPP

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

Summary: Review Request: openfire -- Instant messaging and groupchat server 
based on XMPP


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
   Flag||fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 07:17 EST ---
Giving up on local Jabber server.

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

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


[Bug 389471] Review Request: afflib - Library to support the Advanced Forensic Format

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

Summary: Review Request: afflib - Library to support the Advanced Forensic 
Format


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 07:50 EST ---
Spec URL: http://kwizart.fedorapeople.org/SPECS/afflib.spec
SRPMS: http://kwizart.fedorapeople.org/SRPMS/afflib-3.1.3-2.fc9.kwizart.src.rpm
Description: Library to support the Advanced Forensic Format

Changelog
- Rebuild with newer libewf and enable-libewf=yes
- Add pkg-config support in afflib-devel.
- Add a patch to remove ldconfig call when building the package.
- Add libtermcap-devel

This package requires libewf = 20080305
I will update libewf to 20080315 for F-9 now.
(until then it is available here : http://rpms.kwizart.net/fedora/development/ )




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

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


[Bug 437321] Review Request: libarchive - A library for handling streaming archive formats

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

Summary: Review Request: libarchive - A library for handling streaming archive 
formats


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 08:41 EST ---
Looks fine now. 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, or are watching someone who is.

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


[Bug 437960] New: Review Request: mac-robber - Tool to create a timeline of file activity for mounted file systems

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




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

   Summary: Review Request: mac-robber - Tool to create a timeline
of file activity for mounted file systems
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://kwizart.fedorapeople.org/SPECS/mac-robber.spec
SRPM URL: http://kwizart.fedorapeople.org/SRPMS/mac-robber-1.00-1.fc9.src.rpm
Description: Tool to create a timeline of file activity for mounted file systems

This package is needed by the sleuthkit

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

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


[Bug 437960] Review Request: mac-robber - Tool to create a timeline of file activity for mounted file systems

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

Summary: Review Request: mac-robber - Tool to create a timeline of file 
activity for mounted file systems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||401101
  nThis||




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

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


[Bug 401101] Review Request: sleuthkit - The Sleuth Kit (TSK)

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

Summary: Review Request: sleuthkit - The Sleuth Kit (TSK)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||437960




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

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


[Bug 437321] Review Request: libarchive - A library for handling streaming archive formats

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

Summary: Review Request: libarchive - A library for handling streaming archive 
formats


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 09:15 EST ---
New Package CVS Request
===
Package Name: libarchive
Short Description: A library for handling streaming archive formats
Owners: tbzatek
Branches: devel
InitialCC: tbzatek
Cvsextras Commits: yes

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

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


[Bug 437966] New: Review Request: aimage - Advanced Disk Imager

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




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

   Summary: Review Request: aimage - Advanced Disk Imager
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://kwizart.fedorapeople.org/SPECS/aimage.spec
SRPM URL: 
http://kwizart.fedorapeople.org/SRPMS/aimage-3.1.0-1.fc9.kwizart.src.rpm
Description: Advanced Disk Imager

rpmlint is quiet -  build tested for F-9

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

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


[Bug 416471] Review Request: xsel -- manipulate the X selection

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

Summary: Review Request: xsel -- manipulate the X selection


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 09:02 EST ---
Each time you do a new, please post the full url, it helps when
getting it by mail and using for example wget from the command 
line to get it.

Without looking at the content, it looks like the release is wrong.
It should be along
0.1.svn20080107

See
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-d97a3f40b6dd9d2288206ac9bd8f1bf9b791b22a

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

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


[Bug 428567] Review Request: ETL-devel - Extended Template Library

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

Summary: Review Request: ETL-devel - Extended Template Library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 09:32 EST ---
(In reply to comment #41)
 ETL/synfig/synfigstudio don't use that function. From google it looks like it
 was part of glib at one point but was renamed.

Maybe there is a missing -lstdc++ somewhere ?

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

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


[Bug 437966] Review Request: aimage - Advanced Disk Imager

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

Summary: Review Request: aimage - Advanced Disk Imager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||389471




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 09:30 EST ---
Need to have afflib reviewed first

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

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


[Bug 436239] Review Request: joda-time - Java date and time API

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

Summary: Review Request: joda-time - Java date and time API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:24 EST ---
Okay. tzdata replacing seems to be working good.

One thing:
- From rpmlint:
---
$ rpmlint joda-time
joda-time.noarch: W: incoherent-version-in-changelog 1.5.2-3 
1.5.2-3.tzdata2008a.fc9
---
  Please change %changelog to
---
%changelog
* Mon Mar 17 2008 Conrad Meyer [EMAIL PROTECTED] - 1.5.2-3.tz2008a
---
  before importing this to CVS.

---
 This package (joda-time) is APPROVED by me
---

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

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


[Bug 437400] Review Request: msr-tools - tools to read/write processor model specific registers

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

Summary: Review Request: msr-tools - tools to read/write processor model 
specific registers


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:48 EST ---
1.) Package currently won't build:

install rdmsr %{buildroot}%{_sbindir}/rdmsr
install wrmsr %{buildroot}%{_sbindir}/wrmsr

Did you mean install -D there?

2.) Some cosmetical things:

msr-tools.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 36)
msr-tools.src: W: summary-ended-with-dot Collection of tools for reading/writing
CPU model specific registers.
msr-tools.x86_64: W: incoherent-version-in-changelog 1.1.2 1.1.2-1.fc8

the last one is the same as bug #437397 comment #3 -- the changelog header
should end with  - 1.1.2-1 (no dist tag there, contrary to what rpmlint output
above suggests)

I see no more issues with the package. I'll approve it once these are addressed.

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

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


[Bug 437400] Review Request: msr-tools - tools to read/write processor model specific registers

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

Summary: Review Request: msr-tools - tools to read/write processor model 
specific registers


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:49 EST ---
Oh, pardon me -- one more minor thing: -n %{name}-%{version} as %setup macro
argument is not necessary.

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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:51 EST ---
New Package CVS Request
===
Package Name: perl-Template-Timer
Short Description: Template::Timer Perl module
Owners: cweyl
Branches: F-7, F-8, devel
InitialCC: perl-sig

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

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:53 EST ---
New Package CVS Request
===
Package Name: perl-Test-WWW-Mechanize-Catalyst
Short Description: Test::WWW::Mechanize for Catalyst
Owners: cweyl
Branches: F-7, F-8, devel
InitialCC: perl-sig

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

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


[Bug 435829] Review Request: tomcat6 - Apache Servlet/JSP Engine, RI for Servlet 2.5/JSP 2.1 API

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

Summary: Review Request: tomcat6 -  Apache Servlet/JSP Engine, RI for Servlet 
2.5/JSP 2.1 API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 10:57 EST ---
I'll take this.

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

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


[Bug 437397] Review Request: pmtools - power management debugging tools

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

Summary: Review Request: pmtools - power management debugging tools


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:01 EST ---
1.) %setup -n %{name}-%{version}

%{name}-%{version} is the default value. %setup -q would be sufficient (-q is
always mandatory).

2.) * Thu Mar 13 2008 Dave Jones [EMAIL PROTECTED]20071116

This should have been:
* Thu Mar 13 2008 Dave Jones [EMAIL PROTECTED] - 20071116-1

3.) The package depends on headers from kernel source; Currently they are looked
for in ../include; this is from acpidump Makefile:

KERNEL_INCLUDE := ../include
CFLAGS += -Wall -Wstrict-prototypes -Wdeclaration-after-statement -Os -s
-D_LINUX -DDEFINE_ALTERNATE_TYPES -I$(KERNEL_INCLUDE)

Maybe one solution to this is to depend on kernel-devel and try to guess path to
the includes with something like CPPFLAGS=-I$(ls -d /usr/src/kernels/*/include
|tail -n1). Pay extra attention for the tool not to be dependent on a specific
kernel. It's likely you better know what to do there anyways :)

Another problem in above two lines is that -Os -s are not allowed. Patch them
away; and probably take a look at another Makefiles for similar problems.

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

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


[Bug 401101] Review Request: sleuthkit - The Sleuth Kit (TSK)

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

Summary: Review Request: sleuthkit - The Sleuth Kit (TSK)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||389471




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

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


[Bug 401101] Review Request: sleuthkit - The Sleuth Kit (TSK)

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

Summary: Review Request: sleuthkit - The Sleuth Kit (TSK)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:03 EST ---
Spec URL: 
http://kwizart.fedorapeople.org/SPEC/sleuthkit.spec
SRPM URL: 
http://kwizart.fedorapeople.org/SRPMS/sleuthkit-2.51-1.fc9.kwizart.src.rpm
Description: The Sleuth Kit (TSK)

Changelog
- Update to 2.51
- Add libewf/afflib BR
- Requires mac-robber external package.
- Remove internal perl-Date-Manip.


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

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


[Bug 389471] Review Request: afflib - Library to support the Advanced Forensic Format

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

Summary: Review Request: afflib - Library to support the Advanced Forensic 
Format


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||401101
  nThis||




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

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


[Bug 435829] Review Request: tomcat6 - Apache Servlet/JSP Engine, RI for Servlet 2.5/JSP 2.1 API

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

Summary: Review Request: tomcat6 -  Apache Servlet/JSP Engine, RI for Servlet 
2.5/JSP 2.1 API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:08 EST ---
Please fix item(s) mared by X:
MUST:
* package is named appropriately
 - match upstream tarball or project name
 - try to match previous incarnations in other distributions/packagers for
consistency
 - specfile should be %{name}.spec
 - non-numeric characters should only be used in Release (ie. cvs or
   something)
 - for non-numerics (pre-release, CVS snapshots, etc.), see
   http://fedoraproject.org/wiki/Packaging/NamingGuidelines#PackageRelease
 - if case sensitivity is requested by upstream or you feel it should be
   not just lowercase, do so; otherwise, use all lower case for the name
* is it legal for Fedora to distribute this?
 - OSI-approved
 - not a kernel module
 - not shareware
 - is it covered by patents?
 - it *probably* shouldn't be an emulator
 - no binary firmware
* license field matches the actual license.
* license is open source-compatible.
 - use acronyms for licences where common
* specfile name matches %{name}
* verify source and patches (md5sum matches upstream, know what the patches do)
 - if upstream doesn't release source drops, put *clear* instructions on
   how to generate the the source drop; ie. 
  # svn export blah/tag blah
  # tar cjf blah-version-src.tar.bz2 blah
* skim the summary and description for typos, etc.
* correct buildroot
 - should be:
   %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
We're keeping the buildroot as NVR as per discussed in the meeting with Fedora
and JPackage folks.
* if %{?dist} is used, it should be in that form (note the ? and %
locations)
* license text included in package and marked with %doc
* keep old changelog entries; use judgement when removing (too old?
useless?)
* packages meets FHS (http://www.pathname.com/fhs/)
* rpmlint on this package.srpm gives no output
 - justify warnings if you think they shouldn't be there
* changelog should be in one of these formats:

  * Fri Jun 23 2006 Jesse Keating [EMAIL PROTECTED] - 0.6-4
  - And fix the link syntax.

  * Fri Jun 23 2006 Jesse Keating [EMAIL PROTECTED] 0.6-4
  - And fix the link syntax.

  * Fri Jun 23 2006 Jesse Keating [EMAIL PROTECTED]
  - 0.6-4
  - And fix the link syntax.

* Packager tag should not be used
* Vendor tag should not be used
* Distribution tag should not be used
* use License and not Copyright 
* Summary tag should not end in a period
* if possible, replace PreReq with Requires(pre) and/or Requires(post)
X specfile is legible
 - in tomcat6-6.0.conf, JAVA_HOME is set to /usr/lib/jvm/java-icedtea, shouldn't
it be set to /usr/lib/jvm/java and set alternatives to set icedtea to be the 
jvm?
 - in tomcat6-6.0.init, should a check to see if the file
/etc/rc.d/init.d/functions exists before sourcing it?
 - should icedtea be listed as Requires as well?
 - is jpackage-utils a Requires as well?
 - is this the correct link for FHS 2.3:
http://www.pathname.com/fhs/pub/fhs-2.3.html
* package successfully compiles and builds on at least x86
X BuildRequires are proper
 - builds in mock will flush out problems here
 - the following packages don't need to be listed in BuildRequires:
   bash
   bzip2
   coreutils
   cpio
   diffutils
   fedora-release (and/or redhat-release)
   gcc
   gcc-c++
   gzip
   make
   patch
   perl
   redhat-rpm-config
   rpm-build
   sed
   tar
   unzip
   which
X sed doesn't need to be listed as a BR.
* summary should be a short and concise description of the package
* description expands upon summary (don't include installation
instructions)
* make sure lines are = 80 characters
* specfile written in American English
* make a -doc sub-package if necessary
 - see
  
http://fedoraproject.org/wiki/Packaging/Guidelines#head-9bbfa57478f0460c6160947a6bf795249488182b
* packages including libraries should exclude static libraries if possible
* don't use rpath
* config files should usually be marked with %config(noreplace)
* GUI apps should contain .desktop files
* should the package contain a -devel sub-package?
* use macros appropriately and consistently
 - ie. %{buildroot} and %{optflags} vs. $RPM_BUILD_ROOT and $RPM_OPT_FLAGS
* don't use %makeinstall
* install section must begin with rm -rf $RPM_BUILD_ROOT or %{buildroot}
* locale data handling correct (find_lang)
 - if translations included, add BR: gettext and use %find_lang %{name} at the
   end of %install
* consider using cp -p to preserve timestamps
* split Requires(pre,post) into two separate lines
* package should probably not be relocatable
* package contains code
 - see http://fedoraproject.org/wiki/Packaging/Guidelines#CodeVsContent
 - in general, there should be no offensive content
* package should own all directories and files
* 

[Bug 437981] New: Review Request: eigen - A lightweight C++ template library for vector and matrix math

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




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

   Summary: Review Request: eigen -  A lightweight C++ template
library for vector and matrix math
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/eigen/eigen.spec
SRPM URL: 
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/eigen/eigen-1.0.5-1.src.rpm
Description:
Eigen is a lightweight C++ template library for vector and matrix math, a.k.a. 
linear algebra.

$ rpmlint *.rpm
eigen.src: W: invalid-license GPLv2+ with exceptions
eigen-devel.noarch: W: invalid-license GPLv2+ with exceptions
bah. :)

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

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


[Bug 436036] Review Request: jna - Pure Java access to native libraries

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

Summary: Review Request: jna - Pure Java access to native libraries


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:27 EST ---
For -javadoc subpackages:
  - It seems Java related packages use -javadoc for subpackages,
not -javadocs
  - And please use Documentation simply for Group.


  This package (jna) is APPROVED by me


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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:29 EST ---
I'll review this.

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

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


[Bug 431098] Review Request: gambas2 - IDE based on a basic interpreter with object extensions

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

Summary: Review Request: gambas2 - IDE based on a basic interpreter with object 
extensions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:36 EST ---
Well, it seems that 2.3.0 is already released

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:37 EST ---
Hmmm, this seems to want graphviz and latex to build the documentation (it uses 
them on my fully populated system, at least), so I think those should be 
BRed.

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:41 EST ---
Oh, and what rpmlint are you using? That License tag is valid according to the 
Fedora license list, and the rpmlint-0.82-2.fc8 I'm using doesn't spit out 
those false warnings. (I get empty output from rpmlint, which is good. :-) )

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

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


[Bug 435829] Review Request: tomcat6 - Apache Servlet/JSP Engine, RI for Servlet 2.5/JSP 2.1 API

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

Summary: Review Request: tomcat6 -  Apache Servlet/JSP Engine, RI for Servlet 
2.5/JSP 2.1 API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:40 EST ---
* packages meets FHS (http://www.pathname.com/fhs/)

I changed from /srv in the upstream JPackage spec as I don't think Fedora allows
this, but I can't find any reference right now.

X specfile is legible
 - in tomcat6-6.0.conf, JAVA_HOME is set to /usr/lib/jvm/java-icedtea, shouldn't
it be set to /usr/lib/jvm/java and set alternatives to set icedtea to be the 
jvm?

I think I should get rid of the specific icedtea reference. But fnasser
mentioned it had to be built and run on icedtea. However, I don't think we
should exclude GCJ, but it hasn't be tested by me on there.

X BuildRequires are proper

Will build again in mock.

X sed doesn't need to be listed as a BR.

OK

X package should build in mock

Package built in mock for me when I posted it. This is due to the recent change
to java-1.6.0-openjdk. Removing the specific requirement on icedtea will fix 
this.


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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:48 EST ---
Built/checked on fc7.  Doh on the graphviz (my bad for not using mock).

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 11:53 EST ---
Don't forget texlive-latex too, it's used to convert the formulas in the 
doxygen documentation to images for the HTML.

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 12:02 EST ---
Spec URL: http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/eigen/eigen.spec
SRPM URL: 
http://kdeforge.unl.edu/apt/kde-redhat/SOURCES/eigen/eigen-1.0.5-2.src.rpm

%changelog
* Tue Mar 18 2008 Rex Dieter [EMAIL PROTECTED] 1.0.5-2
- BR: graphviz tex(latex)
- -devel: Provides: %%name (compat with other distros)

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

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


[Bug 197974] Tracking bug for reviews stalled pending the adoption of guidelines

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

Summary: Tracking bug for reviews stalled pending the adoption of guidelines
Alias: F-GUIDELINES

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||436356
  nThis||




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

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


[Bug 436356] Review Request: openoffice.org-extendedPDF - Create PDF with hyperlinks, bookmarks and more

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

Summary: Review Request: openoffice.org-extendedPDF -  Create PDF with 
hyperlinks, bookmarks and more


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||197974




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 12:46 EST ---
Fair enough, lets set to blocked on F-GUIDELINES, that seems a reasonable way to
document the status of this for 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, or are watching someone who is.

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


[Bug 431098] Review Request: gambas2 - IDE based on a basic interpreter with object extensions

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

Summary: Review Request: gambas2 - IDE based on a basic interpreter with object 
extensions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 12:46 EST ---
For license issue:
Please check the following:
---
GPLv2_or_QPL/gb.image/src/qcolor.cpp
GPLv2_or_QPL/gb.image/src/qcolor.h
GPLv2_or_QPL/gb.image/src/qimage.h
GPLv2_or_QPL/gb.image/src/qpoint.cpp
GPLv2_or_QPL/gb.image/src/qpoint.h
GPLv2_or_QPL/gb.image/src/qrect.cpp
GPLv2_or_QPL/gb.image/src/qrect.h
GPLv2_or_QPL/gb.image/src/qsize.cpp
GPLv2_or_QPL/gb.image/src/qsize.h
GPLv2_or_QPL/gb.qt/src/qtxembed.cpp
GPLv2_or_QPL/gb.qt/src/qtxembed.h
GPLv2plus/examples/examples/Database/PictureDatabase/ModuleDatabase.module
GPLv2plus/examples/examples/Games/RobotFindsKitten/COPYING
GPLv2plus/examples/examples/Games/RobotFindsKitten/readme.txt
GPLv2plus/examples/examples/OpenGL/GambasGears/Module1.module
GPLv2plus/examples/examples/Printing/Printing/ModulePrintTwoImages.module
GPLv2plus/examples/examples/Printing/Printing/ModulePrintTwoImgWordWrap.module
GPLv2plus/help/help/tree/jsTree.css
GPLv2plus/help/help/tree/jsTree.js
GPLv2plus/help/help/tree/tree.html
--

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:04 EST ---
MUST Items:
+ rpmlint output: none
+ named and versioned according to the Package Naming Guidelines:
+ spec file name matches base package name
+ Packaging Guidelines:
  + License GPLv2+ with exceptions OK, matches actual license
  + No known patent problems
  + No emulator, no firmware, no binary-only or prebuilt components
  + Complies with the FHS
  + proper changelog, tags, BuildRoot, Requires, BuildRequires, Summary, 
Description
  + no non-UTF-8 characters
  + relevant documentation included
  + nothing to compile, so RPM_OPT_FLAGS are irrelevant
  + nothing to compile (noarch package), so no debuginfo package
  + no static libraries nor .la files
  + no duplicated system libraries
  + no binaries = no rpaths
  + no configuration files, so %config guideline doesn't apply
  + no init scripts, so init script guideline doesn't apply
  + no GUI executables, so no .desktop file needed
  + ... and thus no desktop-file-install needed either
  + no timestamp-clobbering file commands
  + _smp_mflags used
  + scriptlets are valid
  + not a web application, so web application guideline doesn't apply
  + no conflicts
+ complies with all the legal guidelines
+ license included as %doc
+ source matches upstream:
  MD5SUM: 960d7e5fb6542270eae4d53ca99b607c
  SHA1SUM: 32400616c4866eab158115c556df330b077644a3
+ builds on at least one arch (F8 i386)
+ no known non-working arches, so no ExcludeArch needed
+ BuildRequires are complete (cmake, doxygen, graphviz, tex(latex))
+ no translations, so translation/locale guidelines don't apply
+ no shared libraries, so no ldconfig call needed
+ package not relocatable
+ ownership correct (owns package-specific directories, doesn't own directories 
owned by another package)
+ no duplicate files in %files
+ permissions correct, defattr used correctly
+ %clean section present and correct
+ macros used where possible
+ no non-code content
+ no large documentation files, so no -doc package needed
+ %doc files not required at runtime
+ all header files in -devel
+ no static libraries, so no -static package needed
+ no .pc files, so no Requires: pkgconfig needed
+ no .so symlinks vs. plugins
+ no main package (as this is a headers-only library), -devel Provides: %{name} 
= %{version}-%{release} instead, OK
+ no .la files
+ no GUI executables, so no .desktop file needed
+ buildroot is deleted at the beginning of %install
+ all filenames are valid UTF-8

SHOULD Items:
+ license included as %doc
+ no translations for description and summary provided by upstream
* skipping mock test
* skipping all arch test
* skipping functionality test
+ scriptlets are sane
+ no subpackages other than -devel, so Usually, subpackages other than devel 
should require the base package using a fully versioned dependency. is 
irrelevant
+ no .pc files, so placement of .pc files is irrelevant
+ no file dependencies

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, or are watching someone who is.

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


[Bug 431098] Review Request: gambas2 - IDE based on a basic interpreter with object extensions

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

Summary: Review Request: gambas2 - IDE based on a basic interpreter with object 
extensions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:07 EST ---
The rest issues for 2.2.1-5:

* Requires(post/postun): desktop-file-utils
  - What is this? (there is no gambas2 i386 rpm)

* Timestamps
  - When using install or cp commands, please add -p option
to keep timestamps on installed files.

* /sbin/ldconfig call for gambas-runtime
  - Is this needed? It seems no libraries are installed under
ld default paths.

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

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


[Bug 226559] Merge Review: xhtml1-dtds

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

Summary: Merge Review: xhtml1-dtds


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:14 EST ---
From my point of view it's fine,

Daniel

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:10 EST ---
New Package CVS Request
===
Package Name: eigen
Short Description: A lightweight C++ template library for vector and matrix math
Owners: rdieter, kkofler
Branches: EL-5 F-7 F-8
Cvsextras Commits: yes

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

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


[Bug 437321] Review Request: libarchive - A library for handling streaming archive formats

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

Summary: Review Request: libarchive - A library for handling streaming archive 
formats


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:26 EST ---
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, or are watching someone who is.

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


[Bug 437481] Review Request: ocaml-newt - OCaml library for using newt text mode window system

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

Summary: Review Request: ocaml-newt - OCaml library for using newt text mode 
window system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:28 EST ---
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, or are watching someone who is.

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


[Bug 437917] Review Request: perl-Array-RefElem - Set up array elements as aliases

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

Summary: Review Request: perl-Array-RefElem - Set up array elements as aliases
Alias: perl-Array-RefElem

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:32 EST ---
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, or are watching someone who is.

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


[Bug 437918] Review Request: perl-Devel-LexAlias - Alias lexical variables

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

Summary: Review Request: perl-Devel-LexAlias - Alias lexical variables
Alias: perl-Devel-LexAlias

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:34 EST ---
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, or are watching someone who is.

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:36 EST ---
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, or are watching someone who is.

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:44 EST ---
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, or are watching someone who is.

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


[Bug 436356] Review Request: openoffice.org-extendedPDF - Create PDF with hyperlinks, bookmarks and more

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

Summary: Review Request: openoffice.org-extendedPDF -  Create PDF with 
hyperlinks, bookmarks and more


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:51 EST ---
Sounds good. Thanks for your understanding. 
(Clearing the cvs flag here until it's ready for cvs)


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

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


[Bug 437481] Review Request: ocaml-newt - OCaml library for using newt text mode window system

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

Summary: Review Request: ocaml-newt - OCaml library for using newt text mode 
window system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:55 EST ---
 - including 'README' file in %doc of main package.

I didn't do this because after some discussion on fedora-devel-list
and for the new package guidelines we decided not to include the
README file and others like it twice, and only in -devel.

 - moving './configure' call from the %prep section to the %build section, 
 since
 its part of the build process, not source unpackaging/patching process.

Done.

---

Build for F-9: http://koji.fedoraproject.org/koji/taskinfo?taskID=521096

The F-8 build is waiting for camlidl to get pushed to F-8 updates.

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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 13:39 EST ---
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, or are watching someone who is.

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


[Bug 435829] Review Request: tomcat6 - Apache Servlet/JSP Engine, RI for Servlet 2.5/JSP 2.1 API

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

Summary: Review Request: tomcat6 -  Apache Servlet/JSP Engine, RI for Servlet 
2.5/JSP 2.1 API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 14:13 EST ---
SRPM URL: 
http://dwalluck.fedorapeople.org/tomcat6/tomcat6-6.0.16-1jpp.4.fc9.src.rpm

It builds for me in mock with `BuildRequires: java-devel = 1.6.0'. If we leave
it unversioned, gcj is pulled in and it does not currently build with gcj.

I commented out JAVA_HOME in tomcat6.conf, it should now use the default one.


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

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


[Bug 436239] Review Request: joda-time - Java date and time API

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

Summary: Review Request: joda-time - Java date and time API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 14:19 EST ---
OK. I'll wait on requesting CVS until we arrive at some conclusion about the 
status of java packaging.

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

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


[Bug 438024] New: Review Request: freeimage - Multi-format image decoder library

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




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

   Summary: Review Request: freeimage - Multi-format image decoder
library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.atrpms.net/~hdegoede/freeimage.spec
SRPM URL: http://people.atrpms.net/~hdegoede/freeimage-3.10.0-1.fc9.src.rpm
Description:
FreeImage is a library for developers who would like to support popular
graphics image formats like PNG, BMP, JPEG, TIFF and others as needed by
today's multimedia applications.

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

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


[Bug 438024] Review Request: freeimage - Multi-format image decoder library

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

Summary: Review Request: freeimage - Multi-format image decoder library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||435399
  nThis||




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

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


[Bug 433187] Review Request: boinc-client - A platform for distributed computing

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

Summary: Review Request: boinc-client - A platform for distributed computing
Alias: boinc-client-review

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 14:44 EST ---
Well, for 5.10.45-2:

* File lists/scriptlet order for log files under %_localstatedir/log/boinc


%postun
# SELinux support
if [ $1 -eq 0 ]; then  # final removal
  semanage fcontext -d -t var_log_t %{_localstatedir}/lib/boinc/.*\.log
2/dev/null || :
fi

  - As %_localstatedir/lib/boinc is in %files lists, at %postun
%_localstatedir/lib/boinc/{boinc,error}.log are already deleted
at this time.

Then:
- Should {boinc,error}.log be removed at final removal?
  If not (for example, you think that some sysadmins don't want
  to have those log files automatically deleted), then
  these 2 files should not be in %files list

- But this contradicts my comment 13... i.e. if these two
   files are not in %files list, then when boinc_client is installed
   at first time these two files does not exists, then
   semanage or restorecon on %post does nothing...

- If you think these two files can be removed at final removal,
  then the above scriptlet is not needed.


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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 15:26 EST ---
imported, built for rawhide.

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

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


[Bug 438040] New: Review Request: GMT-coastlines-high - High resolution coastline data for GMT

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




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

   Summary: Review Request: GMT-coastlines-high - High resolution
coastline data for GMT
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-coastlines-high.spec
SRPM URL: 
http://www.cora.nwra.com/~orion/fedora/GMT-coastlines-high-1.9-1.src.rpm
Description:
High resolution coastline data for GMT.

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

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


[Bug 438039] New: Review Request: GMT-coastlines - Coastline data for GMT

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




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

   Summary: Review Request: GMT-coastlines - Coastline data for GMT
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-coastlines.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-coastlines-1.9-1.src.rpm
Description:
Crude, low, and intermediate resolutions coastline data for GMT.


GMT package has a build dependency on this for the %check stage.

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

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


[Bug 437981] Review Request: eigen - A lightweight C++ template library for vector and matrix math

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

Summary: Review Request: eigen -  A lightweight C++ template library for vector 
and matrix math


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 15:36 EST ---
eigen-1.0.5-2.fc8 has been submitted as an update for Fedora 8

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

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


[Bug 438039] Review Request: GMT-coastlines - Coastline data for GMT

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

Summary: Review Request: GMT-coastlines - Coastline data for GMT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||438043
  nThis||




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

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


[Bug 438041] New: Review Request: GMT-coastlines-full - Full resolution coastline data for GMT

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




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

   Summary: Review Request: GMT-coastlines-full - Full resolution
coastline data for GMT
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT-coastlines-full.spec
SRPM URL: 
http://www.cora.nwra.com/~orion/fedora/GMT-coastlines-full-1.9-1.src.rpm
Description:
Full resolution coastline data for GMT.

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

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


[Bug 438043] New: Review Request: GMT - Generic Mapping Tools

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




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

   Summary: Review Request: GMT - Generic Mapping Tools
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.cora.nwra.com/~orion/fedora/GMT.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/GMT-4.2.1-1.fc8.src.rpm
Description:
GMT is an open source collection of ~60 tools for manipulating geographic and
Cartesian data sets (including filtering, trend fitting, gridding, projecting,
etc.) and producing Encapsulated PostScript File (EPS) illustrations ranging
from simple x-y plots via contour maps to artificially illuminated surfaces
and 3-D perspective views.  GMT supports ~30 map projections and transforma-
tions and comes with support data such as coastlines, rivers, and political
boundaries.

GMT is developed and maintained by Paul Wessel and Walter H. F.  Smith with
help from a global set of volunteers, and is supported by the National
Science Foundation.

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

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


[Bug 437917] Review Request: perl-Array-RefElem - Set up array elements as aliases

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

Summary: Review Request: perl-Array-RefElem - Set up array elements as aliases
Alias: perl-Array-RefElem

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:06 EST ---
Thanks for the review! :)

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

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


[Bug 437923] Review Request: perl-Test-WWW-Mechanize-Catalyst - Test::WWW::Mechanize for Catalyst

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

Summary: Review Request: perl-Test-WWW-Mechanize-Catalyst - 
Test::WWW::Mechanize for Catalyst


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:07 EST ---
Thanks for the review! :)

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

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


[Bug 437919] Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical goodness

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

Summary: Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical 
goodness


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


Bug 437919 depends on bug 437917, which changed state.

Bug 437917 Summary: Review Request: perl-Array-RefElem - Set up array elements 
as aliases
https://bugzilla.redhat.com/show_bug.cgi?id=437917

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 437918] Review Request: perl-Devel-LexAlias - Alias lexical variables

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

Summary: Review Request: perl-Devel-LexAlias - Alias lexical variables
Alias: perl-Devel-LexAlias

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:08 EST ---
Thanks for the review! :)

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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:07 EST ---
Thanks for the review!

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

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


[Bug 437919] Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical goodness

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

Summary: Review Request: perl-Catalyst-Controller-BindLex - Stash your lexical 
goodness


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


Bug 437919 depends on bug 437918, which changed state.

Bug 437918 Summary: Review Request: perl-Devel-LexAlias - Alias lexical 
variables
https://bugzilla.redhat.com/show_bug.cgi?id=437918

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 241596] Review Request: maildrop - Mail delivery agent with filtering abilities

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

Summary: Review Request: maildrop - Mail delivery agent with filtering abilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:08 EST ---
Thanks Mamoru!

New Package CVS Request
===
Package Name: maildrop
Short Description: Mail delivery agent with filtering abilities
Owners: athimm
Branches: F-7 F-8
InitialCC:
Cvsextras Commits: no (too many people in this group)


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

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


[Bug 438069] New: Get java-1.6.0-openjdk in EPEL-4 and EPEL-5

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




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

   Summary: Get java-1.6.0-openjdk in EPEL-4 and EPEL-5
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: [EMAIL PROTECTED],fedora-package-
[EMAIL PROTECTED],[EMAIL PROTECTED],[EMAIL PROTECTED]


This is to track progress on getting OpenJDK Java to EPEL.

+++ This bug was initially created as a clone of Bug #433070 +++

-- Additional comment from [EMAIL PROTECTED] on 2008-03-11 12:09 EST --
Would it be possible to also add this package to EPEL5? 

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 10:48 EST --
Erik: Currently it depends on lesstif (has to be branched for EPEL, not a big
deal), freetype = 2.3.0 (not a BR, but complains during the build. probably
would work even with older one that's in RHEL?), and newer jpackage-utils and
tzdata-java than are in RHEL. Probably there can be a way found to work this
around, I'll experiment a bit when my initial build is done (and will publish a
repo with this, if anyone's interested).

Moreover, it can be bootstrapped with gcj-1.5.0, but only 1.4.2 is avaliable.
This can be probably worked around by manually adding java-1.6.0-openjdk-devel
to the EPEL build root. I am bootstrapping with one from f9 and it seems to be
sufficient.

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 17:01 EST --
Erik: Here's my build of openjdk for RHEL-5, with packages that are not in EPEL:
[1]. Dependencies were grabbed from dist-f9, tzdata was changed due to #437150.
Should we open another bug for efforts to get that in EPEL, so we don't spam
people that were interested in review request into rawhide?

[1] http://netbsd.sk/~lkundrak/openjdk-el5/ (will probably be subject to move to
http://people.redhat.com/lkundrak/openjdk-el5/ but I am over quota and am
waiting for extension approval :)

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 17:20 EST --
Great job!
I'll try out your repo in a couple of days to install a package called Alfresco
which requires Java 1.6 on a CentOS 5 machine. I'll let you know if I notice
anything strange.

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 17:25 EST --
Would having java-1.6.0-openjdk in EPEL cause conflicts if it were introduced
later in RHEL?

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 17:34 EST --
Thomas: Not necessarily. It could be removed from EPEL for that release. To
allow smooth update you would just bump the revision to be bigger that one of
the package in EPEL. We can agree on some revision number and keep the revision
number of the EPEL package smaller than it.

-- Additional comment from [EMAIL PROTECTED] on 2008-03-12 17:52 EST --
I updated the Java release notes for Fedora 9:

http://fedoraproject.org/wiki/Docs/Beats/Java


-- Additional comment from [EMAIL PROTECTED] on 2008-03-17 11:40 EST --
(In reply to comment #22)
 Erik: Here's my build of openjdk for RHEL-5, with packages that are not in 
 EPEL:
 [1].

I've just tried installing a fresh CentOS 5 system with your repo enabled during
the installation and your packages were automatically pulled in.

However, when installing Tomcat5 I get the following error :
  Updating  : tomcat5  ### [48/74] 
/usr/bin/build-jar-repository: error: Could not find ecj Java extension for 
this JVM
/usr/bin/build-jar-repository: error: Some specified jars were not found for
this jvm

This failure makes it impossible to run a tomcat based web-application. I've
tried rebuilding eclipse(-ecj) against java-1.6.0-openjdk, but this fails.

-- Additional comment from [EMAIL PROTECTED] on 2008-03-17 12:16 EST --
(In reply to comment #22)
 Should we open another bug for efforts to get that in EPEL, so we don't spam
 people that were interested in review request into rawhide?(In reply to
comment #27)

(In reply to comment #27)
 However, when installing Tomcat5 I get the following error :
   Updating  : tomcat5  ### [48/74] 
 /usr/bin/build-jar-repository: error: Could not find ecj Java extension for
this JVM
 /usr/bin/build-jar-repository: error: Some specified jars were not found for
 this jvm

Works for me, on fairly minimal RHEL-5. 

 
 This failure makes it impossible to run a tomcat based web-application. I've
 tried rebuilding eclipse(-ecj) against java-1.6.0-openjdk, but this fails.



-- Additional comment from [EMAIL PROTECTED] on 2008-03-18 15:56 EST --
Has anyone looked at building on EL4?

-- 
Configure bugmail: 

[Bug 438069] Get java-1.6.0-openjdk in EPEL-4 and EPEL-5

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

Summary: Get java-1.6.0-openjdk in EPEL-4 and EPEL-5
Alias: openjdk-epel

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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:55 EST ---
Note that for RHEL you *could*, theoretically, build against the system 
openmotif.

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

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


[Bug 438043] Review Request: GMT - Generic Mapping Tools

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

Summary: Review Request: GMT - Generic Mapping Tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 16:54 EST ---
I did a gmt package too. It is there:

http://www.environnement.ens.fr/perso/dumas/fc-srpms/gmt-4.2.1-1.fc8.src.rpm
http://www.environnement.ens.fr/perso/dumas/fc-srpms/gmt-data-4.2-1.src.rpm

I stuffed all the resolutions in one package.
The version of the data should certainly be 1.9.1 (like you do), it is 
4.2 in my gmt-data package because it was derived from the preceding one.

Also I don't remember why I choosed lower case package name, in any case
it is like mandriva and debian packages.

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

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


[Bug 433070] Review Request: java-1.6.0-openjdk - The OpenJDK 1.6.0 runtime environment

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

Summary: Review Request: java-1.6.0-openjdk - The OpenJDK 1.6.0 runtime 
environment


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:01 EST ---
Progress on RHEL packaging - bug #438069

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

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


[Bug 438069] Get java-1.6.0-openjdk in EPEL-4 and EPEL-5

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

Summary: Get java-1.6.0-openjdk in EPEL-4 and EPEL-5
Alias: openjdk-epel

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|fedora-package- |
   |[EMAIL PROTECTED]   |
  Component|Package Review  |java-1.6.0-openjdk




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

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


[Bug 426026] Review Request: gnome-menu-extended - Gnome Menu with KDE directory

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

Summary: Review Request: gnome-menu-extended - Gnome Menu with KDE directory


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:22 EST ---
The link to the RFE bug : https://bugzilla.redhat.com/show_bug.cgi?id=438072

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

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


[Bug 241596] Review Request: maildrop - Mail delivery agent with filtering abilities

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

Summary: Review Request: maildrop - Mail delivery agent with filtering abilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:22 EST ---
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, or are watching the QA contact.

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


[Bug 416471] Review Request: xsel -- manipulate the X selection

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

Summary: Review Request: xsel -- manipulate the X selection


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:24 EST ---
New version is out. See http://www.vergenet.net/~conrad/software/xsel/ for 
tar.gz

Busy with work. Will update rpm tomorrow.

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

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


[Bug 437397] Review Request: pmtools - power management debugging tools

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

Summary: Review Request: pmtools - power management debugging tools


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:24 EST ---
../include is in the top level of the source tarball. It's a userspace-ified
version set of the kernel headers.

spec and srpm updated for other points.

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

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


[Bug 437400] Review Request: msr-tools - tools to read/write processor model specific registers

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

Summary: Review Request: msr-tools - tools to read/write processor model 
specific registers


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:26 EST ---
Thanks, Updated.

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

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


[Bug 437924] Review Request: perl-Template-Timer - Template::Timer Perl module

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

Summary: Review Request: perl-Template-Timer - Template::Timer Perl module
Alias: perl-Template-Timer

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||438083
  nThis||




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

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


[Bug 438083] New: Review Request: perl-Catalyst-View-TT - Template Toolkit View Class

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




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

   Summary: Review Request: perl-Catalyst-View-TT - Template Toolkit
View Class
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Catalyst-View-TT/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SRPM URL: 
http://fedora.biggerontheinside.net/review/perl-Catalyst-View-TT-0.26-1.fc8.src.rpm
SPEC URL: http://fedora.biggerontheinside.net/review/perl-Catalyst-View-TT.spec

Description:
This is the Catalyst view base class for the Template Toolkit.

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

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


[Bug 433547] Review Request: nagios-nsca - nagios passive check daemon

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

Summary: Review Request: nagios-nsca - nagios passive check daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 17:56 EST ---
Xavier asked me to have a quick look.  Sorry I couldn't get to it yesterday.

A few meta-issues first:

Xavier, if you indeed are reviewing this, you should assign the ticket to
yourself and set the fedora-review flag to '?'.

The summary of the ticket should match the submitted package name.  Was it de

I don't have any particular issues with the reviewer providing fixed packages,
but  Michael does need to respond to the stuff in comments 8 and 9 before this
can move anywhere.

I am unfamiliar with nagios and related software, so I can really only look at
the basic packaging, which looks fine.  I'm looking at the package linked in
comment 9.  The scriptlets aren't exactly the same as the recommended ones, but
the differences are inconsequential.  Do note, however, that the || : bit only
has an effect as the last executed command in a scriptlet; its sole purpose is
to prevent the scriptlet from returning a nonzero exit code.

Also note that I tend to just drop the bits of my review template that don't
apply, such as debuginfo and compiler flag bits for a noarch package, and the
GUI app bit for a non-gui app.  They're just placeholders to remind me to check
things when they apply.

Finally, I see some source files that are obviously GPLv2+, and others (nsca.c,
send_nsca.c) which say GPLv2.  Someone should ping upstream and see if they
intend GPLv2 only for the final product.  Otherwise I don't think GPLv2+ is
accurate for the license tag.

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

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


[Bug 438083] Review Request: perl-Catalyst-View-TT - Template Toolkit View Class

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

Summary: Review Request: perl-Catalyst-View-TT - Template Toolkit View Class


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





--- Additional Comments From [EMAIL PROTECTED]  2008-03-18 18:03 EST ---
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=521583


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

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


  1   2   >