golang / dep

Go dependency management tool experiment (deprecated)
https://golang.github.io/dep/
BSD 3-Clause "New" or "Revised" License
12.87k stars 1.05k forks source link

Installation failure using install.sh in MacOS #2234

Closed r-pai closed 3 years ago

r-pai commented 4 years ago

As mentioned in the dep README.md, I was unable to install dep using brew as it was giving me error as mac os is old and stuff. I tried using install.sh and it failed. The output of the install script is added in the 4th question answer.

The issue is the script uses 'mktemp' command to create a temporary file. The OSX's mktemp tool has no default template and requires a template to be specified but in Linux , mktemp (GNU) does not require a template to be specified.

I fixed and installed dep in my mac using install.sh . Here is the fix https://github.com/r-pai/dep/commit/ca8601e1311e3e1f4fa375ff9e47b2ee60657006 Please let me know if I can submit a PR for this.

Issue Details

  1. What version of Go (go version) and dep (git describe --tags) are you using?? Ans) go version go1.12.9 darwin/amd64

  2. What dep installation command did you run? Ans) curl https://raw.githubusercontent.com/golang/dep/master/install.sh | sh

  3. What did you expect to see? Ans) dep to be installed

  4. What did you see instead? Ans) % Total % Received % Xferd Average Speed Time Time Time Current Dload Upload Total Spent Left Speed 100 5230 100 5230 0 0 4637 0 0:00:01 0:00:01 --:--:-- 4640 ARCH = amd64 OS = darwin Will install into /Users/mac/go/bin Fetching https://github.com/golang/dep/releases/latest.. Release Tag = v0.5.4 Fetching https://github.com/golang/dep/releases/tag/v0.5.4.. usage: mktemp [-d] [-q] [-t prefix] [-u] template ... mktemp [-d] [-q] [-u] -t prefix_

mvdan commented 3 years ago

Dep was officially deprecated earlier this year, and the proposal to archive this repository was accepted. As such, I'm closing outstanding issues before archiving the repository. For any further comments, please use the proposal thread on the Go issue tracker. Thanks!