[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #37 from Fedora Update System upda...@fedoraproject.org  
2009-10-03 14:55:09 EDT ---
givaro-3.2.15-0.2.rc1.fc10 has been pushed to the Fedora 10 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||3.2.15-0.2.rc1.fc10
 Resolution||ERRATA




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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #38 from Fedora Update System upda...@fedoraproject.org  
2009-10-03 15:06:06 EDT ---
givaro-3.2.15-0.2.rc1.fc11 has been pushed to the Fedora 11 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|3.2.15-0.2.rc1.fc10 |3.2.15-0.2.rc1.fc11




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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #36 from Fedora Update System upda...@fedoraproject.org  
2009-09-15 03:44:24 EDT ---
givaro-3.2.15-0.2.rc1.fc10 has been pushed to the Fedora 10 testing repository.
 If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update givaro'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F10/FEDORA-2009-9580

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #35 from Fedora Update System upda...@fedoraproject.org  
2009-09-15 03:43:21 EDT ---
givaro-3.2.15-0.2.rc1.fc11 has been pushed to the Fedora 11 testing repository.
 If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update givaro'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F11/FEDORA-2009-9573

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #33 from Fedora Update System upda...@fedoraproject.org  
2009-09-14 06:30:55 EDT ---
givaro-3.2.15-0.2.rc1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/givaro-3.2.15-0.2.rc1.fc11

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #34 from Fedora Update System upda...@fedoraproject.org  
2009-09-14 06:31:03 EDT ---
givaro-3.2.15-0.2.rc1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/givaro-3.2.15-0.2.rc1.fc10

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #32 from Kevin Fenzi ke...@tummy.com  2009-09-14 00:56:30 EDT ---
cvs done.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #29 from D Haley my...@yahoo.com  2009-09-12 02:26:34 EDT ---
SPEC URL : http://dhd.selfip.com/427e/givaro-2.rc1.spec
SRPM URL : http://dhd.selfip.com/427e/givaro-3.2.15-0.2.rc1.fc10.src.rpm 


F10:http://koji.fedoraproject.org/koji/taskinfo?taskID=1672622
F11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1672627

$ rpmlint `cat tmp | awk '{print $2}'`
givaro.i386: W: shared-lib-calls-exit /usr/lib/libgivaro.so.0.0.4
e...@glibc_2.0
givaro-devel.i386: W: no-documentation
givaro-static.i386: W: no-documentation
5 packages and 0 specfiles checked; 0 errors, 3 warnings.
$ cat tmp
Wrote: /home/makerpm/rpmbuild/SRPMS/givaro-3.2.15-0.2.rc1.fc10.src.rpm
Wrote: /home/makerpm/rpmbuild/RPMS/i386/givaro-3.2.15-0.2.rc1.fc10.i386.rpm
Wrote:
/home/makerpm/rpmbuild/RPMS/i386/givaro-devel-3.2.15-0.2.rc1.fc10.i386.rpm
Wrote:
/home/makerpm/rpmbuild/RPMS/i386/givaro-static-3.2.15-0.2.rc1.fc10.i386.rpm
Wrote:
/home/makerpm/rpmbuild/RPMS/i386/givaro-debuginfo-3.2.15-0.2.rc1.fc10.i386.rpm

changelog:
* Sun Sep 12 2009 D Haley mycae(a!t)yahoo.com - 3.2.15-0.2.rc1
- Change to GPL+ from GPL2 per bugzilla comment

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #30 from Martin Gieseking martin.giesek...@uos.de  2009-09-12 
03:07:14 EDT ---
Sorry for the overlooked license ambiguity. Since this has been fixed,
everything seems to be fine now.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


D Haley my...@yahoo.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #31 from D Haley my...@yahoo.com  2009-09-12 04:01:54 EDT ---
New Package CVS Request
===
Package Name: givaro
Short Description: C++ library for arithmetic and algebraic computations
Owners: mycae
Branches: F-10 F-11 EL-5
InitialCC: baz

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

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




--- Comment #24 from Martin Gieseking martin.giesek...@uos.de  2009-09-08 
04:20:41 EDT ---
Since you've already done a good job bringing this package in shape, there
isn't much left to do. If nobody else wants to do the re-review, I can do it
later today.
However, I'm not quite happy with the call of exit() in a shared library. This
is simply bad style and should be avoided by all means. I'm not sure about
Fedora's policy about this, though. 
Here is a patch that replaces the call of exit() by throwing an exception: 
http://mgieseki.fedorapeople.org/givaro/givaro-exit.patch

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #25 from Jussi Lehtola jussi.leht...@iki.fi  2009-09-08 05:21:23 
EDT ---
(In reply to comment #24)
 Since you've already done a good job bringing this package in shape, there
 isn't much left to do. If nobody else wants to do the re-review, I can do it
 later today.
 However, I'm not quite happy with the call of exit() in a shared library. This
 is simply bad style and should be avoided by all means. I'm not sure about
 Fedora's policy about this, though. 
 Here is a patch that replaces the call of exit() by throwing an exception: 
 http://mgieseki.fedorapeople.org/givaro/givaro-exit.patch  

Calling exit() is not forbidden (in fact it is quite common in scientific
packages), so you can omit the rpmlint warning. You should not apply any
patches that make functional changes, since that will break compatibility with
upstream. You can ask upstream to make the change, though...

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #26 from Martin Gieseking martin.giesek...@uos.de  2009-09-08 
05:52:24 EDT ---
Actually, throwing an exception doesn't change the code functionality in a
significant way. It still stops the application if it's not caught but provides
better memory handling. 
To me, using exit() in shared libraries is pretty amateurish style, especially
in combination with C++. I also haven't seen it in any of the scientific
packages I use. But anyway, if it's OK to leave it in the package, I can live
with it.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|al...@users.sourceforge.net |martin.giesek...@uos.de
   Flag|fedora-review?  |fedora-review+




--- Comment #27 from Martin Gieseking martin.giesek...@uos.de  2009-09-08 
14:04:09 EDT ---
Here comes the review.

$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm
givaro.x86_64: W: shared-lib-calls-exit /usr/lib64/libgivaro.so.0.0.4
e...@glibc_2.2.5
givaro-devel.x86_64: W: no-documentation
givaro-devel.x86_64: E: non-executable-script /usr/share/givaro/givaro-makefile
0644 /bin/sh
givaro-static.x86_64: W: no-documentation
5 packages and 0 specfiles checked; 1 errors, 3 warnings.

- call of exit() can be ignored according to comment #25
- missing docs in -devel and -static is OK
- #!/bin/sh in makefile already removed according to comment #23


-
keys used in following checklist:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
- GPLv2 boilerplates missing (upstream notified) 

[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: files containing the text of the license(s) must be included in %doc.
- COPYING added to %doc of base package

[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
- md5 hash is f0b754ee54bcac0e866f2979b57393ba

[+] MUST: The package MUST successfully compile and build into binary rpms.
- builds in koji
  https://koji.fedoraproject.org/koji/taskinfo?taskID=1662967

[.] MUST: If the package does not successfully compile, ...
[+] MUST: All build dependencies must be listed in BuildRequires
[.] MUST: The spec file MUST handle locales properly.
- no locales

[+] MUST: Packages containing .so files must call ldconfig in %post and
%postun.
- base package contains libgivaro.so.0.0.4
- ldconfig called in %post and %postun

[.] MUST: If the package is designed to be relocatable, ...
- package not relocatable

[+] MUST: A package must own all directories that it creates.
[+] MUST: A package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
- givaro-makefile contains #!/bin/sh that must be removed
- already done according to comment #23 

[+] MUST: %clean section must contain rm -rf %{buildroot}.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
- no large docs

[+] MUST: Files in %doc must not affect the runtime of the application.
[+] MUST: Header files must be in a -devel package.
[+] MUST: Static libraries must be in a -static package.
- libgivaro.a

[.] MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
- no .pc files

[+] MUST: .so without suffix must go in a -devel package.
- libgivaro.so

[+] MUST: devel packages must require  %{name} = %{version}-%{release}
[+] MUST: Packages must NOT contain any .la libtool archives
- .la file removed in %install

[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file
- no GUI

[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot}.
[+] MUST: All filenames in rpm packages must be valid UTF-8.


[+] SHOULD: The reviewer should test that the package builds in mock.
- builds in mock

[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
[+] SHOULD: The reviewer should test that the package functions as described.
- I tested some of the functions provided by the library and they worked as
expected
- givaro-config prints the correct flags

[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
call of ldconfig in %post and %postun

[+] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
- subpackage -static requires base and -devel package
- Requires: %{name} = %{version}-%{release} is redundant


The package is 

[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #28 from Jussi Lehtola jussi.leht...@iki.fi  2009-09-08 14:21:41 
EDT ---
Umh..

src/kernel/zpz/StaticElement.h states it is under LGPLv2+, the rest of the
files just say see the copyright file which doesn't exist. There is a COPYING
that contains the GPLv3 license, thus the license tag should read GPL+.

https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


D Haley my...@yahoo.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW




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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #18 from D Haley my...@yahoo.com  2009-08-22 23:04:23 EDT ---
Hi Conrad,


I have notified upstream via email wrt. to the exit call. They now provide
versioned tarballs at the following location:

http://www-lmc.imag.fr/CASYS/LOGICIELS/givaro/Downloads/:


Dear Prof. Roch and Gautier,

Recently givaro has been submitted for package review[1] for inclusion in the
Fedora linux repositories. However small error was noticed during an automated
source-code check. As a requirement of packaging, upstream (you) must be
notified of this error. 

File src/kernel/zpz/givzpz16table1.C is included in a shared library
(libgivaro.so), and Line 46 makes a call to the exit() function.

Calling exit() in a shared library results in incorrect de-allocation of system
resources.

If it is possible to update this file in the next version of givaro to not use
exit(), this would be most helpful.


[1] https://bugzilla.redhat.com/show_bug.cgi?id=475065

Kind Regards.

D. Haley

===

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #19 from Conrad Meyer kon...@tylerc.org  2009-08-22 23:08:49 EDT 
---
Hi, I also no longer have time for this package. If you would like to take it,
you are quite welcome. Otherwise, it should be closed.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #20 from D Haley my...@yahoo.com  2009-08-23 00:32:59 EDT ---
I'm happy to try to push this through

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #21 from Conrad Meyer kon...@tylerc.org  2009-08-23 00:38:54 EDT 
---
Ok, sounds good!

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #22 from D Haley my...@yahoo.com  2009-08-23 01:34:12 EDT ---
SPEC URL: http://dhd.selfip.com/427e/givaro.spec
SRPM URL: http://dhd.selfip.com/427e/givaro-3.2.15-0.1.rc1.fc10.src.rpm

F10: http://koji.fedoraproject.org/koji/taskinfo?taskID=1626651
F11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1626660

rpmlint:
$ rpmlint `cat tmp`
givaro.i386: W: shared-lib-calls-exit /usr/lib/libgivaro.so.0.0.4
e...@glibc_2.0
givaro-devel.i386: W: no-documentation
givaro-devel.i386: E: non-executable-script /usr/share/givaro/givaro-makefile
0644
givaro-static.i386: W: no-documentation
5 packages and 0 specfiles checked; 1 errors, 3 warnings.
$ cat tmp
/home/makerpm/rpmbuild/SRPMS/givaro-3.2.15-0.1.rc1.fc10.src.rpm
/home/makerpm/rpmbuild/RPMS/i386/givaro-3.2.15-0.1.rc1.fc10.i386.rpm
/home/makerpm/rpmbuild/RPMS/i386/givaro-devel-3.2.15-0.1.rc1.fc10.i386.rpm
/home/makerpm/rpmbuild/RPMS/i386/givaro-static-3.2.15-0.1.rc1.fc10.i386.rpm
/home/makerpm/rpmbuild/RPMS/i386/givaro-debuginfo-3.2.15-0.1.rc1.fc10.i386.rpm

Notes:
*Upstream has been notified about GPL headers, and as above, exit() usage
*Tarball has been re-downloaded from new download area on givaro website
*makefile has been placed into %{_datadir}/%{name}/
*Trailing slashes have been added
*givaro-config.in has been patched to produce correct output in case of
multiple flags

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #23 from D Haley my...@yahoo.com  2009-08-23 01:39:35 EDT ---
Whoops. Sorry for the noise, but I have fixed that E, with sed -i 's|#!
/bin/sh||'  $RPM_BUILD_ROOT%{_datadir}/%{name}/givaro-makefile in my current
spec.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #16 from D Haley my...@yahoo.com  2009-07-26 05:05:15 EDT ---
ping?

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #17 from Conrad Meyer kon...@tylerc.org  2009-07-26 05:42:17 EDT 
---
Pong. Haven't had time :(.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-31 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from D Haley my...@yahoo.com  2009-05-31 05:24:14 EDT ---
I recommend emailing the three lead devs, probably in the one email. Their
addresses are on each of their profile sites. I'm sure they would be happy to
help, considering the time it would have taken to write their system. Also
their input here would clear up a few matters relatively quickly.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


D Haley my...@yahoo.com changed:

   What|Removed |Added

 CC||my...@yahoo.com




--- Comment #9 from D Haley my...@yahoo.com  2009-05-30 23:05:39 EDT ---
Here is my shot at this.

General comments:
* givaro-makefile is a makefile and not an executable shell script. Remove the
$!/bin/sh from it and move the file into %doc.
*Why are the header files split into two sections (gmp++ and %{name})? This may
be related to a later point I make about the configure script.


*  MUST: rpmlint must be run on every package. The output should be posted in
the review.[1]]

$ rpmlint ../SRPMS/givaro-3.2.13-2.fc10.src.rpm
../RPMS/i386/givaro-3.2.13-2.fc10.i386.rpm givaro.spec 
givaro.i386: W: shared-lib-calls-exit /usr/lib/libgivaro.so.0.0.2
e...@glibc_2.0
2 packages and 1 specfiles checked; 0 errors, 1 warnings.

Please inform upstream that it is not a good idea to do this, or patch this
out (if possible). Offending file is: src/kernel/zpz/givzpz16table1.C: Line 46.
Could be replaced with an exception or some kind of return code propagation. No
idea what the easiest solution is. If upstream is informed that they shouldn't
do this, and replies with something sensible, I don't see this as a block.

* MUST: The package must be named according to the Package Naming Guidelines .
OK

* MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
OK

* MUST: The package must meet the Packaging Guidelines .
As far as I can see it is OK.

* MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
OK
* MUST: The License field in the package spec file must match the actual
license. 


* MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
OK: Please consider adding AUTHORS and ChangeLog (not a blocker)
* MUST: The spec file must be written in American English.
OK
* MUST: The spec file for the package MUST be legible. 
OK
* MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
 FAIL: URL provided gives 404 error. (The requested URL
/CASYS/LOGICIELS/givaro/givaro-3.2.13.tar.gz was not found on this server.)
Please update URL or request upstream to not remove old tarballs.
* MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
 OK. Was able to compile.
 * MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. 
N/A
* MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
OK.
* MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
OK
* MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. 
OK
* MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. 
N/A
* MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. 
OK
* MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK
* MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line. 
OK
* MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
OK
* MUST: Each package must consistently use macros. 
OK
* MUST: The package must contain code, or permissable 

[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Conrad Meyer kon...@tylerc.org  2009-05-30 23:41:12 EDT 
---
(In reply to comment #9)
 Here is my shot at this.
 
 General comments:
 * givaro-makefile is a makefile and not an executable shell script. Remove the
 $!/bin/sh from it and move the file into %doc.

Why should we put this in %doc? How is it remotely helpful?

 *Why are the header files split into two sections (gmp++ and %{name})? This 
 may
 be related to a later point I make about the configure script.

gmp++ is C++ bindings for gmp, I think.

 *  MUST: rpmlint must be run on every package. The output should be posted in
 the review.[1]]
 
 $ rpmlint ../SRPMS/givaro-3.2.13-2.fc10.src.rpm
 ../RPMS/i386/givaro-3.2.13-2.fc10.i386.rpm givaro.spec 
 givaro.i386: W: shared-lib-calls-exit /usr/lib/libgivaro.so.0.0.2
 e...@glibc_2.0
 2 packages and 1 specfiles checked; 0 errors, 1 warnings.
 
 Please inform upstream that it is not a good idea to do this, or patch 
 this
 out (if possible). Offending file is: src/kernel/zpz/givzpz16table1.C: Line 
 46.
 Could be replaced with an exception or some kind of return code propagation. 
 No
 idea what the easiest solution is. If upstream is informed that they shouldn't
 do this, and replies with something sensible, I don't see this as a block.

This is poor coding practice by upstream, but it isn't a blocker.

 ...
 Please consider adding AUTHORS and ChangeLog (not a blocker)

Ok (though this is not something I feel strongly about).

 ...
 * MUST: The sources used to build the package must match the upstream source,
 as provided in the spec URL. Reviewers should use md5sum for this task. If no
 upstream URL can be specified for this package, please see the Source URL
 Guidelines for how to deal with this.
  FAIL: URL provided gives 404 error. (The requested URL
 /CASYS/LOGICIELS/givaro/givaro-3.2.13.tar.gz was not found on this server.)
 Please update URL or request upstream to not remove old tarballs.

Again, upstream's fault...

 ...
 * SHOULD: The reviewer should test that the package functions as described. A
 package should not segfault instead of running, for example.
 Would it be better to patch the -config file to return 
 /usr/include/givaro/
 for --cflags rather than /usr/include ? Also it would be good if 
 `givaro-config
 --cflags --libs` did not return endlines. (add -n to lines 58 and 62, also add
 leading space to linker  include flags). Finally this may be because of the
 gmp++ bit?

Something like that sounds necessary, yes.

 The examples given on the website are a bit bogus -- requires some
 preprocessor that doesn't exist (404 again). Could you pack a trivial example
 that compiles into doc directory?

Sorry, I'm not familiar with the use of this library, I'm just interested in
getting SAGE (and dependencies) packaged and in Fedora.

 ...

Thanks for the review!

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from D Haley my...@yahoo.com  2009-05-31 01:00:28 EDT ---
Some quick replies:

 Why should we put this in %doc? How is it remotely helpful?
Maybe %doc is not quite the right place, but as far as I can see it is only an
example makefile, hence documentation, or similar. It certainly should not be
executable, nor should it be mislabelled as a shell script and definitely does
not belong in /usr/bin.

 This is poor coding practice by upstream, but it isn't a blocker.
Still, please try to contact upstream and if they reply put a copy of the
relevant reply here. or state that they are non-responsive. I note that te

 Again, upstream's fault...
This is a blocker. I cannot confirm the md5sum without this, which is required
for review - we need to build against an upstream tarball somewhere, and agree
on its location. http://www-lmc.imag.fr/CASYS/LOGICIELS/givaro/Downloads/ may
have what we need.

Sorry, I'm not familiar with the use of this library, I'm just interested in
getting SAGE (and dependencies) packaged and in Fedora.

I need some way to confirm that the system is working properly -- do you have a
suggestion? 

gmp++ is C++ bindings for gmp, I think
So is this two separate projects bundled into one? I am totally confused.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from D Haley my...@yahoo.com  2009-05-31 01:31:50 EDT ---
Additional:

None of the source files have a proper GPL header, as documented in the GPL
licence How to Apply These Terms to Your New Programs. Please notify
upstream. I am uncertain as to whether this is a blocker or not.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Conrad Meyer kon...@tylerc.org  2009-05-31 01:37:26 EDT 
---
(In reply to comment #12)
 Additional:
 
 None of the source files have a proper GPL header, as documented in the GPL
 licence How to Apply These Terms to Your New Programs. Please notify
 upstream. I am uncertain as to whether this is a blocker or not.  

Upstream doesn't have any sort of bugtracking system or email contact I can
find (other than directly emailing all six developers).

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

2009-05-30 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Conrad Meyer kon...@tylerc.org  2009-05-31 01:36:43 EDT 
---
(In reply to comment #11)
 Some quick replies:
 
  Why should we put this in %doc? How is it remotely helpful?
 Maybe %doc is not quite the right place, but as far as I can see it is only an
 example makefile, hence documentation, or similar. It certainly should not be
 executable, nor should it be mislabelled as a shell script and definitely does
 not belong in /usr/bin.

Ah, sorry, didn't realize I was putting it in /usr/bin. I would prefer to not
install it anywhere, I think.

  This is poor coding practice by upstream, but it isn't a blocker.
 Still, please try to contact upstream and if they reply put a copy of the
 relevant reply here. or state that they are non-responsive. I note that te

Sorry, I don't see any sort of contact info or bug tracking system on the home
page.

  Again, upstream's fault...
 This is a blocker. I cannot confirm the md5sum without this, which is required
 for review - we need to build against an upstream tarball somewhere, and agree
 on its location. http://www-lmc.imag.fr/CASYS/LOGICIELS/givaro/Downloads/ may
 have what we need.

Eh, it won't be an issue *after* review though. I can update to the most recent
tarball and you can verify against that md5sum (or, please, sha1sum).

 Sorry, I'm not familiar with the use of this library, I'm just interested in
 getting SAGE (and dependencies) packaged and in Fedora.
 
 I need some way to confirm that the system is working properly -- do you have 
 a
 suggestion? 

Maybe look at some of their examples on the webpage?

 gmp++ is C++ bindings for gmp, I think
 So is this two separate projects bundled into one? I am totally confused.  

It's developed by the givaro folks, and they distribute it with givaro. I don't
believe it's a library that anyone else is supposed to use (at least not yet).
But you can try asking them if you want a better explanation.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #8 from Conrad Meyer kon...@tylerc.org  2009-03-13 01:15:45 EDT 
---
Alex: I didn't respond to comment #3 because that is a preference thing I and I
prefer it the way I have it. I would welcome a review.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Alex Lancaster al...@users.sourceforge.net changed:

   What|Removed |Added

 AssignedTo|al...@users.sourceforge.net |ti...@math.uh.edu




--- Comment #5 from Alex Lancaster al...@users.sourceforge.net  2009-03-09 
23:13:44 EDT ---
Reassigning to tibbs, given that he's set the review flag... ;)  

tibbs: feel free to reassign back to me if you don't end up doing the review.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

 AssignedTo|ti...@math.uh.edu   |al...@users.sourceforge.net




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-03-10 00:32:05 EDT 
---
Erm, what?  I'm just going through and setting review flags on the tickets
where this seems to have been forgotten.  (Basically, any review ticket that is
assigned to someone.)  Doing bugzilla cleanup doesn't mean that I actually want
to do the review.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #7 from Alex Lancaster al...@users.sourceforge.net  2009-03-10 
01:25:37 EDT ---
(In reply to comment #6)
 Erm, what?  I'm just going through and setting review flags on the tickets
 where this seems to have been forgotten.  (Basically, any review ticket that 
 is
 assigned to someone.)  Doing bugzilla cleanup doesn't mean that I actually 
 want
 to do the review.  

OK, I had assigned it to me because I had intended to do the review at some
point in the future, but I don't like to set the flag until I'm actually in the
process of doing the review in case somebody else wants to take it if I don't
get around to it.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Alex Lancaster al...@users.sourceforge.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||al...@users.sourceforge.net
 AssignedTo|nob...@fedoraproject.org|al...@users.sourceforge.net




--- Comment #4 from Alex Lancaster al...@users.sourceforge.net  2009-02-19 
04:30:08 EDT ---
Can you update taking into account comment #3? and I'll review it.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #3 from Michael Schwendt bugs.mich...@gmx.net  2009-02-11 
06:44:16 EDT ---
Just a comment, not a review: In your %files sections, you're advised to mark
recursively included paths with a trailing slash. That makes it trivial to
distinguish files and directories.

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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


Conrad Meyer kon...@tylerc.org changed:

   What|Removed |Added

 Blocks||476299




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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #2 from Rex Dieter rdie...@math.unl.edu  2008-12-11 14:55:10 EDT 
---
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=993866

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

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


[Bug 475065] Review Request: givaro - C++ library for arithmetic and algebraic computations

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


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





--- Comment #1 from Conrad Meyer [EMAIL PROTECTED]  2008-12-07 19:14:04 EDT 
---
New URLs:
http://konradm.fedorapeople.org/fedora/SPECS/givaro.spec
http://konradm.fedorapeople.org/fedora/SRPMS/givaro-3.2.13-2.fc9.src.rpm

Fixed a multilibs conflict (%{_includedir}/givaro-config.h).

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

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