Skip to content

Conversation

@boegel
Copy link
Member

@boegel boegel commented Aug 24, 2020

This will come in useful when building certain software (GCC, binutils, ...) in an environment where you don't want to pick up anything from the host environment, for example when cross-compiling or when installing software on top of an alternative environment like Gentoo Prefix.

@boegel boegel added this to the next release (4.3.0) milestone Aug 24, 2020
@casparvl
Copy link
Contributor

Looks pretty straightforward to me. Just to clarify your ideas here: you don't envision the --sysroot option will do anything on the framework side I guess? I mean, I guess we mostly want to define this option in framework so that we can query its setting in the relevant EasyBlocks and customize behaviour accordingly there?

@boegel
Copy link
Member Author

boegel commented Aug 24, 2020

@casparvl Exactly, see easybuilders/easybuild-easyblocks#2142 (Perl) and easybuilders/easybuild-easyblocks#2143 (binutils) for example.

I'm not planning anything specific on the framework side for now, although it's possible we could enhance the sanity check when --sysroot is set, to make sure nothing outside the specified sysroot is picked up (by checking ldd output, etc.).

Whether that's a viable approach remains to be seen, but it could be interesting...

@casparvl
Copy link
Contributor

I'm not planning anything specific on the framework side for now, although it's possible we could enhance the sanity check when --sysroot is set, to make sure nothing outside the specified sysroot is picked up (by checking ldd output, etc.).

Interesting idea. I guess we could try and see with a couple of ldd's if 'all' cases where it picks up something from outside the sysroot are 'wrong'. I mean, the things outside the sysroot it should pick up on are drivers and such, but those shouldn't typically show up with an ldd I guess... It IS very hard to otherwise find out that you're linking against stuff outside of your sysroot...

Copy link
Contributor

@casparvl casparvl left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit b74b830 into easybuilders:develop Aug 25, 2020
@boegel boegel deleted the sysroot branch August 25, 2020 09:40
@boegel boegel added the EESSI Related to EESSI project label Aug 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

EESSI Related to EESSI project enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants