[Octave-bug-tracker] [bug #56267] addpath() on Windows fails to recognize same subdirs with different fileseps and case

[Markus Mützel]

Follow-up Comment #9, bug #56267 (project octave):

With the attached path, I see the following on Windows 10 64bit:

>> canonicalize_file_name ('C:\WINDOWS')
ans = C:\Windows
>> canonicalize_file_name ('C:\WINDOWS\SYSTEM32')
ans = C:\Windows\System32
>> canonicalize_file_name ('c:\WiNdOwS/SYSWOW64')
ans = c:\Windows\SysWOW64


It doesn't work for UNC paths though. However, I don't know whether we are
supporting these anyway.

It also doesn't change the case of the drive letter. Maybe we should always
return a capital letter?

I am also not sure whether it is OK to conditionally include windows.h in the
core file.

Octave 6 crashes when running the test suite (with and without the patch).
Therefore, it is hard to judge whether it is save to apply this change. The
tests that were running until the crash looked ok.

(file #47557)
    _______________________________________________________

Additional Item Attachment:

File name: bug56267_win32_canonicalize_case.patch Size:1 KB
   
<https://savannah.gnu.org/file/bug56267_win32_canonicalize_case.patch?file_id=47557>



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?56267>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/