Re: emacs-30 f050b9c5033: Fix Tramp IPv6 handling in tests

[Madhu <enometh@meer.net>]

* Michael Albinus via Mailing list for Emacs changes <20240722075709.1C23CC41F06@vcs2.savannah.gnu.org> :
Wrote on Mon, 22 Jul 2024 03:57:08 -0400 (EDT):
> branch: emacs-30
> commit f050b9c5033ef92ac299d3da30774bc228fd0e08
> Author: Michael Albinus <michael.albinus@gmx.de>
> diff --git a/lisp/net/tramp-integration.el b/lisp/net/tramp-integration.el
> --- a/lisp/net/tramp-integration.el
> +++ b/lisp/net/tramp-integration.el
> @@ -275,9 +275,14 @@ NAME must be equal to `tramp-current-connection'."
>  ;;; Integration of shortdoc.el:
>
>  (with-eval-after-load 'shortdoc
> -  (dolist (elem '((file-remote-p
> +  (dolist (elem `((file-remote-p
>  		   :eval (file-remote-p "/ssh:user@host:/tmp/foo")
> -		   :eval (file-remote-p "/ssh:user@host:/tmp/foo" 'method))
> +		   :eval (file-remote-p "/ssh:user@host:/tmp/foo" 'method)
> +		   :eval (file-remote-p "/ssh:user@[::1]#1234:/tmp/foo" 'host)
> +		   ;; We don't want to see the text properties.
> +		   :no-eval (file-remote-p "/sudo::/tmp/foo" 'user)
> +		   :result ,(substring-no-properties
> +			     (file-remote-p "/sudo::/tmp/foo" 'user)))

just a caveat that substring-no-properties can end up being called on
nil here.  I agree it is unlikely, but I managed to trigger it with the
`swiper' package, where an unconditional (require 'tramp) in the midst
of code kept failing.