Skip to content

Fixed missed inline import for subprocess.Popen - #1785

Merged
embray merged 1 commit into
astropy:masterfrom
jwoillez:fix_setup_helpers_Popen
Nov 14, 2013
Merged

embray merged 1 commit into
astropy:masterfrom
jwoillez:fix_setup_helpers_Popen

Conversation

@jwoillez

Copy link
Copy Markdown
Member

@astrofrog

Copy link
Copy Markdown
Member

@embray @eteq - if possible I think we should get this in the final 0.3.0

@astrofrog

Copy link
Copy Markdown
Member

(but the RC has already been tagged, so we shouldn't delay that, let's just backport this to 0.3.x once the branch is created, then include it in the final version)

@eteq

eteq commented Nov 14, 2013

Copy link
Copy Markdown
Member

I agree with @astrofrog this should go into 0.3.0 - @embray, do you want to merge this yourself so you can backport it, or can we just merge it and you'll run the script before the final release?

@embray

embray commented Nov 14, 2013

Copy link
Copy Markdown
Member

Oops. This may well be my fault in fact. Yes, it's because of this sort of thing that we make release candidates :)

(This might be a good place to mention that the 0.3 release candidate is out and people should download it and use it: https://testpypi.python.org/pypi/astropy/0.3rc1)

embray added a commit that referenced this pull request Nov 14, 2013
Fixed missed inline import for subprocess.Popen
@embray
embray merged commit 7a9abfd into astropy:master Nov 14, 2013
embray added a commit that referenced this pull request Nov 14, 2013
Fixed missed inline import for subprocess.Popen
@jwoillez
jwoillez deleted the fix_setup_helpers_Popen branch November 18, 2013 09:33
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

Successfully merging this pull request may close these issues.

4 participants