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

[Michael Albinus <michael.albinus@gmx.de>]

Michael Albinus <michael.albinus@gmx.de> writes:

Hi Madhu,

>>>  ;;; 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.
>
> In that case (file-remote-p returns nil) the whole block doesn't make
> sense. How did they manage to manipulate Tramp this way? Setting
> tramp-mode to nil? Manipulating file-name-handler-alist?

I've added a sanity check. Pushed to the emacs-30 branch.

Best regards, Michael.