Skip to content

clarify argument splitting in 'rocq makefile' - #22322

Open
RalfJung wants to merge 1 commit into
rocq-prover:masterfrom
RalfJung:rocq-makefile-arg-splitting
Open

clarify argument splitting in 'rocq makefile'#22322
RalfJung wants to merge 1 commit into
rocq-prover:masterfrom
RalfJung:rocq-makefile-arg-splitting

Conversation

@RalfJung

Copy link
Copy Markdown
Contributor

The docs say that -arg "-w all" passes the argument -w all to rocq compile, but that's not quite correct -- when stated like that, I would interpret this as meaning that a single argument, "-w all", is passed to rocq compile. What actually happens is that two arguments, "-w" "all", are passed. So this adjusts the docs to clarify that.

What is still missing is a description of how to pass arbitrary arguments. For instance if I wanted to pass arg' arg as a single argument, as in the shell invocation rocq compile "arg\' arg", I do not know how I would do that. Lucky enough this does not usually seem to be necessary...

@RalfJung
RalfJung requested a review from a team as a code owner July 29, 2026 10:26
@coqbot-app coqbot-app Bot added the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Jul 29, 2026
But note, that single-quotes in a ``_RocqProject`` file are only special
characters if they appear in the string following ``-arg``. And on their own
they don't quote spaces. For example ``-arg 'foo bar'`` in ``_RocqProject`` is
equivalent to ``-arg foo "bar'"`` (in ``_RocqProject`` notation). ``-arg "'foo

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part here is still confusing me, or maybe it is wrong. Somehow the first ', the one before foo, is apparently getting lost?

@RalfJung
RalfJung force-pushed the rocq-makefile-arg-splitting branch from c544bdb to 8efa6dc Compare July 29, 2026 10:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant