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 the default tmp dir (fixes a few tests) #1

Closed
wants to merge 1 commit into from
Closed

Fix the default tmp dir (fixes a few tests) #1

wants to merge 1 commit into from

Conversation

chilts
Copy link

@chilts chilts commented Jun 22, 2012

Hiya,

I noticed the tests weren't completely working so I've fixed the default /tmp dir (should be /tmp/) since otherwise files of names /tmpclike-oX4RWS-postfix were failing to be created - should have been /tmp/clike-oX4RWS-postfix.

Cheers,
Andy

@raszi raszi closed this in 1155724 Jun 25, 2012
@raszi
Copy link
Owner

raszi commented Jun 25, 2012

It was a better idea to fix the tests, because the environment variables may or may not contain a tailing slash.

Thanks for reporting!

@silkentrance silkentrance mentioned this pull request Apr 28, 2020
@chilts
Copy link
Author

chilts commented Apr 30, 2020

Holy shit ... that's cool! Thanks for merging and releasing. :)

@silkentrance
Copy link
Collaborator

lol, just saw that this was a very old issue. damn you github notifications 😁

Repository owner locked as resolved and limited conversation to collaborators Apr 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants