Skip to content

Build arm64 version of pylauncher.exe#27360

Merged
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:pylauncher_arm64
Jul 20, 2026
Merged

Build arm64 version of pylauncher.exe#27360
sbc100 merged 1 commit into
emscripten-core:mainfrom
sbc100:pylauncher_arm64

Conversation

@sbc100

@sbc100 sbc100 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

The executable comes it at just 6144 bytes.

@sbc100
sbc100 requested a review from dschuff July 15, 2026 17:51
@sbc100
sbc100 force-pushed the pylauncher_arm64 branch from d6f2878 to b908775 Compare July 15, 2026 17:58
@sbc100
sbc100 force-pushed the pylauncher_arm64 branch from b908775 to a81273b Compare July 15, 2026 18:00
@sbc100
sbc100 requested a review from juj July 15, 2026 18:04

@dschuff dschuff left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do we have users that are asking for this? Are they just building their own LLVM and Binaryen?

Comment thread .github/workflows/ci.yml
steps:
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
- name: Setup MSVC
uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is it bad to use a devenv from some random person? Is there really not an official one?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I seems like github actions doesn't have way to get cl.exe in the PATH by default.

There is another implementation at https://github.com/egor-tensin/vs-shell.

We could also try to take the approach we used on circleci and write out own bat file: https://github.com/emscripten-core/emscripten/blob/main/.circleci/setup_vs2022.bat

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems fine, especially keeping the version pinned by hash.

@sbc100

sbc100 commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator Author

do we have users that are asking for this? Are they just building their own LLVM and Binaryen?

@juj was asking about it in #24858 (comment)

set OUT=pylauncher.exe
set MACHINE=X64

if /i "%~1"=="arm64" (

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I will take your word for it that this is the best way to do this 😅

Comment thread .github/workflows/ci.yml
steps:
- uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0
- name: Setup MSVC
uses: ilammy/msvc-dev-cmd@0b201ec74fa43914dc39ae48a89fd1d8cb592756 # v1.13.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems fine, especially keeping the version pinned by hash.

@sbc100
sbc100 merged commit 1c6fa69 into emscripten-core:main Jul 20, 2026
40 checks passed
@sbc100
sbc100 deleted the pylauncher_arm64 branch July 20, 2026 22:14
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.

2 participants