Skip to content

Revert "Merge pull request #42242 from eeckstein/escapeinfo" #42562

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

Merged
merged 1 commit into from
Apr 22, 2022

Conversation

compnerd
Copy link
Member

This reverts commit c05e064, reversing
changes made to c1534d5.

This caused a regression on Windows.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

This reverts commit c05e064, reversing
changes made to c1534d5.

This caused a regression on Windows.
@compnerd
Copy link
Member Author

CC: @eeckstein @atrick

@compnerd
Copy link
Member Author

@swift-ci please test

@shahmishal shahmishal merged commit b4fa721 into swiftlang:main Apr 22, 2022
@eeckstein
Copy link
Contributor

@compnerd what regression?

@eeckstein
Copy link
Contributor

Anything else than the test fail, which is fixed by #42555?

@eeckstein
Copy link
Contributor

eeckstein commented Apr 22, 2022

Never mind, I saw your comment in the other PR. Unfortunately the XFAIL for disabling the test is missing a ':'

@compnerd compnerd deleted the 42242 branch April 22, 2022 05:06
@compnerd
Copy link
Member Author

https://ci-external.swift.org/job/oss-swift-windows-toolchain-x86_64-vs2019/547/console

Why should the test be an XFAIL? If you are XFAILing it, do you have a timeline for repairing it?

@eeckstein
Copy link
Contributor

eeckstein commented Apr 22, 2022

The XFAIL was only the workaround from the build wrangler. The problem is a wrong check-pattern. I'll remove the XFAIL in my next attempt to land the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants