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

Add linux/arm64 targets #815

Merged
merged 3 commits into from
May 9, 2024
Merged

Add linux/arm64 targets #815

merged 3 commits into from
May 9, 2024

Conversation

runesoerensen
Copy link
Contributor

@runesoerensen runesoerensen commented Apr 19, 2024

This PR simply adds the linux/arm64 targets required to package and use the various buildpacks in this repository.

It depends on #814, which introduces arm64 support through updated inventory logic and version resolution that factors in the host OS and arch.

@runesoerensen runesoerensen added enhancement New feature or request skip changelog labels Apr 19, 2024
@runesoerensen runesoerensen self-assigned this Apr 19, 2024
@runesoerensen runesoerensen force-pushed the support-arm64 branch 2 times, most recently from 409ba6d to a147938 Compare April 23, 2024 14:45
@runesoerensen runesoerensen marked this pull request as ready for review April 23, 2024 14:47
@runesoerensen runesoerensen force-pushed the use-artifact-inventory branch from 2db2178 to 59cfa15 Compare April 25, 2024 20:46
@runesoerensen runesoerensen requested a review from a team as a code owner April 25, 2024 20:46
@edmorley edmorley removed the request for review from a team April 28, 2024 13:04
@runesoerensen runesoerensen changed the base branch from use-artifact-inventory to main April 29, 2024 21:56
@runesoerensen
Copy link
Contributor Author

runesoerensen commented May 8, 2024

I've added changelog entries as well as targets for nodejs-npm-install - I'm assuming the missing linux/arm64 target in that buildpack.toml was an oversight. Let me know otherwise!

I've removed the "skip changelog" label, and requested another review as there are changes to several files since this was initially approved by @joshwlewis - so maybe just add a 👍 if that all looks good before I merge it? :)

@runesoerensen runesoerensen requested a review from joshwlewis May 8, 2024 16:54
Copy link
Member

@joshwlewis joshwlewis left a comment

Choose a reason for hiding this comment

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

Nice work, let's do it!

@joshwlewis joshwlewis merged commit 08e9f1c into main May 9, 2024
26 checks passed
@joshwlewis joshwlewis deleted the support-arm64 branch May 9, 2024 14:47
colincasey pushed a commit that referenced this pull request May 10, 2024
## heroku/nodejs

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

### Changed

- Updated `heroku/nodejs-corepack` to `3.1.0`.
- Updated `heroku/nodejs-engine` to `3.1.0`.
- Updated `heroku/nodejs-npm-engine` to `3.1.0`.
- Updated `heroku/nodejs-npm-install` to `3.1.0`.
- Updated `heroku/nodejs-pnpm-engine` to `3.1.0`.
- Updated `heroku/nodejs-pnpm-install` to `3.1.0`.
- Updated `heroku/nodejs-yarn` to `3.1.0`.

## heroku/nodejs-corepack

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-engine

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.1.0`.
- Updated `heroku/nodejs-function-invoker` to `3.1.0`.
- Updated `heroku/nodejs-npm` to `3.1.0`.

## heroku/nodejs-function-invoker

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-npm-install

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-pnpm-engine

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-pnpm-install

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

## heroku/nodejs-yarn

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
colincasey pushed a commit that referenced this pull request May 10, 2024
## heroku/nodejs

### Added

- Support for `arm64` and multi-arch images. ([#815](#815))

### Changed

- Updated `heroku/nodejs-corepack` to `3.2.0`.
- Updated `heroku/nodejs-engine` to `3.2.0`.
- Updated `heroku/nodejs-npm-engine` to `3.2.0`.
- Updated `heroku/nodejs-npm-install` to `3.2.0`.
- Updated `heroku/nodejs-pnpm-engine` to `3.2.0`.
- Updated `heroku/nodejs-pnpm-install` to `3.2.0`.
- Updated `heroku/nodejs-yarn` to `3.2.0`.

## heroku/nodejs-corepack

- No changes.

## heroku/nodejs-engine

- No changes.

## heroku/nodejs-function

### Changed

- Updated `heroku/nodejs-engine` to `3.2.0`.
- Updated `heroku/nodejs-function-invoker` to `3.2.0`.
- Updated `heroku/nodejs-npm` to `3.2.0`.

## heroku/nodejs-function-invoker

- No changes.

## heroku/nodejs-npm

- No changes.

## heroku/nodejs-npm-engine

- No changes.

## heroku/nodejs-npm-install

- No changes.

## heroku/nodejs-pnpm-engine

- No changes.

## heroku/nodejs-pnpm-install

- No changes.

## heroku/nodejs-yarn

- No changes.

Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
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 this pull request may close these issues.

2 participants