Skip to content

[wasi/wasm] EmccCompile.cs: quote the compiler path#97427

Merged
lewing merged 1 commit intodotnet:mainfrom
radical:fix-emcc-path-quoting
Jan 24, 2024
Merged

[wasi/wasm] EmccCompile.cs: quote the compiler path#97427
lewing merged 1 commit intodotnet:mainfrom
radical:fix-emcc-path-quoting

Conversation

@radical
Copy link
Copy Markdown
Member

@radical radical commented Jan 23, 2024

This broke in PR #95051 which changed the task to use a custom path to
the compiler instead of using emcc from PATH.

Fixes https://github.com/dotnet/AspNetCore-ManualTests/issues/2594 .

This broke in PR#95051 which changed the task to use a custom path to
the compiler instead of using `emcc` from `PATH`.

Fixes dotnet/AspNetCore-ManualTests#2594 .
@radical radical added the arch-wasm WebAssembly architecture label Jan 23, 2024
@ghost
Copy link
Copy Markdown

ghost commented Jan 23, 2024

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

This broke in PR#95051 which changed the task to use a custom path to
the compiler instead of using emcc from PATH.

Fixes https://github.com/dotnet/AspNetCore-ManualTests/issues/2594 .

Author: radical
Assignees: -
Labels:

arch-wasm

Milestone: -

@ghost ghost added the area-Build-mono label Jan 23, 2024
@ghost ghost assigned radical Jan 23, 2024
@lewing
Copy link
Copy Markdown
Member

lewing commented Jan 23, 2024

thanks @radical

@radical
Copy link
Copy Markdown
Member Author

radical commented Jan 23, 2024

/backport to release/9.0-preview1

@github-actions
Copy link
Copy Markdown
Contributor

Started backporting to release/9.0-preview1: https://github.com/dotnet/runtime/actions/runs/7633027574

@lewing
Copy link
Copy Markdown
Member

lewing commented Jan 24, 2024

#97044 is causing Release LibraryTests_EAT causing the lane to hang

@lewing lewing merged commit 109211e into dotnet:main Jan 24, 2024
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

arch-wasm WebAssembly architecture area-Build-mono

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants