On Feb 13 10:08, Jeremy Drake via Cygwin-patches wrote: > 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).
The new behaviour makes more sense, actually. Pushed! Would you mind to send a patch with a release message we can add to release/3.6.0? Thanks, Corinna
