Skip to content

[v18] Remove 'wasm-pack' Build Dependency#62174

Merged
rhammonds-teleport merged 1 commit intobranch/v18from
rhammonds/backport-58618-branch/v18
Dec 11, 2025
Merged

[v18] Remove 'wasm-pack' Build Dependency#62174
rhammonds-teleport merged 1 commit intobranch/v18from
rhammonds/backport-58618-branch/v18

Conversation

@rhammonds-teleport
Copy link
Copy Markdown
Contributor

Backport #58618 to branchv/18

* Get a Teleport build working without wasm-pack

* * seed docker images with wasm-opt, and wasm-bindgen-cli
* Replace potentially confusing 'CARGO_TARGET_wasm' variable.
* Replace oneoff .gitignore with a corresponding entry in the top level .gitignore.
* Replace 'printf' used to create package.json file with a cleaner multiline string.
* Remove unnecessary 'NEED_VERSION'
* Use awk to find wasm-bindgen version without using 'cargo' since it doesn't seem to be available on ARM64 runners

* Move 'WASM_BINDGEN_VERSION' definition out of versions.mk

* Pass '--locked' flag to wasm-opt install
@rhammonds-teleport rhammonds-teleport added the no-changelog Indicates that a PR does not require a changelog entry label Dec 11, 2025
@rhammonds-teleport rhammonds-teleport marked this pull request as ready for review December 11, 2025 19:24
@github-actions github-actions bot requested a review from bl-nero December 11, 2025 19:25
Copy link
Copy Markdown
Contributor

@fheinecke fheinecke left a comment

Choose a reason for hiding this comment

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

Please cut a dev build prior to merge to ensure that there is no branch-specific logic that breaks because of this

@rhammonds-teleport
Copy link
Copy Markdown
Contributor Author

Please cut a dev build prior to merge to ensure that there is no branch-specific logic that breaks because of this

Of course! https://github.com/gravitational/teleport.e/actions/runs/20143189013/job/57817214307
Some networking issues are affecting a few downstream jobs, but it looks like the actual builds have passed.

@rhammonds-teleport rhammonds-teleport added this pull request to the merge queue Dec 11, 2025
Merged via the queue into branch/v18 with commit 400895e Dec 11, 2025
47 of 48 checks passed
@rhammonds-teleport rhammonds-teleport deleted the rhammonds/backport-58618-branch/v18 branch December 11, 2025 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants