Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Test HybridGlobalization with AOT on CI #104027

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

Test HybridGlobalization with AOT on CI #104027

wants to merge 3 commits into from

Conversation

ilonatommy
Copy link
Member

@ilonatommy ilonatommy commented Jun 26, 2024 •
edited
Loading

Investigating #103229 (comment) that is connected with old issue #51746. In case the problem still exists and no solution is found, log a proper issue for it and close #51746 that is similar but not same and is no longer valid.

Investigating #103229 (comment) that is connected with old issue #51746.

@ilonatommy ilonatommy added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Jun 26, 2024
@ilonatommy
Copy link
Member Author

/azp run runtime-wasm

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ilonatommy
Copy link
Member Author

ilonatommy commented Jun 26, 2024 •
edited
Loading

error (cannot repro locally):
e.g. WasmTestOnNodeJS-ST-Common.Tests / WasmTestOnChrome-ST-Common.Tests

MSBuild version 17.11.0-preview-24279-02+b963c24ef for .NET
  ** Building a proxy for the original test project, to AOT on helix. In order to do that, this recreates the original inputs for the *wasm* part of the build. See C:\helix\work\workitem\e\publish\ProxyProjectForAOTOnHelix.proj, and C:\helix\work\workitem\e\publish\ProxyProjectForAOTOnHelix.props. **
C:\helix\work\correlation\build\wasm-shared\WasmApp.Common.targets(508,5): error : Specified Emscripten sdk at $(EMSDK_PATH)=C:\helix\work\correlation\build\\emsdk\ is missing some paths: $(EmscriptenPythonToolsPath)=C:\helix\work\correlation\build\emsdk\python\ . SDK is required for AOT'ing assemblies.

This was referenced Jun 26, 2024
Closed
Closed
Open
Open
Closed
Closed
@ilonatommy
Copy link
Member Author

/azp run runtime-wasm

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@build-analysis build-analysis bot mentioned this pull request Jul 4, 2024
Open
3 tasks
@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Tagging subscribers to this area: @directhex, @matouskozak
See info in area-owners.md if you want to be subscribed.

@dotnet-policy-service Dotnet Policy Service
Copy link
Contributor

Draft Pull Request was automatically closed for 30 days of inactivity. Please let us know if you'd like to reopen it.

@github-actions github-actions bot locked and limited conversation to collaborators Sep 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@akoeplinger akoeplinger Awaiting requested review from akoeplinger akoeplinger will be requested when the pull request is marked ready for review akoeplinger is a code owner

@maraf maraf Awaiting requested review from maraf maraf will be requested when the pull request is marked ready for review maraf is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

@ilonatommy ilonatommy

Labels
area-Infrastructure-mono NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[wasm] Handle multiple python directories in emsdk
2 participants
@ilonatommy @ViktorHofer