tests/README: make "Run" section foolproof #1352

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@MarcelRaad
Member

This is probably obvious to linux users, but wasn't to me as a mainly Windows user when I first read this and the error messages are not very helpful ("don't know how to make target test" if you forget ./configure and "cannot get curl version" if you forget make).

@MarcelRaad MarcelRaad tests/README: make "Run" section foolproof
curl must be built before building the tests.
fcfa061

@MarcelRaad, thanks for your PR! By analyzing the history of the files in this pull request, we identified @bagder, @dfandrich and @yangtse to be potential reviewers.

@bagder
bagder approved these changes Mar 23, 2017 View changes

👍

@MarcelRaad MarcelRaad added a commit to MarcelRaad/curl that referenced this pull request Mar 24, 2017
@MarcelRaad MarcelRaad tests/README: make "Run" section foolproof
curl must be built before building the tests.

Closes curl#1352
d145634
@MarcelRaad MarcelRaad added a commit that closed this pull request Mar 24, 2017
@MarcelRaad MarcelRaad tests/README: make "Run" section foolproof
curl must be built before building the tests.

Closes #1352
d145634
@MarcelRaad MarcelRaad deleted the MarcelRaad:tests_readme_run branch Mar 24, 2017
Member

Thanks @bagder !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment