Skip to content

fix: allow multiple (dirs ..) stanzas#13734

Draft
anmonteiro wants to merge 2 commits intoocaml:mainfrom
anmonteiro:anmonteiro/multiple-dirs
Draft

fix: allow multiple (dirs ..) stanzas#13734
anmonteiro wants to merge 2 commits intoocaml:mainfrom
anmonteiro:anmonteiro/multiple-dirs

Conversation

@anmonteiro
Copy link
Collaborator

fixes #6249

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@rgrinberg
Copy link
Member

I don't think we should making the evaluation of stanzas in a dune file dependant on the order.

@anmonteiro
Copy link
Collaborator Author

thank you, i agree. do you have a suggestion on how we'd handle something like:

(dirs foo)
(dirs :standard \ foo)
(dirs foo)

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.

Feature request: allow multiple (dirs ... ) stanzas

2 participants