On 8/2/2022 4:19 AM, Corinna Vinschen wrote:
On Aug 2 13:12, Takashi Yano wrote:
Then, what about the v2 patch attached?
symlink_wsl is doing the right thing, as Ken points out.
Actually, I was suggesting that symlink_wsl was doing the wrong thing in case
the user used an uppercase drive letter. WSL doesn't recognize /mnt/C, so I
don't think symlink_wsl should convert /cygdrive/C to /mnt/C.
Therefore, the more effort the code makes to be clever, the higher
chances are that an incorrect conversion takes place.
Agreed.
I'm tempted to go even further and say that Cygwin shouldn't ever
convert the cygdrive prefix to /mnt, on the grounds that users who
care about WSL interoperability can simply use /mnt as their
cygdrive prefix. But maybe that ship has sailed.
In hindsight this might have been a step too far. I was trying to allow
interoperability and reduce the number of problems based on different
drive letter handling. And, well, this is the first time a user has
a problem due to that :}
The ship hasn't sailed entirely. We can revert this decision for 3.4
and just keep the /mnt conversion in check_reparse_point_target for
backward compat. Or we just fix the problem at hand and otherwise
keep the code as is...?
The latter seems like the safest solution, rather than risk breaking some other
use case.
Ken
--
Problem reports: https://cygwin.com/problems.html
FAQ: https://cygwin.com/faq/
Documentation: https://cygwin.com/docs.html
Unsubscribe info: https://cygwin.com/ml/#unsubscribe-simple
