Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@kemingy
Copy link
Member

@kemingy kemingy commented Dec 15, 2022

Signed-off-by: Keming <[email protected]>
Signed-off-by: Keming <[email protected]>
@kemingy kemingy marked this pull request as ready for review December 15, 2022 08:24
@kemingy
Copy link
Member Author

kemingy commented Dec 15, 2022

PTAL @VoVAllen

The merge operation overrides /home/envd/.bashrc. But I don't know why.

Signed-off-by: Keming <[email protected]>
@kemingy
Copy link
Member Author

kemingy commented Dec 15, 2022

@VoVAllen
Copy link
Member

Is the problem fixed in this pr or not?

@kemingy
Copy link
Member Author

kemingy commented Dec 15, 2022

Is the problem fixed in this pr or not?

Fixed by putting the "conda activate envd" into the last step of Python. BTW, "conda init bash" will still be written to root bashrc. That may need more refactoring. But I prefer to keep it as it is in v0. V1 doesn't have this issue.

Copy link
Member

@VoVAllen VoVAllen left a comment

Choose a reason for hiding this comment

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

/lgtm

@muniu-bot muniu-bot bot added the lgtm label Dec 15, 2022
@kemingy kemingy merged commit 31eec50 into tensorchord:main Dec 15, 2022
@kemingy kemingy deleted the fix_v0_bash branch December 15, 2022 09:41
frostming pushed a commit to frostming/envd that referenced this pull request Dec 16, 2022
* fix: v0 bashrc file

Signed-off-by: Keming <[email protected]>

* reorder py llb

Signed-off-by: Keming <[email protected]>

* fix go emb

Signed-off-by: Keming <[email protected]>

Signed-off-by: Keming <[email protected]>
Signed-off-by: Frost Ming <[email protected]>
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.

bug: v0 dev env doesn't activate envd conda env by default

2 participants