Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix mktemp invocation in ui makefile #907

Closed
stuartnelson3 opened this issue Jul 16, 2017 · 4 comments · Fixed by #971
Closed

Fix mktemp invocation in ui makefile #907

stuartnelson3 opened this issue Jul 16, 2017 · 4 comments · Fixed by #971

Comments

@stuartnelson3
Copy link
Contributor

stuartnelson3 commented Jul 16, 2017

on macOS:

# mktemp ./elm-XXXXXXXXXX.js
./elm-XXXXXXXXXX.js
~/s/g/p/a/u/app (master *%=)
# ls
CONTRIBUTING.md   README.md         elm-stuff         favicon.ico       script.js         tests
Makefile          elm-package.json  elm-XXXXXXXXXX.js index.html        src

mktemp isn't using the provided template.

@mxinden
Copy link
Member

mxinden commented Jul 16, 2017

@stuartnelson3 Strange, seems to work fine with Linux. @w0rm are you facing the same issues?

@w0rm
Copy link
Member

w0rm commented Jul 16, 2017

@mxinden yeah, I could reproduce this on macOS

make build-all
cd /Users/w0rm/Work/Go/src/github.com/prometheus/alertmanager/ui/app && /Applications/Xcode.app/Contents/Developer/usr/bin/make script.js
>> building elm-env docker image
>> format front-end code
Processing multiple files...
>> building script.js
Success! Compiled 1 module.                                         
Successfully generated ./elm-XXXXXXXXXX.js
make: go-bindata: No such file or directory
make: *** [ui/bindata.go] Error 1

@w0rm
Copy link
Member

w0rm commented Sep 1, 2017

@mxinden @stuartnelson3 any idea how to fix this on macOS?

@stuartnelson3
Copy link
Contributor Author

Not sure off the top of my head, but I guess you can check the macos man page for mktemp

hh pushed a commit to ii/alertmanager that referenced this issue Apr 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants