----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71299/#review217264 -----------------------------------------------------------
Fix it, then Ship it! support/setup_dev.sh Lines 1 (patched) <https://reviews.apache.org/r/71299/#comment304566> Can we name this with a hyphen, `setup-dev.sh`? The majority of the support scripts use hyphen rather than underscore. support/setup_dev.sh Lines 7 (patched) <https://reviews.apache.org/r/71299/#comment304565> Wrong script name? - James Peach On Aug. 16, 2019, 7:42 p.m., Benjamin Bannier wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71299/ > ----------------------------------------------------------- > > (Updated Aug. 16, 2019, 7:42 p.m.) > > > Review request for mesos, Benno Evers and Till Toenshoff. > > > Bugs: MESOS-9630 > https://issues.apache.org/jira/browse/MESOS-9630 > > > Repository: mesos > > > Description > ------- > > This patch breaks the installation of developer tools (i.e., linter > configuration files and git hooks) out of `./bootstrap`. This not only > simplifies and streamlines the setup, but will allow us to add > developer-only features without breaking users who are just interested > in building a distribution tarball. > > > Diffs > ----- > > bootstrap 7be2cc95c7ace55d710315427f38284cc5b2af4c > bootstrap.bat 584b5c3ca228ff485b22473cd167d3f454d6dad4 > docs/advanced-contribution.md 573138d3642e901f46ee35db58299627f188b94c > docs/beginner-contribution.md 471f5dd0da30d921cc3b29484d2b02f164f8ca75 > docs/clang-format.md 4289813c2ca78e1b963aa22a596abd6e8aa3a28b > support/gitignore 7218eda0d78b8e6fc4568f215016961bd7a11a1b > support/llvm/README.md 188930224ba8ff53dbef8770bc56625be936bb76 > support/setup_dev.sh PRE-CREATION > > > Diff: https://reviews.apache.org/r/71299/diff/3/ > > > Testing > ------- > > > Thanks, > > Benjamin Bannier > >
