Skip to content

Ability to externalize WASM was broken #3345

Closed
@mhdawson

Description

Bug Description

This PR introduced the ability to externalize WASM, which is needed by distros to be able to rebuild all components shipped: #2643

PR #3074 broke this by removing some of the implementation needed.

Reproducible By

Build as specified in the undici CONTRIBUTING.md - https://github.com/nodejs/undici/blob/main/CONTRIBUTING.md#building-for-externally-shared-node-builtins

and validate that the wasm is not in the expected place

Expected Behavior

After building as described in the undici CONTRIBUTING.md - https://github.com/nodejs/undici/blob/main/CONTRIBUTING.md#building-for-externally-shared-node-builtins

the wasm binary should be in the expected place

Logs & Screenshots

Environment

Additional context

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions