Page MenuHomePhabricator

[SECP256K1] Move the autotools Travis build to it's own script

Authored by Fabien on Jan 8 2020, 14:35.



This will make it easier to follow when introducing the cmake build into

There is no change in behavior.

Test Plan

Run the script, run the travis build.

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

deadalnix requested changes to this revision.Jan 8 2020, 17:32
deadalnix added inline comments.
60 ↗(On Diff #15223)

contrib doesn't seems to be the right place for this project. There is code in there.

This revision now requires changes to proceed.Jan 8 2020, 17:32

Address feedbacks (also from D4856):

  • Move the scripts out of contrib/
  • Set the locale (the linter didn't throw because secp256k1 is excluded, I will update this in another diff)
  • Use set -x instead of set -v for consistency
  • Don't export script scoped variables.
This revision is now accepted and ready to land.Jan 9 2020, 11:31