search

JohnCremona / eclib

The eclib package includes mwrank (for 2-descent on elliptic curves over Q) and modular symbol code used to create the elliptic curve database.
GNU General Public License v2.0
21 stars 16 forks source link

updated xcode version for Travis #51

Closed JohnCremona closed 5 years ago

JohnCremona commented 5 years ago

Let's see if this works for Travis...

JohnCremona commented 5 years ago

This failed: the osx configure failed to find ntl. Help would be appreciated.

JohnCremona commented 5 years ago

@isuruf do you know what is wrong here? The configure script cannot find ntl when Travis runs.

isuruf commented 5 years ago

Looks like OSX works, but linux fails. Can you print config.log?

JohnCremona commented 5 years ago

@isuruf you are right I misread the output notification. I don't know how to recover the file config.log from travis.

JohnCremona commented 5 years ago

I don't understand this. I had assumed that it was failing because of the publicised end of support for xcode 6.4 but as you saw it is the Linux build which fails. Is it possible that the latest version of miniconda puts NTL somewhere different? The error from running configure is that it cannot find NTL. But nothing to do with NTL has changed.

isuruf commented 5 years ago

Here's what I did to print the config.log https://github.com/isuruf/eclib/commit/9a35372bce480fb949cd5f0ab36ac4bfc0a2794e

Error is this,

configure:18912: result: no
configure:18912: checking NTL/ZZ.h presence
configure:18912: g++ -E -I/home/travis/local/include  -I/home/travis/local/include conftest.cpp
In file included from /usr/include/c++/4.8/type_traits:35:0,
                 from /home/travis/local/include/NTL/tools.h:25,
                 from /home/travis/local/include/NTL/ZZ.h:19,
                 from conftest.cpp:26:
/usr/include/c++/4.8/bits/c++0x_warning.h:32:2: error: #error This file requires compiler and library support for the ISO C++ 2011 standard. This support is currently experimental, and must be enabled with the -std=c++11 or -std=gnu++11 compiler options.
 #error This file requires compiler and library support for the \
JohnCremona commented 5 years ago

Changing from trusty to xenial did the trick. Thanks for suggesting that.