bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

[Drew Adams]

emacs -Q
 
Set `dired-listing-switches' to "-alF".
 
M-x dired-create-directory RET aaaaaaa RET
 
The new directory is displayed without the trailing /.  You need to hit
`g' to get the proper display.
 

In GNU Emacs 24.0.50.1 (i386-mingw-nt5.1.2600)
 of 2010-05-23 on G41R2F1
Windowing system distributor `Microsoft Corp.', version 5.1.2600
configured using `configure --with-gcc (3.4) --no-opt --cflags -Ic:/xpm/include'
 

bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

[Drew Adams]

I should add:

1. This is a regression wrt Emacs 20.  The bug was introduced in Emacs 21.

2. Displaying the new dir without the trailing / means that Dired actions will
not work correctly for that entry.  For example, you cannot use `x' to delete it
after flagging it `D'.  You must first use `g' to update the display to get the
/.

bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

[Eli Zaretskii]

> From: "Drew Adams" <drew.adams@oracle.com>
> Date: Fri, 28 May 2010 10:15:05 -0700
> Cc: 
> 
> 1. This is a regression wrt Emacs 20.  The bug was introduced in Emacs 21.

??? Emacs 20 on MS-Windows didn't support the -F switch at all,
AFAICS.  So I don't understand how can this be a regression wrt
Emacs 20.  What am I missing?

> 2. Displaying the new dir without the trailing / means that Dired actions will
> not work correctly for that entry.  For example, you cannot use `x' to delete it
> after flagging it `D'.  You must first use `g' to update the display to get the
> /.

Allright, allright, you convinced me.  No need for more evidence.  I
even see the bug (and will fix it if no one beats me to it).

bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

[Drew Adams]

> > 1. This is a regression wrt Emacs 20.  The bug was 
> introduced in Emacs 21.
> 
> ??? Emacs 20 on MS-Windows didn't support the -F switch at all,
> AFAICS.  So I don't understand how can this be a regression wrt
> Emacs 20.  What am I missing?

You're right. In emacs -q for Emacs 20 the / is not supported either.
I was checking with my own setup.

> > 2. Displaying the new dir without the trailing / means that 
> > Dired actions will not work correctly for that entry.
> > For example, you cannot use `x' to delete it after flagging
> > it `D'.  You must first use `g' to update the display to
> > get the /.
> 
> Allright, allright, you convinced me.  No need for more evidence.  I
> even see the bug (and will fix it if no one beats me to it).

Thank you.

bug#6294: 24.0.50; `dired-create-directory' needs to add / when -alF

[Eli Zaretskii]

> From: "Drew Adams" <drew.adams@oracle.com>
> Date: Fri, 28 May 2010 09:52:00 -0700
> Cc: 
> 
> emacs -Q
>  
> Set `dired-listing-switches' to "-alF".
>  
> M-x dired-create-directory RET aaaaaaa RET
>  
> The new directory is displayed without the trailing /.  You need to hit
> `g' to get the proper display.

Fixed in revno 100474.