nix store sign: Use required attribute

edolstra · edolstra · commit 3bd2b76f6e93 · 2025-10-16T16:35:13.000+02:00
diff --git a/src/nix/sigs.cc b/src/nix/sigs.cc
@@ -104,6 +104,7 @@ struct CmdSign : StorePathsCommand
             .labels = {"file"},
             .handler = {&secretKeyFile},
             .completer = completePath,
+            .required = true,
         });
     }
 
@@ -114,9 +115,6 @@ struct CmdSign : StorePathsCommand
 
     void run(ref<Store> store, StorePaths && storePaths) override
     {
-        if (secretKeyFile.empty())
-            throw UsageError("you must specify a secret key file using '-k'");
-
         SecretKey secretKey(readFile(secretKeyFile));
         LocalSigner signer(std::move(secretKey));
 

Original file line number	Diff line number	Diff line change
`@@ -104,6 +104,7 @@ struct CmdSign : StorePathsCommand`
`104`	`104`	`.labels = {"file"},`
`105`	`105`	`.handler = {&secretKeyFile},`
`106`	`106`	`.completer = completePath,`
	`107`	`+ .required = true,`
`107`	`108`	`});`
`108`	`109`	`}`
`109`	`110`
`@@ -114,9 +115,6 @@ struct CmdSign : StorePathsCommand`
`114`	`115`
`115`	`116`	`void run(ref<Store> store, StorePaths && storePaths) override`
`116`	`117`	`{`
`117`		`- if (secretKeyFile.empty())`
`118`		`- throw UsageError("you must specify a secret key file using '-k'");`
`119`		`-`
`120`	`118`	`SecretKey secretKey(readFile(secretKeyFile));`
`121`	`119`	`LocalSigner signer(std::move(secretKey));`
`122`	`120`