Skip to content

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
@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