[Bug 245697] Review Request: perl-Gearman-Server - Function call router and load balancer

2007-07-02 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-Gearman-Server - Function call router and load 
balancer
Alias: perl-Gearman-Server

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 02:08 EST ---
I added a BuildRequires for perl-Gearman-Server (which is not in the repo yet) 
and enabled the tests, and 
it seems to work in mock.

New version:

http://rubenkerkhof.com/packages/perl-Gearman-Client-Async.spec
http://rubenkerkhof.com/packages/perl-Gearman-Client-Async-0.94-2.fc7.src.rpm


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

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


[Bug 236486] Review Request: Profugus - automatically migrates Xen virtual machines from one system to another based on CPU time

2007-07-02 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: Profugus - automatically migrates Xen virtual machines 
from one system to another based on CPU time


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 02:20 EST ---
As I understand nobody really wants to help me out and my project stuck as many
others. Well, anyway i'm gonna continue developing it... just for fun

Here are links of new release:
SRPM Url : http://downloads.sourceforge.net/profugus/profugus-0.2-2.src.rpm
Spec Url :http://downloads.sourceforge.net/profugus/profugus-0.2-2.spec

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

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


[Bug 236486] Review Request: Profugus - automatically migrates Xen virtual machines from one system to another based on CPU time

2007-07-02 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: Profugus - automatically migrates Xen virtual machines 
from one system to another based on CPU time


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 02:40 EST ---
Umm.. while I can review this, the trouble for me is that
I don't understand Xen *completely*.

Is this program surely working for you? Perhaps I cannot understand
how to use this immediately.

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

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


[Bug 236486] Review Request: Profugus - automatically migrates Xen virtual machines from one system to another based on CPU time

2007-07-02 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: Profugus - automatically migrates Xen virtual machines 
from one system to another based on CPU time


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 02:47 EST ---
it works for me, but you know that it's better to test it on different systems
if you have any question fell free to ask, also i can send steps how i test
that, so if you have 2 systems with xen installed i can provide information how
to test it.

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

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


[Bug 242311] Review Request: perl-Time-Duration - rounded or exact English expression of durations

2007-07-02 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-Time-Duration - rounded or exact English 
expression of durations


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 02:54 EST ---
Ruben: Good point, ChangeLog should indeed be in %doc, it has been added.
SRPM: http://marcbradshaw.co.uk/packages/SRPMS/perl-Time-Duration-1.04-4.src.rpm

Jeffrey: I would be happy to have a co-maintainer. Thanks.

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

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


[Bug 246227] Review Request: brutus-keyring - small and simple keyring daemon based upon libgcrypt

2007-07-02 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: brutus-keyring - small and simple keyring daemon based 
upon libgcrypt
Alias: brutus-keyring

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 03:18 EST ---
No, I've noticed that, but I would really like to keep the header if it isn't to
annoying. 

It is not a matter of being a legalese nitpicker. I just prefer to have a
license notice on the top of each and every file (except for the really
insignificant ones) to avoid any possible legal ambiguities.

I'll certainly remove it if it is a problem, but I would rather keep the license
notice if possible.

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

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


[Bug 236486] Review Request: Profugus - automatically migrates Xen virtual machines from one system to another based on CPU time

2007-07-02 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: Profugus - automatically migrates Xen virtual machines 
from one system to another based on CPU time


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 03:32 EST ---
Actually I have only one machine :) However now I can trust you.

By the first glance at 0.2-2:
For packaging of 0.2-2, some issues must be fixed.

* Version
  - On rpm packaging, 0.2-2 is considered as Version (as hyphen is
not allowed for rpm version, for this case the rpm version is 0.2.2)
So the EVR (Epoch:Version:Release) is
0.2.2-number%{?dist}.
* Descriptsion
  - contains too long line (must not exceed 79 characters)
* Arch
  - This package contains only text files and architecture must be
noarch.
* Macros
  - Please use macros (please check %_sysconfdir, %_mandir,
%_initrddir for example)
* scripts
---
mkdir -p %{buildroot}/etc/rc.d/rc3.d %{buildroot}/etc/rc.d/rc5.d/
---
  - Why is this needed?
* Permission
  - The permissions of the following files are incorrect.
-
E: profugus script-without-shebang /etc/profugus.conf
W: profugus spurious-executable-perm /usr/share/man/man8/profugus.8.gz
-
* File entry
  - /etc/profugus.conf is listed twice.
* Requires
  - Add proper %Requires(post), etc (check: Services subsection of
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets )
* defattr
  Now we recommend: %defattr(-,root,root,-)

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

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


[Bug 246227] Review Request: brutus-keyring - small and simple keyring daemon based upon libgcrypt

2007-07-02 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: brutus-keyring - small and simple keyring daemon based 
upon libgcrypt
Alias: brutus-keyring

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 03:40 EST ---
(In reply to comment #13)
 No, I've noticed that, but I would really like to keep 
 the header if it isn't to annoying. 

Well, okay. I don't force you to remove them.

---
Requires:ORBit2*-devel*
---
  - Oops.. I just wanted to emphasize -devel.
The name is ORBit2-devel. Sorry for confusing you..
  - By the way did you modify the Source tarball itself silently?

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

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


[Bug 246432] New: Review Request: blitz : C++ class library for matrix scientific computing

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




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

   Summary: Review Request: blitz : C++ class library for matrix
scientific computing
   Product: Fedora
   Version: devel
  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://astro-sfg.fis.ucm.es/~spr/blitz.spec
SRPM URL: http://astro-sfg.fis.ucm.es/~spr/blitz-0.9-0.1.src.rpm
Description: Blitz++ is a C++ class library for scientific computing which 
provides performance on par with Fortran 77/90. It uses template techniques to 
achieve high performance. Blitz++ provides dense arrays and vectors, random 
number generators, and small vectors

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

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


[Bug 244346] Review Request: mailutils - Collection of GNU mail-related utilities

2007-07-02 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: mailutils - Collection of GNU mail-related utilities
Alias: mailutils-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 05:14 EST ---
(In reply to comment #11)

Patrice: Much thanks for your comments.

 You don't use some features, like
 libgsasl, guile, or emacs-mh. I think these should be shipped.

That sounds sane.  I will add those.

 I think it is bad to conflict with nmh. I think that
 you should use 
 --with-mh-bindir
 instead.

This was intentional.  If you look at the spec file, I already use
--with-mh-bindir=%{_bindir}, that overrides the default bin/mh location
of mh compatibility files.  The original reason was that rpmlint objected
having a subdirectory of bin.  I tend to agree with rpmlint -- I would not
call having binaries such weird places as subdirectories of bin a good
practice.  Moreover, I think there is no use of having both mh-s installed,
at a time, and conflicts are about the best and most standard way to deal
with situations such as this.

If not conflicting with nmh is important for you, and you insist on removing
the conflict, please either suggest an alternative resolution, or explain
where would you put the conflicting files and why.

 Some suggestions:
 
 * put the %post and friends before the %files section. There is no
   reason, except that it is how all the other spec files are done
   in fedora.

That's fair, I'll move that.

 * use wildcards for man pages and info pages in %files, like
 %{_infodir}/%{name}.info*
 
 %{_mandir}/man1/popauth.1*
 
 * don't use .gz in the install-info %preun, install-info is able to
   figure out itself what compression it should use.

I was already wondering wheter manual pages are always gzipped.  Thus,
thanks for the explanation and be sure I'll change it.  I will probably have
to change it also in some other packages.

 * use 
 rm $RPM_BUILD_ROOT/%{_libdir}/%{name}/*.a
 instead of
 rm -f $RPM_BUILD_ROOT/%{_libdir}/%{name}/*.a
 that way build will break if something changed upstream.

Makes sense.  Will change that.

 * It seems to me that COPYING.LESSER should be in libs %doc
   I also guess that devel subpackage is LGPL licensed.
   I would also add COPYING in all the subpackages, since 
   the doc subpackage may not be installed.
 
   Alternatively you could have all packages depend on -doc
   since it is tiny, and more like a common package that really
   a doc package in my opinion.

I was thinking about the second option already, though it was for different
reasons and first one also makes sense.  I'll think for a while about this
and pick one of the solutions for the next revision of the package.

 * since this package is heavily split it would be nice, in my 
   opinion, to try to have a similar layout than in other repos. 
   I haven't really found other repos packaging mailutils, except
   for alt linux, but I may have missed something. Maybe you could
   use the split from the upstream specfile, as long as it makes sense
   (no -dev subpackage, for example).

Will look at this later.

The revised packages with most of your suggestions incorporated will be
available either in today's evening (CET), or tomorrow.

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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 05:31 EST ---
I added a note to bug 239004.

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

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


[Bug 243951] Review Request: pbm2l7k - Converts PBM stream to Lexmark 7000, 7200 and 5700 printer language

2007-07-02 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: pbm2l7k - Converts PBM stream to Lexmark 7000, 7200 
and 5700 printer language
Alias: pbm2l7k-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 05:48 EST ---
Thanks Marek and Jason. The package was imported into CVS.

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

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


[Bug 226294] Merge Review: php

2007-07-02 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: php
Alias: php

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


Bug 226294 depends on bug 246210, which changed state.

Bug 246210 Summary: php-dbase conflict with php-common
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=246210

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED



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

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


[Bug 243947] Review Request: c2050 - Converts bitcmyk data to Lexmark 2050 printer language

2007-07-02 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: c2050 - Converts bitcmyk data to Lexmark 2050 printer 
language
Alias: c2050-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 05:57 EST ---
Thanks Marek and Kevin. The package was imported into CVS.

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

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


[Bug 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets

2007-07-02 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: R-widgetTools-1.12.0-2 - Tools to support the 
construction of tcltk widgets


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 05:59 EST ---
I discovered by testing the package that I made some mistakes on the files (I
had lost the html interface)


There is a corrected version who is working:

SPEC: http://pingoured.dyndns.org/public/RPM/R-widgetTools/R-widgetTools.spec
SRPM:
http://pingoured.dyndns.org/public/RPM/R-widgetTools/R-widgetTools-1.12.0-7.fc6.src.rpm

Thanks

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

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


[Bug 243831] Review Request: rsyslog - the enhanced syslogd for Linux and Unix

2007-07-02 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: rsyslog - the enhanced syslogd for Linux and Unix


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 06:12 EST ---
Kevin, are there any news after last FESCo discussion?

(In reply to comment #37)
 cvs done. 
 
 Please note that FESCo is planning on discussing if rsyslog should replace
 sysklogd in it's thursday meeting. You might hold off on building until 
after
 that. Also, be very carefull when pushing to F-7. I don't think we want to
 replace sysklogd in a released version. 



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

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


[Bug 246227] Review Request: brutus-keyring - small and simple keyring daemon based upon libgcrypt

2007-07-02 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: brutus-keyring - small and simple keyring daemon based 
upon libgcrypt
Alias: brutus-keyring

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 06:42 EST ---
(In reply to comment #14)
 (In reply to comment #13)
  No, I've noticed that, but I would really like to keep 
  the header if it isn't to annoying. 
 
 Well, okay. I don't force you to remove them.

Thanks :-)

 ---
 Requires:ORBit2*-devel*
 ---
   - Oops.. I just wanted to emphasize -devel.
 The name is ORBit2-devel. Sorry for confusing you..

Silly me...


   - By the way did you modify the Source tarball itself silently?

Sigh, yes. The spec file revision number didn't make it to the tar-ball version
string. I've now changed my ways and will increase the third version digit for
each spec file modification so that any source change will be reflected by the
tar-ball version.

New files here:

Spec URL:
http://www.omesc.com/sites/default/files/downloads/dist/brutus-keyring/Fedora%207/SPECS/brutus-keyring.spec

SRPM URL:
http://www.omesc.com/sites/default/files/downloads/dist/brutus-keyring/Fedora%207/SRPMS/brutus-keyring-0.9.1-1.fc7.src.rpm


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

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


[Bug 229180] Review Request: texlive-texmf - Architecture independent parts of the TeX formatting system

2007-07-02 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: texlive-texmf - Architecture independent parts of the 
TeX formatting system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 08:10 EST ---
I just uploaded the new texlive packages:

http://people.redhat.com/jnovy/files/texlive/

Main features/differences from the previous version:
1. dropped former texjive zip list for the texmf trees and texlive now uses the
zips generated from the scheme-tetex.tpm from upstream
2. reintroduced texlive-errata updating scheme

Tom, this means the pdflatex will work for you since fmtutil now regenerates all
fmt files correctly due to correct dependency resolution.

Seems like we have the first functional texlive release based on scheme-tetex 
now.

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

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


[Bug 242416] Review Request: texlive - Binaries for the TeX formatting system

2007-07-02 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: texlive - Binaries for the TeX formatting system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 08:19 EST ---
Matej, could you please try it again with the new (0.4) packages:

http://people.redhat.com/jnovy/files/texlive/

It should build successfully.

Main features/differences from the previous version:
1. dropped former texjive zip list for the texmf trees and texlive now uses the
zips generated from the scheme-tetex.tpm from upstream
2. reintroduced texlive-errata updating scheme


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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 08:50 EST ---
(In reply to comment #41)
 Okay,
 
 * For this package, the last remaining is (I just found):
 ---
 /usr/share/doc/ppl/NEWS:  ISO-8859 English text
 /usr/share/doc/ppl/TODO:  ISO-8859 English text
 ---
 Please change to UTF-8 (by iconv for example).

Done.  I have simply changed

http://www.cs.unipr.it/ppl/Download/ftp/releases/0.9/RedHat/FC6/ppl-0.9-docfiles.patch



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

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


[Bug 226291] Merge Review: perl-XML-Simple

2007-07-02 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: perl-XML-Simple


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 09:40 EST ---
I don't think you're doing anything wrong.  It looks like the package was in
extras pre-FC5, but was moved to core in FC5.  I'll request that the package be
added to those branches in CVS.  Thanks for catching this.

Package Change Request
==
Package Name: 
New Branches: FC-5, FC-6, F-7, devel

Guys, some of the branches for this package were apparently lost when extras was
merged.  Perhaps because the package went from extras - core.  I assume the
proper thing to do is request that it be added to those branches, like I did
above.  If not, please do whatever the right thing is.


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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 09:43 EST ---
Well, please create a new spec/srpm and upload anyway.

And please do a pre-review of other person's review request or
submit a new package review request.

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

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


[Bug 246460] New: qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

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




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

   Summary: qtpfsgui - A Qt4 graphical user interface that provides
a workflow for HDR imaging
   Product: Fedora
   Version: devel
  Platform: All
   URL: http://www.silfreed.net/download/repo/packages/qtpfsgui/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://www.silfreed.net/download/repo/packages/qtpfsgui/qtpfsgui.spec
SRPM URL: 
http://www.silfreed.net/download/repo/packages/qtpfsgui/qtpfsgui-1.8.9-2.src.rpm


%description
Qtpfsgui is a Qt4 graphical user interface that provides a workflow for
HDR imaging.


$ rpmlint RPMS/qtpfsgui-*1.8.9-2*.rpm SRPMS/qtpfsgui-1.8.9-2.src.rpm
$ echo $?
0

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

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


[Bug 246460] qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

2007-07-02 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: qtpfsgui - A Qt4 graphical user interface that provides a workflow for 
HDR imaging


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 246460] qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

2007-07-02 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: qtpfsgui - A Qt4 graphical user interface that provides a workflow for 
HDR imaging


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 10:01 EST ---
Currently working on bug 241403 as my primary sponsorship bug.

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

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


[Bug 220743] Review Request: postgis - Geographic Information Systems Extensions to PostgreSQL

2007-07-02 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: postgis - Geographic Information Systems Extensions to 
PostgreSQL


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 10:46 EST ---
Package Change Request
==
Package Name: postgis
New Branches : F-7 EL-5 

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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 10:48 EST ---
(In reply to comment #45)
 Well, please create a new spec/srpm and upload anyway.

New SPEC:
http://www.cs.unipr.it/ppl/Download/ftp/releases/0.9/RedHat/FC6/ppl.spec

New SRPM:
http://www.cs.unipr.it/ppl/Download/ftp/releases/0.9/RedHat/FC6/ppl-0.9-9.src.rpm

 And please do a pre-review of other person's review request or
 submit a new package review request.

I am trying to select a project for which I have a minimum of qualification, but
I am having problems: all such projects seem to have requests that are either
stuck (e.g., https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227933,
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227570) or by people that
is unlikely to need my reviewe (e.g., 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208679).
I am confused about how to proceed: any hints?
Meanwhile I am reading again the packaging guidelines.



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

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


[Bug 226387] Merge Review: samba

2007-07-02 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: samba


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 11:38 EST ---
It seems that winbind name changes must be applied for all releases...

Consider the system boot/shutdown process. According to /etc/inittab,
l0:0:wait:/etc/rc.d/rc 0 or
l6:6:wait:/etc/rc.d/rc 6

In /etc/rc.d/rc script we have such a fragment:

---start here
# First, run the KILL scripts.
for i in /etc/rc$runlevel.d/K* ; do
check_runlevel $i || continue

# Check if the subsystem is already up.
subsys=${i#/etc/rc$runlevel.d/K??}
[ -f /var/lock/subsys/$subsys -o -f /var/lock/subsys/$subsys.init ] \
|| continue

# Bring the subsystem down.
if LC_ALL=C egrep -q (killproc |action ) $i ; then
$i stop
else
action $Stopping $subsys:  $i stop
fi
done

---end here---

For winbind we have /etc/rc.d/rc0.d/K35winbind symlink. The subsys value,
computed from this filename, is winbind. But when we check
/var/lock/subsys/$subsys , we find nothing, as /var/lock/subsys/winbindd
exists there (double-d at the end instead of one).

IOW, it seems that with winbindd name, the init script will not be invoked for
stop at halt/reboot . And the similar when changing between runlevels (from 3
to 5 etc.)



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

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


[Bug 226387] Merge Review: samba

2007-07-02 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: samba


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #154749|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 11:42 EST ---
Created an attachment (id=158345)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=158345action=view)
.spec file changes to work wiith winbindd to winbind renaming

This variant seems more safe. First, just duplicate new subsys lock, if old
is present, then remove the old lock, if it still exists.

No idea what about FC6 (and RHEL?) .

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

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


[Bug 243831] Review Request: rsyslog - the enhanced syslogd for Linux and Unix

2007-07-02 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: rsyslog - the enhanced syslogd for Linux and Unix


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 11:52 EST ---
(In reply to comment #39)

There was high level discussion about features in general, but nothing specific
to rsyslog. ;( I will ask for it to be added to the schedule for this week. 

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

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


[Bug 235501] Review Request: jsdoc - Produces javadoc-style documentation from JavaScript sourcefiles

2007-07-02 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: jsdoc - Produces javadoc-style documentation from 
JavaScript sourcefiles


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:07 EST ---
JSDoc-1.10.2-4.fc7 has been pushed to the Fedora 7 testing repository.  If 
problems still persist, please make note of it in this bug report.

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

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


[Bug 221884] Review Request: pyBackPack (GTK+ Python backup tool)

2007-07-02 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: pyBackPack (GTK+ Python backup tool)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:11 EST ---
pybackpack-0.5.1-2.fc7 has been pushed to the Fedora 7 stable repository.  If 
problems still persist, please make note of it in this bug report.

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

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


[Bug 221884] Review Request: pyBackPack (GTK+ Python backup tool)

2007-07-02 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: pyBackPack (GTK+ Python backup tool)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |ERRATA
   Fixed In Version||0.5.1-2.fc7




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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:11 EST ---
Well, would you choose from newer review requests of others,
not choosing older ones? (Currently the newest review requests is
bug 246460)


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

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


[Bug 245080] Review Request: libntlm - library that implement NTLM authentication derived from Samba sources.

2007-07-02 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: libntlm - library that implement NTLM authentication  
derived from Samba sources.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:12 EST ---
libntlm-0.3.13-3.fc7 has been pushed to the Fedora 7 stable repository.  If 
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are 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 245080] Review Request: libntlm - library that implement NTLM authentication derived from Samba sources.

2007-07-02 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: libntlm - library that implement NTLM authentication  
derived from Samba sources.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |ERRATA
   Fixed In Version||0.3.13-3.fc7




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

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


[Bug 239428] Review Request: slim - Simple Login Manager

2007-07-02 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: slim - Simple Login Manager


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:16 EST ---
Ping Andres,

Please put a CVS Admin request on this bug.

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

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


[Bug 171801] Review Request: libspectrum, lib765 and libdsk - libraries required for the fuse-emulator

2007-07-02 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: libspectrum, lib765 and libdsk - libraries required 
for the fuse-emulator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|fedora-extras-  |
   |[EMAIL PROTECTED] |
 CC||fedora-package-
   ||[EMAIL PROTECTED]
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |
 CC||fedora-extras-
   ||[EMAIL PROTECTED]
   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 12:58 EST ---
Package Change Request
==
Package Name: libspectrum
Updated Fedora Owners: [EMAIL PROTECTED],[EMAIL PROTECTED]


Package Change Request
==
Package Name: libdsk
Updated Fedora Owners: [EMAIL PROTECTED],[EMAIL PROTECTED]


Package Change Request
==
Package Name: lib765
Updated Fedora Owners: [EMAIL PROTECTED],[EMAIL PROTECTED]


Note, the review requests appear to have taken place for all three packages
under the same Review bug, so I'm placing all 3 CVS requests here. The current
maintainer ([EMAIL PROTECTED]) appears to be AWOL at the moment due to
personal reasons. I was given explicit permission to take over fuse-emulator
and fuse-emulator-libs but lib765, libdsk and libspectrum also need updating
in order to maintain the former. The original maintainer should be left as
co-maintainer in case he wishes to resume in the future.

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

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


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

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

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


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 13:04 EST ---
OK: I will try to write a review for

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


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

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


[Bug 239428] Review Request: slim - Simple Login Manager

2007-07-02 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: slim - Simple Login Manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 13:45 EST ---
New Package CVS Request
===
Package Name: slim
Short Description: Simple Login Manager
Owners: [EMAIL PROTECTED]
Branches: F-7
InitialCC: 


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

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


[Bug 246386] Review Request: ois - Open Input System, OO gaming input library

2007-07-02 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: ois - Open Input System, OO gaming input library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 199405] Review Request: vtk - The Visualization Toolkit - A high level 3D visualization library

2007-07-02 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: vtk - The Visualization Toolkit - A high level 3D 
visualization library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 13:52 EST ---
Axel -

  Are you sure the %bcond_with/without macros work in the Fedora build system? 
Don't seem to do anything for me in mock.

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

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


[Bug 241403] Review Request: qgis - A user friendly Open Source Geographic Information System

2007-07-02 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: qgis - A user friendly Open Source Geographic 
Information System


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:03 EST ---
Now I am sponsoring you.

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

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


[Bug 246460] qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

2007-07-02 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: qtpfsgui - A Qt4 graphical user interface that provides a workflow for 
HDR imaging


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:05 EST ---
Being sponsored by [EMAIL PROTECTED] (bug 241403)

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

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


[Bug 246460] qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

2007-07-02 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: qtpfsgui - A Qt4 graphical user interface that provides a workflow for 
HDR imaging


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:09 EST ---
Very rapid comment:
* Don't write the packages to Requires explicitly which are selected
  by the dependency check of the libraries checked by rpmbuild itself.
  Perhaps all the Requires currently written is not needed.

( Currently I may have no time to review this)

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

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


[Bug 246046] gtksourceview2 - GtkSourceView v2.x

2007-07-02 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: gtksourceview2 - GtkSourceView v2.x


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:26 EST ---
 These are a bit strange.  Upstream has these in their Makefile.am as::
  languages_DATA = $(LANGUAGES) language.rng language2.rng language.dtd \
  check.sh convert.py

After consulting upstream, I've removed these.

 * Missing BuildRequire: pcre-devel

I've patched configure now to use the GRegex provided by GLib 2.13.x,
and it builds fine in mock without pcre-devel


 * Only a subset of the *.lang files pass the check.sh script. 

The check script fails for old-format language specs. The library can handle
both formats.


 In this case, the Guidelines specify that the latest version should be
 [basename] and previous versions should be [basename][version]. 

The guidelines are misguided here...

New versions up at 

http://people.redhat.com/mclasen/gtksourceview/gtksourceview2.spec
http://people.redhat.com/mclasen/gtksourceview/gtksourceview2-1.90.1-4.fc8.src.rpm



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

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


[Bug 246351] Review Request: libopensync-plugin-gnokii - plugin for using gnokii with opensync

2007-07-02 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: libopensync-plugin-gnokii - plugin for using gnokii 
with opensync


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 246289] Review Request: atanks - Remake of a classic DOS game Scorched Earth

2007-07-02 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: atanks - Remake of a classic DOS game Scorched Earth


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:31 EST ---
cvs done.

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

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


[Bug 197488] Review Request: uread - Utilities for unformatted fortran files

2007-07-02 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: uread - Utilities for unformatted fortran files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:35 EST ---
cvs done.

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

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


[Bug 222648] Review Request: audacious-plugin-fc - Future Composer plugin for Audacious

2007-07-02 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: audacious-plugin-fc - Future Composer plugin for 
Audacious


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:39 EST ---
cvs done.

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

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


[Bug 246302] Review Request: sub2srt - Convert files in .sub format to .srt

2007-07-02 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: sub2srt - Convert files in .sub format to .srt


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:42 EST ---
cvs done.

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

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


[Bug 245431] Review Request: libtommath - portable number theoretic multiple-precision integer library

2007-07-02 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: libtommath -  portable number theoretic 
multiple-precision integer library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:47 EST ---
cvs done.

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

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


[Bug 246460] Review request: qtpfsgui - A Qt4 graphical user interface that provides a workflow for HDR imaging

2007-07-02 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: qtpfsgui - A Qt4 graphical user interface that 
provides a workflow for HDR imaging


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|qtpfsgui - A Qt4 graphical  |Review request: qtpfsgui - A
   |user interface that provides|Qt4 graphical user interface
   |a workflow for HDR imaging  |that provides a workflow for
   ||HDR imaging




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:47 EST ---
(In reply to comment #3)
 Very rapid comment:
 * Don't write the packages to Requires explicitly which are selected
   by the dependency check of the libraries checked by rpmbuild itself.
   Perhaps all the Requires currently written is not needed.

Sorry about that; fixed up now.

 ( Currently I may have no time to review this)

No problems.  Thanks for the sponsorship.


Spec URL: 
http://www.silfreed.net/download/repo/packages/qtpfsgui/qtpfsgui.spec
SRPM URL: 
http://www.silfreed.net/download/repo/packages/qtpfsgui/qtpfsgui-1.8.9-3.src.rpm

%changelog
* Mon Jul 02 2007 Douglas E. Warner [EMAIL PROTECTED] 1.8.9-3
- removing explicit Requires


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

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


[Bug 243631] Review Request: msmtp - an SMTP client

2007-07-02 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: msmtp - an SMTP client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:50 EST ---
cvs done. 

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

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


[Bug 245655] Review Request: perl-Gearman - Distributed job system

2007-07-02 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-Gearman - Distributed job system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:54 EST ---
cvs done.
Note that you are supposed to set the fedora-cvs flag to ?, not +. :) 

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

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


[Bug 242416] Review Request: texlive - Binaries for the TeX formatting system

2007-07-02 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: texlive - Binaries for the TeX formatting system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:57 EST ---
1.) Local build works for me (F7, x86_64)

2.) I don't unserstand the advantage of texlive-errata updating scheme?

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

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


[Bug 245697] Review Request: perl-Gearman-Server - Function call router and load balancer

2007-07-02 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-Gearman-Server - Function call router and load 
balancer
Alias: perl-Gearman-Server

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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 14:58 EST ---
Comment #3 looks like it's in the wrong review request? 

cvs done. 
Note again that you should set fedora-cvs to ? and a cvsadmin will set it to +
when done. 

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

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


[Bug 245432] Review Request: libtomcrypt - comprehensive, portable cryptographic toolkit

2007-07-02 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: libtomcrypt - comprehensive, portable cryptographic 
toolkit


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:00 EST ---
cvs done.

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

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


[Bug 229322] Review Request: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool from being a single point of failure

2007-07-02 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: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool 
from being a single point of failure


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:01 EST ---
SPEC provided and spec in SRPM difer:
[EMAIL PROTECTED] fedora]$ diff postgresql-pgpool-ha.spec
/usr/src/redhat/SPECS/postgresql-pgpool-ha.spec
14d13
 BuildArch:noarch

Which is right?

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

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


[Bug 235249] Review Request: vdrift-data-minimal - VDrift minimal data set

2007-07-02 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: vdrift-data-minimal - VDrift minimal data set


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:02 EST ---
Known issue, see vdrift review bug:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234490

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

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


[Bug 245743] Review Request: python-isprelink - Python module to determine if a file has been prelinked

2007-07-02 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: python-isprelink - Python module to determine if a 
file has been prelinked


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:05 EST ---
cvs done.

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

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


[Bug 222475] Review Request: sofia-sip - Sofia SIP User-Agent library

2007-07-02 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: sofia-sip - Sofia SIP User-Agent library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:07 EST ---
cvs done.

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

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


[Bug 229322] Review Request: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool from being a single point of failure

2007-07-02 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: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool 
from being a single point of failure


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:09 EST ---
Hello,

(In reply to comment #9)

 Which is right?

Ouch. BuildArch:  noarch is ok.

Regards, Devrim

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

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


[Bug 222478] Review Request: sofsip-cli - SIP VoIP/IM example client

2007-07-02 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: sofsip-cli - SIP VoIP/IM example client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:10 EST ---
cvs done.

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

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


[Bug 199405] Review Request: vtk - The Visualization Toolkit - A high level 3D visualization library

2007-07-02 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: vtk - The Visualization Toolkit - A high level 3D 
visualization library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:12 EST ---
Yes, the bcond macros could at most deactivate the Java builds, but the error is
due to the Java build being attempted. You can see from the build logs that the
bcond macros didn't mess up anything.

Could you reproduce the failure of the Java builds?


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

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


[Bug 229322] Review Request: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool from being a single point of failure

2007-07-02 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: pgpool-ha : Pgpool-HA uses heartbeat to keep pgpool 
from being a single point of failure


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:18 EST ---
Ok.

But. . .

[EMAIL PROTECTED] SPECS]$ rpmbuild -ba postgresql-pgpool-ha.spec
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.35249
+ umask 022
+ cd /usr/src/redhat/BUILD
+ LANG=C
+ export LANG
+ unset DISPLAY
+ cd /usr/src/redhat/BUILD
+ rm -rf pgpool-ha-1.1.0
+ /usr/bin/bzip2 -dc /usr/src/redhat/SOURCES/pgpool-ha-1.1.0.tar.bz2
+ tar -xf -
+ STATUS=0
+ '[' 0 -ne 0 ']'
+ cd pgpool-ha-1.1.0
++ /usr/bin/id -u
+ '[' 500 = 0 ']'
++ /usr/bin/id -u
+ '[' 500 = 0 ']'
+ /bin/chmod -Rf a+rX,u+w,g-w,o-w .
+ exit 0
Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.35249
+ umask 022
+ cd /usr/src/redhat/BUILD
+ cd pgpool-ha-1.1.0
+ LANG=C
+ export LANG
+ unset DISPLAY
+ CFLAGS='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic
-fasynchronous-unwind-tables'
+ export CFLAGS
+ CXXFLAGS='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic
-fasynchronous-unwind-tables'
+ export CXXFLAGS
+ FFLAGS='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic
-fasynchronous-unwind-tables'
+ export FFLAGS
++ find . -name config.guess -o -name config.sub
+ ./configure --build=i686-redhat-linux-gnu --host=i686-redhat-linux-gnu
--target=noarch-redhat-linux-gnu --program-prefix= --prefix=/usr
--exec-prefix=/usr --bindir=/usr/bin --sbindir=/usr/sbin --sysconfdir=/etc
--datadir=/usr/share --includedir=/usr/include --libdir=/usr/lib
--libexecdir=/usr/libexec --localstatedir=/var --sharedstatedir=/usr/com
--mandir=/usr/share/man --infodir=/usr/share/info --bindir=/usr/bin
--sysconfdir=/etc --mandir=/usr/share/man --libdir=/usr/lib
checking for a BSD-compatible install... /usr/bin/install -c
checking for /usr/lib/ocf/resource.d/heartbeat... yes
checking for psql... /usr/bin/psql
checking for pgpool... /usr/bin/pgpool
configure: creating ./config.status
config.status: creating src/Makefile
+ make -C src
make: Entering directory `/usr/src/redhat/BUILD/pgpool-ha-1.1.0/src'
sed -e '[EMAIL PROTECTED]@#/usr/bin/psql#g'  pgpool.monitor.in | \
sed -e '[EMAIL PROTECTED]@#/usr/bin/pgpool#g'  pgpool.monitor
sed -e '[EMAIL PROTECTED]@#/usr/bin/psql#g'  pgpool.in | \
sed -e '[EMAIL PROTECTED]@#/usr/bin/pgpool#g'  pgpool
make: Leaving directory `/usr/src/redhat/BUILD/pgpool-ha-1.1.0/src'
+ exit 0
Executing(%install): /bin/sh -e /var/tmp/rpm-tmp.52313
+ umask 022
+ cd /usr/src/redhat/BUILD
+ cd pgpool-ha-1.1.0
+ LANG=C
+ export LANG
+ unset DISPLAY
+ rm -rf /var/tmp/postgresql-pgpool-ha-1.1.0-1.fc7-root-limb
+ cd src
+ make install
/usr/bin/install -c -m 755 pgpool /usr/lib/ocf/resource.d/heartbeat/
/usr/bin/install: cannot create regular file
`/usr/lib/ocf/resource.d/heartbeat/pgpool': Permission denied
make: *** [install] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.52313 (%install)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.52313 (%install)


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

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


[Bug 245694] Review Request: php-pear-Phlickr - Phlickr is a PHP5 based api kit used with the Flickr API

2007-07-02 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: php-pear-Phlickr - Phlickr is a PHP5 based api kit 
used with the Flickr API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:14 EST ---
cvs done.

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

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


[Bug 199405] Review Request: vtk - The Visualization Toolkit - A high level 3D visualization library

2007-07-02 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: vtk - The Visualization Toolkit - A high level 3D 
visualization library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:47 EST ---
I could reproduce the java failure if I explicitly set with_java.  I fixed it by
removing %{?_smp_flags} from the make command.  Looks like a parallel build
dependency issue - probably trying to compile the wrappers before they are all
generated.

Also, why not build with OSMesa?  I believe it is only FC-5 that didn't have
OSMesa support.  Compiles fine for me on devel.

Next, you might consider providing a -mpi version like is done with paraview. 
Probably not a big win though since I think most mpi users will compile their 
own.

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

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


[Bug 246386] Review Request: ois - Open Input System, OO gaming input library

2007-07-02 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: ois - Open Input System, OO gaming input library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 15:52 EST ---
Imported and build, closing.

Note I'm also building this F-7 updates-testing


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

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


[Bug 225669] Merge Review: ctags

2007-07-02 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: ctags


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:13 EST ---
+ rpmlint
W: ctags summary-ended-with-dot A C programming language indexing and/or
cross-reference tool.
E: ctags tag-not-utf8 %changelog
E: ctags non-utf8-spec-file ctags.spec
W: ctags summary-ended-with-dot A C programming language indexing and/or
cross-reference tool.
E: ctags tag-not-utf8 %changelog
E: ctags-debuginfo tag-not-utf8 %changelog

Comment:
 * fix utf-8 with iconv(1)
 * Both Summary and description should be reworked, they don't match the package
as of
   today.

+ naming : ok

+ guidelines
 * changelog is 10 years old (sic!), could be a be shorter?
 * disttag is missing
 * source url is wrong, see:
  
http://fedoraproject.org/wiki/Packaging/SourceURL#head-e27982f18a3bfd26b5b6ecbee113d2d8f3f006f2
 * I prefer this buildroot:
   %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
 * add  %{?_smp_mflags} to make
 * preserve timestamp on man files (cp -p/install -p)
 * don't use %makeinstall
  
http://fedoraproject.org/wiki/Packaging/Guidelines#head-fcaf3e6fcbd51194a5d0dbcfbdd2fcb7791dd002
 * don't shiop INSTALL
 the rest seems ok.

+ License: ok
+ American English: ok, however as noted the summary and description is outdated
and should be fixed.
+ legible: ok
+ src: md54sum: 9026a6c6950751bc4fd1be37e8a2070f : ok
+ build: ok (with correct build opts)
+ buildreq: ok
+ dirs and dups: ok
+ I prefer %defattr(...) as %defattr(-, root, root, -)
+ clean: ok
+ debuginfo: ok
Then a issue maybe outside the scope of this review, I really like to see etags
enabled and
included. Perhaps in a ctags-etags subpackage like this:

%package etags
Summary: Exuberant Ctags for emacs tag format
Group: Development/Tools
Requires: ctags = %{version}-%{release}
%description
This package will generate tags in a format which GNU Emacs understand,
it's a alternativ implementation of the GNU etags program.
Note: some command line options is not compatible with GNU etags.

Then change %configure line to include --enable-etags .

There are several other /usr/bin/etags coming from emacs and xemacs,
hence you must use alternatives to setup /usr/bin/etags.ctags as
alternativ. I can help with the details if you want, it goes
something like this:

move files (after install):

mv /usr/bin/etags to /usr/bin/etags.ctags
mv /usr/share/man/man1/etags.1 /usr/share/man/man1/etags.ctags.1

set up alternatives with slave (in %post and %preun):

alternatives --install /usr/bin/etags etags /usr/bin/etags.ctags 20 \
 --slave /usr/share/man/man1/etags.1.gz etags-etagsman
/usr/share/man/man1/etags.ctags.1.gz

alternatives --remove etags /usr/bin/etags.ctags

With low prio as 20, xemacs and emacs should ne affected.
and included the files with

%files etags
%defattr(-, root, root, -)
%doc COPYING
%{_bindir}/etags.ctags
%{_mandir}/man1/etags*






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

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


[Bug 199405] Review Request: vtk - The Visualization Toolkit - A high level 3D visualization library

2007-07-02 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: vtk - The Visualization Toolkit - A high level 3D 
visualization library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:14 EST ---
(In reply to comment #62)
 I could reproduce the java failure if I explicitly set with_java.

Yes, that's the default if you don't pass anything on the command line.

 I fixed it by removing %{?_smp_flags} from the make command.  Looks like a
 parallel build dependency issue

Argh, I hate %{?_smp_flags}. I only added it because I was hoping to get the
review happen due to comment #5. I should ask the FPC to forbid smp_flags.

Anyway thanks for spotting this!

 Also, why not build with OSMesa?

But, it's already built with it. The default it to build with java and OSMesa.

 Next, you might consider providing a -mpi version

Yes, but let's first make the first entry :)

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

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


[Bug 245743] Review Request: python-isprelink - Python module to determine if a file has been prelinked

2007-07-02 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: python-isprelink - Python module to determine if a 
file has been prelinked


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:17 EST ---
Packages imported and builds submitted.

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

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


[Bug 246524] New: Review Request: libquikgrid - The QuikGrid generator and contouring routine library

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




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

   Summary: Review Request: libquikgrid -  The QuikGrid generator
and contouring routine library
   Product: Fedora
   Version: devel
  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://miskatonic.cs.nmsu.edu/pub/libquikgrid.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/libquikgrid-4.0-1.src.rpm

Description:
This library contains the grid generator (called xpand) and contouring routine
(called contour) in the QuikGrid program.

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

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


[Bug 246525] New: Review Request: mini - A high-performance terrain rendering library

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




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

   Summary: Review Request: mini - A high-performance terrain
rendering library
   Product: Fedora
   Version: devel
  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://miskatonic.cs.nmsu.edu/pub/mini.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/mini-8.1-1.src.rpm

Description:
The Mini library is the core of the high-performance terrain renderer
which is described in the paper Real-Time Generation of Continuous Levels of 
Detail for Height Fields.

The Mini library applies a view-dependent mesh simplification scheme to render
large-scale terrain data at real-time. For this purpose, a quadtree 
representation of a height field is built. This quadtree is also utilized for 
fast view frustum culling and geomorphing.

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

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


[Bug 246526] New: Review Request: minizip - Minizip manipulates files from a .zip archive

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




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

   Summary: Review Request: minizip -  Minizip manipulates files
from a .zip archive
   Product: Fedora
   Version: devel
  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://miskatonic.cs.nmsu.edu/pub/minizip.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/minizip-1.2.3-1.src.rpm

Description:
Minizip manipulates files from a .zip archive

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

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


[Bug 246527] New: Review Request: vtp - VTP (Virtual Terrain Project) libraries and applications

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




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

   Summary: Review Request: vtp - VTP (Virtual Terrain Project)
libraries and applications
   Product: Fedora
   Version: devel
  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://miskatonic.cs.nmsu.edu/pub/vtp.spec
SRPM URL: http://miskatonic.cs.nmsu.edu/pub/vtp-070331-1.src.rpm

Description: 
The VTP (Virtual Terrain Project) software consists of a collection of
applications and libraries related to virtual terrain generation
and display. The main libraries include vtdata and vtlib, while
the primary applications are VTBuilder and Enviro.

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

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


[Bug 246524] Review Request: libquikgrid - The QuikGrid generator and contouring routine library

2007-07-02 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: libquikgrid -  The QuikGrid generator and contouring 
routine library
Alias: libquikgrid

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||libquikgrid




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

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


[Bug 246525] Review Request: mini - A high-performance terrain rendering library

2007-07-02 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: mini - A high-performance terrain rendering library
Alias: mini

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||mini




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

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


[Bug 246526] Review Request: minizip - Minizip manipulates files from a .zip archive

2007-07-02 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: minizip -  Minizip manipulates files from a .zip 
archive
Alias: minizip

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||minizip




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

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


[Bug 246527] Review Request: vtp - VTP (Virtual Terrain Project) libraries and applications

2007-07-02 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: vtp - VTP (Virtual Terrain Project) libraries and 
applications
Alias: vtp

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||246524, 246525, 246526
  Alias||vtp




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

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


[Bug 246525] Review Request: mini - A high-performance terrain rendering library

2007-07-02 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: mini - A high-performance terrain rendering library
Alias: mini

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||246527
  nThis||




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

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


[Bug 246524] Review Request: libquikgrid - The QuikGrid generator and contouring routine library

2007-07-02 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: libquikgrid -  The QuikGrid generator and contouring 
routine library
Alias: libquikgrid

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||246527
  nThis||




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

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


[Bug 246526] Review Request: minizip - Minizip manipulates files from a .zip archive

2007-07-02 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: minizip -  Minizip manipulates files from a .zip 
archive
Alias: minizip

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||246527
  nThis||




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

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


[Bug 199405] Review Request: vtk - The Visualization Toolkit - A high level 3D visualization library

2007-07-02 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: vtk - The Visualization Toolkit - A high level 3D 
visualization library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:31 EST ---
(In reply to comment #63)
 (In reply to comment #62)
  Also, why not build with OSMesa?
 
 But, it's already built with it. The default it to build with java and OSMesa.
 

Hmm, guess I really just don't understand %bcond.

  Next, you might consider providing a -mpi version
 
 Yes, but let's first make the first entry :)

Yeah, yeah... :-)

BTW - I usually do make VERBOSE=1 with my cmake packages.

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

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


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

2007-07-02 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: tcl-tcldict - Tcl dictionary extensiuon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 217806] Review Request: tk-tktreectrl - flexible listbox widget for Tk

2007-07-02 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: tk-tktreectrl - flexible listbox widget for Tk


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 193712] Review Request: sos

2007-07-02 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: sos


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 231814] Review Request: avarice - Program for interfacing the Atmel JTAG ICE to GDB

2007-07-02 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: avarice - Program for interfacing the Atmel JTAG ICE 
to GDB


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||ail.com)




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

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


[Bug 231835] Review Request: uisp - Universal In-System Programmer for Atmel AVR and 8051

2007-07-02 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: uisp - Universal In-System Programmer for Atmel AVR 
and 8051


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||ail.com)




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

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


[Bug 243665] Review Request: perl-Geo-IP - Efficient GeoIP bindings for Perl

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

Summary: Review Request: perl-Geo-IP - Efficient GeoIP bindings for Perl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||tenment.com)




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

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


[Bug 231835] Review Request: uisp - Universal In-System Programmer for Atmel AVR and 8051

2007-07-02 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: uisp - Universal In-System Programmer for Atmel AVR 
and 8051


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |ail.com)|




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:56 EST ---
Thanks for the review! I am very busy this week, but will finish the package
during the weekend.

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

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


[Bug 231814] Review Request: avarice - Program for interfacing the Atmel JTAG ICE to GDB

2007-07-02 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: avarice - Program for interfacing the Atmel JTAG ICE 
to GDB


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |ail.com)|




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 16:57 EST ---
Thanks for the review! I am very busy this week, but will finish the package
during the weekend.

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

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


[Bug 245655] Review Request: perl-Gearman - Distributed job system

2007-07-02 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-Gearman - Distributed job system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 17:03 EST ---
Thanks Kev, I must have been sleeping

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

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


[Bug 245697] Review Request: perl-Gearman-Server - Function call router and load balancer

2007-07-02 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-Gearman-Server - Function call router and load 
balancer
Alias: perl-Gearman-Server

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


Bug 245697 depends on bug 245655, which changed state.

Bug 245655 Summary: Review Request: perl-Gearman - Distributed job system
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245655

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 246526] Review Request: minizip - Minizip manipulates files from a .zip archive

2007-07-02 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: minizip -  Minizip manipulates files from a .zip 
archive
Alias: minizip

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 17:03 EST ---
Hi
I think that since minizip is included in the contrib directory of zlib, you
should request the zlib maintainer to enable minizip...
Mostly because he may enable it sooner of later if possible...

This seems to be stated here:
http://www.winimage.com/zLibDll/minizip.html

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

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


[Bug 246356] Review Request: perl-Gearman-Client-Async - Asynchronous Client for the Gearman distributed job system

2007-07-02 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-Gearman-Client-Async - Asynchronous Client for 
the Gearman distributed job system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 17:15 EST ---
I added a BuildRequires for perl-Gearman-Server (#245697, will build that one 
toninght for rawhide) and 
enabled the tests, and 
all test pass in mock for devel.

New version:

http://rubenkerkhof.com/packages/perl-Gearman-Client-Async.spec
http://rubenkerkhof.com/packages/perl-Gearman-Client-Async-0.94-2.fc7.src.rpm

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

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


[Bug 245708] Review Request: scsi-target-utils - SCSI target daemon and tools

2007-07-02 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: scsi-target-utils  - SCSI target daemon and tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 17:21 EST ---
MUST:
 - Package meets naming and packaging guidelines - ***

Upstream package name appears to be 'tgt'. While scsi-target-utils is certainly
more informative, is there a particular reason for the disconnect? I also notice
the upstream project name is 'stgt' while upstream source is 'tgt'.

 - Spec file matches base package name. - OK
 - Spec has consistant macro usage. - OK
 - Meets Packaging Guidelines. - OK
 - License - OK
 - License field in spec matches - OK
 - License file included in package - ***

A copy of GPL2 is not included. Doesn't appear to be included upstream, either.

 - Spec in American English - OK
 - Spec is legible. - OK
 - Sources match upstream md5sum: - OK

 - Package needs ExcludeArch - No
 - BuildRequires correct - OK
 - Spec handles locales/find_lang - N/A
 - Package is relocatable and has a reason to be. - N/A
 - Package has %defattr and permissions on files is good. - N/A
 - Package has a correct %clean section. - OK
 - Package has correct buildroot
  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) - OK
 - Package is code or permissible content. - OK
 - Doc subpackage needed/used. - N/A
 - Packages %doc files don't affect runtime. - N/A

 - Headers/static libs in -devel subpackage. - N/A
 - Spec has needed ldconfig in post and postun - N/A
 - .pc files in -devel subpackage/requires pkgconfig - N/A
 - .so files in -devel subpackage. - N/A
 - -devel package Requires: %{name} = %{version}-%{release} - N/A
 - .la files are removed. - N/A

 - Package is a GUI app and has a .desktop file - N/A

 - Package compiles and builds on at least one arch. - OK
 - Package has no duplicate files in %files. - OK
 - Package doesn't own any directories other packages own. - OK
 - Package owns all the directories it creates. - OK
 - No rpmlint output. ***

Source rpm:

W: scsi-target-utils strange-permission tgtd.init 0755 - can be ignored

Binary rpm:
E: scsi-target-utils init-script-without-chkconfig-postin /etc/rc.d/init.d/tgtd
E: scsi-target-utils init-script-without-chkconfig-preun /etc/rc.d/init.d/tgtd

See below.

E: scsi-target-utils incoherent-subsys /etc/rc.d/init.d/tgtd tgtd]

Typo?

W: scsi-target-utils no-reload-entry /etc/rc.d/init.d/tgtd

See below.

W: scsi-target-utils incoherent-init-script-name tgtd

Not sure what to make of this warning.

 - final provides and requires are sane:

See below about chkconfig. Otherwise reasonable.

SHOULD Items:

 - Should build in mock. - OK
 - Should build on all supported archs - only tested x86_64
 - Should function as described. - did not test
 - Should have sane scriptlets. - ***

Missing the proper scriptlets for adding/removing scripts. See:

 http://fedoraproject.org/wiki/Packaging/ScriptletSnippets

See the section on chkconfig Requires: there as well; as it is now, they're 
wrong.

 - Should have subpackages require base package with fully versioned depend. - 
N/A
 - Should have dist tag - OK
 - Should package latest version - OK

Other comments:

1. Versioning - this uses 0.1 for the package version. Upstream versioning is
done by date; the package should reflect that.

2. The init script - various issues

1) doesn't actually do anything sane if it fails to start
2) doesn't have a reload entry (if it's possible)
3) doesn't use the proper LSB return codes

3. Xen support - is not built. Should it be?


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

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


[Bug 245697] Review Request: perl-Gearman-Server - Function call router and load balancer

2007-07-02 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-Gearman-Server - Function call router and load 
balancer
Alias: perl-Gearman-Server

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


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

2007-07-02 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: tcl-tcldict - Tcl dictionary extensiuon


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-07-02 17:41 EST ---
Imported and built on rawhide.  Thanks!

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

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


[Bug 246537] New: Review Request: perl-Test-Output - Utilities to test STDOUT and STDERR messages

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




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

   Summary: Review Request: perl-Test-Output - Utilities to test
STDOUT and STDERR messages
   Product: Fedora
   Version: devel
  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://ftp.kspei.com/pub/steve/rpms/perl-Test-Output/perl-Test-Output.spec
SRPM URL: http://ftp.kspei.com/pub/steve/rpms/perl-Test-Output-0.10-1.src.rpm
Description:
Test::Output provides a simple interface for testing output sent to STDOUT
or STDERR. A number of different utilies are included to try and be as
flexible as possible to the tester.

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

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


  1   2   >