correct the patch and apply it earlier in the R-INLA install#12927
correct the patch and apply it earlier in the R-INLA install#12927boegel merged 4 commits intoeasybuilders:developfrom
Conversation
|
@deniskristak @Darkless012 Can you clarify the |
|
@branfosj @boegel @Darkless012 it truly does look like a mistake..sorry for that, mustve left it there accidentaly |
|
Test report by @branfosj |
|
Test report by @branfosj |
|
@boegelbot please test @ generoso |
@deniskristak Thanks for clarifying! |
For reference: this test is from the system with no |
|
@boegel: Request for testing this PR well received on generoso PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 847721070 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
|
Going in, thanks @branfosj! |
(created using
eb --new-pr)On a system with no
rsyncI get:/bin/bash: line 9: rsync: command not foundduring the install offmesher. Testing shows that theR-INLA-21.05.02-skip-binaries.patchwas being applied after this component was installed, so the patching out ofrsynchad no impact.There is one issue though, which is why this is marked as a draft. As the patching happens earlier,
GMRFLibfileopenmp.cgetsINLA_LINK_WITH_OPENBLASadded before it is built. This is invalidccode and breaks the build. I do not know what this part of the patch was trying to do, so I've commented it out.