Skip to content

[3.24.0] backport #15156#15229

Merged
shonfeder merged 2 commits into
ocaml:3.24.0-rcfrom
Alizter:backport-15156
Jun 21, 2026
Merged

[3.24.0] backport #15156#15229
shonfeder merged 2 commits into
ocaml:3.24.0-rcfrom
Alizter:backport-15156

Conversation

@Alizter

@Alizter Alizter commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Backport #15156 onto 3.24.0-rc.

Also includes the test commit from #15148 (test: bash bin pform) as a prerequisite, since the test file runnable-argument.t modified by #15156 was introduced in #15148 and is not yet on 3.24.0-rc.

Alizter added 2 commits June 18, 2026 07:50
(cherry picked from commit bf27262)

Signed-off-by: Ali Caglayan <alizter@gmail.com>
Value.string_of_path now uses Path.reach_for_running so that pforms
render bare-basename paths with a leading "./". This avoids bash and
other shells (in (bash ...) and (system ...) actions) treating
same-directory binaries as PATH lookups.

Also rewrites the %{bin:NAME} entry in doc/concepts/variables.rst.

(cherry picked from commit 2851a23)

Signed-off-by: Ali Caglayan <alizter@gmail.com>
@Alizter Alizter requested a review from shonfeder June 18, 2026 06:56
@Alizter Alizter mentioned this pull request Jun 18, 2026
30 tasks
@shonfeder

Copy link
Copy Markdown
Member

Great. Thanks for adding this!

@shonfeder shonfeder merged commit 49b6157 into ocaml:3.24.0-rc Jun 21, 2026
30 checks passed
@Alizter Alizter added this to the 3.24.0 milestone Jul 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants