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

GIT-INFO: suggest using autoreconf instead of buildconf following 85868537d #7033

Closed
wants to merge 1 commit into from

Conversation

@starrify
Copy link
Contributor

@starrify starrify commented May 8, 2021

A minor change to resolve the conflicts in documentations.

Someone that is new to this project may, as I earlier did myself, start from docs/INSTALL.md and observe these lines:

curl/docs/INSTALL.md

Lines 22 to 26 in 1763ace

## Building from git
If you get your code off a git repository instead of a release tarball, see
the `GIT-INFO` file in the root directory for specific instructions on how to
proceed.

Then in GIT-INFO there are:

curl/GIT-INFO

Lines 13 to 18 in 1763ace

To build in environments that support configure, after having extracted
everything from git, do this:
./buildconf
./configure
make

However ./buildconf has been obsoleted as part of 8586853:

curl/buildconf

Lines 3 to 4 in 1763ace

echo "*** Do not use buildconf. Instead, just use: autoreconf -fi" >&2
exec ${AUTORECONF:-autoreconf} -fi "${@}"

Therefore this PR attempts to update GIT-INFO to have the recommended approach.

@bagder
bagder approved these changes May 8, 2021
@bagder bagder closed this in df269fe May 8, 2021
@bagder
Copy link
Member

@bagder bagder commented May 8, 2021

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants