On Thu, 13 Feb 2025, Corinna Vinschen wrote: > On Feb 13 15:15, Corinna Vinschen wrote: > > > I think this looks good, including patch 3. > > To wit: > > $ mount | grep drvmount > C:/drvmount on /mnt/c/drvmount type ntfs (binary,posix=0,noumount,auto) > $ mount C:/drvmount /home/corinna/drv > $ mount | grep drvmount > C:/drvmount on /home/corinna/drv type ntfs (binary,user) > C:/drvmount on /mnt/c/drvmount type ntfs (binary,posix=0,noumount,auto) > $ df /mnt/c/drvmount > Filesystem 1K-blocks Used Available Use% Mounted on > C:/drvmount 4175868 56744 4119124 2% /mnt/c/drvmount > $ df /home/corinna/drv > Filesystem 1K-blocks Used Available Use% Mounted on > C:/drvmount 4175868 56744 4119124 2% /home/corinna/drv > > > If you're fine with that, > > I'll push all three patches.
Sounds good to me. It's less code, and the code it calls (cygdrive_posix_path) is more efficient because it doesn't have to loop through all mounts. As long as you're ok with the change in behavior that: mount C: /c mount will now show C: on both /c and /cygdrive/c, instead of just /c as before (this is the behavior all of that de-duplication mess was trying to replicate).
