Commitbb2007c

committed

Set correct permissions when downloading tool

The extraction of some tools failed because the parent folder ofsome files wasn't created with useful permissions.Here we correctly set the permission such as the folder hasexec and write permission for the current user

1 parenta064c29 commitbb2007cCopy full SHA for bb2007c

File tree

1 file changed

-1

lines changed

tools
- download.go

1 file changed

-1

lines changed

`‎tools/download.go`

Lines changed: 8 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -423,7 +423,8 @@ func extractTarGz(body []byte, location string) (string, error) {`
`423`	`423`	`info:=header.FileInfo()`
`424`	`424`
`425`	`425`	`// Create parent folder`
`426`		`-iferr=os.MkdirAll(filepath.Dir(path),info.Mode());err!=nil {`
	`426`	`+dirmode:=info.Mode()\|os.ModeDir\|0700`
	`427`	`+iferr=os.MkdirAll(filepath.Dir(path),dirmode);err!=nil {`
`427`	`428`	`returnlocation,err`
`428`	`429`	`}`
`429`	`430`
`@@ -514,6 +515,12 @@ func extractBz2(body []byte, location string) (string, error) {`
`514`	`515`	`path:=filepath.Join(location,strings.Replace(header.Name,basedir,"",-1))`
`515`	`516`	`info:=header.FileInfo()`
`516`	`517`
	`518`	`+// Create parent folder`
	`519`	`+dirmode:=info.Mode()\|os.ModeDir\|0700`
	`520`	`+iferr=os.MkdirAll(filepath.Dir(path),dirmode);err!=nil {`
	`521`	`+returnlocation,err`
	`522`	`+}`
	`523`	`+`
`517`	`524`	`ifinfo.IsDir() {`
`518`	`525`	`iferr=os.MkdirAll(path,info.Mode());err!=nil {`
`519`	`526`	`returnlocation,err`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Commitbb2007c

File tree

1 file changed

1 file changed

`‎tools/download.go`

0 commit comments