Commit 9e85678
authored
[devbox.Open] drop ensureNixInstalled function call (#1453)
## Summary
Problem:
Commands like `devbox version` are installing nix.
Cause:
By inserting a `debug.PrintStack()` one can
[see](https://gist.github.com/savil/0fff37124abae3d1c0088404c05a73cc)
that `devbox.Open` is invoked
for `devbox version` from a few places. In the latest 0.5.12 release, we
added
a call to `ensureNixInstalled` in `devbox.Open`. Hence even though we
don't set `PreRunE: ensureNixInstalled`
in the command, the nix install is triggered.
The original motivation for adding this call in `devbox.Open` was that
`nix` is pretty widely used in the `Devbox`
library, and we should add `ensureNixInstalled` as a check.
## How was it tested?
compiles
will rely on tests to pass1 parent 4f014e2 commit 9e85678
1 file changed
+0
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
130 | 130 | | |
131 | 131 | | |
132 | 132 | | |
133 | | - | |
134 | | - | |
135 | | - | |
136 | | - | |
137 | 133 | | |
138 | 134 | | |
139 | 135 | | |
| |||
0 commit comments