Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Sep 28, 2020

Requires to avoid that this pattern is added again for every extension that is being installed, since EasyBlock.rpath_filter_dirs is passed down to Toolchain.prepare every time, which leads to:

== 2020-09-28 09:03:43,588 toolchain.py:973 DEBUG Combined RPATH filter: '/lib.*,/usr.*,/tmp/eb-fx9yc8f8.*,
/tmp/easybuild/build/RbundleBioconductor/3.11/foss-2020a-R-4.0.0.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.
*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,
.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*,.*/lib(64)?/stubs/?.*'

reported by @terjekv

@boegel boegel added the bug fix label Sep 28, 2020
@boegel boegel added this to the next release (4.3.1) milestone Sep 28, 2020
@boegel boegel requested a review from ocaisa September 28, 2020 18:18
Copy link
Member

@migueldiascosta migueldiascosta left a comment

Choose a reason for hiding this comment

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

lgtm

@migueldiascosta
Copy link
Member

Going in, thanks @boegel!

@migueldiascosta migueldiascosta merged commit 8291a1b into easybuilders:develop Oct 8, 2020
@boegel boegel deleted the lib_stubs_rpath_filter branch October 8, 2020 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants