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

Added jhbuild easyblock #1004

Closed
wants to merge 1 commit into from
Closed

Conversation

damianam
Copy link
Member

@damianam damianam commented Sep 30, 2016

This PR adds a jhbuild easyblock. Useful for installing a variety of packages (X.Org, GTK, GIMP, etc). A test PR in https://github.com/hpcugent/easybuild-easyconfigs will follow (next week). This is a first step to clean easybuilders/easybuild-easyconfigs#3463

Test easyconfig PR: easybuilders/easybuild-easyconfigs#3641

@boegel boegel modified the milestone: v3.x Nov 15, 2016
def __init__(self, *args, **kwargs):
"""Skip the install step."""
super(EB_jhbuild, self).__init__(*args, **kwargs)
self.cfg['skipsteps'] = "install"
Copy link
Member Author

Choose a reason for hiding this comment

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

I believe this is a bug that will create a broken dumped file, with skipsteps = 'install' instead of skipsteps = ['install']

@damianam
Copy link
Member Author

@boegel I guess this one won't make it to 3.1.0, right?

@damianam
Copy link
Member Author

@boegel, what should we do with this? It is not that useful for us anymore since we use your X11, but throwing it away is kind of a waste.

@boegel
Copy link
Member

boegel commented Aug 9, 2017

@damianam Well, if it's not going to be used we should close the PR?

If we ever need to get back to it, we can revisit this PR...

@damianam
Copy link
Member Author

damianam commented Aug 9, 2017

Sounds reasonable. I am closing it.

@damianam damianam closed this Aug 9, 2017
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