[Bug 580319] Review Request: gretl - Gnu Regression, Econometrics and Time-series Library

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from hannes  2010-08-29 02:33:17 
EDT ---
No I didn't ask him but I wouldn't bother if he is interested in maintaining
it. I will not stick to it like glue if he likes to finish the work. I just
thought that it would be good to get gretl into fedora. Since there wasn't
really any spec or something in here I didn't think that I overtake this
package because there was nothing I could use.

Perhaps we should just wait a bit if he is showing up and then talk to him what
he wants to do. 

I didn't want to offend him or anyone else, just want to do some packaging of
programs which are useful for me.

Best Regards

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


[Bug 580319] Review Request: gretl - Gnu Regression, Econometrics and Time-series Library

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Martin Gieseking  2010-08-29 
02:22:21 EDT ---
(In reply to comment #6)
> Usually, you should take over a package without asking for permission.

I meant: you should NOT take over a package without asking for permission. :)

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


[Bug 580319] Review Request: gretl - Gnu Regression, Econometrics and Time-series Library

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Martin Gieseking  changed:

   What|Removed |Added

 CC||martin.giesek...@uos.de

--- Comment #6 from Martin Gieseking  2010-08-29 
02:20:55 EDT ---
Hannes, did you ask Lee whether he's still interested in maintaining gretl?
Usually, you should take over a package without asking for permission.

As far as I see, Lee isn't a member of the packager group yet. Thus, this bug
should have been blocked with FE-NEEDSPONSOR.

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


[Bug 628280] Review Request: apache-commons-collections - rename of jakarta-commons-collections

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

carl  changed:

   What|Removed |Added

 CC||dev...@gunduz.org
 Blocks||177841(FE-NEEDSPONSOR)

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


[Bug 628280] New: Review Request: apache-commons-collections - rename of jakarta-commons-collections

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apache-commons-collections - rename of 
jakarta-commons-collections

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

   Summary: Review Request: apache-commons-collections - rename of
jakarta-commons-collections
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: carlgr...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


!!! NOTE: This is a re-review because the package has been renamed from
jakarta-commons-collections

Spec URL: http://carlwithak.info/apache-commons-collections.spec
SRPM URL:
http://carlwithak.info/apache-commons-collections-3.2.1-4.fc13.src.rpm
Description: Commons-Collections seek to build upon the JDK classes by
providing new interfaces, implementations and utilities.

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


[Bug 589867] Review Request: logcheck - analyzes logfiles and sends email

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Matthias Runge  2010-08-28 
17:59:15 EDT ---
lockfile-progs (as last remaining dependency) is in -testing now. All other
dependencies are approved and hit the repositories.

Can we proceed with the review now?

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


[Bug 471805] Review Request: aespipe - AES-based encryption tool for tar/cpio and loop-aes images

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #24 from Jiri Hladky  2010-08-28 17:41:21 EDT 
---
Hi Mamoru-san,

thanks a lot for reviewing the package. I have replaced %define with %global
and made sure that everything works fine.

Spec URL: http://jhladky.fedorapeople.org/aespipe-2.4b-3.spec
MD5: 6b1af633ea681808fb30c1a1f27fc487

SRPM URL: http://jhladky.fedorapeople.org/aespipe-2.4b-3.fc12.src.rpm
MD5: 820348afe0fde4724df8ca3e77397a96

Thanks
Jirka

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


[Bug 471805] Review Request: aespipe - AES-based encryption tool for tar/cpio and loop-aes images

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jiri Hladky  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #25 from Jiri Hladky  2010-08-28 17:42:18 EDT 
---
New Package CVS Request
===
Package Name: aespipe
Short Description: AES-based encryption tool for tar/cpio and loop-aes images
Owners: jhladky
Branches: F-12 F-13 EL-5 EL-6
InitialCC: jhladky

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


[Bug 587056] Review Request: hntool - Open source hardening tool for Unix

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Filipe Rosset  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2010-08-28 17:00:12

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


[Bug 625939] Review Request: tintwizard - A GUI wizard which generates config files for tint2 panels

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Germán Racca  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #6 from Germán Racca  2010-08-28 15:29:34 EDT ---
Many thanks Ankur!! :)


New Package SCM Request
===
Package Name: tintwizard
Short Description: A GUI wizard which generates config files for tint2 panels
Owners: skytux
Branches: f12 f13 f14
InitialCC:

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

[Bug 628202] New: Review Request: gretl - A tool for econometric analysis

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gretl - A tool for econometric analysis

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

   Summary: Review Request: gretl - A tool for econometric
analysis
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: johannes.l...@googlemail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://hannes.fedorapeople.org/gretl.spec
SRPM URL: http://hannes.fedorapeople.org/gretl-1.9.1-1.fc13.src.rpm
Description:A cross-platform software package for econometric analysis, written
in the C programming language.

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


[Bug 580319] Review Request: gretl - Gnu Regression, Econometrics and Time-series Library

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

hannes  changed:

   What|Removed |Added

 CC||johannes.l...@googlemail.co
   ||m

--- Comment #5 from hannes  2010-08-28 11:33:11 
EDT ---
Since there weren't any developments on this bug report I filed another one.
https://bugzilla.redhat.com/show_bug.cgi?id=628202

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


[Bug 570017] Review Request: k4dirstat - KDE4 version of kdirstat

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Kalev Lember  changed:

   What|Removed |Added

   Flag||needinfo?(laurent.rineau__f
   ||ed...@normalesup.org)

--- Comment #13 from Kalev Lember  2010-08-28 09:58:18 EDT 
---
Laurent, could you please respond?

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


[Bug 521707] Review Request: python-zc.buildout - System for managing Python development buildouts

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #21 from Kalev Lember  2010-08-28 09:57:52 EDT 
---
Jonathan, could you please respond?

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


[Bug 480607] Review Request: bluecove - Implementation of JSR-82 Java Bluetooth API

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #18 from Tareq Al Jurf  2010-08-28 
09:46:55 EDT ---
No, no problem.
I've thought you forgot it or something like that.

Regards :)

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


[Bug 624865] Review Request: minion - Fast, Scalable Constraint Solving

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #15 from Fedora Update System  
2010-08-28 08:27:27 EDT ---
minion-0.10-4.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/minion-0.10-4.fc13

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


[Bug 624865] Review Request: minion - Fast, Scalable Constraint Solving

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 624865] Review Request: minion - Fast, Scalable Constraint Solving

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Fedora Update System  
2010-08-28 08:09:48 EDT ---
minion-0.10-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/minion-0.10-4.fc14

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


[Bug 624463] Review Request: perl-Snowball-Swedish - Porter's stemming algorithm for Swedish

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 624463] Review Request: perl-Snowball-Swedish - Porter's stemming algorithm for Swedish

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Fedora Update System  2010-08-28 
08:01:04 EDT ---
perl-Snowball-Swedish-1.2-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/perl-Snowball-Swedish-1.2-2.fc14

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


[Bug 624463] Review Request: perl-Snowball-Swedish - Porter's stemming algorithm for Swedish

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System  2010-08-28 
08:01:09 EDT ---
perl-Snowball-Swedish-1.2-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/perl-Snowball-Swedish-1.2-2.fc13

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


[Bug 624461] Review Request: perl-Snowball-Norwegian - Porter's stemming algorithm for Norwegian

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

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


[Bug 624461] Review Request: perl-Snowball-Norwegian - Porter's stemming algorithm for Norwegian

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Fedora Update System  2010-08-28 
08:00:19 EDT ---
perl-Snowball-Norwegian-1.2-2.fc13 has been submitted as an update for Fedora
13.
https://admin.fedoraproject.org/updates/perl-Snowball-Norwegian-1.2-2.fc13

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


[Bug 624461] Review Request: perl-Snowball-Norwegian - Porter's stemming algorithm for Norwegian

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Fedora Update System  2010-08-28 
08:00:13 EDT ---
perl-Snowball-Norwegian-1.2-2.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/perl-Snowball-Norwegian-1.2-2.fc14

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


[Bug 624461] Review Request: perl-Snowball-Norwegian - Porter's stemming algorithm for Norwegian

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Iain Arnell  2010-08-28 07:47:40 EDT ---
I didn't hear back from upstream about license yet, so I've imported it with
"GPLv2 or Artistic".

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


[Bug 624463] Review Request: perl-Snowball-Swedish - Porter's stemming algorithm for Swedish

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Iain Arnell  2010-08-28 07:47:46 EDT ---
I didn't hear back from upstream about license yet, so I've imported it with
"GPLv2 or Artistic".

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


[Bug 627936] Review Request: bowtie - An ultrafast, memory-efficient short read aligner

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Thomas Spura  changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org

--- Comment #2 from Thomas Spura  2010-08-28 
06:56:36 EDT ---
- Hmm, there are no CFLAGS defintions in the Makefile...
  How about: make %{?_smp_mflags} -p EXTRA_FLAGS="%{optflags}"
  Then the debuginfo-without-sources error is gone and the rest should be
ignorable.


- When you don't want to build this for EL5, you can delete quite a lot of
stuff:
  See: bug #603233 comment 11

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


[Bug 429444] Review Request: libmspack - A library for compression and decomepression

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Dan Horák  changed:

   What|Removed |Added

 CC||d...@danny.cz
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #14 from Dan Horák  2010-08-28 04:49:50 EDT ---
Package Change Request
==
Package Name: libmspack
Owners: sharkcz

I'm going to unorphan this one after being orphaned yesterday -
http://lists.fedoraproject.org/pipermail/devel/2010-August/142091.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 627590] Review Request: openoffice.org-diafilter - Dia Format importer and gallery extension for OpenOffice.org

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Xavier Bachelot  changed:

   What|Removed |Added

 CC||xav...@bachelot.org

--- Comment #3 from Xavier Bachelot  2010-08-28 04:11:39 
EDT ---
David, just a drive-by note, rpmlint should be run on the binary rpms too, not
only the srpm. This is usually from the binaries the most significant warnings
and errors come from.

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


[Bug 614036] Review Request: stdair - C++ Standard Airline IT Library - FE-NEEDSPONSOR

2010-08-28 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Martin Gieseking  changed:

   What|Removed |Added

 CC||martin.giesek...@uos.de

--- Comment #4 from Martin Gieseking  2010-08-28 
03:49:07 EDT ---
Here are some more comments:

- the package doesn't build in mock because of missing 
  BuildRequires:soci-mysql-devel

- The tarball bundles library libextracppunit. It's not permitted to link
  against bundled libraries in Fedora. Instead, remove it and use the
  corresponding library provided by a separate package. extracc isn't part of
  Fedora yet, but already under review (bug #616881).
  Also see https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries

- replace %define with %global
 
https://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define
 

- As far as I see, there are no info files. Hence, drop Requires(post) and
  Requires(preun).

- since you can't add any new packages to Fedora < 12, remove the %if 
  statements. It's sufficient to use the following for all targets: 
  BuildArch:  noarch
  BuildRequires:  tex(latex)

- it's not necessary to remove INSTALL in %prep

- I recommend to replace
rm -f $RPM_BUILD_ROOT%{_libdir}/lib%{name}.la
rm -f $RPM_BUILD_ROOT%{_libdir}/libextracppunit.la
rm -rf %{mydocs} && mkdir -p %{mydocs}
  with
rm -f $RPM_BUILD_ROOT%{_libdir}/*.la
mkdir -p %{mydocs}

  (you don't need to remove %{mydocs} as it doesn't exist here yet)

- move the man3 manpages to the devel package as it contains information for
  developers

- drop AUTHORS, ChangeLog, NEWS, and README from the doc package
  (files must not be added multiple times, COPYING is an exception here)

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