Skip to content

Relax FileCheck directives for skip_clean_llbuild.test #66062

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
May 24, 2023

Conversation

edymtt
Copy link
Contributor

@edymtt edymtt commented May 22, 2023

This is needed short term to allow the test to pass when running on Python 3.11 and later and avoid errors like

<stdin>:44:1: note: non-matching line after previous match is here
<string>:1: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
^

Long term we want to replace uses of the obsolete pipes.quotes function with shlex.quotes.

Addresses rdar://109664710

This is needed short term to allow the test to pass when running on
Python 3.11 and later and avoid errors like

```
<stdin>:44:1: note: non-matching line after previous match is here
<string>:1: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
^
```

Long term we want to replace uses of the obsolete `pipes.quotes`
function with `shlex.quotes`.

Addresses rdar://109664710
@edymtt edymtt requested a review from shahmishal May 22, 2023 19:06
@edymtt
Copy link
Contributor Author

edymtt commented May 22, 2023

@swift please test

@edymtt
Copy link
Contributor Author

edymtt commented May 22, 2023

@swift-ci please test Apple Silicon

@edymtt
Copy link
Contributor Author

edymtt commented May 22, 2023

@swift-ci please build toolchain

edymtt added a commit to edymtt/swift that referenced this pull request May 22, 2023
This is needed short term to allow the test to pass when running on
Python 3.11 and later and avoid errors like

```
<stdin>:44:1: note: non-matching line after previous match is here
<string>:1: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
^
```

Long term we want to replace uses of the obsolete `pipes.quotes`
function with `shlex.quotes`.

Addresses rdar://109664710

(cherry picked from commit 1534925,
 swiftlang#66062)
edymtt added a commit to edymtt/swift that referenced this pull request May 22, 2023
This is needed short term to allow the test to pass when running on
Python 3.11 and later and avoid errors like

```
<stdin>:44:1: note: non-matching line after previous match is here
<string>:1: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
^
```

Long term we want to replace uses of the obsolete `pipes.quotes`
function with `shlex.quotes`.

Addresses rdar://109664710

(cherry picked from commit 1534925,
 swiftlang#66062)
@edymtt
Copy link
Contributor Author

edymtt commented May 23, 2023

Apple Silicon test is failing for unrelated failures, but this test is now passing

PASS: Swift-validation(macosx-arm64) :: BuildSystem/skip_clean_llbuild.test (9209 of 16340)
...
Failed Tests (2):
  Swift(macosx-arm64) :: Concurrency/Runtime/async_taskgroup_discarding.swift
  Swift(macosx-arm64) :: Python/python_lint.swift

@edymtt
Copy link
Contributor Author

edymtt commented May 23, 2023

@swift-ci please test

@edymtt
Copy link
Contributor Author

edymtt commented May 23, 2023

@swift-ci please build toolchain

@edymtt
Copy link
Contributor Author

edymtt commented May 23, 2023

@swift-ci please test Apple Silicon

@edymtt
Copy link
Contributor Author

edymtt commented May 24, 2023

The macOS toolchain job is failing with an unrelated failure when configuring LLDB

-- LLDB.framework: resources subdirectory is 'Versions/A/Resources'
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE) 
CMake Error at test/CMakeLists.txt:153 (message):
  Couldn't find libcxx build in
  '/Users/ec2-user/jenkins/workspace/swift-PR-toolchain-macos/branch-main/buildbot_osx/libcxx-macosx-x86_64'.
  To run the test-suite for a standalone LLDB build please build libcxx and
  point LLDB_TEST_LIBCXX_ROOT_DIR to it.

@edymtt
Copy link
Contributor Author

edymtt commented May 24, 2023

The Apple Silicon bot is now hitting a failure when testing llbuild

...
PASS: Swift-validation(macosx-arm64) :: BuildSystem/skip_clean_llbuild.test (9208 of 16339)
...
Failed Tests (1):
  llbuild :: SwiftBuildTool/swift-compiler-whole-module-optimization.swift-build

@edymtt
Copy link
Contributor Author

edymtt commented May 24, 2023

Also verified the test still passes for the regular macOS PR testing

PASS: Swift-validation(macosx-x86_64) :: BuildSystem/skip_clean_llbuild.test (9275 of 16397)
PASS: Swift-validation(iphonesimulator-x86_64) :: BuildSystem/skip_clean_llbuil.test (9274 of 16397)
PASS: Swift-validation(watchsimulator-x86_64) :: BuildSystem/skip_clean_llbuild.test (9274 of 16397)

@edymtt edymtt merged commit 1aaaef9 into swiftlang:main May 24, 2023
NuriAmari pushed a commit to NuriAmari/swift that referenced this pull request May 28, 2023
…6062)

This is needed short term to allow the test to pass when running on
Python 3.11 and later and avoid errors like

```
<stdin>:44:1: note: non-matching line after previous match is here
<string>:1: DeprecationWarning: 'pipes' is deprecated and slated for removal in Python 3.13
^
```

Long term we want to replace uses of the obsolete `pipes.quotes`
function with `shlex.quotes`.

Addresses rdar://109664710
@edymtt edymtt deleted the edymtt/relax-skip_clean_llbuild.test branch May 31, 2023 14:37
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.

1 participant