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

add cuda.py #145

Merged
merged 9 commits into from
Mar 21, 2013
Merged

add cuda.py #145

merged 9 commits into from
Mar 21, 2013

Conversation

fgeorgatos
Copy link
Collaborator

Signed-off-by: Fotis Georgatos fotis.georgatos@uni.lu

boegel and others added 2 commits March 8, 2013 17:17
Signed-off-by: Fotis Georgatos <fotis.georgatos@uni.lu>
cmd = install_script + " --prefix=" + self.installdir
else:
# the following would require to include "osdependencies = 'libglut'" because of -samples
# installparams = "-samplespath=%s/samples/ -toolkitpath=%s -samples -toolkit" % (self.installdir, self.installdir))
Copy link
Member

Choose a reason for hiding this comment

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

@fgeorgatos: Please create an issue to fix this. We should actually (optionally) install the samples and use them as a kind of test suite for CUDA, to verify that it works (it should probably be enabled by default). We can dynamically require libglut as a dependency if the samples are being built/installed.

Copy link
Contributor

Choose a reason for hiding this comment

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

I like the idea of the samples install being optional, and passed in via the CUDA-5.0.35.eb file. I think it should be enabled by default but that's not a strong opinion. Then the choice of whether to install the samples is just a change to the EasyConfig.

Copy link
Member

Choose a reason for hiding this comment

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

I agree that installing the samples should be enabled by default. The libglut dependency should be easy to fulfill with EasyBuild.

Copy link
Member

Choose a reason for hiding this comment

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

A dedicated issue was created to enhance the easyblock for the samples, see #147.

@ajdecon
Copy link
Contributor

ajdecon commented Mar 19, 2013

Except for the permissions issue, this looks fine to me. (not sure why this exists... was a problem observed during an install?)

boegel and others added 5 commits March 20, 2013 19:40
…around on issue on systems where an X11 term is being used
don't use full path for Perl install scripts, unset DISPLAY env var
…ir, not needed (leftovers from easyblock that was used as a base for this one)
check unsetting of DISPLAY, remove chmod'ing that's obsolete
@georgets
Copy link
Contributor

Looks good!

@boegel
Copy link
Member

boegel commented Mar 21, 2013

This has seen extensive testing together with easybuilders/easybuild-easyconfigs#164, and after a couple of rounds of improvements, should now be ready for production, so merging it in.

CUDA support in toolchains in the framework is in the works.

boegel added a commit that referenced this pull request Mar 21, 2013
@boegel boegel merged commit 113d0a5 into easybuilders:develop Mar 21, 2013
@fgeorgatos fgeorgatos deleted the contrib_CUDA_easyblock branch March 27, 2013 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants