GNU bug report logs - #73763
31.0.50; [PATCH] Remove unnecessary trailing white space from SRC revision number

Previous Next

Package: emacs;

Reported by: Alex Bochannek <alex <at> bochannek.com>

Date: Sat, 12 Oct 2024 04:25:01 UTC

Severity: minor

Tags: patch

Found in version 31.0.50

Done: Sean Whitton <spwhitton <at> spwhitton.name>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Alex Bochannek <alex <at> bochannek.com>
Subject: bug#73763: closed (Re: bug#73763: 31.0.50; [PATCH] Remove
 unnecessary trailing white space from SRC revision number)
Date: Sun, 13 Oct 2024 08:38:02 +0000
[Message part 1 (text/plain, inline)]
Your bug report

#73763: 31.0.50; [PATCH] Remove unnecessary trailing white space from SRC revision number

which was filed against the emacs package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 73763 <at> debbugs.gnu.org.

-- 
73763: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=73763
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Sean Whitton <spwhitton <at> spwhitton.name>
To: Alex Bochannek <alex <at> bochannek.com>
Cc: 73763-done <at> debbugs.gnu.org
Subject: Re: bug#73763: 31.0.50; [PATCH] Remove unnecessary trailing white
 space from SRC revision number
Date: Sun, 13 Oct 2024 16:36:28 +0800
Hello,

Thank you, installed on master.

-- 
Sean Whitton

[Message part 3 (message/rfc822, inline)]
From: Alex Bochannek <alex <at> bochannek.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 31.0.50; [PATCH] Remove unnecessary trailing white space from SRC
 revision number
Date: Fri, 11 Oct 2024 20:45:53 -0700
[Message part 4 (text/plain, inline)]
Hello!

I am using the SRC back end to VC and noticed `^J' after the revision
number in the mode line. This appears to be due to the way the output of
the `src list' command is converted to a string, which keeps the
newline.

The following patch has been tested under macOS Sequoia 15.0.1 and
should be equally applicable to other platforms.

[0001-Remove-unnecessary-trailing-white-space-from-SRC-rev.patch (text/x-patch, inline)]
From 37d0c418b633f588c8162704630e422afecbf5df Mon Sep 17 00:00:00 2001
From: Alex Bochannek <awb <at> awb-mbp-m3.local>
Date: Fri, 11 Oct 2024 20:32:41 -0700
Subject: [PATCH] Remove unnecessary trailing white space from SRC revision
 number

---
 lisp/vc/vc-src.el | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lisp/vc/vc-src.el b/lisp/vc/vc-src.el
index 27f58cb3369..ff19b0f7696 100644
--- a/lisp/vc/vc-src.el
+++ b/lisp/vc/vc-src.el
@@ -222,8 +222,9 @@ vc-src-command
 (defun vc-src-working-revision (file)
   "SRC-specific version of `vc-working-revision'."
   (let ((result (ignore-errors
-		  (with-output-to-string
-		    (vc-src-command standard-output file "list" "-f{1}" "@")))))
+                  (string-trim-right
+                   (with-output-to-string
+                     (vc-src-command standard-output file "list" "-f{1}" "@"))))))
     (if (zerop (length result)) "0" result)))
 
 ;;;
-- 
2.39.5 (Apple Git-154)

[Message part 6 (text/plain, inline)]
-- 
Alex.

This bug report was last modified 313 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.