Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow Rose Apps to set PYTHONPATH=_PYTHONPATH before running #2725

Closed

Conversation

wxtim
Copy link
Contributor

@wxtim wxtim commented Sep 8, 2023

Allows Cylc Wrapper to cached the users PYTHONPATH before switching to a Cylc 8 environment.

Partner of cylc/cylc-flow#5727

The Rose end of cylc/cylc-flow#5124

Tagging @dpmatthews who may want to be involved in functionally testing this, although I'll propose a user test when I put up a companion PR for Cylc. Probably won't want to be a main reviewer.

This PR is self-sufficient. It won't break anything if it goes in without the companion Cylc PR.

Allows Cylc Wrapper to cached the users PYTHONPATH before
switching to a Cylc 8 environment.
@wxtim wxtim requested a review from oliver-sanders September 8, 2023 09:21
@wxtim wxtim self-assigned this Sep 8, 2023
@wxtim wxtim requested a review from MetRonnie September 8, 2023 09:21
@wxtim wxtim added the small label Sep 8, 2023
@wxtim wxtim added this to the 2.2.0 milestone Sep 8, 2023
@wxtim wxtim marked this pull request as draft September 26, 2023 10:21
@wxtim wxtim closed this Oct 5, 2023
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.

1 participant