bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Allen Li]

Reproduce:

1. emacs -Q
2. M-x minibuffer-electric-default-mode RET
3. C-x C-f /ssh:some-host:

Expected:

The ~/ prepopulated in the minibuffer should be greyed out.

Actual:

On typing the second :, the ~/ stops being greyed out.

In GNU Emacs 26.1 (build 1, x86_64-pc-linux-gnu, GTK+ Version
3.22.24), modified by Debian
Windowing system distributor 'The X.Org Foundation', version 11.0.11906000
System Description: Debian GNU/Linux rodete (upgraded from: Ubuntu 14.04 LTS)

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Eli Zaretskii]

> From: Allen Li <darkfeline@felesatra.moe>
> Date: Mon, 27 Aug 2018 16:53:18 -0700
> 
> Reproduce:
> 
> 1. emacs -Q
> 2. M-x minibuffer-electric-default-mode RET
> 3. C-x C-f /ssh:some-host:
> 
> Expected:
> 
> The ~/ prepopulated in the minibuffer should be greyed out.
> 
> Actual:
> 
> On typing the second :, the ~/ stops being greyed out.

I seem to be unable to reproduce this here.  Is it important for
some-host to name an existing address?

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Allen Li]

On Mon, Aug 27, 2018 at 11:05 PM Eli Zaretskii <eliz@gnu.org> wrote:
>
> > From: Allen Li <darkfeline@felesatra.moe>
> > Date: Mon, 27 Aug 2018 16:53:18 -0700
> >
> > Reproduce:
> >
> > 1. emacs -Q
> > 2. M-x minibuffer-electric-default-mode RET
> > 3. C-x C-f /ssh:some-host:
> >
> > Expected:
> >
> > The ~/ prepopulated in the minibuffer should be greyed out.
> >
> > Actual:
> >
> > On typing the second :, the ~/ stops being greyed out.
>
> I seem to be unable to reproduce this here.  Is it important for
> some-host to name an existing address?

No, I had to play with it to figure out the right conditions, you need
to invoke find-file twice:

1. emacs -Q
2. M-x minibuffer-electric-default-mode RET
3. C-x C-f /ssh:some-host: C-g
3. C-x C-f /ssh:some-host:

Expected:

The ~/ prepopulated in the minibuffer should be greyed out.

Actual:

On typing the second :, the ~/ stops being greyed out.

I can reproduce this consistently now, can you try again?

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Michael Albinus]

Allen Li <darkfeline@felesatra.moe> writes:

> On Mon, Aug 27, 2018 at 11:05 PM Eli Zaretskii <eliz@gnu.org> wrote:
>>
>> > From: Allen Li <darkfeline@felesatra.moe>
>> > Date: Mon, 27 Aug 2018 16:53:18 -0700
>> >
>> > Reproduce:
>> >
>> > 1. emacs -Q
>> > 2. M-x minibuffer-electric-default-mode RET
>> > 3. C-x C-f /ssh:some-host:
>> >
>> > Expected:
>> >
>> > The ~/ prepopulated in the minibuffer should be greyed out.
>> >
>> > Actual:
>> >
>> > On typing the second :, the ~/ stops being greyed out.
>>
>> I seem to be unable to reproduce this here.  Is it important for
>> some-host to name an existing address?
>
> No, I had to play with it to figure out the right conditions, you need
> to invoke find-file twice:
>
> 1. emacs -Q
> 2. M-x minibuffer-electric-default-mode RET
> 3. C-x C-f /ssh:some-host: C-g
> 3. C-x C-f /ssh:some-host:
>
> Expected:
>
> The ~/ prepopulated in the minibuffer should be greyed out.
>
> Actual:
>
> On typing the second :, the ~/ stops being greyed out.
>
> I can reproduce this consistently now, can you try again?

Maybe it is a Tramp problem. If I do the following, I can reproduce it:

# emacs -Q -l tramp

OTOH, the following Emacs invocation doesn't show the problem:

# emacs -Q -l tramp --eval '(setq tramp-mode nil)'

All tests performed with Emacs 27.0.50, but this shouldn't matter. Will
dig further.

Best regards, Michael.

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Michael Albinus]

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

Hi Allen,

> Maybe it is a Tramp problem. If I do the following, I can reproduce it:
>
> # emacs -Q -l tramp
>
> OTOH, the following Emacs invocation doesn't show the problem:
>
> # emacs -Q -l tramp --eval '(setq tramp-mode nil)'
>
> All tests performed with Emacs 27.0.50, but this shouldn't matter. Will
> dig further.

Finally, I could trap it. Tramp's rfn-shadow hook ceased to work after
Tramp has been changed to lexical binding.

Fixed in the emacs-26 branch. Could you, pls, check?

Best regards, Michael.

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Eli Zaretskii]

> From: Michael Albinus <michael.albinus@gmx.de>
> Cc: eliz@gnu.org,  32550@debbugs.gnu.org
> Date: Tue, 04 Sep 2018 12:02:19 +0200
> 
> > Maybe it is a Tramp problem. If I do the following, I can reproduce it:
> >
> > # emacs -Q -l tramp
> >
> > OTOH, the following Emacs invocation doesn't show the problem:
> >
> > # emacs -Q -l tramp --eval '(setq tramp-mode nil)'
> >
> > All tests performed with Emacs 27.0.50, but this shouldn't matter. Will
> > dig further.
> 
> Finally, I could trap it. Tramp's rfn-shadow hook ceased to work after
> Tramp has been changed to lexical binding.
> 
> Fixed in the emacs-26 branch. Could you, pls, check?

Seems to work fine here, thanks.

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Allen Li]

On Tue, Sep 4, 2018 at 3:02 AM Michael Albinus <michael.albinus@gmx.de> wrote:
> Fixed in the emacs-26 branch. Could you, pls, check?
>
> Best regards, Michael.

Also confirmed, thanks for the quick fix

bug#32550: 26.1; minibuffer-electric-default-mode breaks greying out find-file prefix

[Michael Albinus]

Version: 26.1

Hi Allen & Eli,

Allen Li <darkfeline@felesatra.moe> writes:

> On Tue, Sep 4, 2018 at 3:02 AM Michael Albinus <michael.albinus@gmx.de> wrote:
>> Fixed in the emacs-26 branch. Could you, pls, check?
>>
>> Best regards, Michael.
>
> Also confirmed, thanks for the quick fix

Thanks for confirmation, I'm closing the bug.

Best regards, Michael.