Skip to content

runtests: use the local curl for verifying - #3198

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/runtests-vcurl
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/runtests-vcurl

Conversation

@bagder

@bagder bagder commented Oct 31, 2018

Copy link
Copy Markdown
Member

... revert the mistaken change brought in commit 8440616.

Reported-by: Alessandro Ghedini
Bug: https://curl.haxx.se/mail/lib-2018-10/0118.html

... revert the mistaken change brought in commit 8440616.

Reported-by: Alessandro Ghedini
Bug: https://curl.haxx.se/mail/lib-2018-10/0118.html

@MarcelRaad MarcelRaad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! I was just wondering about that too when I tried to build on a fresh machine without curl.

@bagder

bagder commented Oct 31, 2018

Copy link
Copy Markdown
Member Author

I'm thinking of trying removing the curl tool from the travis image before the tests are run, to make sure this kind of mistake will be caught in the future...

bagder added a commit that referenced this pull request Oct 31, 2018
... to make sure the test suite runs with its newly build tool!

Bug: #3198
@bagder bagder closed this in 8effa8c Oct 31, 2018
@bagder
bagder deleted the bagder/runtests-vcurl branch October 31, 2018 12:50
bagder added a commit that referenced this pull request Nov 1, 2018
... to make sure the test suite runs with its newly build tool!

Bug: #3198
bagder added a commit that referenced this pull request Nov 2, 2018
on Linux. To make sure the test suite runs with its newly build tool and
doesn't require an external one present.

Bug: #3198
Closes #3200
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants