Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade: lcalc 2.0.5 to allow building on Cygwin #33043

Closed
mkoeppe opened this issue Dec 18, 2021 · 9 comments
Closed

Upgrade: lcalc 2.0.5 to allow building on Cygwin #33043

mkoeppe opened this issue Dec 18, 2021 · 9 comments

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Dec 18, 2021

In Cygwin, building Sage 9.5.beta8 fails
while building lcalc 2.0.4.

See for instance github-actions run 4568970532
with our cygwin-standard configuration:

/usr/bin/bash ../../libtool  --tag=CXX   --mode=link g++ -std=gnu++11  -g -O2 -version-info 1:1:0 -L/opt/sage-676f077d2977748d2c17251384443b4059bee4a9/lib -Wl,-rpath,/opt/sage-676f077d2977748d2c17251384443b4059bee4a9/lib -L/opt/sage-676f077d2977748d2c17251384443b4059bee4a9/lib -Wl,-rpath,/opt/sage-676f077d2977748d2c17251384443b4059bee4a9/lib  -o libLfunction.la -rpath /opt/sage-676f077d2977748d2c17251384443b4059bee4a9/lib Ldokchitser.lo Lelliptic.lo Lgamma.lo Lglobals.lo Lmisc.lo Lnumbertheory.lo Lriemannsiegel.lo   -lpari 
libtool:   error: can't build x86_64-pc-cygwin shared library unless -no-undefined is specified
make[4]: *** [Makefile:472: libLfunction.la] Error 1

Here we upgrade to lcalc 2.0.5, which has the fix from

CC: @orlitzky @slel

Component: porting: Cygwin

Keywords: lcalc, upgrade

Author: Michael Orlitzky

Branch/Commit: a5a14ec

Reviewer: Matthias Koeppe

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

@mkoeppe mkoeppe added this to the sage-9.5 milestone Dec 18, 2021
@mkoeppe

This comment has been minimized.

@orlitzky
Copy link
Contributor

Author: Michael Orlitzky

@orlitzky
Copy link
Contributor

Commit: a5a14ec

@orlitzky
Copy link
Contributor

New commits:

a5a14ecTrac #33043: upgrade lcalc to v2.0.5.

@orlitzky
Copy link
Contributor

Branch: u/mjo/ticket/33043

@mkoeppe
Copy link
Member Author

mkoeppe commented Dec 19, 2021

Reviewer: Matthias Koeppe

@slel
Copy link
Member

slel commented Dec 23, 2021

Changed keywords from none to lcalc, upgrade

@slel

This comment has been minimized.

@slel slel changed the title lcalc build fails on cygwin-standard Upgrade: lcalc 2.0.5 to allow building on Cygwin Dec 23, 2021
@vbraun
Copy link
Member

vbraun commented Dec 28, 2021

Changed branch from u/mjo/ticket/33043 to a5a14ec

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants