Add shortName for quiet flag

[shivaraj-bh]

Motivation

This PR adds q as a shortName for the quiet flag for nix
command, overriding

nix/src/libmain/common-args.cc

Lines 23 to 28 in 26c3fc1

	addFlag({
	.longName = "quiet",
	.description = "Decrease the logging verbosity level.",
	.category = loggingCategory,
	.handler = {[]() { verbosity = verbosity > lvlError ? (Verbosity) (verbosity - 1) : lvlError; }},
	});

.
Since this flag can be repeated, it simplifies usage by allowing nix -qq ... instead of nix --quiet --quiet ....

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

[cole-h]

How does this interact with nix-store --query, which has -q as a short flag?

[shivaraj-bh]

How does this interact with nix-store --query, which has -q as a short flag?

I overlooked that. That explains why the CI is failing.

In that case, adding the flag with shortName to only nix command should work? i.e in

nix/src/nix/main.cc

Line 104 in 26c3fc1

NixArgs() : MultiCommand("", RegisterCommand::getCommandsFor({})), MixCommonArgs("nix")

[edolstra]

I feel that --quiet is probably not used often enough to justify having it as a one-character flag. Especially since the new CLI by default isn't very verbose anyway.

[tomberek]

Discussed in meeting:

• implementation is not quite right for overriding the common-args
• random historical note: "-q" for query came from the original RPM CLI interface
• "--quiet" already exists
• "-q" for quiet is not likely to be used often

Decision: CLOSE, reopen if we have additional user requests for the short flag or more use-cases come to light

[Ericson2314]

https://github.com/NixOS/nix/blob/master/CONTRIBUTING.md#making-changes-to-nix See also the guidance here re misc new features. Ideally one maintainers could handle this async, but we're not very good at that, so instead these things gum up our delicate process. Sorry.