You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by jo...@apache.org on 2013/04/26 16:40:44 UTC

[1/3] git commit: [#6069] ticket:317 Change code snapshots to zip files

Updated Branches:
  refs/heads/master c01c95eb1 -> b9aaa6149


[#6069] ticket:317 Change code snapshots to zip files


Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/352b74b8
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/352b74b8
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/352b74b8

Branch: refs/heads/master
Commit: 352b74b8b981b1ffdcaaa4d2fab522ab4dc41d87
Parents: c01c95e
Author: Yuriy Arhipov <yu...@yandex.ru>
Authored: Mon Apr 15 16:05:37 2013 +0400
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Fri Apr 26 14:36:07 2013 +0000

----------------------------------------------------------------------
 Allura/allura/model/repository.py                |    4 ++--
 ForgeGit/forgegit/model/git_repo.py              |    6 +++---
 ForgeGit/forgegit/tests/model/test_repository.py |   14 +++++++-------
 ForgeSVN/forgesvn/model/svn.py                   |   10 +++++++---
 ForgeSVN/forgesvn/tests/model/test_repository.py |    4 ++--
 5 files changed, 21 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/352b74b8/Allura/allura/model/repository.py
----------------------------------------------------------------------
diff --git a/Allura/allura/model/repository.py b/Allura/allura/model/repository.py
index 8a95993..53d166f 100644
--- a/Allura/allura/model/repository.py
+++ b/Allura/allura/model/repository.py
@@ -272,7 +272,7 @@ class Repository(Artifact, ActivityObject):
         return filename
 
     def tarball_url(self, revision):
-        filename = '%s%s' % (self.tarball_filename(revision), '.tar.gz')
+        filename = '%s%s' % (self.tarball_filename(revision), '.zip')
         r = os.path.join(self.tool,
                          self.project.shortname[:1],
                          self.project.shortname[:2],
@@ -283,7 +283,7 @@ class Repository(Artifact, ActivityObject):
 
     def get_tarball_status(self, revision):
         pathname = os.path.join(self.tarball_path, self.tarball_filename(revision))
-        filename = '%s%s' % (pathname, '.tar.gz')
+        filename = '%s%s' % (pathname, '.zip')
         tmpfilename = '%s%s' % (pathname, '.tmp')
 
         if os.path.isfile(filename):

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/352b74b8/ForgeGit/forgegit/model/git_repo.py
----------------------------------------------------------------------
diff --git a/ForgeGit/forgegit/model/git_repo.py b/ForgeGit/forgegit/model/git_repo.py
index 5c79e34..2511ce0 100644
--- a/ForgeGit/forgegit/model/git_repo.py
+++ b/ForgeGit/forgegit/model/git_repo.py
@@ -345,11 +345,11 @@ class GitImplementation(M.RepositoryImplementation):
         if not os.path.exists(self._repo.tarball_path):
             os.makedirs(self._repo.tarball_path)
         archive_name = self._repo.tarball_filename(commit)
-        filename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.tar.gz'))
+        filename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.zip'))
         tmpfilename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.tmp'))
         try:
-            with gzip.open(tmpfilename, 'w') as fp:
-                self._git.archive(fp, format='tar', treeish=commit, prefix=archive_name + '/')
+            with open(tmpfilename, 'wb') as archive_file:
+                self._git.archive(archive_file, format='zip', treeish=commit, prefix=archive_name + '/')
             os.rename(tmpfilename, filename)
         finally:
             if os.path.exists(tmpfilename):

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/352b74b8/ForgeGit/forgegit/tests/model/test_repository.py
----------------------------------------------------------------------
diff --git a/ForgeGit/forgegit/tests/model/test_repository.py b/ForgeGit/forgegit/tests/model/test_repository.py
index 46c2daa..1614069 100644
--- a/ForgeGit/forgegit/tests/model/test_repository.py
+++ b/ForgeGit/forgegit/tests/model/test_repository.py
@@ -258,12 +258,12 @@ class TestGitRepo(unittest.TestCase, RepoImplTestBase):
         self.assertEqual(new_tree.other_ids, orig_tree.other_ids)
 
     def test_tarball(self):
-        if os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz"):
-            os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz")
+        if os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip"):
+            os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip")
         assert_equal(self.repo.tarball_path, '/tmp/tarball/git/t/te/test/testgit.git')
-        assert_equal(self.repo.tarball_url('HEAD'), 'file:///git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz')
+        assert_equal(self.repo.tarball_url('HEAD'), 'file:///git/t/te/test/testgit.git/test-src-git-HEAD.zip')
         self.repo.tarball('HEAD')
-        assert os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz")
+        assert os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip")
 
     def test_all_commit_ids(self):
         cids = list(self.repo.all_commit_ids())
@@ -292,15 +292,15 @@ class TestGitRepo(unittest.TestCase, RepoImplTestBase):
                 'name': u'README'}])
 
     def test_tarball_status(self):
-        if os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz"):
-            os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz")
+        if os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip"):
+            os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip")
         if os.path.isfile("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tmp"):
             os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tmp")
         if os.path.isdir("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD/"):
             os.removedirs("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD/")
         self.repo.tarball('HEAD')
         assert_equal(self.repo.get_tarball_status('HEAD'), 'ready')
-        os.rename("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tar.gz",
+        os.rename("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.zip",
                   "/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tmp")
         assert_equal(self.repo.get_tarball_status('HEAD'), 'busy')
         os.remove("/tmp/tarball/git/t/te/test/testgit.git/test-src-git-HEAD.tmp")

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/352b74b8/ForgeSVN/forgesvn/model/svn.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/model/svn.py b/ForgeSVN/forgesvn/model/svn.py
index de05fd0..8fdf628 100644
--- a/ForgeSVN/forgesvn/model/svn.py
+++ b/ForgeSVN/forgesvn/model/svn.py
@@ -29,6 +29,7 @@ from datetime import datetime
 import tempfile
 import tarfile
 from shutil import rmtree
+from zipfile import ZipFile
 
 import tg
 import pysvn
@@ -641,7 +642,7 @@ class SVNImplementation(M.RepositoryImplementation):
             os.makedirs(self._repo.tarball_path)
         path = os.path.join(self._repo.tarball_path, commit)
         archive_name = self._repo.tarball_filename(commit)
-        filename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.tar.gz'))
+        filename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.zip'))
         tmpfilename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.tmp'))
         if os.path.exists(path):
             rmtree(path)
@@ -649,8 +650,11 @@ class SVNImplementation(M.RepositoryImplementation):
             self._svn.export(self._url,
                              path,
                              revision=pysvn.Revision(pysvn.opt_revision_kind.number, commit))
-            with tarfile.open(tmpfilename, "w:gz") as tar:
-                tar.add(path, arcname=archive_name)
+            with ZipFile(tmpfilename, 'w') as tarball_zip:
+                for dirname, subdirs, files in os.walk(path):
+                    for f in files:
+                        tarball_zip.write(os.path.join(dirname, f), os.path.relpath(os.path.join(dirname, f), path))
+
             os.rename(tmpfilename, filename)
         finally:
             rmtree(path)

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/352b74b8/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 e8fb14e..8107829 100644
--- a/ForgeSVN/forgesvn/tests/model/test_repository.py
+++ b/ForgeSVN/forgesvn/tests/model/test_repository.py
@@ -289,9 +289,9 @@ class TestSVNRepo(unittest.TestCase, RepoImplTestBase):
 
     def test_tarball(self):
         assert_equal(self.repo.tarball_path, '/tmp/tarball/svn/t/te/test/testsvn')
-        assert_equal(self.repo.tarball_url('1'), 'file:///svn/t/te/test/testsvn/test-src-1.tar.gz')
+        assert_equal(self.repo.tarball_url('1'), 'file:///svn/t/te/test/testsvn/test-src-1.zip')
         self.repo.tarball('1')
-        assert os.path.isfile("/tmp/tarball/svn/t/te/test/testsvn/test-src-1.tar.gz")
+        assert os.path.isfile("/tmp/tarball/svn/t/te/test/testsvn/test-src-1.zip")
 
     def test_is_empty(self):
         assert not self.repo.is_empty()


[3/3] git commit: [#6069] Bumped version of ForgeHg

Posted by jo...@apache.org.
[#6069] Bumped version of ForgeHg

Signed-off-by: Cory Johns <cj...@slashdotmedia.com>


Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/b9aaa614
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/b9aaa614
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/b9aaa614

Branch: refs/heads/master
Commit: b9aaa6149cbe42edb9110b617584160492879353
Parents: 49933de
Author: Cory Johns <cj...@slashdotmedia.com>
Authored: Fri Apr 26 14:36:38 2013 +0000
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Fri Apr 26 14:36:38 2013 +0000

----------------------------------------------------------------------
 requirements-sf.txt |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/b9aaa614/requirements-sf.txt
----------------------------------------------------------------------
diff --git a/requirements-sf.txt b/requirements-sf.txt
index 0923185..351f054 100644
--- a/requirements-sf.txt
+++ b/requirements-sf.txt
@@ -4,7 +4,7 @@ akismet==0.2.0
 amqplib==0.6.1
 kombu==1.0.4
 coverage==3.5a1-20110413
-ForgeHg==0.1.8
+ForgeHg==0.1.9
 ForgePastebin==0.2.6
 mechanize==0.2.4
 MySQL-python==1.2.3c1


[2/3] git commit: [#6069] ticket:324 added prefix to zip tarball

Posted by jo...@apache.org.
[#6069] ticket:324 added prefix to zip tarball


Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/49933de9
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/49933de9
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/49933de9

Branch: refs/heads/master
Commit: 49933de99f876d44db7663094145143ff50d55a4
Parents: 352b74b
Author: Yuriy Arhipov <yu...@yandex.ru>
Authored: Thu Apr 25 04:10:47 2013 +0400
Committer: Cory Johns <cj...@slashdotmedia.com>
Committed: Fri Apr 26 14:36:08 2013 +0000

----------------------------------------------------------------------
 ForgeSVN/forgesvn/model/svn.py                   |   12 +++++++-----
 ForgeSVN/forgesvn/tests/model/test_repository.py |    3 +++
 2 files changed, 10 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/49933de9/ForgeSVN/forgesvn/model/svn.py
----------------------------------------------------------------------
diff --git a/ForgeSVN/forgesvn/model/svn.py b/ForgeSVN/forgesvn/model/svn.py
index 8fdf628..2994aeb 100644
--- a/ForgeSVN/forgesvn/model/svn.py
+++ b/ForgeSVN/forgesvn/model/svn.py
@@ -29,7 +29,7 @@ from datetime import datetime
 import tempfile
 import tarfile
 from shutil import rmtree
-from zipfile import ZipFile
+from zipfile import ZipFile, ZIP_DEFLATED
 
 import tg
 import pysvn
@@ -640,8 +640,8 @@ class SVNImplementation(M.RepositoryImplementation):
     def tarball(self, commit):
         if not os.path.exists(self._repo.tarball_path):
             os.makedirs(self._repo.tarball_path)
-        path = os.path.join(self._repo.tarball_path, commit)
         archive_name = self._repo.tarball_filename(commit)
+        path = os.path.join(self._repo.tarball_path, archive_name)
         filename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.zip'))
         tmpfilename = os.path.join(self._repo.tarball_path, '%s%s' % (archive_name, '.tmp'))
         if os.path.exists(path):
@@ -651,9 +651,11 @@ class SVNImplementation(M.RepositoryImplementation):
                              path,
                              revision=pysvn.Revision(pysvn.opt_revision_kind.number, commit))
             with ZipFile(tmpfilename, 'w') as tarball_zip:
-                for dirname, subdirs, files in os.walk(path):
-                    for f in files:
-                        tarball_zip.write(os.path.join(dirname, f), os.path.relpath(os.path.join(dirname, f), path))
+               for root, dirs, files in os.walk(path):
+                    for name in files:
+                        file_to_zip = os.path.join(root, name)
+                        arcname = file_to_zip[len(os.path.dirname(path)):].strip('/')
+                        tarball_zip.write(file_to_zip, arcname, compress_type=ZIP_DEFLATED)
 
             os.rename(tmpfilename, filename)
         finally:

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/49933de9/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 8107829..68e884c 100644
--- a/ForgeSVN/forgesvn/tests/model/test_repository.py
+++ b/ForgeSVN/forgesvn/tests/model/test_repository.py
@@ -21,6 +21,7 @@ import unittest
 import pkg_resources
 from itertools import count, product
 from datetime import datetime
+from zipfile import ZipFile
 
 from collections import defaultdict
 from pylons import tmpl_context as c
@@ -292,6 +293,8 @@ class TestSVNRepo(unittest.TestCase, RepoImplTestBase):
         assert_equal(self.repo.tarball_url('1'), 'file:///svn/t/te/test/testsvn/test-src-1.zip')
         self.repo.tarball('1')
         assert os.path.isfile("/tmp/tarball/svn/t/te/test/testsvn/test-src-1.zip")
+        tarball_zip = ZipFile('/tmp/tarball/svn/t/te/test/testsvn/test-src-1.zip', 'r')
+        assert_equal(tarball_zip.namelist(), ['test-src-1/README'])
 
     def test_is_empty(self):
         assert not self.repo.is_empty()