You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@mesos.apache.org by Kevin Klues <kl...@gmail.com> on 2015/12/11 05:51:20 UTC
Review Request 41243: Update how we find the .git directory in
bootstrap
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/41243/
-----------------------------------------------------------
Review request for mesos and Benjamin Hindman.
Bugs: MESOS-4125
https://issues.apache.org/jira/browse/MESOS-4125
Repository: mesos
Description
-------
When building from git, bootstrap will (among other things) install
pre-commit and post-rewrite hooks into the .git/hooks directory of the
mesos tree. However the current implementation always assumes that .git
exists in the same directory as the bootstrap file. This may not always
be true.
Most notably, it is not true if the mesos tree is included as a
submodule inside another project. When included as a submodule, .git is
no longer a directory, but rather a file whose text contains a pointer
back to the actual location of the .git folder inside the containing
project. To get at this directory, we need to run 'git rev-parse
--git-dir' instead of simply assuming that the local .git is the proper
directory.
Diffs
-----
bootstrap 89d986fd95dc16bbb79623ef92e3b14a2e7009f9
Diff: https://reviews.apache.org/r/41243/diff/
Testing
-------
Thanks,
Kevin Klues