You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by tv...@apache.org on 2013/03/12 21:04:51 UTC
[5/50] [abbrv] git commit: [#5557] Remove permissive
pre-revprop-change for new commits to not allow history change by default
[#5557] Remove permissive pre-revprop-change for new commits to not allow history change by default
Signed-off-by: Cory Johns <jo...@geek.net>
Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/6049825c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/6049825c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/6049825c
Branch: refs/heads/si/5453
Commit: 6049825c7817ba38045040eac5edc02cc7673a9e
Parents: a6c59e4
Author: Cory Johns <jo...@geek.net>
Authored: Wed Mar 6 23:03:37 2013 +0000
Committer: Tim Van Steenburgh <tv...@gmail.com>
Committed: Thu Mar 7 23:05:28 2013 +0000
----------------------------------------------------------------------
ForgeSVN/forgesvn/model/svn.py | 8 --------
ForgeSVN/forgesvn/tests/model/test_repository.py | 10 ++--------
2 files changed, 2 insertions(+), 16 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/6049825c/ForgeSVN/forgesvn/model/svn.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/model/svn.py b/ForgeSVN/forgesvn/model/svn.py
index 7aba890..0b94028 100644
--- a/ForgeSVN/forgesvn/model/svn.py
+++ b/ForgeSVN/forgesvn/model/svn.py
@@ -522,14 +522,6 @@ class SVNImplementation(M.RepositoryImplementation):
with open(fn, 'wb') as fp:
fp.write(text)
os.chmod(fn, 0755)
- # create a blank pre-revprop-change file if one doesn't
- # already exist to allow remote modification of revision
- # properties (see http://svnbook.red-bean.com/en/1.1/ch05s02.html)
- fn = os.path.join(self._repo.fs_path, self._repo.name, 'hooks', 'pre-revprop-change')
- if not os.path.exists(fn):
- with open(fn, 'wb') as fp:
- fp.write('#!/bin/sh\n')
- os.chmod(fn, 0755)
def _revno(self, oid):
return int(oid.split(':')[1])
http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/6049825c/ForgeSVN/forgesvn/tests/model/test_repository.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/tests/model/test_repository.py b/ForgeSVN/forgesvn/tests/model/test_repository.py
index 2e32676..5174c00 100644
--- a/ForgeSVN/forgesvn/tests/model/test_repository.py
+++ b/ForgeSVN/forgesvn/tests/model/test_repository.py
@@ -131,10 +131,7 @@ class TestSVNRepo(unittest.TestCase, RepoImplTestBase):
shutil.rmtree(dirname)
repo.init()
repo._impl.clone_from('file://' + repo_path)
- assert os.path.exists('/tmp/testsvn/hooks/pre-revprop-change')
- assert os.access('/tmp/testsvn/hooks/pre-revprop-change', os.X_OK)
- with open('/tmp/testsvn/hooks/pre-revprop-change') as f: c = f.read()
- self.assertEqual(c, '#!/bin/sh\n')
+ assert not os.path.exists('/tmp/testsvn/hooks/pre-revprop-change')
assert os.path.exists('/tmp/testsvn/hooks/post-commit')
assert os.access('/tmp/testsvn/hooks/post-commit', os.X_OK)
with open('/tmp/testsvn/hooks/post-commit') as f: c = f.read()
@@ -180,10 +177,7 @@ class TestSVNRepo(unittest.TestCase, RepoImplTestBase):
shutil.rmtree(dirname)
repo.init()
repo._impl.clone_from('file://' + repo_path)
- assert os.path.exists('/tmp/testsvn/hooks/pre-revprop-change')
- assert os.access('/tmp/testsvn/hooks/pre-revprop-change', os.X_OK)
- with open('/tmp/testsvn/hooks/pre-revprop-change') as f: c = f.read()
- self.assertEqual(c, '#!/bin/sh\n')
+ assert not os.path.exists('/tmp/testsvn/hooks/pre-revprop-change')
assert os.path.exists('/tmp/testsvn/hooks/post-commit')
assert os.access('/tmp/testsvn/hooks/post-commit', os.X_OK)
with open('/tmp/testsvn/hooks/post-commit') as f: c = f.read()