Skip to content

fix(stdune): relativize ordinary same-drive Windows paths - #16285

Open
Alizter wants to merge 1 commit into
ocaml:mainfrom
Alizter:fix-windows-path-reach
Open

Alizter wants to merge 1 commit into
ocaml:mainfrom
Alizter:fix-windows-path-reach

Conversation

@Alizter

@Alizter Alizter commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Description

Relativize ordinary drive-rooted Windows paths when the source and destination
are on the same drive.

Conservatively keep current-drive-rooted, drive-relative, UNC, extended-length,
device, malformed, ADS, and different-drive paths absolute. The characterization
tests in #16279 cover these boundaries.

This follows #16278 and #16279 and completes the ordinary drive-path portion of
the Windows support left out of #15077. It supersedes the automatically closed
#16280.

@Alizter
Alizter force-pushed the fix-windows-path-reach branch from ccd2649 to b276a81 Compare August 31, 2026 13:39
@Alizter
Alizter force-pushed the fix-windows-path-reach branch 2 times, most recently from 1dd99b2 to 7674fa5 Compare August 31, 2026 14:40
@Alizter Alizter added the windows Issues that relate to Dune on Microsoft Windows label Aug 31, 2026
@Alizter
Alizter force-pushed the fix-windows-path-reach branch from 7674fa5 to b023e63 Compare August 31, 2026 14:54
@Alizter
Alizter marked this pull request as ready for review August 31, 2026 16:16
Keep other Windows namespaces and unsupported path forms absolute.

Signed-off-by: Ali Caglayan <alizter@gmail.com>
@Alizter
Alizter force-pushed the fix-windows-path-reach branch from b023e63 to 1befac5 Compare August 31, 2026 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

windows Issues that relate to Dune on Microsoft Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant