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

Add to glossary and clarify garbage collection #2071

Merged

Conversation

sseefried
Copy link
Contributor

While trying to understand garbage collection it was not immediately
clear that only the runtime dependency closure of output paths
would be kept (instead of the build-time dependency closure).
This commit attempts to clarify this by expanding some of the
glossary definitions and extending the Garbage Collection
section.

While trying to understand garbage collection it was not immediately
clear that only the runtime dependency closure of output paths
would be kept (instead of the build-time dependency closure).
This commit attempts to clarify this by expanding some of the
glossary definitions and extending the Garbage Collection
section.
@sseefried
Copy link
Contributor Author

@7c6f434c Thanks!

What is the procedure now? Are you the one that merges it?

@7c6f434c
Copy link
Member

I don't have commit access to the Nix repo, though, just wanted to record that I have verified that the change is pure improvement.

Hm, I cannot even request reviews by GitHub means…

Sigh. @shlevy is it small enough to merge quickly?

@shlevy shlevy merged commit 3d748d3 into NixOS:master Apr 12, 2018
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.

None yet

3 participants