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

README Build from Source: m4 (step 5) is needed to run make configure (step 3) #33553

Closed
seblabbe opened this issue Mar 24, 2022 · 11 comments
Closed

Comments

@seblabbe
Copy link
Contributor

In the current README (https://gitlab.com/sagemath/sage/-/blob/develop/README.md), in the Instructions to Build from Source section, it says at step 3 to:

3. [Git] If you cloned the Sage repository using git, bootstrap the
source tree using:

$ make configure

On a bare debian linux version, the above fails becaue of the absence of m4.

The installation of m4 is suggested only at step 5:

5. [Linux, Cygwin] Install the required minimal build prerequisites.


Compilers: gcc, gfortran, g++ (a recent enough matching
set of these three will avoid building Sage-specific compilers).
See the Installation Manual for a discussion of suitable compilers.


Build tools: GNU make, GNU m4, perl (including
ExtUtils::MakeMaker), ranlib, git, tar, bc.


Python 3.4 or later, or Python 2.7, a full installation including
urllib; but ideally version 3.7.x, 3.8.x, or 3.9.x, which will
avoid having to build Sage's own copy of Python 3.

Component: documentation

Author: Matthias Koeppe

Branch/Commit: f1af3cb

Reviewer: Sébastien Labbé

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

@seblabbe seblabbe added this to the sage-9.6 milestone Mar 24, 2022
@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 24, 2022

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 24, 2022

Author: Matthias Koeppe

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 24, 2022

New commits:

1872edcREADME.md: Move installation step on minimal build prereqs earlier
eb94fcfREADME.md: Also mention bootstrapping prerequisites

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 24, 2022

Commit: eb94fcf

@seblabbe
Copy link
Contributor Author

Changed commit from eb94fcf to f1af3cb

@seblabbe
Copy link
Contributor Author

comment:3

The change 9. to 10. needed to remove one space. I added a commit to fix this in case this changes the layout (I have more experience with !ReST than with markdown, so I don't know).


New commits:

f1af3cb33553: removed one space after `10.`

@seblabbe
Copy link
Contributor Author

@seblabbe
Copy link
Contributor Author

Reviewer: Sébastien Labbé

@seblabbe
Copy link
Contributor Author

comment:4

For me it is a positive review. Feel free to change to positive review if you agree with my commit.

@mkoeppe
Copy link
Contributor

mkoeppe commented Mar 25, 2022

comment:5

Thank you!

@vbraun
Copy link
Member

vbraun commented Apr 2, 2022

Changed branch from public/33553 to f1af3cb

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