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

share lib/ and bin/ directories #5396

Closed
oliver-sanders opened this issue Mar 3, 2023 · 2 comments · Fixed by #5525
Closed

share lib/ and bin/ directories #5396

oliver-sanders opened this issue Mar 3, 2023 · 2 comments · Fixed by #5525
Assignees
Labels
Milestone

Comments

@oliver-sanders
Copy link
Member

Problem

Workflows often build or install libraries and executables at runtime. Where should these be put?

You could put them into the bin/ and lib/ directories, but these would be overwritten by reinstallation, so you would have to list every resource you want to self-manage into the .cylcignore file which is fiddly.

Proposed Solution

Add support for a bin/ and lib/ directory in the share/ directory (i.e. put these in job's PATH and PYTHONPATH vars). This would help to colocate files generated at runtime and avoid the reinstall problem.

@oliver-sanders oliver-sanders added small question Flag this as a question for the next Cylc project meeting. labels Mar 3, 2023
@oliver-sanders oliver-sanders added this to the cylc-8.x milestone Mar 3, 2023
@oliver-sanders oliver-sanders removed the question Flag this as a question for the next Cylc project meeting. label Mar 16, 2023
@oliver-sanders
Copy link
Member Author

Meeting 2023-03-16: Do it!

@oliver-sanders oliver-sanders modified the milestones: cylc-8.x, cylc-8.2.0 Mar 16, 2023
@wxtim wxtim self-assigned this May 9, 2023
@wxtim
Copy link
Member

wxtim commented May 9, 2023

Do we need to create these folders on install, or is that the user's problem?

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 a pull request may close this issue.

2 participants