revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-20 20:40:56 +01:00
Code Issues Releases Wiki Activity

Add explicit "/" when installing hooks

Browse source 
Summary: There's no guarantee that the local path has a trailing "/". We
should probably guarantee that at some point, but just add one
unconditionally for now.

Auditors: btrahan
This commit is contained in:
epriestley 2013-12-02 16:17:19 -08:00
parent ba16df0fed
commit 50b5a0c8b7
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
View file

@ -311,9 +311,9 @@ final class PhabricatorRepositoryPullEngine
$path = $repository->getLocalPath();
if ($repository->isWorkingCopyBare()) {
$path .= 'hooks/pre-receive';
$path .= '/hooks/pre-receive';
} else {
$path .= '.git/hooks/pre-receive';
$path .= '/.git/hooks/pre-receive';
}
$this->installHook($path);
@ -390,7 +390,7 @@ final class PhabricatorRepositoryPullEngine
*/
private function installMercurialHook() {
$repository = $this->getRepository();
$path = $repository->getLocalPath().'.hg/hgrc';
$path = $repository->getLocalPath().'/.hg/hgrc';
$root = dirname(phutil_get_library_root('phabricator'));
$bin = $root.'/bin/commit-hook';
@ -430,7 +430,7 @@ final class PhabricatorRepositoryPullEngine
*/
private function installSubversionHook() {
$repository = $this->getRepository();
$path = $repository->getLocalPath().'hooks/pre-commit';
$path = $repository->getLocalPath().'/hooks/pre-commit';
$this->installHook($path);
}