Tides malfunctions in version 7.0beta

[miguelmarco]

As was pointed in #19781, tides_mpfr does not work properly in the last beta.

I could trace the problem to the loading of libgmp. I attach a branch that seems to fix it, but it could be that now it is using system's libgmp/libmpfr .

Can somebody test it in a platform without libgmp installed system wide?

CC: @vbraun @jdemeyer

Component: packages: optional

Keywords: tides

Author: Miguel Marco

Branch: 0396add

Reviewer: Volker Braun

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

[miguelmarco]

Branch: u/mmarco/tides_malfunctions_in_version_7_0beta

[miguelmarco]

Changed keywords from none to tides

[miguelmarco]

Author: Miguel Marco

[miguelmarco]

Commit: 12c3e8a

[miguelmarco]

New commits:

12c3e8a

Don't use -L and -I flags

[jdemeyer]

comment:3

You most likely just need to add $LDFLAGS to the command line.

[miguelmarco]

comment:5

You mean during the compilation of the library, the compilation of each instance or at running it?

[jdemeyer]

comment:6

I mean in the command line that you changed in your patch.

[vbraun]

comment:7

You can just test it using ldd /path/to/binary

You should always add $LDFLAGS when creating a binary. That includes shared libraries.

Static libraries (libtides.a) aren't executable so you don't need $LDFLAGS.

[sagetrac-git]

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

6981184	Added $LDFLAGS
0396add	Put -L directive before -l

[sagetrac-git]

Changed commit from 12c3e8a to 0396add

[miguelmarco]

comment:9

I think it is fixed now.

[vbraun]

Reviewer: Volker Braun

[vbraun]

Changed branch from u/mmarco/tides_malfunctions_in_version_7_0beta to 0396add

[slel]

Changed commit from 0396add to none