[PATCH] imap-send: improve error messages for missing configuration

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The error message 'no imap store specified' was confusing because
it referred to 'store' when the actual missing configuration was
'imap.folder'. Similarly, the host error message provided no
guidance on how to fix the issue.

Improve both error messages to:
- Clearly state what configuration is missing
- Provide hints showing the exact git config commands needed
- Include examples of typical values

This helps users quickly understand and resolve configuration issues
when using git imap-send.

Signed-off-by: Jörg Thalheim <joerg@xxxxxxxxxxx>
---
 imap-send.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/imap-send.c b/imap-send.c
index 2e812f5a6e..a2d6f6d3f6 100644
--- a/imap-send.c
+++ b/imap-send.c
@@ -1539,13 +1539,17 @@ int cmd_main(int argc, const char **argv)
 		server.port = server.use_ssl ? 993 : 143;
 
 	if (!server.folder) {
-		fprintf(stderr, "no imap store specified\n");
+		fprintf(stderr, "error: no imap folder specified\n");
+		fprintf(stderr, "hint: set the target folder with 'git config imap.folder <folder>'\n");
+		fprintf(stderr, "      (e.g., 'git config imap.folder Drafts')\n");
 		ret = 1;
 		goto out;
 	}
 	if (!server.host) {
 		if (!server.tunnel) {
-			fprintf(stderr, "no imap host specified\n");
+			fprintf(stderr, "error: no imap host specified\n");
+			fprintf(stderr, "hint: set the imap host with 'git config imap.host <host>'\n");
+			fprintf(stderr, "      (e.g., 'git config imap.host imaps://imap.example.com')\n");
 			ret = 1;
 			goto out;
 		}
-- 
2.49.0





[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux