port venv.nu and venv_deactivate.nu

- copy the already ported activation script from `conda.nu`
- add `load-env` and `def --env`
- remove duplication for PATH/Path because nu now supports case insensitive access
- remove $nu.path use $env.PATH
- remove PATH serialization to string/from string
- use a record instead of a table for `load-env`

modules/virtual_environments/venv/venv.nu

@@ -1,33 +1,18 @@
-def venv [venv-dir] {
- let venv-abs-dir = ($venv-dir | path expand)
- let venv-name = ($venv-abs-dir | path basename)
- let old-path = ($nu.path | str collect (path-sep))
- let new-path = (if (windows?) { (venv-path-windows $venv-abs-dir) } { (venv-path-unix $venv-abs-dir) })
- let new-env = [[name, value];
- [VENV_OLD_PATH $old-path]
- [VIRTUAL_ENV $venv-name]]
-
- $new-env | append $new-path
-}
-
-def venv-path-unix [venv-dir] {
- let venv-path = ([$venv-dir "bin"] | path join)
- let new-path = ($nu.path | prepend $venv-path | str collect (path-sep))
- [[name, value]; [PATH $new-path]]
-}
-
-def venv-path-windows [venv-dir] {
- # 1. Conda on Windows needs a few additional Path elements
- # 2. The path env var on Windows is called Path (not PATH)
- let venv-path = ([$venv-dir "Scripts"] | path join)
- let new-path = ($nu.path | prepend $venv-path | str collect (path-sep))
- [[name, value]; [Path $new-path]]
-}
-
-def windows? [] {
- (sys).host.name == "Windows"
+def --env venv [venv_dir] {
+ let venv_abs_dir = ($venv_dir | path expand)
+ let venv_name = ($venv_abs_dir | path basename)
+ let old_path = $env.PATH
+ let new_path = (venv-path $venv_abs_dir)
+ let new_env = ({VENV_OLD_PATH: $old_path, VIRTUAL_ENV: $venv_name} | merge $new_path)
+ load-env $new_env
 }
 
-def path-sep [] {
- if (windows?) { ";" } { ":" }
+def "venv-path" [venv_dir] {
+ let env_path = [
+ $venv_dir,
+ ([$venv_dir, "bin"] | path join)
+ ]
+ return {
+ PATH: ($env.PATH | prepend $env_path)
+ }
 }

modules/virtual_environments/venv/venv_deactivate.nu

@@ -1,4 +1,3 @@
-let path-name = (if ((sys).host.name == "Windows") { "Path" } { "PATH" })
-let-env $path-name = $nu.env.VENV_OLD_PATH
-unlet-env VIRTUAL_ENV
-unlet-env VENV_OLD_PATH
+$env.PATH = $env.VENV_OLD_PATH
+hide-env VIRTUAL_ENV
+hide-env VENV_OLD_PATH