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

Compact input derivation diffs #86

Merged

Conversation

9999years
Copy link
Collaborator

Before:

• The input derivation named `bash-5.2p32` differs
  • These two derivations have already been compared
• The input derivation named `curl-8.9.1` differs
  • These two derivations have already been compared
• The input derivation named `mirrors-list` differs
  • These two derivations have already been compared
• The input derivation named `stdenv-linux` differs
  • These two derivations have already been compared

After:

• Input derivations differ but have already been compared
    bash-5.2p32
    curl-8.9.1
    mirrors-list
    stdenv-linux

This change reduced the output from a large diff of two production NixOS systems from 190,769 lines to 142,871 lines, for a 25% reduction in output.

9999years and others added 2 commits October 22, 2024 16:50
Before:

    • The input derivation named `bash-5.2p32` differs
      • These two derivations have already been compared
    • The input derivation named `curl-8.9.1` differs
      • These two derivations have already been compared
    • The input derivation named `mirrors-list` differs
      • These two derivations have already been compared
    • The input derivation named `stdenv-linux` differs
      • These two derivations have already been compared

After:

    • Input derivations differ but have already been compared
        bash-5.2p32
        curl-8.9.1
        mirrors-list
        stdenv-linux

This change reduced the output from a large diff of two production NixOS
systems from 190,769 lines to 142,871 lines, for a 25% reduction in
output.
@Gabriella439 Gabriella439 merged commit 83aa9a9 into Gabriella439:main Oct 25, 2024
3 checks passed
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