Commita2cd130

authored

Merge pull request#697 from cblegare/master

Remove trailing slash on drive path

2 parents28cbb95 +280e573 commita2cd130Copy full SHA for a2cd130

File tree

+13

-2

lines changed

+13

-2

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -24,5 +24,6 @@ Contributors are:`
`24`	`24`	`-Alexis Horgix Chotard`
`25`	`25`	`-Piotr Babij <piotr.babij _at_ gmail.com>`
`26`	`26`	`-Mikuláš Poul <mikulaspoul _at_ gmail.com>`
	`27`	`+-Charles Bouchard-Légaré <cblegare.atl _at_ ntis.ca>`
`27`	`28`
`28`	`29`	`Portions derived from other open source works and are clearly marked.`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -278,7 +278,7 @@ def _to_relative_path(cls, parent_repo, path):`
`278`	`278`	`ifnotpath.startswith(working_tree_linux):`
`279`	`279`	`raiseValueError("Submodule checkout path '%s' needs to be within the parents repository at '%s'"`
`280`	`280`	`% (working_tree_linux,path))`
`281`		`-path=path[len(working_tree_linux)+1:]`
	`281`	`+path=path[len(working_tree_linux.rstrip('/'))+1:]`
`282`	`282`	`ifnotpath:`
`283`	`283`	`raiseValueError("Absolute submodule path '%s' didn't yield a valid relative path"%path)`
`284`	`284`	`# end verify converted relative path makes sense`

Lines changed: 11 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`
`11`	`11`	`importgit`
`12`	`12`	`fromgit.cmdimportGit`
`13`		`-fromgit.compatimportstring_types`
	`13`	`+fromgit.compatimportstring_types,is_win`
`14`	`14`	`fromgit.excimport (`
`15`	`15`	`InvalidGitRepositoryError,`
`16`	`16`	`RepositoryDirtyError`
`@@ -911,3 +911,13 @@ def test_branch_renames(self, rw_dir):`
`911`	`911`	`parent_repo.submodule_update(to_latest_revision=True,force_reset=True)`
`912`	`912`	`assertsm_mod.commit()==sm_pfb.commit,"Now head should have been reset"`
`913`	`913`	`assertsm_mod.head.ref.name==sm_pfb.name`
	`914`	`+`
	`915`	`+@skipIf(notis_win,"Specifically for Windows.")`
	`916`	`+deftest_to_relative_path_with_super_at_root_drive(self):`
	`917`	`+classRepo(object):`
	`918`	`+working_tree_dir='D:\\'`
	`919`	`+super_repo=Repo()`
	`920`	`+submodule_path='D:\\submodule_path'`
	`921`	`+relative_path=Submodule._to_relative_path(super_repo,submodule_path)`
	`922`	`+msg='_to_relative_path should be "submodule_path" but was "%s"'%relative_path`
	`923`	`+assertrelative_path=='submodule_path',msg`

Comments

(0)