Skip to content

Add some checks on .install root and rootexec fields - #7039

Draft
rjbou wants to merge 6 commits into
ocaml:masterfrom
rjbou:dot-install-root-checks
Draft

Add some checks on .install root and rootexec fields#7039
rjbou wants to merge 6 commits into
ocaml:masterfrom
rjbou:dot-install-root-checks

Conversation

@rjbou

@rjbou rjbou commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

This PR adds some checks on the target optional field : it shouldn't begin with another field prefix. There a warning when it is the case (and failure on strict mode)
From #6938 (comment)

Todo :

@rjbou rjbou added this to the 2.6.0~alpha2 milestone Jul 15, 2026
@rjbou
rjbou force-pushed the dot-install-root-checks branch from d8ef23f to be6cbbf Compare July 15, 2026 17:47
@kit-ty-kate

Copy link
Copy Markdown
Member

I would argue that having this check would be at best unnecessary and at worst annoying for users generating their .install file themselves. I can honestly see value in giving users the possibility to simply do:

root: [
  ... # everything here
]

it would make protyping a lot easier for small tools and scripts (e.g. generating opam packages from other package managers)

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.

3 participants