Homec4science

Support SVN pre-commit hoooks

Authored by epriestley <git@epriestley.com> on Dec 3 2013, 00:45.

Description

Support SVN pre-commit hoooks

Summary:
Ref T4189. This adds SVN support, which was a little more messy than I though. Principally, we can not use PHABRICATOR_USER for Subversion, because it strips away the entire environment for "security reasons".

Instead, use --tunnel-user plus svnlook author to figure out the author.

Also fix "ssh://" clone URIs, which needs to be "svn+ssh://".

Test Plan:

  • Made SVN commits through the hook.
  • Made Git commits, too, to make sure I didn't break anything.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4189

Differential Revision: https://secure.phabricator.com/D7683

Details

Committed
epriestley <git@epriestley.com>Dec 3 2013, 00:45
Pushed
aubortJan 31 2017, 17:16
Parents
rPH618b5cbbc4e8: Install pre-commit hooks in Git repositories
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH017d6ccd0793: Support SVN pre-commit hoooks (authored by epriestley <git@epriestley.com>).Dec 3 2013, 00:45