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

Target specific sources #62

Closed
pedrolamarao opened this issue Oct 25, 2023 · 2 comments · Fixed by #72
Closed

Target specific sources #62

pedrolamarao opened this issue Oct 25, 2023 · 2 comments · Fixed by #72
Labels
enhancement New feature or request
Milestone

Comments

@pedrolamarao
Copy link
Owner

Provide DSL for configuring target specific sources.
It is enough to allow including sources by pattern.

@pedrolamarao pedrolamarao added the enhancement New feature or request label Oct 25, 2023
@pedrolamarao pedrolamarao added this to the M4 milestone Oct 25, 2023
@pedrolamarao
Copy link
Owner Author

Do we really need more than this?

tasks.named<MetalCompileTask>("compile-main-cxx") {
    when (target.get()) {
        "i686-elf" -> include("x86_32-elf/*")
        "x86_64-elf" -> include("x86_64-elf/*")
    }
}

@pedrolamarao pedrolamarao modified the milestones: M4, M5 Nov 7, 2023
@pedrolamarao
Copy link
Owner Author

Yes, we do: we must apply the same include/exclude patterns to both compile objects and compile commands tasks.

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

Successfully merging a pull request may close this issue.

1 participant