"Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > ret |= !cp; > + if (ret) { > + error(_("gpg failed to sign the data:\n%s"), > + gpg_status.len ? gpg_status.buf : "(no gpg output)"); > + strbuf_release(&gpg_status); > + return -1; > + } > strbuf_release(&gpg_status); > - if (ret) > - return error(_("gpg failed to sign the data")); Good. As we are worried about error messages that are too terse, dumping everything to the output would be a vast improvement. Hopefully gpg_status.len would to be thousands of bytes long, and this is not a codepath that is triggered remotely anyway. Will queue. Thanks.