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

./configure: line 13402: syntax error near unexpected token `else' #424

Closed
paveljanik opened this issue Oct 31, 2016 · 5 comments · Fixed by #752
Closed

./configure: line 13402: syntax error near unexpected token `else' #424

paveljanik opened this issue Oct 31, 2016 · 5 comments · Fixed by #752

Comments

@paveljanik
Copy link
Contributor

Probably after merging #399, autogen.sh generates this code here:

  if test "$cross_compiling" = yes; then
  else
  cat >conftest.$ac_ext <<_ACEOF

ie. else immediately following then. On other systems, there is : (true) there.

@theuni Do you have an idea?

@gmaxwell
Copy link
Contributor

This has been open forever without any other reports, I assume it got fixed. Feel free to reopen if otherwise.

@duhanov
Copy link

duhanov commented May 18, 2020

Same error!

@duhanov
Copy link

duhanov commented May 18, 2020

CentOS release 6.7 (Final)

@elichai
Copy link
Contributor

elichai commented May 18, 2020

I confirmed it in a docker, the problem is that CentOS 6 ships with an old autoconf, and after installing a third party update I was able to compile and pass the tests.
CentOS 6 got released a decade ago, stopped getting full updates 3 years ago and will be officially unmaintained in a few months.
The result of that is that I couldn't even clone this repo, because yum install git gave me 1.7.1, which github doesn't support, so I had to get git from a third party source.

So I think that not supporting CentOS 6 out of the box is quite reasonable seeing that even github doesn't support it out of the box (oh, and don't get me started on the SSL problems I had because all the CAs are out of date and it uses old openssl which I had to upgrade both)

real-or-random added a commit to real-or-random/secp256k1 that referenced this issue May 18, 2020
real-or-random added a commit to real-or-random/secp256k1 that referenced this issue May 18, 2020
@real-or-random
Copy link
Contributor

Yes, no need to support old very old versions of autoconf. But I assume the fix is trivial.

@paveljanik @duhanov @elichai Can you check if #752 fixes this issue?

real-or-random added a commit that referenced this issue May 22, 2020
5e8747a autoconf: Use ":" instead of "dnl" as a noop (Tim Ruffing)

Pull request description:

  Fixes #424.

Top commit has no ACKs.

Tree-SHA512: a83664afbc6ca1254c4767161bfbec82f3489a8a248ba7a5a46ed9ec2a39232cf92f504accadd4dbb1a6ea4791dbf7f0e1f030e51f02f49eb9a38a2e509ee6c2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants