Skip to content
This repository was archived by the owner on Nov 11, 2024. It is now read-only.

Fix py.test rootdir on OSX - #194

Merged
embray merged 1 commit into
astropy:masterfrom
embray:osx-tmpdir-fix
Oct 1, 2015
Merged

embray merged 1 commit into
astropy:masterfrom
embray:osx-tmpdir-fix

Conversation

@embray

@embray embray commented Sep 30, 2015

Copy link
Copy Markdown
Member

This is a followup to #189 needed for things to work properly in OSX, where tempfile.mkdtemp does not return a real path.

This will also need to be backported to the v1.0.x branch.

…in OSX, where tempfile.mkdtemp does *not* return a real path.
@embray embray added this to the v1.0.4 milestone Sep 30, 2015
@embray

embray commented Sep 30, 2015

Copy link
Copy Markdown
Member Author

Mysterious appveyor fail on Python 2.6...

@embray

embray commented Oct 1, 2015

Copy link
Copy Markdown
Member Author

Incidentally, the PR in which I tested this against Astropy, astropy/astropy#4202, passed. So I went ahead and merged it. Not sure what the build problems are that I'm having here but I think it's just a build system problem and not a real problem.

embray added a commit that referenced this pull request Oct 1, 2015
@embray
embray merged commit 01a97a2 into astropy:master Oct 1, 2015
@embray
embray deleted the osx-tmpdir-fix branch October 1, 2015 14:03
embray added a commit that referenced this pull request Oct 1, 2015
Fix py.test rootdir on OSX
Conflicts:
	astropy_helpers/commands/_test_compat.py
embray added a commit to embray/astropy that referenced this pull request Oct 1, 2015
@embray

embray commented Oct 1, 2015

Copy link
Copy Markdown
Member Author

Backported in 61cb890, and also ported to astropy/astropy#4020

embray added a commit to embray/astropy that referenced this pull request Oct 1, 2015
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Dec 17, 2015
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Jun 12, 2016
drdavella pushed a commit to astropy/pytest-remotedata that referenced this pull request Sep 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants