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

Use $TMP instead of /tmp #46

Closed
lstor opened this issue Jan 5, 2015 · 5 comments
Closed

Use $TMP instead of /tmp #46

lstor opened this issue Jan 5, 2015 · 5 comments

Comments

@lstor
Copy link

lstor commented Jan 5, 2015

...because $TMP should work on Linux, Mac and Windows :)

@twolfson
Copy link
Owner

twolfson commented Jan 5, 2015

I wasn't really considering Windows but I guess if it makes sense then we can add it. For what it's worth, that one line install is based off of git-extras so you should complain to them too =P

https://github.com/tj/git-extras/wiki/Installation#clone--tarball

I will take a look at fixing this by the end of the weekend.

@lstor
Copy link
Author

lstor commented Jan 5, 2015

Yeah, the one line install is easy to fix manually anyway. I was thinking more of the "My colors don't work as advertised"-script.

With a tiny bit of extra care, I don't see why these things shouldn't work on MsysGit and/or cygwin. And it's pretty neat!

@twolfson
Copy link
Owner

twolfson commented Jan 7, 2015

I am looking into this now. It looks like msysgit's $TMP is /tmp. If cygwin is the same, then I might close this as a wontfix =/

@twolfson
Copy link
Owner

twolfson commented Jan 7, 2015

Yep, cygwin seems to be the same. Can you elaborate more on your situation such that /tmp doesn't exist?

@twolfson twolfson closed this as completed Jan 7, 2015
@lstor
Copy link
Author

lstor commented Jan 7, 2015

Hmm, it worked now. Must have been something I did on Monday -- my bad. Thanks for looking into it, though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants