diff --git a/AUTHORS b/AUTHORS index 2b67d6cb3..98167ea7e 100644 --- a/AUTHORS +++ b/AUTHORS @@ -22,5 +22,6 @@ Contributors are: -Anson Mansfield -Ken Odegard -Alexis Horgix Chotard +-Piotr Babij Portions derived from other open source works and are clearly marked. diff --git a/git/repo/base.py b/git/repo/base.py index d3bdc9831..9ed3f7141 100644 --- a/git/repo/base.py +++ b/git/repo/base.py @@ -931,12 +931,16 @@ def _clone(cls, git, url, path, odb_default_type, progress, **kwargs): if not osp.isabs(path) and git.working_dir: path = osp.join(git._working_dir, path) + repo = cls(path, odbt=odbt) + + # retain env values that were passed to _clone() + repo.git.update_environment(**git.environment()) + # adjust remotes - there may be operating systems which use backslashes, # These might be given as initial paths, but when handling the config file # that contains the remote from which we were clones, git stops liking it # as it will escape the backslashes. Hence we undo the escaping just to be # sure - repo = cls(path, odbt=odbt) if repo.remotes: with repo.remotes[0].config_writer as writer: writer.set_value('url', Git.polish_url(repo.remotes[0].url)) diff --git a/git/test/test_repo.py b/git/test/test_repo.py index 312e67f92..86fb2f51f 100644 --- a/git/test/test_repo.py +++ b/git/test/test_repo.py @@ -201,6 +201,15 @@ def _assert_empty_repo(self, repo): pass # END test repos with working tree + @with_rw_directory + def test_clone_from_keeps_env(self, rw_dir): + original_repo = Repo.init(osp.join(rw_dir, "repo")) + environment = {"entry1": "value", "another_entry": "10"} + + cloned = Repo.clone_from(original_repo.git_dir, osp.join(rw_dir, "clone"), env=environment) + + assert_equal(environment, cloned.git.environment()) + def test_init(self): prev_cwd = os.getcwd() os.chdir(tempfile.gettempdir())