-
Notifications
You must be signed in to change notification settings - Fork 1.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add "meson run" command #13885
base: master
Are you sure you want to change the base?
Add "meson run" command #13885
Conversation
docs/markdown/Commands.md
Outdated
*Since 1.7.0* `run` is an alias of `devenv` with the difference that if `--bin` | ||
is not specified it defaults to the project's main executable. It is an error if | ||
more than one executable is installable and `--bin` is not specified or is | ||
ambiguous. Since an executable is required, `meson run` will never start an |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is badly worded and difficult to understand. What is "the project's main executable"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ultimately I would like to add e.g. meson.set_run_target(exe)
to let define from meson.build what it the project's run command. For now if there is one and only one installable exe it picks it, otherwise it's an error.
How would you rephrase?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we don't have such an API then we shouldn't write documentation as if we do -- we can always reword the documentation if and when we get such an API.
If you're suggesting to create a guess mode then it should just say that straight out: "--bin is required unless the project only contains a single installable executable".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rephrased the doc now that we don't have --bin in devenv anymore. Is it better?
ec2e185
to
bf3c40f
Compare
540300b
to
df52e10
Compare
CI issue seems unrelated |
No description provided.