Opened 9 years ago
Closed 7 years ago
#14648 closed enhancement (duplicate)
Old FSF config.(guess|sub) scripts do not recognize Cygwin64
Reported by: | jpflori | Owned by: | jdemeyer |
---|---|---|---|
Priority: | major | Milestone: | sage-duplicate/invalid/wontfix |
Component: | packages: standard | Keywords: | spkg cygwin cygwin64 |
Cc: | kcrisman, dimpase, leif, tscrim | Merged in: | |
Authors: | Reviewers: | Jean-Pierre Flori | |
Report Upstream: | Reported upstream. Developers acknowledge bug. | Work issues: | |
Branch: | Commit: | ||
Dependencies: | #15365, #15580, #15015 | Stopgaps: |
Description (last modified by )
The FSF config.guess and config.sub shipped with MPIR 2.6.0 and some other packages are very old (timestamp says 2003 for the MPIR ones) and fail to recognize Cygwin64. Latest upstream files at:
- http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.guess;hb=HEAD
- http://git.savannah.gnu.org/gitweb/?p=config.git;a=blob_plain;f=config.sub;hb=HEAD
solves this issue.
MPIR report at:
Other spkg affected:
Change History (59)
comment:1 Changed 9 years ago by
comment:2 Changed 9 years ago by
sqlite spkg (even the latest upstream packaged in 5.10.beta*) fails for the same reason.
comment:3 Changed 9 years ago by
- Cc leif added
comment:4 follow-up: ↓ 5 Changed 9 years ago by
Same problem when configuring ECL, it fails in the provided gmp subdir (that we don't even use...).
comment:5 in reply to: ↑ 4 ; follow-up: ↓ 7 Changed 9 years ago by
Replying to jpflori:
Same problem when configuring ECL, it fails in the provided gmp subdir (that we don't even use...).
I proposed a patch to allow removing the GMP folder a couple of times... 8/
comment:6 Changed 9 years ago by
- Description modified (diff)
- Keywords mpir removed
- Summary changed from MPIR fails to configure on Cygwin64 to Old FSF config.(guess|sub) scripts do not recognize Cygwin64
I'm changing this ticket to only take care of the outdated fsf config.* scripts.
The further problems with MPIR should go elsewher.
comment:7 in reply to: ↑ 5 Changed 9 years ago by
Replying to leif:
Replying to jpflori:
Same problem when configuring ECL, it fails in the provided gmp subdir (that we don't even use...).
I proposed a patch to allow removing the GMP folder a couple of times... 8/
ROFL, three years ago: #9493.
(No idea whether the patch w.r.t. install-sh
still applies or is at all still necessary.)
comment:8 Changed 9 years ago by
- Milestone changed from sage-5.11 to sage-5.12
comment:9 Changed 8 years ago by
- Cc tscrim added
comment:10 Changed 8 years ago by
Given the above, shall we put the new scripts as a patch and start the review?
comment:11 Changed 8 years ago by
- Branch set to u/tscrim/ticket/14648
- Commit set to f5a57d0c790c6e6afd67fb02ce23a1ac31db8fb5
Here's my attempt at a clean patch.
New commits:
c0a09c5 | Make sure tinfo is correctly linked in when needed on Cygwin.
|
45bb195 | Update ncurses to dev version 5.9-20131221.
|
f5a57d0 | Merge branch 'u/jpflori/ticket/15617' of trac.sagemath.org:sage into cygwin64build
|
comment:12 Changed 8 years ago by
- Commit changed from f5a57d0c790c6e6afd67fb02ce23a1ac31db8fb5 to 815819c36ab1f278464c68e2ef5b3749d23da82c
Branch pushed to git repo; I updated commit sha1. New commits:
815819c | Initial attempt.
|
comment:13 Changed 8 years ago by
Now with the correct thing pushed.
comment:14 Changed 8 years ago by
This patch is very bad. MPIR's config.guess
is a custom script, it is not the autotools' version. See also the thread you linked to.
For example, it detects my machine as sandybridge-unknown-linux-gnu
. With this patch, this becomes x86_64-unknown-linux-gnu
.
Also, Initial attempt.
is obviously a bad commit message.
comment:15 Changed 8 years ago by
Hey Travis, I'll up my working (hopefully not only for me) branch a little later today. In this very ticket I'll only update the FSF scripts. In another one I'll put the magic needed for MPIR to build the correct ABI.
comment:16 Changed 8 years ago by
- Branch changed from u/tscrim/ticket/14648 to u/jpflori/ticket/14648
- Commit changed from 815819c36ab1f278464c68e2ef5b3749d23da82c to 0ac6c60c8ed2b1607645d771f76beeff1b02d82d
comment:17 Changed 8 years ago by
- Dependencies set to #15365
comment:18 Changed 8 years ago by
- Commit changed from 0ac6c60c8ed2b1607645d771f76beeff1b02d82d to c311252891e2ef51244de0ab6ceeee73eedfeacc
comment:19 Changed 8 years ago by
- Commit changed from c311252891e2ef51244de0ab6ceeee73eedfeacc to b922e3b00fededdb77b25210d247d252b3136230
comment:20 Changed 8 years ago by
- Commit changed from b922e3b00fededdb77b25210d247d252b3136230 to 9ba3d7fb03b7e55aa3f116a82840e5b07e472ea1
comment:21 Changed 8 years ago by
IML needs updating.
comment:22 Changed 8 years ago by
Since #15580, you could also consider copying config.guess
from $SAGE_ROOT/config/config.guess
.
comment:23 Changed 8 years ago by
Yes, that's a very good idea!
comment:24 Changed 8 years ago by
Be sure to check which version of Automake you require (config.guess
belongs to Automake) and edit configure.ac
accordingly.
comment:25 Changed 8 years ago by
I guess there is some magic in configure which checks for consistency between configure.ac and config.guess?
comment:26 Changed 8 years ago by
I meant edit $SAGE_ROOT/configure.ac
, in particular the line
AM_INIT_AUTOMAKE([1.9.6 foreign])
And no, I don't think autotools cares about the version of config.guess
, it just requires it to be available.
comment:27 Changed 8 years ago by
After a quick look through automake versions it would be 1.10.
comment:28 follow-up: ↓ 29 Changed 8 years ago by
FTR, this currently doesn't work for me on my cygwin32 and ubuntu machines. I believe it's the same issue as comment:14:
checking build system type... Invalid configuration `k8-pc-linux-gnu': machine `k8-pc' not recognized configure: error: /bin/bash ./config.sub k8-pc-linux-gnu failed Error configuring MPIR (with CFLAGS unset).
comment:29 in reply to: ↑ 28 Changed 8 years ago by
Replying to tscrim:
FTR, this currently doesn't work for me on my cygwin32 and ubuntu machines. I believe it's the same issue as comment:14:
checking build system type... Invalid configuration `k8-pc-linux-gnu': machine `k8-pc' not recognized configure: error: /bin/bash ./config.sub k8-pc-linux-gnu failed Error configuring MPIR (with CFLAGS unset).
I assume the previous version of the pkg does work?
comment:30 Changed 8 years ago by
(And you are using my branch and not yours?)
comment:31 Changed 8 years ago by
Yes (and yes).
comment:32 Changed 8 years ago by
- Commit changed from 9ba3d7fb03b7e55aa3f116a82840e5b07e472ea1 to 79275bdb80d4b50a86dace1ba083da6b34a2b873
comment:33 Changed 8 years ago by
- Dependencies changed from #15365 to #15365, #15580
- Status changed from new to needs_review
comment:34 Changed 8 years ago by
For the k8 issue, did any previous version work?
comment:35 Changed 8 years ago by
Ok, I guess the problem is with https://github.com/wbhart/mpir/commit/3b410a80565eb71fd32a86630d550098ebc85f82
In fact, the MPIR wrapper should be modified to map k8 and so on to something else.
comment:36 Changed 8 years ago by
- Status changed from needs_review to needs_work
comment:37 Changed 8 years ago by
- Milestone changed from sage-6.1 to sage-6.2
comment:38 Changed 8 years ago by
Latest sqlite is wroking on Cygwin64 (or rather "at least it builds now"). See #16098.
comment:39 Changed 8 years ago by
Latest MPIR at #15015 will work as well.
comment:40 Changed 8 years ago by
ECL fixed at #9493.
So only dealing with IML here.
comment:41 Changed 8 years ago by
- Commit changed from 79275bdb80d4b50a86dace1ba083da6b34a2b873 to d3a4456c68cd7ecbe13f7ada905002c7cd8440f3
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
d3a4456 | Use uptodate config.* script for IML.
|
comment:42 Changed 8 years ago by
- Description modified (diff)
- Status changed from needs_work to needs_review
comment:43 Changed 8 years ago by
Thats a kind of vile hack to use some of the Sage autoconf output. Did you contact IML upstream?
comment:44 Changed 8 years ago by
Note that this vile hack just use standard files distributed by autoconf, not some files specifically crafted (well, the exact version is) by configure when it's run.
Not contacted IML though, I though the project was dead. If not we should really work more with them to deal with the BLAS/linking hell in their build system.
comment:45 follow-up: ↓ 46 Changed 8 years ago by
Well we could give it a try. If there is no response then we should just make our own tarball with updated autoconf. Its not exactly a moving target ;-)
I agree that the hack works, but its ugly and fragile if we change anything in the sage autoconf files (e.g. move to a different directory).
comment:46 in reply to: ↑ 45 Changed 8 years ago by
Replying to vbraun:
Well we could give it a try. If there is no response then we should just make our own tarball with updated autoconf. Its not exactly a moving target ;-)
I'll drop them an email (without specific patches first, just getting the temperature there).
I agree that the hack works, but its ugly and fragile if we change anything in the sage autoconf files (e.g. move to a different directory).
Sure, I realize I should at least check the return code for such cases.
comment:47 follow-up: ↓ 48 Changed 8 years ago by
Sent an email upstream.
comment:48 in reply to: ↑ 47 Changed 8 years ago by
comment:49 follow-up: ↓ 51 Changed 8 years ago by
Maybe... At least the mail address given here: https://cs.uwaterloo.ca/~astorjoh/iml.html#bug Don't know where the authors are working now.
comment:50 Changed 8 years ago by
- Commit changed from d3a4456c68cd7ecbe13f7ada905002c7cd8440f3 to 4638ee169c6e0b2d4f0a35ad8142a09bf00adbd8
Branch pushed to git repo; I updated commit sha1. New commits:
4638ee1 | Check that FSF config.* scripts are correctly copied from SAGE_ROOT/config.
|
comment:51 in reply to: ↑ 49 ; follow-up: ↓ 52 Changed 8 years ago by
Replying to jpflori:
Maybe... At least the mail address given here: https://cs.uwaterloo.ca/~astorjoh/iml.html#bug Don't know where the authors are working now.
Got a reply and Arne would be very happy to make a new release. Now it's time to think about what we should push upstream with regard to the build system (it may be a good time to provide support for different BLAS implem than ATLAS), bug fixes and so on.
comment:52 in reply to: ↑ 51 ; follow-up: ↓ 53 Changed 8 years ago by
Replying to jpflori:
Replying to jpflori:
Maybe... At least the mail address given here: https://cs.uwaterloo.ca/~astorjoh/iml.html#bug Don't know where the authors are working now.
Got a reply and Arne would be very happy to make a new release.
\o/ (after years of apparent death)
Now it's time to think about what we should push upstream with regard to the build system (it may be a good time to provide support for different BLAS implem than ATLAS), bug fixes and so on.
Best post to sage-devel, such that the usual suspectives will notice, and/or cc some of those involved in the previous IML tickets (#748 (!), #9568 ...).
comment:53 in reply to: ↑ 52 Changed 8 years ago by
- Report Upstream changed from N/A to Reported upstream. Developers acknowledge bug.
comment:54 Changed 8 years ago by
Thanks for the head up on IML :) the laundry list as far as I remember is
- getting configure to accept other cblas. Cannot use gentoo patches as they rely on cblas installing .pc files which is not standard across implementation.
- remove the "repl" subfolder which doesn't do anything and create problems on OS X https://raw.githubusercontent.com/cschwan/sage-on-gentoo/master/sci-libs/iml/files/iml-1.0.3-repl_removal.patch
- fix undefined symbols in nullspace.c https://raw.githubusercontent.com/cschwan/sage-on-gentoo/master/sci-libs/iml/files/iml-1.0.3-fix-undefined-symbol.patch
- I have an extra fix in the s-o-g tree for automake 1.13 that probably should go in https://raw.githubusercontent.com/cschwan/sage-on-gentoo/master/sci-libs/iml/files/iml-1.0.3-automake-1.13.patch
comment:55 Changed 8 years ago by
- Milestone changed from sage-6.2 to sage-6.3
comment:56 Changed 8 years ago by
Better late than never, I've forwaded our patches (except the emamples one, what is that for? quicker test?) and two new ones for inclusion in a potential new release of IML (one for shared libs on Cygwin (who cares? :)) and on to add a --with-cblas configure flag to pass the ld flags for linking to cblas, falling back to the old atlas detection code in case of failure).
Best, JP
comment:57 Changed 8 years ago by
- Milestone changed from sage-6.3 to sage-6.4
comment:58 Changed 7 years ago by
- Branch u/jpflori/ticket/14648 deleted
- Commit 4638ee169c6e0b2d4f0a35ad8142a09bf00adbd8 deleted
- Dependencies changed from #15365, #15580 to #15365, #15580, #15015
- Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
- Reviewers set to Jean-Pierre Flori
- Status changed from needs_review to positive_review
All packages are now recent enough except for MPIR which will get updated in #15015.
comment:59 Changed 7 years ago by
- Resolution set to duplicate
- Status changed from positive_review to closed
There is a planned update to MPIR spkg at #13948.