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

buildconf: invoke 'autoreconf -fi' instead #5853

Closed
wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Aug 25, 2020

The custom script isn't necessary but remains for simplicity, and just
invokes autoreconf.

@bagder bagder added the build label Aug 25, 2020
@mback2k
Copy link
Member

mback2k commented Aug 25, 2020

My buildbots using testcurl report this error: testcurl: buildconf was NOT successful

@bagder
Copy link
Member Author

bagder commented Aug 25, 2020

Ah, yes thanks I need to address that!

The custom script isn't necessary anymore - but remains for simplicity,
and just invokes autoreconf.

Closes #5853
@bagder bagder closed this in 8586853 Aug 27, 2020
@bagder bagder deleted the bagder/buildconf-removed branch August 27, 2020 20:54
mback2k added a commit to mback2k/curl that referenced this pull request Aug 29, 2020
Also make buildconf exit with the return code of autoreconf.

Follow up to curl#5853
mback2k added a commit that referenced this pull request Aug 29, 2020
Also make buildconf exit with the return code of autoreconf.

Reviewed-by: Daniel Stenberg

Follow up to #5853
Closes #5890
emilengler added a commit to emilengler/curl that referenced this pull request Sep 6, 2020
This is a minor issue but those asteriskes at the end were
most likely forgotten by curl#5853
emilengler added a commit to emilengler/curl that referenced this pull request Sep 6, 2020
This is a minor issue but those asterisks at the end were
most likely forgotten by curl#5853

Closes curl#5925
momoka0122y added a commit to momoka0122y/curl that referenced this pull request Sep 19, 2021
bagder pushed a commit that referenced this pull request Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants