fix buggy test in #29445

[dimpase]

configure:17249: checking whether hyperellcharpoly bug is fixed
  ***   too many arguments: ...lcharpoly(x^10+x^9+x^8+x,0*Mod(1,3))
  ***                                                   ^-----------
configure:17257: result: no; cannot use system pari/GP with known bug

also should fix the error in another ticket #29494 touching this stuff

CC: @mkoeppe @kliem @fchapoton @videlec @kedlaya

Component: build: configure

Author: Dima Pasechnik

Branch/Commit: bf8c1d9

Reviewer: Vincent Delecroix

Issue created by migration from https://trac.sagemath.org/ticket/29554

[dimpase]

comment:1

{{{
configure:17249: checking whether hyperellcharpoly bug is fixed
  ***   too many arguments: ...lcharpoly(x^10+x^9+x^8+x,0*Mod(1,3))
  ***                                                   ^-----------
configure:17257: result: no; cannot use system pari/GP with known bug

}}}

[dimpase]

comment:2

missing extra pair of []:

-        bug_check=$(echo "hyperellcharpoly([x^10 + x^9 + x^8 + x,0]*Mod(1,3))" | $GP -qf 2>> config.log)
+        bug_check=`echo "hyperellcharpoly([[x^10 + x^9 + x^8 + x,0]]*Mod(1,3))" | $GP -qf 2>> config.log`

[dimpase]

Author: Dima Pasechnik

[dimpase]

Branch: u/dimpase/packages/parifixes

[dimpase]

Commit: 526d15c

[dimpase]

New commits:

526d15c

fix buggy tests in #29445 and in #29494

[sagetrac-git]

Changed commit from 526d15c to bf8c1d9

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

bf8c1d9

use macro to check pari/gp deps

[videlec]

Attachment: config.log

[videlec]

comment:9

configure is now happy with my pari. However there is something that I don't understand in config.log line 2023. matpermanent is not found. Why is there a search for it if it does not matter?

[dimpase]

comment:10

Replying to @videlec:

configure is now happy with my pari. However there is something that I don't understand in config.log line 2023. matpermanent is not found. Why is there a search for it if it does not matter?

This is normal functioning of AC_SEARCH_LIBS macro.

matpermanent is used to detect that libpari is at least verision 2.11. First, an attempt is made to link without -lpari, adn it fails. The next attempt (line 2107), -lpari is added, and test passes.

[videlec]

comment:11

If you feel like testing on my computer was good enough for a review, you can set to positive review.

[videlec]

Reviewer: Vincent Delecroix

[dimpase]

comment:12

Thanks, I think it's good.

[kliem]

comment:13

Thanks for the quick fix.

I started test runs a minute ago: https://github.com/kliem/sage-test-27122/actions/runs/85835472.

I will leave a message here tomorrow either way, but I don't expect any trouble.

[kliem]

comment:14

Replying to @kliem:

Thanks for the quick fix.

I started test runs a minute ago: https://github.com/kliem/sage-test-27122/actions/runs/85835472.

I will leave a message here tomorrow either way, but I don't expect any trouble.

Yes. Seems to work fine.

[vbraun]

Changed branch from u/dimpase/packages/parifixes to bf8c1d9