cmd/abigen: require either --abi or --combined-json (#31045)

This PR addresses issue #30768 , which highlights that running cmd/abigen/abigen --pkg my_package example.json (erroneously omitting the --abi flag) generates an empty binding, when it should fail explicitly. --------- Co-authored-by: jwasinger <[email protected]>
ethereum · Jan 20, 2025 · cc814d6 · cc814d6
1 parent ea31bd9
commit cc814d6
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/cmd/abigen/main.go b/cmd/abigen/main.go
@@ -98,6 +98,9 @@ func abigen(c *cli.Context) error {
 	if c.String(pkgFlag.Name) == "" {
 		utils.Fatalf("No destination package specified (--pkg)")
 	}
+	if c.String(abiFlag.Name) == "" && c.String(jsonFlag.Name) == "" {
+		utils.Fatalf("Either contract ABI source (--abi) or combined-json (--combined-json) are required")
+	}
 	var lang bind.Lang
 	switch c.String(langFlag.Name) {
 	case "go":