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 family presets #14

Merged
merged 5 commits into from
Nov 23, 2023
Merged

Target family presets #14

merged 5 commits into from
Nov 23, 2023

Conversation

dherman
Copy link
Collaborator

@dherman dherman commented Nov 22, 2023

Add new target family presets to add-target command:

  • neon add-target can take a preset, e.g. neon add-target linux
  • we won't yet change the "targets" section format but we'll get ready to in the future:
    • neon add-target creates the template package directory for a target
    • neon tarball in the future won't ever take a --target and create a temp dir, it'll require --in-dir
    • For now, add neon rust-target to do the node2rust conversion for a project, and we'll use that in neon-actions/build in a future iteration
    • To be ready for that, this repo will always use --in-dir instead of --target
    • neon add-target creates the template package directory for a target
image image

David Herman added 5 commits November 23, 2023 00:00
- family.json is the database of presets
- add-target help info describes the presets
- `neon add-target` can take a present e.g. `neon add-target linux`
- we won't yet change the `"targets"` section format but we'll get ready to in the future:
  * `neon tarball` in the future won't ever take a --target and create a temp dir, it'll require `--in-dir`
  * so for now let's add `neon rust-target` to do the node2rust conversion for a project, and we'll use that in neon-actions/build in a future iteration
  * to be ready for that, this repo will always use `--in-dir` instead of `--target`
  * in the next commit in this PR, let's make `neon add-target` create the template package directory for a target
@dherman dherman marked this pull request as ready for review November 23, 2023 07:47
@dherman dherman merged commit bf34a89 into main Nov 23, 2023
2 checks passed
@dherman dherman deleted the target-families branch November 23, 2023 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant