Skip to content
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

Support --target-dir in cargo make invocations #252

Open
wmmc88 opened this issue Dec 10, 2024 · 0 comments
Open

Support --target-dir in cargo make invocations #252

wmmc88 opened this issue Dec 10, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@wmmc88
Copy link
Collaborator

wmmc88 commented Dec 10, 2024

Currently, this is not supported for 2 reasons:

  1. the option is not currently parsed by our cargo-make plugin:
    impl ParseCargoArgs for CompilationOptions {

This is a simple enhancement^

  1. The implementation of find_top_level_cargo_manifest does not work when a --target-dir is supplied:
    pub fn find_top_level_cargo_manifest() -> PathBuf {

AFAIU, this 2nd issue is blocked by having support for mutually-exclusive cargo features. The pre-RFC for that idea is here. See previous discussion on this issue: rust-lang/cargo#13484

For the time being, as far as i can tell, there is no path forward to support --target-dir builds until progress is made on mutually exclusive cargo features

@wmmc88 wmmc88 added the enhancement New feature or request label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant