Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds
--profile
option tobuild
andrun
subcommands.Consideration
cargo
has some default profiles, and one of that for developing nameddev
outputstarget/debug/*
.In this PR I named
debug
for that, but if you mind, it's better to change before merge.FAQ
valid_profile
returnResult<Option<String>, String>
,Result<core::builder::ninja::build::Mode, String>
or something like that?poac
is aspiring to be likecargo
,poac
will support user-defined profiles.Result<String, String>
with return value"debug"
ifvalid_profile(None, false)
orvalid_profile(None, None)
?clean
subcommand.clean
needs to recognize unspecified ,--profile=debug
, and release mode (--profile=release
and/or--release
).None
returned byvalid_profile
indicates above unspecified .