[ictce/5.3.0] gputools/0.28 for R/3.0.2 (REVIEW)#546
[ictce/5.3.0] gputools/0.28 for R/3.0.2 (REVIEW)#546boegel merged 11 commits intoeasybuilders:developfrom
Conversation
Signed-off-by: Fotis Georgatos <[email protected]>
|
Automatic reply from Jenkins: Can I test this? |
|
forgot to mention that, this issue relies on |
|
nope: mea culpa, #549 has goolf only, |
|
Jenkins: ok to test |
|
@fgeorgatos: This needs a couple of kinks worked out. 0 & 4) The versioning makes little sense, you're supposed to be able to order versions... How about this? version = '0.28'
versionsuffix = '-<commit>'
|
Signed-off-by: Fotis Georgatos <[email protected]>
…onfigs into contrib_Rgputools
Signed-off-by: Fotis Georgatos <[email protected]>
There was a problem hiding this comment.
aha!! #1150 has the fix for this ;-) thanks, @pescobar !
addendum: the potential fix is now proposed instead inside the R easyblock:
easybuilders/easybuild-easyblocks#495
|
@fgeorgatos: this needs more work before it can go in, exactly because it's so tedious as you describe... |
|
Test report by @fgeorgatos |
|
@fgeorgatos: ping on this? I'm cleaning up the oldest PRs, in an effort to reduce the pile of open PRs. Is this still relevant? Any idea of the amount of extra effort required to get it ready for merge? |
|
I'd expect this to be mergeable, esp. if for ictce/5.5.0; where could we see the current build status?! |
|
@fgeorgatos please take a look at fgeorgatos#37 @fgeorgatos I'm still experiencing problems when testing this: Where is |
|
libcuda.so should be part of the nvidia driver |
style fixes in Rgputools easyconfig, rename to 'gputools'
|
You really exercise my memory when you ask me something about a build, 3 years later :-p |
|
Test report by @boegel |
add checksum for gputools
|
Test report by @boegel |
rename gputools easyconfig to get versionsuffix aspect right
|
Green lights everywhere, going in, thanks @fgeorgatos! |
|
fyi. bug #1 made it's appearance again on my side ; here's how it manifests itself: I have my workaround, as described above, however I feel obliged to report... |
|
btw. |
|
@fgeorgatos can you provide a more complete error/log for the unpack issue? what's the exact problem with the unpack? It seems we have something related in |
|
@boegel: well, the case had been described here: |
Hi,
this one looks surprisingly simple but it proved to be very tedious business (spin-off of #282); reason is you MUST:
first, be careful with the exact git commit ID that corresponds to v0.28, this is not exactly a 1-1 relationship :-P
Provide a workaround for tar/pax dogfight in R packages easybuild-easyblocks#313
dyn.load("/hello/big/world"), while R_LIB does get populatedlibcuda.so.1e5cb024works OK, while next commits (all called v0.28!) break down somewhere along the commit sequence.Let's focus on the
e5cb024case for now, that should be the one to get out working first.I need your help w. arguing the possible workarounds, I have a specified manual reproducible case at this point.
Replacing #282. Using a test case here may be recommended.