Skip to content

Conversation

@bedroge
Copy link
Collaborator

@bedroge bedroge commented Jun 30, 2020

No description provided.


- Answer the simple questions, e.g. about the installation path, and wait
for a couple of hours for the script to complete.

Copy link
Contributor

Choose a reason for hiding this comment

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

The installation path can also be set in the environment as $EPREFIX

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Haven't tried this myself (does that also skip the other questions about wishing the script luck?), but added a note about it.

Copy link
Contributor

Choose a reason for hiding this comment

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

It does not skip the questions, but saves time typing

Copy link
Contributor

Choose a reason for hiding this comment

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

And it avoids accidentally installing to a different location... :)

+1 for defining $EPREFIX up front

I also had to do this, because some /etc/profile.d scripts in the system I'm playing on define $LD_LIBRARY_PATH & co:

export SHELL="/bin/bash --norc --noprofile

The Prefix bootstrap script gives up when it sees things like $LD_LIBRARY_PATH and $CPATH defined in a freshly created shell.

peterstol
peterstol previously approved these changes Jun 30, 2020
Copy link
Contributor

@peterstol peterstol left a comment

Choose a reason for hiding this comment

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

Looks good to me

README.md Outdated

- Run the bootstrap script inside the container:
```
singularity exec ./centos8.simg ./bootstrap-prefix.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

singularity exec should use consistent image name (prefix.simg or prefix.sif)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Made the names consistent with the other command. Still using .simg here since I was running it with an older 2.x version on Ubuntu, but could use .sif (Singularity 3.x) as well.

Comment on lines 5 to 9
dnf install -y gcc gcc-c++ diffutils

%environment
export LC_ALL=C
export PATH=/usr/games:$PATH
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
dnf install -y gcc gcc-c++ diffutils
%environment
export LC_ALL=C
export PATH=/usr/games:$PATH
dnf install -y gcc gcc-c++ diffutils wget
# Add Gentoo Prefix Bootstrap
wget https://gitweb.gentoo.org/repo/proj/prefix.git/plain/scripts/bootstrap-prefix.sh -P /usr/local/bin
chmod +x /usr/local/bin/bootstrap-prefix.sh
%environment
export LC_ALL=C
export PATH=/usr/games:/usr/local/bin:$PATH

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

For now this will probably work, but I think it's better to include our own version/copy of the bootstrap script. That's a version that we tested (and possibly we want to modify it a bit). Always taking the latest version from the Gentoo git might give unexpected errors/results.

Copy link
Contributor

Choose a reason for hiding this comment

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

Jup, agreed. And we've seen that the bootstrap script isn't terribly stable...

@bedroge
Copy link
Collaborator Author

bedroge commented Jul 2, 2020

Merged my instructions into the new README, and improved some things (e.g. the def file).

boegel and others added 4 commits July 2, 2020 18:50
…t $SHELL may contain something like '/bin/bash --noprofile --norc'
fix determining shell in bootstrap_startscript
Make bootstrap script executable

Co-authored-by: Peter Stol <[email protected]>
Copy link
Contributor

@peterstol peterstol left a comment

Choose a reason for hiding this comment

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

First version tested

@peterstol peterstol merged commit a93b84d into master Aug 14, 2020
@bedroge bedroge deleted the install_prefix branch August 14, 2020 08:46
poksumdo pushed a commit to poksumdo/compatibility-layer that referenced this pull request Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants