Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[wasm] Fix AOT for projects with space in path/name #92880

Closed
wants to merge 4 commits into from

Conversation

maraf
Copy link
Member

@maraf maraf commented Oct 2, 2023

Relates to #89073 #92399

@ghost ghost assigned maraf Oct 2, 2023
@maraf maraf added the arch-wasm WebAssembly architecture label Oct 2, 2023
@maraf maraf added this to the 9.0.0 milestone Oct 2, 2023
@ghost
Copy link

ghost commented Oct 2, 2023

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

Issue Details

null

Author: maraf
Assignees: maraf
Labels:

arch-wasm, area-Build-mono

Milestone: -

lewing
lewing previously approved these changes Oct 2, 2023
Copy link
Member

@lewing lewing left a comment

Choose a reason for hiding this comment

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

Just don't try %20 in the path

@lewing lewing dismissed their stale review October 2, 2023 14:36

meant to comment

Co-authored-by: kasperk81 <83082615+kasperk81@users.noreply.github.com>
@@ -409,7 +409,7 @@
Condition="'$(EmccInitialHeapSize)' == ''"
DependsOnTargets="_CheckEmccIsExpectedVersion">
<ItemGroup>
<_AOTObjectFile Include="%(_BitcodeFile.ObjectFile)" />
<_AOTObjectFile Include="&quot;%(_BitcodeFile.ObjectFile)&quot;" />
Copy link
Member

Choose a reason for hiding this comment

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

Add the quotes where this is being used instead of here.

@maraf maraf closed this Oct 31, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Nov 30, 2023
@maraf maraf deleted the WasmAotSpace branch January 18, 2024 11:14
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.

4 participants