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.

Sign up for GitHub

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

Jump to bottom

Fix headless build #1133

Merged
merged 2 commits into from
Feb 18, 2022
Merged

Fix headless build #1133

merged 2 commits into from
Feb 18, 2022

Conversation

prabhuramachandran
Copy link
Member

  • Ensure that tests can run when toolkit is null.
  • Do not generate docs on build as this will fail on headless servers.

@prabhuramachandran
Do not generate docs on build.
f7a5834 
The process of building the docs requires a UI and on a Linux machine
this will require an X-server.  The docs are not packaced in the wheel
so this is entirely pointless and worse, this prevents installation of
Mayavi on a headless server.
@prabhuramachandran
Ensure that tests can run when toolkit is null.
3ee2c0a 
This is essential to be able to test on headless servers.
@prabhuramachandran prabhuramachandran merged commit c75bb7d into enthought:master Feb 18, 2022
@prabhuramachandran prabhuramachandran deleted the fix-headless-build branch February 18, 2022 11:32
@prabhuramachandran prabhuramachandran mentioned this pull request Jul 11, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rahulporuri rahulporuri rahulporuri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@prabhuramachandran @rahulporuri