Commit2ba897b

committed

fix(repo): make it serializable with pickle

It's entirely untested if this repo still does the right thing,but I'd think it does.Fixes#504

1 parentae6e26e commit2ba897bCopy full SHA for 2ba897b

File tree

+26

-3

lines changed

+26

-3

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ Changelog`
`6`	`6`	`=============================`
`7`	`7`
`8`	`8`	* `tag.commit` will now resolve commits deeply.
	`9`	+* `Repo` objects can now be pickled, which helps with multi-processing.
`9`	`10`
`10`	`11`	* `DiffIndex.iter_change_type(...)` produces better results when diffing
`11`	`12`	`2.0.8 - Features and Bugfixes`

Lines changed: 20 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -213,6 +213,17 @@ def _deplete_buffer(fno, handler, buf_list, wg=None):`
`213`	`213`
`214`	`214`	`defdashify(string):`
`215`	`215`	`returnstring.replace('_','-')`
	`216`	`+`
	`217`	`+`
	`218`	`+defslots_to_dict(self,exclude=()):`
	`219`	`+returndict((s,getattr(self,s))forsinself.__slots__ifsnotinexclude)`
	`220`	`+`
	`221`	`+`
	`222`	`+defdict_to_slots_and__excluded_are_none(self,d,excluded=()):`
	`223`	`+fork,vind.items():`
	`224`	`+setattr(self,k,v)`
	`225`	`+forkinexcluded:`
	`226`	`+setattr(self,k,None)`
`216`	`227`
`217`	`228`	`## -- End Utilities -- @}`
`218`	`229`
`@@ -235,7 +246,15 @@ class Git(LazyMixin):`
`235`	`246`	`"""`
`236`	`247`	`__slots__= ("_working_dir","cat_file_all","cat_file_header","_version_info",`
`237`	`248`	`"_git_options","_environment")`
`238`		`-`
	`249`	`+`
	`250`	`+_excluded_= ('cat_file_all','cat_file_header','_version_info')`
	`251`	`+`
	`252`	`+def__getstate__(self):`
	`253`	`+returnslots_to_dict(self,exclude=self._excluded_)`
	`254`	`+`
	`255`	`+def__setstate__(self,d):`
	`256`	`+dict_to_slots_and__excluded_are_none(self,d,excluded=self._excluded_)`
	`257`	`+`
`239`	`258`	`# CONFIGURATION`
`240`	`259`	`# The size in bytes read from stdout when copying git's output to another stream`
`241`	`260`	`max_chunk_size=1024*64`

Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -93,7 +93,6 @@ class Repo(object):`
`93`	`93`
`94`	`94`	`'git_dir' is the .git repository directory, which is always set."""`
`95`	`95`	`DAEMON_EXPORT_FILE='git-daemon-export-ok'`
`96`		`-__slots__= ("working_dir","_working_tree_dir","git_dir","_bare","git","odb")`
`97`	`96`
`98`	`97`	`# precompiled regex`
`99`	`98`	`re_whitespace=re.compile(r'\s+')`

Lines changed: 5 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,8 @@`
`4`	`4`	`#`
`5`	`5`	`# This module is part of GitPython and is released under`
`6`	`6`	`# the BSD License: http://www.opensource.org/licenses/bsd-license.php`
	`7`	`+importpickle`
	`8`	`+`
`7`	`9`	`fromgit.test.libimport (`
`8`	`10`	`patch,`
`9`	`11`	`TestBase,`
`@@ -104,13 +106,15 @@ def test_tree_from_revision(self):`
`104`	`106`
`105`	`107`	`# try from invalid revision that does not exist`
`106`	`108`	`self.failUnlessRaises(BadName,self.rorepo.tree,'hello world')`
	`109`	`+`
	`110`	`+deftest_pickleable(self):`
	`111`	`+pickle.loads(pickle.dumps(self.rorepo))`
`107`	`112`
`108`	`113`	`deftest_commit_from_revision(self):`
`109`	`114`	`commit=self.rorepo.commit('0.1.4')`
`110`	`115`	`assertcommit.type=='commit'`
`111`	`116`	`assertself.rorepo.commit(commit)==commit`
`112`	`117`
`113`		`-deftest_commits(self):`
`114`	`118`	`mc=10`
`115`	`119`	`commits=list(self.rorepo.iter_commits('0.1.6',max_count=mc))`
`116`	`120`	`assertlen(commits)==mc`

Comments

(0)