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.

Sign up for GitHub

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

Jump to bottom

lib.modules: fix test line using empty "$@" #346677

Merged
merged 1 commit into from
Oct 5, 2024
Merged

lib.modules: fix test line using empty "$@" #346677

merged 1 commit into from
Oct 5, 2024
+1 −1

Commits on Oct 5, 2024

  1. lib.modules: fix test line using empty "$@" 

    Probably a missed left over from somewhere in
the commit 58f385f.
As can be seen in that commit where this line was introduced,
"$@" was also just emptied by the last `set` call in line 169.
This line is currently valid, but breaks suddenly when somewhere earlier
a `set --` instruction is used in the future.
Neither in commit 58f385f
nor in PR NixOS#197547
have I found anything stating that this "defect" was intentional.
    @Zocker1999NET
    Zocker1999NET committed Oct 5, 2024
    Configuration menu
    Copy the full SHA
    f35a61d View commit details
    Browse the repository at this point in the history