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

bootstrap should use the autotools installed in SAGE_LOCAL/bin if available #21214

Closed
mkoeppe opened this issue Aug 10, 2016 · 12 comments
Closed

Comments

@mkoeppe
Copy link
Member

mkoeppe commented Aug 10, 2016

... this is so that it does not have to rely on downloading the generated files, nor on an external installation of the autotools (which is convenient on Mac OS X because one has to remove fink from PATH).

This patch sources src/bin/sage-env in order to augment PATH just before calling the autotools.

Depends on #21047

CC: @jdemeyer @videlec @embray @nexttime @vbraun @jpflori @dimpase

Component: build

Author: Matthias Koeppe

Branch/Commit: 2f30faf

Reviewer: Dima Pasechnik

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

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 10, 2016

@mkoeppe

This comment has been minimized.

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 10, 2016

Author: Matthias Koeppe

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 10, 2016

New commits:

2f30fafbootstrap: Get autotools from our own package into PATH

@mkoeppe
Copy link
Member Author

mkoeppe commented Aug 10, 2016

Commit: 2f30faf

@mkoeppe mkoeppe added this to the sage-7.4 milestone Aug 10, 2016
@dimpase
Copy link
Member

dimpase commented Sep 4, 2016

comment:4

src/bin/ != SAGE_LOCAL/bin/; thus either the branch or the ticket description is wrong.

@mkoeppe

This comment has been minimized.

@dimpase
Copy link
Member

dimpase commented Sep 4, 2016

comment:6

the title still says "bootstrap should use the autotools installed in SAGE_LOCAL/bin if available".

@nexttime
Copy link
Mannequin

nexttime mannequin commented Sep 4, 2016

comment:7

Replying to @dimpase:

the title still says "bootstrap should use the autotools installed in SAGE_LOCAL/bin if available".

... which is correct.

@dimpase
Copy link
Member

dimpase commented Sep 4, 2016

Reviewer: Dima Pasechnik

@dimpase
Copy link
Member

dimpase commented Sep 4, 2016

comment:8

ok, good, sorry for confusion.

@vbraun
Copy link
Member

vbraun commented Sep 4, 2016

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

3 participants