Skip to content

ops: don't re-apply already applied operations#377

Closed
quad wants to merge 1 commit intomasterfrom
ssr/20230531/skip-envar-reapply
Closed

ops: don't re-apply already applied operations#377
quad wants to merge 1 commit intomasterfrom
ssr/20230531/skip-envar-reapply

Conversation

@quad
Copy link
Copy Markdown
Contributor

@quad quad commented May 31, 2023

(This solution is horrible as it interacts with the HERMIT_ENV_OPS envar setup in the activate script. I'd appreciate suggestions for a better approach!)

Hermit environment variable operations can be overrode by the user after
an environment has already been opened. If this is the case, then we
shouldn't re-apply the operations; we should instead expressed respect
the wish of a mangled environment by the user.

This is relevant for constructs like Python virtual environments.

Context: https://square.slack.com/archives/C01RTEH4MFV/p1685522543591719

@quad quad requested a review from alecthomas May 31, 2023 09:50
Hermit environment variable operations can be overrode by the user after
an environment has already been opened. If this is the case, then we
shouldn't re-apply the operations; we should instead expressed respect
the wish of a mangled environment by the user.

This is relevant for constructs like Python virutal environments.

Context: https://square.slack.com/archives/C01RTEH4MFV/p1685522543591719
@quad quad force-pushed the ssr/20230531/skip-envar-reapply branch from f62ba38 to 3d9ca9b Compare May 31, 2023 09:54
@quad
Copy link
Copy Markdown
Contributor Author

quad commented Jul 3, 2023

Closed in favour of #250.

@quad quad closed this Jul 3, 2023
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.

1 participant