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

[release/8.0-staging] [aot] mono_aot_split_options: reset state after a backslash #108213

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 24, 2024

Backport of #108067 to release/8.0-staging

/cc @lambdageek

Customer Impact

  • Customer reported
  • Found internally

Customers targeting WebAssembly, Android or iOS with projects in a directory containing a " may run into crashes or errors during AOT compilation.

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

Manual testing

Risk

Low

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

  • If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.

@lambdageek lambdageek added the Servicing-consider Issue for next servicing release review label Sep 24, 2024
@lambdageek lambdageek added this to the 8.0.x milestone Sep 24, 2024
@lambdageek lambdageek requested review from steveisok and removed request for SamMonoRT September 24, 2024 19:09
@lambdageek lambdageek self-assigned this Sep 24, 2024
@jeffschwMSFT jeffschwMSFT removed the Servicing-consider Issue for next servicing release review label Sep 26, 2024
@lambdageek
Copy link
Member

@jeffschwMSFT usually you leave a comment when you twiddle the labels. Did we decide this is not going to meet the bar for 8.0 or did you forget to add the "servicing-approved" label?

@jeffschwMSFT
Copy link
Member

we discussed in teams. we will take this for 9 GA. without a customer it does not directly reach the bar

@lambdageek lambdageek closed this Sep 26, 2024
@jkotas jkotas deleted the backport/pr-108067-to-release/8.0-staging branch October 10, 2024 07:59
@github-actions github-actions bot locked and limited conversation to collaborators Nov 9, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants