Cannot use UNC paths on Windows VMs about nuitka HOT 8 CLOSED

So I merged your PR, and will put it on my factory branch, I don't think I really want to make this part of a relatively late hotfix, since it risks breaking existing setups. I tried Python 2.7 and 3.10 where Python is a symlink and it continued to work, that is something I care about. Moving it out to a separate function is bound to make the code maybe more readable.

from nuitka.

Part of the current pre-release.

from nuitka.

Screenshots and no version information, I am sure you can do better, and I have indeed changed something there recently for #2732 and this code for resolving symlinks is needed, because many tools don't work with non-real paths. I think the issue solution there was to indeed avoid resolve something different.

from nuitka.

I fixed getFilenameRealPath to undo the damage in case drive letters get lost, but it may not be general enough to cover these mounts in the middle of things. You are more than welcome to make it walk the path up if it should be necessary.

from nuitka.

Sorry, I liked to make bug reports completely reproduceable,
but VMs in Windows world are strange, there is no reliable way to make it reproduceable by some unified way (VirtualBox, Hyper-V, Windows Sandbox):

• Windows sanboxes are strangely broken for now
• Working VirtualBox are incompatible with working Hyper-V
• ...

So, I would like to know, what virtualization have you on your windows stand, to make reproduceable examples with vagrant (for example).

I catch the issue sometimes ago, make quick fix "works for me", and was testing on my different projects during several weeks. So, I miss the #2732 and unfortunately, I dont see linked commits that should fix this issue for checking.

I tried to check latest pre-release commit ea9093d

and the issue still persists somehow:

nuitka-crash-report.zip

What tools needed real UNC path here? Why just not

...
            newpath = path
            if not result.startswith('UNC'):
                newpath = os.path.join(
                    os.path.dirname(path), result.rstrip("\r\n")
                )
            _real_path_windows_cache[path] = newpath
...

from nuitka.

The commit history of the nuitka.utils.FileOperations.py has it. Due to rebasing for perfect git bisection, I don't use commit hashes as much as others do.

That change handles top level, but can surely be extended.

Like I said, many tools cannot work with symlinks, and need short paths in 8.3 style to work reliably. This includes tools from big companies. The realpath does more than it's supposed to do there, but it cannot do only one thing. Reverting it to drive letter paths, was easy for me. I imagine you can make it more general.

from nuitka.

Thanks for your PRs, these are all super useful, keep em coming :)

from nuitka.

Part of the 2.2 release I just made.

from nuitka.