Fix duplicate repos_path check

tyru · tyru · commit 56a9bc27263c · 2017-09-25T00:30:46.000+09:00
If the same repos_path[] element exists in a different profile,
check fails wrongly.
diff --git a/lockjson/lockjson.go b/lockjson/lockjson.go
@@ -102,8 +102,8 @@ func validate(lockJSON *LockJSON) error {
 	}
 
 	// Validate if duplicate profiles[]/repos_path[] exist
-	dup = make(map[string]bool, len(lockJSON.Profiles)*10)
 	for _, profile := range lockJSON.Profiles {
+		dup = make(map[string]bool, len(lockJSON.Profiles)*10)
 		for _, reposPath := range profile.ReposPath {
 			if _, exists := dup[reposPath]; exists {
 				return errors.New("duplicate 'repos_path' (" + reposPath + ") in profile '" + profile.Name + "'")

Original file line number	Diff line number	Diff line change
`@@ -102,8 +102,8 @@ func validate(lockJSON *LockJSON) error {`
`102`	`102`	`}`
`103`	`103`
`104`	`104`	`// Validate if duplicate profiles[]/repos_path[] exist`
`105`		`- dup = make(map[string]bool, len(lockJSON.Profiles)*10)`
`106`	`105`	`for _, profile := range lockJSON.Profiles {`
	`106`	`+ dup = make(map[string]bool, len(lockJSON.Profiles)*10)`
`107`	`107`	`for _, reposPath := range profile.ReposPath {`
`108`	`108`	`if _, exists := dup[reposPath]; exists {`
`109`	`109`	`return errors.New("duplicate 'repos_path' (" + reposPath + ") in profile '" + profile.Name + "'")`