Skip to content

feat(napi): add aarch64-linux-android target#5141

Merged
Brooooooklyn merged 8 commits intomainfrom
07-02-feat_napi_add_aarch64-linux-android_target
Jul 12, 2025
Merged

feat(napi): add aarch64-linux-android target#5141
Brooooooklyn merged 8 commits intomainfrom
07-02-feat_napi_add_aarch64-linux-android_target

Conversation

@Boshen
Copy link
Member

@Boshen Boshen commented Jul 2, 2025

closes #4560

Copy link
Member Author

Boshen commented Jul 2, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@netlify
Copy link

netlify bot commented Jul 2, 2025

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit abdc470
🔍 Latest deploy log https://app.netlify.com/projects/rolldown-rs/deploys/6871fb9b8fc81e0008846e06

@IWANABETHATGUY
Copy link
Member

It seems you also need to create an extra package under packages/rolldown/npm

@Boshen Boshen requested a review from Brooooooklyn July 2, 2025 08:52
@Boshen
Copy link
Member Author

Boshen commented Jul 2, 2025

@Brooooooklyn can you help out and clean up the need for a packages/rolldown/npm directory? It's redundant.

@Boshen Boshen marked this pull request as draft July 2, 2025 08:53
@Brooooooklyn
Copy link
Member

I don't understand why these dependencies are needed: https://github.com/rolldown/rolldown/blob/main/packages/rolldown/package.json#L180-L191

The packages/rolldown/npm directory is not required if there are no optionalDependencies present

@Boshen
Copy link
Member Author

Boshen commented Jul 2, 2025

I don't understand why these dependencies are needed: https://github.com/rolldown/rolldown/blob/main/packages/rolldown/package.json#L180-L191

The packages/rolldown/npm directory is not required if there are no optionalDependencies present

That's why I'm asking you to clean it up 😅 It's a bit confusing right now.

@hyf0
Copy link
Member

hyf0 commented Jul 2, 2025

I don't understand why these dependencies are needed: https://github.com/rolldown/rolldown/blob/main/packages/rolldown/package.json#L180-L191

The packages/rolldown/npm directory is not required if there are no optionalDependencies present

Here's the reason: #654.

@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from fe74e55 to 99f7baf Compare July 11, 2025 03:38
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jul 11, 2025

Open in StackBlitz

@rolldown/browser

npm i https://pkg.pr.new/@rolldown/browser@5141

@rolldown/debug

npm i https://pkg.pr.new/@rolldown/debug@5141

@rolldown/pluginutils

npm i https://pkg.pr.new/@rolldown/pluginutils@5141

rolldown

npm i https://pkg.pr.new/rolldown@5141

@rolldown/binding-android-arm64

npm i https://pkg.pr.new/@rolldown/binding-android-arm64@5141

@rolldown/binding-darwin-arm64

npm i https://pkg.pr.new/@rolldown/binding-darwin-arm64@5141

@rolldown/binding-darwin-x64

npm i https://pkg.pr.new/@rolldown/binding-darwin-x64@5141

@rolldown/binding-freebsd-x64

npm i https://pkg.pr.new/@rolldown/binding-freebsd-x64@5141

@rolldown/binding-linux-arm-gnueabihf

npm i https://pkg.pr.new/@rolldown/binding-linux-arm-gnueabihf@5141

@rolldown/binding-linux-arm64-gnu

npm i https://pkg.pr.new/@rolldown/binding-linux-arm64-gnu@5141

@rolldown/binding-linux-arm64-musl

npm i https://pkg.pr.new/@rolldown/binding-linux-arm64-musl@5141

@rolldown/binding-linux-x64-gnu

npm i https://pkg.pr.new/@rolldown/binding-linux-x64-gnu@5141

@rolldown/binding-linux-x64-musl

npm i https://pkg.pr.new/@rolldown/binding-linux-x64-musl@5141

@rolldown/binding-wasm32-wasi

npm i https://pkg.pr.new/@rolldown/binding-wasm32-wasi@5141

@rolldown/binding-win32-arm64-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-arm64-msvc@5141

@rolldown/binding-win32-ia32-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-ia32-msvc@5141

@rolldown/binding-win32-x64-msvc

npm i https://pkg.pr.new/@rolldown/binding-win32-x64-msvc@5141

commit: d3c250b

@graphite-app
Copy link
Contributor

graphite-app bot commented Jul 11, 2025

Merge activity

  • Jul 11, 6:06 AM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
  • Jul 11, 6:06 AM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..
  • Jul 11, 6:06 AM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
  • Jul 11, 6:06 AM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..
  • Jul 11, 6:08 AM UTC: Graphite disabled "merge when ready" on this PR due to: Unknown error.

@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from 53f476a to 46684ed Compare July 11, 2025 08:12
@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from 3e5c505 to 42763b3 Compare July 11, 2025 09:43
@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from 42763b3 to 8874420 Compare July 11, 2025 09:59
@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from 4ef8a5c to d3c250b Compare July 12, 2025 02:15
@Brooooooklyn Brooooooklyn marked this pull request as ready for review July 12, 2025 06:05
@Brooooooklyn Brooooooklyn force-pushed the 07-02-feat_napi_add_aarch64-linux-android_target branch from ce29359 to abdc470 Compare July 12, 2025 06:07
@Brooooooklyn Brooooooklyn added this pull request to the merge queue Jul 12, 2025
Merged via the queue into main with commit ffa4212 Jul 12, 2025
41 checks passed
@Brooooooklyn Brooooooklyn deleted the 07-02-feat_napi_add_aarch64-linux-android_target branch July 12, 2025 06:52
shulaoda added a commit that referenced this pull request Jul 13, 2025
## [1.0.0-beta.27] - 2025-07-13

### 🚀 Features

- rolldown: support linux-arm64-ohos (#5258) by @Brooooooklyn
- napi: add `aarch64-linux-android` target (#5141) by @Boshen
- rolldown: oxc v0.77.0 (#5255) by @Boshen

### 🐛 Bug Fixes

- `this` in CJS is replaced with undefined but should be `exports`
(#5244) by @IWANABETHATGUY
- normalize path from package.json's realpath before matching
sideEffects field (#5250) by @sapphi-red

### ⚡ Performance

- don't wrap module if it is leaf module and side effects free when
enable `strictExecutionOrder` (#5240) by @IWANABETHATGUY

### ⚙️ Miscellaneous Tasks

- upgrade NAPI-RS cli to 3.0.0-alpha.93 (#5243) by @Brooooooklyn

Co-authored-by: shulaoda <165626830+shulaoda@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support platform for Android

4 participants