Skip to content

fix(path): use External.append_local in Build.to_string for consistent separators#14616

Open
Alizter wants to merge 1 commit into
ocaml:mainfrom
Alizter:push-kqrplkwvnsus
Open

fix(path): use External.append_local in Build.to_string for consistent separators#14616
Alizter wants to merge 1 commit into
ocaml:mainfrom
Alizter:push-kqrplkwvnsus

Conversation

@Alizter
Copy link
Copy Markdown
Collaborator

@Alizter Alizter commented May 20, 2026

Replace Filename.concat with External.append_local when stringifying external build paths with a local suffix. This ensures forward slashes are used on all platforms, removing the need for conditional Windows/Unix test expectations.

@Alizter Alizter added the windows Issues that relate to Dune on Microsoft Windows label May 20, 2026
@Alizter Alizter requested review from nojb and punchagan May 20, 2026 17:09
…t separators

Replace Filename.concat with External.append_local when stringifying external
build paths with a local suffix. This ensures forward slashes are used on all
platforms, removing the need for conditional Windows/Unix test expectations.

Signed-off-by: Ali Caglayan <alizter@gmail.com>
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