Bug#25949 is not fixed on MS-Windows

Bug#25949 is not fixed on MS-Windows

[Eli Zaretskii]

The test for this bug still fails:

  Test files-tests--file-name-non-special--subprocess backtrace:
  Test files-tests--file-name-non-special--subprocess condition:
      (file-error "Setting current directory" "Permission denied" "/:d:/")
     FAILED  7/7  files-tests--file-name-non-special--subprocess

Re: Bug#25949 is not fixed on MS-Windows

[Fabrice Popineau]

[-- Attachment #1: Type: text/plain, Size: 1449 bytes --]

The following patch fixes it :

diff --git a/src/callproc.c b/src/callproc.c
index 70d144f83f..185a05af0d 100644
--- a/src/callproc.c
+++ b/src/callproc.c
@@ -120,14 +120,14 @@ encode_current_directory (void)
   if (NILP (dir))
     dir = build_string ("~");

-  dir = expand_and_dir_to_file (dir, Qnil);
+  dir = expand_and_dir_to_file (remove_slash_colon (dir), Qnil);

   if (NILP (Ffile_accessible_directory_p (dir)))
     report_file_error ("Setting current directory",
                        BVAR (current_buffer, directory));

   /* Remove "/:" from DIR and encode it.  */
-  dir = ENCODE_FILE (remove_slash_colon (dir));
+  dir = ENCODE_FILE (dir);

   if (! file_accessible_directory_p (dir))
     report_file_error ("Setting current directory",


but is it correct ?

Fabrice


2017-08-01 17:24 GMT+02:00 Eli Zaretskii <eliz@gnu.org>:

> The test for this bug still fails:
>
>   Test files-tests--file-name-non-special--subprocess backtrace:
>   Test files-tests--file-name-non-special--subprocess condition:
>       (file-error "Setting current directory" "Permission denied" "/:d:/")
>      FAILED  7/7  files-tests--file-name-non-special--subprocess
>
>


-- 
Fabrice Popineau
-----------------------------
CentraleSupelec
Département Informatique
3, rue Joliot Curie
91192 Gif/Yvette Cedex
Tel direct : +33 (0) 169851950
Standard : +33 (0) 169851212
------------------------------

[-- Attachment #2: Type: text/html, Size: 2724 bytes --]

Re: Bug#25949 is not fixed on MS-Windows

[Eli Zaretskii]

> From: Fabrice Popineau <fabrice.popineau@centralesupelec.fr>
> Date: Tue, 1 Aug 2017 22:26:01 +0200
> Cc: Philipp <p.stephani2@gmail.com>, Emacs developers <emacs-devel@gnu.org>
> 
> The following patch fixes it :

Thanks, but I don't think this is the right fix.  The problem is that
the test sets the default directory to "/:/", which is invalid on
Windows.  It should set the default directory to something like
"/:c:/" instead.

I fixed the test along those lines.

Re: Bug#25949 is not fixed on MS-Windows

[Fabrice Popineau]

[-- Attachment #1: Type: text/plain, Size: 568 bytes --]

2017-08-05 10:41 GMT+02:00 Eli Zaretskii <eliz@gnu.org>:

> > From: Fabrice Popineau <fabrice.popineau@centralesupelec.fr>
> > Date: Tue, 1 Aug 2017 22:26:01 +0200
> > Cc: Philipp <p.stephani2@gmail.com>, Emacs developers <
> emacs-devel@gnu.org>
> >
> > The following patch fixes it :
>
> Thanks, but I don't think this is the right fix.  The problem is that
> the test sets the default directory to "/:/", which is invalid on
> Windows.  It should set the default directory to something like
> "/:c:/" instead.
>
> I fixed the test along those lines.
>

Ok. Thanks.

[-- Attachment #2: Type: text/html, Size: 1154 bytes --]