Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bash profile script to enable micromamba activate #9

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gdevenyi
Copy link

@gdevenyi gdevenyi commented Jun 4, 2024

Need to test.

@gdevenyi gdevenyi marked this pull request as draft June 4, 2024 14:34
@gdevenyi gdevenyi marked this pull request as ready for review June 4, 2024 14:43
@gdevenyi gdevenyi marked this pull request as draft June 4, 2024 14:43
@gdevenyi
Copy link
Author

gdevenyi commented Jun 4, 2024

Challenge here right now is Ubuntu actually doesn't treat login and interactive shells identically (loading of /etc/profile vs /etc/bash.bashrc), meanwhile RedHat does this.

Instead of adding a file in /etc/profile.d, we could trigger a micromamba shell init to the end of /etc/bash.bashrc, but it looks like RedHat installations seem to put the file at /etc/bashrc so I need to figure out how to handle things flexibly

@maresb
Copy link
Collaborator

maresb commented Jun 4, 2024

Thanks a lot @gdevenyi for doing this carefully!

I believe we're currently only testing against Ubuntu. If you figure out a solution can we add RedHat? (Also, if we don't find a solution, do the tests currently pass on RedHat?)

@maresb
Copy link
Collaborator

maresb commented Jun 4, 2024

I'm not really using Ansible these days, so I'm a bit rusty on everything.

@truatpasteurdotfr
Copy link

YMMV, but you may want to exclude root from being initialised just in case something goes wrong within micromamba env.

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.

3 participants