Skip to content

Conversation

c0d1ngm0nk3y
Copy link
Contributor

Prerequisite: New packit release must be consumed to handle the same dependencies with different architectures.

Summary

Adding arm dependencies for download

Use Cases

I want to build an arm image.

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@c0d1ngm0nk3y
Copy link
Contributor Author

@paketo-buildpacks/nodejs-maintainers Probably some adjustment to dependabot are needed. Could you give me some guidance?

@mhdawson
Copy link
Member

@pacostas I think you know the most about freezer so maybe you can take the first look.

@pacostas
Copy link
Contributor

Nice! I think it would be good to apply the multi-arch changes too by following what has been changed on below two PRs:

You will also see on the buildpack.toml file, that is specified the include-files, pre-package and the target attributes. These are necessary for jam and pack, to know what to package.

If you apply the multi-arch changes, then I can run it locally and see what is failing as there are a lot of chances that the error might be due pack does not match properly the arm assets.

@c0d1ngm0nk3y
Copy link
Contributor Author

I think it would be good to apply the multi-arch changes too

see 93da5f2

@c0d1ngm0nk3y c0d1ngm0nk3y added the semver:minor A change requiring a minor version bump label Mar 28, 2025
@c0d1ngm0nk3y
Copy link
Contributor Author

I think it would be good to apply the multi-arch changes too by following what has been changed on below two PRs

I am starting to understand the github-config idea. I hadn't much touch points in the past. But is it not suboptimal to change some files like build.sh, package.sh or create-draft-release.yml and add them to .syncignore. Especially if node-engine would need the very same changes? I think that is the idea behind github-config or what is the planto avoid double maintenance?

@pacostas
Copy link
Contributor

The plan is to sync all the scripts on github-config and remove them from .syncignore, once we ensure that all the buildpacks are able to use these scripts without breaking any of the worfklows. We dont have to go through the whole list paketo-buildpacks/nodejs#995 and open PRs for each buildpack, instead, by ensuring that this PR works with the extension, the npm-install, the node-engine and the nodejs repos, I think we are good to merge it on github-config little by little and later on removing it from .syncignore to be in sync with github-config.

"linux/amd64/bin/detect",
"linux/amd64/bin/run",
"linux/amd64/bin/optimize-memory,
"linux/amd64/bin/inspector,
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like there is a double quote missing here.
Are there any errors that pack throws after fixing the double quote?

@pacostas
Copy link
Contributor

@c0d1ngm0nk3y This PR is superseded by this one #1293

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:minor A change requiring a minor version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants