Andreas Tolfsen ❲:ato❳ Assignee
	Description • 7 years ago

We should probably include the git libexec paths, /usr/libexec/git-core and /usr/local/libexec/git-core, when searching for the git-cinnabar program on Unix platforms.

When following git-cinnabar’s installation instructions, it does not automatically get placed in a directory on PATH.  Instead, it gets placed in git’s libexec path, which is meant for programs to be called by other programs and not directly by users.  Invoking git subcommands happens through a “dispatcher program” that looks in the above-mentioned libexec directories for internal commands.

See my writeup about libexec paths on http://sny.no/2017/05/libexec.

An alternative approach is to invoke `git cinnabar --help` and check its exit code, but expanding the search path we pass to `which.which` seems less invasive.

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 1 • 7 years ago

Testing this patch:

> % which git-cinnabar
> % 
> % ./mach try -b do -p all -u all -t none
> mach try is under development, please file bugs blocking 1149670.
> [1367741 f2aeae1e30d1] try: -b do -p all -u all -t none
> 
> Bundling 1 changesets
> Bundling 2 changesets
> Bundling 2 changesets
> 
> Bundling 1 manifests
> Bundling 1 manifests
> 
> Bundling 0 files
> Bundling 1 files
> remote: adding changesets
> remote: adding manifests
> remote: adding file changes
> remote: added 2 changesets with 1 changes to 1 files (+1 heads)
> remote: recorded push in pushlog
> remote: 
> remote: View your changes here:
> remote:   https://hg.mozilla.org/try/rev/040bf45799b972150278b296d5d2b7fa69c9a85e
> remote:   https://hg.mozilla.org/try/rev/eecee1dfd25c90010dd251591200f5f7e30c44e0
> remote: 
> remote: Follow the progress of your build on Treeherder:
> remote:   https://treeherder.mozilla.org/#/jobs?repo=try&revision=eecee1dfd25c90010dd251591200f5f7e30c44e0
> remote: recorded changegroup in replication log in 0.094s
> To hg::ssh://hg.mozilla.org/try
>  * [new branch]                HEAD -> branches/default/tip
> %

	Mike Hommey [:glandium]
	Comment 4 • 7 years ago
mozreview-review

Comment on attachment 8871263 [details]
Bug 1367741 - Search libexec paths on Unix systems for git-cinnabar;

https://reviewboard.mozilla.org/r/142752/#review147712

It would actually be better to check the exit code for `git cinnabar -h`, because libexec is only one possibility where git might look things up. As a matter of fact, the way I have git and git-cinnabar installed neither has git-cinnabar in $PATH nor in /usr/**/libexec.

	Andreas Tolfsen ❲:ato❳ Assignee
	Comment 5 • 6 years ago

This bug is no longer valid since autotry.py has been removed and
we no longer run a which test on git-cinnabar.