Skip to content

FIX: Interfaces redirecting X crashed if $DISPLAY not defined #1027

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

Merged
merged 7 commits into from
Jan 22, 2015

Conversation

oesteban
Copy link
Contributor

No description provided.

@@ -987,7 +987,8 @@ def run(self, **inputs):
time.sleep(0.2) # give Xvfb time to start
if xvfb_proc.poll() is not None:
raise Exception('Error: Xvfb did not start')
old_displaynum = os.environ['DISPLAY']

# old_displaynum = os.getenv('DISPLAY')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

delete line instead of comment.

@satra
Copy link
Member

satra commented Jan 21, 2015

@oesteban - needs to be remerged with master

Conflicts:
	CHANGES
	nipype/interfaces/base.py
oesteban added a commit that referenced this pull request Jan 22, 2015
FIX: Interfaces redirecting X crashed if $DISPLAY not defined
@oesteban oesteban merged commit 0d3bbdc into nipy:master Jan 22, 2015
@oesteban oesteban deleted the fix/RedirectX branch January 22, 2015 12:07
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.

2 participants