search

necaris / conda.el

Emacs helper library (and minor mode) to work with conda environments
MIT License
153 stars 51 forks source link

Add Unset env feature to deactivate propery #155

Closed ArmCorelin closed 1 day ago

ArmCorelin commented 8 months ago

Changed to use unset and set command when env-update. Because conda is now using those command and I couldn't deactivate without it.

I checked it with ; $ conda shell.posix deactivate

sambrightman commented 8 months ago

The second parameter to setenv should be nil (or omitted) to unset properly. That said, any chance of getting a release with this PR merged?

ArmCorelin commented 8 months ago

@sambrightman Hi, Actually I was aware this is caused conda script is using not only export, using set and unset also. If you need this work immediately, please use my repo. Thanks.

sambrightman commented 8 months ago

The question about releasing was for @necaris.

@ArmCorelin you should adjust the setenv parameter as described.

ArmCorelin commented 8 months ago

@sambrightman OK, I will change that line to (setenv (format "%s" arg) nil)) Thanks

necaris commented 1 day ago

Merged -- apologies for delay!