[Bug 175198] Review Request: perl-Math-Pari

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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-Math-Pari


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





------- Additional Comments From paul@xxxxxxxxxxxx  2006-02-01 06:03 EST -------
(In reply to comment #1)
> 1) Would you mind just hardcoding the pari version? The current construct
>    appears to be error prone: for example, it doesn't handle the current pari
>    development version - 2.2.12 - or any other version component with more
>    than 1 digit.
> 
>   If I assume the Math::Pari version as 2.021201 it will output 2.212 as
>   the pari version (it should output 2.2.12).
> 
>   $ echo 2.021201 | perl -pi -e 'tr/0/./;s/\.+/./;s/\.[^.]*$//'
>   2.212

I'd really prefer to only have to change the one version number in the spec if
possible, since (a) it makes the spec easier to maintain, and (b) it should
ensure that I'm always using the "supported" version of the pari library.

I've updated the expression, which should get it right in all cases now I think.

> 2) There is also a new version available. Could you update?
> 
>    Diff from Math-Pari-2.010702 to Math-Pari-2.010703
>    http://search.cpan.org/diff?from=Math-Pari-2.010702&to=Math-Pari-2.010703

Done:
http://www.city-fan.org/~paul/extras/perl-Math-Pari/perl-Math-Pari-2.010703-1.src.rpm

Spec URL unchanged.


-- 
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-extras-list mailing list
fedora-extras-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-extras-list

[Index of Archives]     [Fedora General Discussion]     [Fedora Art]     [Fedora Docs]     [Fedora Package Review]     [Fedora Desktop]     [Big List of Linux Books]     [Yosemite Backpacking]     [KDE Users]

  Powered by Linux