git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH RESEND] send-email: make produced outputs more readable

From
DSDragan Simic <dsimic@manjaro.org>
Date
Apr 4, 2024, 17:07 UTC
Message-ID
<62553db377c28458883b66bcdc0c58cc0f32d15b.1712250366.git.dsimic@manjaro.org>

When sending multiple patches at once, without confirming the sending of each patch separately, the displayed result statuses of sending each patch become bunched together with the messages produced for the subsequent patch. This unnecessarily makes discerning each of the result statuses a bit difficult, as visible in the sample output excerpt below:

    ...
    MIME-Version: 1.0
    Content-Transfer-Encoding: 8bit
    Result: 250
    OK. Log says:
    ...

As visible above, bunching the "Result: <status-code>" lines together with the messages produced for the subsequent patch makes the output unreadable. Thus, let's add a newline after each displayed result status, to make the outputs more readable, as visible in the sample output excerpt below:

    ...
    MIME-Version: 1.0
    Content-Transfer-Encoding: 8bit
    Result: 250
    OK. Log says:
    ...

This change also adds a newline after the last produced result status, which may be seen as redundant. Though, it doesn't look too bad, and making that last newline not displayed would make the code much more complex, which would not be worth neither the time and effort now, nor the additional maintenance burden in the future.

While there, remove one spotted stray newline in the code.
Signed-off-by: Dragan Simic <dsimic@manjaro.org>
---
Notes:
     * send-email: make produced outputs more readable by separating
       the result statuses from the subsequent patch outputs
    
    This is a resubmission of the patch I submitted about a week and a half
    ago. [1]  The patch subject in the original submission was selected in
    a bit unfortunate way, which this submission corrects, and also improves
    the patch description a bit.  There are no changes to the patch itself.
    
    [1] https://lore.kernel.org/git/6ee28707b9eb8bd8fdfc8756c351455c6bc3bb62.1711447365.git.dsimic@manjaro.org/
 git-send-email.perl | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/git-send-email.perl b/git-send-email.perl
index 821b2b3a135a..62505ab2707c 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -1576,7 +1576,6 @@ sub send_message {
 		print $sm "$header\n$message";
 		close $sm or die $!;
 	} else {
-
 		if (!defined $smtp_server) {
 			die __("The required SMTP server is not properly defined.")
 		}
@@ -1686,9 +1685,9 @@ sub send_message {
 		print $header, "\n";
 		if ($smtp) {
 			print __("Result: "), $smtp->code, ' ',
-				($smtp->message =~ /\n([^\n]+\n)$/s), "\n";
+				($smtp->message =~ /\n([^\n]+\n)$/s), "\n\n";
 		} else {
-			print __("Result: OK\n");
+			print __("Result: OK\n\n");
 		}
 	}
 
Next: Junio C Hamano
Message 1 of 3 in “send-email: make produced outputs more readable”
  1. send-email: make produced outputs more readableDragan Simic, Apr 4, 2024
  2. Junio C HamanoApr 4, 2024
  3. Dragan SimicApr 4, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.