Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Commit

Permalink
dep: update calculatePrune to not assume "/" as separtor
Browse files Browse the repository at this point in the history
dep.calculatePrune assumes "/" is the file separtor. This change fixes
an issue caused by that on Windows.

Fixes #775

Signed-off-by: Ibrahim AshShohail <[email protected]>
  • Loading branch information
ibrasho committed Jun 21, 2017
1 parent 0584d8c commit 07d30f6
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion txn_writer.go
Original file line number Diff line number Diff line change
Expand Up @@ -530,6 +530,7 @@ fail:
return failerr
}

// calculatePrune returns the paths of the packages to be deleted from vendorDir.
func calculatePrune(vendorDir string, keep []string, logger *log.Logger) ([]string, error) {
if logger != nil {
logger.Println("Calculating prune. Checking the following packages:")
Expand All @@ -547,7 +548,7 @@ func calculatePrune(vendorDir string, keep []string, logger *log.Logger) ([]stri
return nil
}

name := strings.TrimPrefix(path, vendorDir+"/")
name := filepath.ToSlash(strings.TrimPrefix(path, vendorDir+string(filepath.Separator)))
if logger != nil {
logger.Printf(" %s", name)
}
Expand Down

0 comments on commit 07d30f6

Please sign in to comment.