Merge pull request #1933 from r2d4/windows-fix

r2d4 · web-flow · commit ad8ea13de039 · 2017-09-11T13:40:18.000-07:00
Use os.Chmod instead of File.Chmod
diff --git a/pkg/minikube/bootstrapper/exec_runner.go b/pkg/minikube/bootstrapper/exec_runner.go
@@ -75,7 +75,7 @@ func (*ExecRunner) Copy(f assets.CopyableFile) error {
 	if err != nil {
 		return errors.Wrapf(err, "error converting permissions %s to integer", perms)
 	}
-	if err := target.Chmod(os.FileMode(perms)); err != nil {
+	if err := os.Chmod(targetPath, os.FileMode(perms)); err != nil {
 		return errors.Wrapf(err, "error changing file permissions for %s", targetPath)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ func (*ExecRunner) Copy(f assets.CopyableFile) error {`
`75`	`75`	`if err != nil {`
`76`	`76`	`return errors.Wrapf(err, "error converting permissions %s to integer", perms)`
`77`	`77`	`}`
`78`		`- if err := target.Chmod(os.FileMode(perms)); err != nil {`
	`78`	`+ if err := os.Chmod(targetPath, os.FileMode(perms)); err != nil {`
`79`	`79`	`return errors.Wrapf(err, "error changing file permissions for %s", targetPath)`
`80`	`80`	`}`
`81`	`81`