BNLNPPS / esi-shell

Apache License 2.0
0 stars 0 forks source link

Add vim text editor to the container #60

Closed rahmans1 closed 2 months ago

rahmans1 commented 2 months ago

Add a light weight text editor to the container. Just chose vim because that's what I am used to.

Resolves #59

buddhasystem commented 2 months ago

Ouch! vim! :) Do we already have nano, at least?

plexoos commented 2 months ago

An editor can be installed... But I don't think that is customary to do so. Look at the various images distributed by major companies, they don't include editors or any additional tools beyond the main application . I guess when people use containers they usually assume a different workflow (although you can install whatever you want when you run it as root).

In our case when we run esi-shell the $HOME dir is mounted inside the container by default. Can we use editors installed on the host to modify files in the user HOME, so the changes are also visible within the container?

plexoos commented 2 months ago

Anyway, that said, it makes a bit more sense to install external tools with the system package manager rather than spack. I would like to reserve the spack environment for only direct dependencies to our primary applications. See next commit