Page MenuHomePhabricator
Files F15391828

D7683.diff
No OneTemporary
Actions

D7683.diff
View Options

Index: scripts/repository/commit_hook.php
===================================================================
--- scripts/repository/commit_hook.php
+++ scripts/repository/commit_hook.php
@@ -4,32 +4,15 @@
$root = dirname(dirname(dirname(__FILE__)));
require_once $root.'/scripts/__init_script__.php';
-$username = getenv('PHABRICATOR_USER');
-if (!$username) {
- throw new Exception(pht('usage: define PHABRICATOR_USER in environment'));
-}
-
-$user = id(new PhabricatorPeopleQuery())
- ->setViewer(PhabricatorUser::getOmnipotentUser())
- ->withUsernames(array($username))
- ->executeOne();
-if (!$user) {
- throw new Exception(pht('No such user "%s"!', $username));
-}
-
if ($argc < 2) {
throw new Exception(pht('usage: commit-hook <callsign>'));
}
+$engine = new DiffusionCommitHookEngine();
+
$repository = id(new PhabricatorRepositoryQuery())
- ->setViewer($user)
+ ->setViewer(PhabricatorUser::getOmnipotentUser())
->withCallsigns(array($argv[1]))
- ->requireCapabilities(
- array(
- // This capability check is redundant, but can't hurt.
- PhabricatorPolicyCapability::CAN_VIEW,
- DiffusionCapabilityPush::CAPABILITY,
- ))
->executeOne();
if (!$repository) {
@@ -37,19 +20,59 @@
}
if (!$repository->isHosted()) {
- // This should be redundant too, but double check just in case.
+ // This should be redundant, but double check just in case.
throw new Exception(pht('Repository "%s" is not hosted!', $callsign));
}
+$engine->setRepository($repository);
+
+
+// Figure out which user is writing the commit.
+
+if ($repository->isGit()) {
+ $username = getenv('PHABRICATOR_USER');
+ if (!strlen($username)) {
+ throw new Exception(pht('usage: PHABRICATOR_USER should be defined!'));
+ }
+} else if ($repository->isSVN()) {
+ // NOTE: In Subversion, the entire environment gets wiped so we can't read
+ // PHABRICATOR_USER. Instead, we've set "--tunnel-user" to specify the
+ // correct user; read this user out of the commit log.
+
+ if ($argc < 4) {
+ throw new Exception(pht('usage: commit-hook <callsign> <repo> <txn>'));
+ }
+
+ $svn_repo = $argv[2];
+ $svn_txn = $argv[3];
+ list($username) = execx('svnlook author -t %s %s', $svn_txn, $svn_repo);
+ $username = rtrim($username, "\n");
+
+ $engine->setSubversionTransactionInfo($svn_txn, $svn_repo);
+} else {
+ throw new Exceptiont(pht('Unknown repository type.'));
+}
+
+$user = id(new PhabricatorPeopleQuery())
+ ->setViewer(PhabricatorUser::getOmnipotentUser())
+ ->withUsernames(array($username))
+ ->executeOne();
+
+if (!$user) {
+ throw new Exception(pht('No such user "%s"!', $username));
+}
+
+$engine->setViewer($user);
+
+
+// Read stdin for the hook engine.
+
$stdin = @file_get_contents('php://stdin');
if ($stdin === false) {
throw new Exception(pht('Failed to read stdin!'));
}
-$engine = id(new DiffusionCommitHookEngine())
- ->setViewer($user)
- ->setRepository($repository)
- ->setStdin($stdin);
+$engine->setStdin($stdin);
$err = $engine->execute();
Index: src/applications/diffusion/controller/DiffusionRepositoryController.php
===================================================================
--- src/applications/diffusion/controller/DiffusionRepositoryController.php
+++ src/applications/diffusion/controller/DiffusionRepositoryController.php
@@ -173,7 +173,12 @@
$serve_ssh = $repository->getServeOverSSH();
if ($serve_ssh !== $serve_off) {
$uri = new PhutilURI(PhabricatorEnv::getProductionURI($repo_path));
- $uri->setProtocol('ssh');
+
+ if ($repository->isSVN()) {
+ $uri->setProtocol('svn+ssh');
+ } else {
+ $uri->setProtocol('ssh');
+ }
$ssh_user = PhabricatorEnv::getEnvConfig('diffusion.ssh-user');
if ($ssh_user) {
Index: src/applications/diffusion/engine/DiffusionCommitHookEngine.php
===================================================================
--- src/applications/diffusion/engine/DiffusionCommitHookEngine.php
+++ src/applications/diffusion/engine/DiffusionCommitHookEngine.php
@@ -5,6 +5,15 @@
private $viewer;
private $repository;
private $stdin;
+ private $subversionTransaction;
+ private $subversionRepository;
+
+
+ public function setSubversionTransactionInfo($transaction, $repository) {
+ $this->subversionTransaction = $transaction;
+ $this->subversionRepository = $repository;
+ return $this;
+ }
public function setStdin($stdin) {
$this->stdin = $stdin;
@@ -39,6 +48,9 @@
case PhabricatorRepositoryType::REPOSITORY_TYPE_GIT:
$err = $this->executeGitHook();
break;
+ case PhabricatorRepositoryType::REPOSITORY_TYPE_SVN:
+ $err = $this->executeSubversionHook();
+ break;
default:
throw new Exception(pht('Unsupported repository type "%s"!', $type));
}
@@ -54,6 +66,13 @@
return 0;
}
+ private function executeSubversionHook() {
+
+ // TODO: Do useful things here, too.
+
+ return 0;
+ }
+
private function parseGitUpdates($stdin) {
$updates = array();
Index: src/applications/diffusion/ssh/DiffusionSSHSubversionServeWorkflow.php
===================================================================
--- src/applications/diffusion/ssh/DiffusionSSHSubversionServeWorkflow.php
+++ src/applications/diffusion/ssh/DiffusionSSHSubversionServeWorkflow.php
@@ -38,7 +38,9 @@
throw new Exception("Expected `svnserve -t`!");
}
- $command = csprintf('svnserve -t');
+ $command = csprintf(
+ 'svnserve -t --tunnel-user=%s',
+ $this->getUser()->getUsername());
$command = PhabricatorDaemon::sudoCommandAsDaemonUser($command);
$future = new ExecFuture('%C', $command);
Index: src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
===================================================================
--- src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
+++ src/applications/repository/engine/PhabricatorRepositoryPullEngine.php
@@ -85,6 +85,8 @@
if ($repository->isHosted()) {
if ($is_git) {
$this->installGitHook();
+ } else if ($is_svn) {
+ $this->installSubversionHook();
} else {
$this->logPull(
pht(
@@ -158,7 +160,7 @@
$root = dirname(phutil_get_library_root('phabricator'));
$bin = $root.'/bin/commit-hook';
- $cmd = csprintf('exec -- %s %s', $bin, $callsign);
+ $cmd = csprintf('exec -- %s %s "$@"', $bin, $callsign);
$hook = "#!/bin/sh\n{$cmd}\n";
@@ -394,5 +396,15 @@
execx('svnadmin create -- %s', $path);
}
+ /**
+ * @task svn
+ */
+ private function installSubversionHook() {
+ $repository = $this->getRepository();
+ $path = $repository->getLocalPath().'hooks/pre-commit';
+
+ $this->installHook($path);
+ }
+
}

File Metadata

Mime Type
text/plain
Expires
Sun, Mar 16, 11:50 AM (3 w, 6 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7706971
Default Alt Text
D7683.diff (6 KB)

Event Timeline

Phabricator · Built by Phacility