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

Prefer targets over stacks #790

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

AidanDelaney
Copy link
Member

When creating a new buildpack, use --targets instead of the deprecated --stacks

fixes: #786

@AidanDelaney AidanDelaney requested a review from a team as a code owner November 27, 2024 05:49
When creating a new buildpack, use --targets instead of the
deprecated --stacks

Signed-off-by: Aidan Delaney <[email protected]>
@AidanDelaney AidanDelaney force-pushed the fix-786/deprecate-stacks branch from 70fd180 to 1491ad1 Compare November 27, 2024 05:49
@gogolok
Copy link
Contributor

gogolok commented Dec 2, 2024

> Installed pack version:  0.31.0
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
WARNING: Expected pack version: 0.36.0

pack-version: '0.31.0'

It's time to update the pack version in the GitHub workflow/actions?

@@ -25,7 +25,7 @@ pack buildpack new examples/node-js \
--api 0.10 \
--path node-js-buildpack \
--version 0.0.1 \
--stacks io.buildpacks.samples.stacks.noble
--targets "linux/amd64"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: While you are here, maybe

diff --git a/content/docs/for-buildpack-authors/tutorials/basic-buildpack/02_building-blocks-cnb.md b/content/docs/for-buildpack-authors/tutorials/basic-buildpack/02_building-blocks-cnb.md
index 55c72761..6ddb7977 100644
--- a/content/docs/for-buildpack-authors/tutorials/basic-buildpack/02_building-blocks-cnb.md
+++ b/content/docs/for-buildpack-authors/tutorials/basic-buildpack/02_building-blocks-cnb.md
@@ -34,7 +34,7 @@ This command will create `node-js-buildpack` directory which contains `buildpack
 - `-a, --api` Buildpack API compatibility of the generated buildpack
 - `-h, --help` Help for 'new'
 - `--path` the location on the filesystem to generate the artifacts
-- `--stacks` Stacks (deprecated) the buildpack will work with
+- `--targets` OS and arch the buildpack will target
 - `-V, --version` the version of the buildpack in buildpack.toml

or similar makes sense?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've bumped this in #793. We will update the text when your changes to pack --targets lands. buildpacks/pack#2296

@AidanDelaney
Copy link
Member Author

Thanks a lot for your work on this

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.

Flag --stacks has been deprecated
2 participants