Skip to content

[v17] Remove 'wasm-pack' Build Dependency#62214

Merged
rhammonds-teleport merged 1 commit intobranch/v17from
rhammonds/backport-58618-branch/v17
Dec 15, 2025
Merged

[v17] Remove 'wasm-pack' Build Dependency#62214
rhammonds-teleport merged 1 commit intobranch/v17from
rhammonds/backport-58618-branch/v17

Conversation

@rhammonds-teleport
Copy link
Copy Markdown
Contributor

* 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
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from bl-nero December 12, 2025 23:47
@rhammonds-teleport rhammonds-teleport added this pull request to the merge queue Dec 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 12, 2025
@rhammonds-teleport rhammonds-teleport added this pull request to the merge queue Dec 15, 2025
Merged via the queue into branch/v17 with commit 1d2e8ed Dec 15, 2025
43 checks passed
@rhammonds-teleport rhammonds-teleport deleted the rhammonds/backport-58618-branch/v17 branch December 15, 2025 14:57
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