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

input: add 'inherit' dependency property #565

Merged
merged 1 commit into from
May 27, 2024

Conversation

rhubert
Copy link
Contributor

@rhubert rhubert commented May 22, 2024

This property controls the inheritance of environment, tools and sandbox
for a dependency. It defaults to true so everything is inherited.

If set to false environment, tools and sandbox are dropped. The
dependency starts with the default-environment, no tools and no sandbox.

This becomes particularly useful when an existing root-package should
become a dependency of another root-package, e.g. for building an
installer.

doc/manual/configuration.rst Outdated Show resolved Hide resolved
pym/bob/input.py Outdated Show resolved Hide resolved
pym/bob/input.py Outdated Show resolved Hide resolved
@rhubert rhubert force-pushed the clean_dep branch 3 times, most recently from f383aaf to ff60e4e Compare May 24, 2024 05:43
@rhubert rhubert changed the title input: add 'clean' dependency property input: add 'inherit' dependency property May 24, 2024
This property controls the inheritance of environment, tools and sandbox
for a dependency. It defaults to true so everything is inherited.

If set to `false` environment, tools and sandbox are dropped. The
dependency starts with the default-environment, no tools and no sandbox.

This becomes particularly useful when an existing root-package should
become a dependency of another root-package, e.g. for building an
installer.
Copy link

codecov bot commented May 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.71%. Comparing base (0025fc2) to head (92710ed).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #565      +/-   ##
==========================================
- Coverage   79.72%   79.71%   -0.01%     
==========================================
  Files          46       46              
  Lines       14693    14723      +30     
==========================================
+ Hits        11714    11737      +23     
- Misses       2979     2986       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rhubert
Copy link
Contributor Author

rhubert commented May 27, 2024

Thanks for fixing this! I did some testing this morning and it works like expected.

@jkloetzke jkloetzke merged commit 277e1ae into BobBuildTool:master May 27, 2024
11 checks passed
@jkloetzke
Copy link
Member

Thanks.

@rhubert rhubert deleted the clean_dep branch September 17, 2024 09:06
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.

2 participants