Skip to content

Revert "Add support and tests for URLComponents.percentEncodedQueryItems" #2941

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

Conversation

shahmishal
Copy link
Member

Reverts #2940

We are seeing following failures on Linux bots:

The following tests FAILED:
1370 - TestFoundation.TestURLComponents-test_queryItems (ILLEGAL)
1376 - TestFoundation.TestURLComponents-test_createURLWithComponents (ILLEGAL)
1379 - TestFoundation.TestURLComponents-test_percentEncodedQueryItems (ILLEGAL)

https://ci.swift.org/job/oss-swift-package-linux-ubuntu-16_04/5723/console
https://ci.swift.org/job/oss-swift-package-amazon-linux-2/953/console
https://ci.swift.org/job/oss-swift-package-centos-7/403/console
https://ci.swift.org/job/oss-swift-package-linux-ubuntu-18_04/3533/console
https://ci.swift.org/job/oss-swift-package-ubuntu-20_04/718/console

@shahmishal shahmishal requested a review from rauhul December 13, 2020 23:37
@shahmishal
Copy link
Member Author

@swift-ci test

@rauhul
Copy link
Member

rauhul commented Dec 14, 2020

What in the world... is it possible to get a backtrace from these failures?

Copy link
Member

@rauhul rauhul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@spevans any new ideas on why this is failing 😓

@shahmishal
Copy link
Member Author

What in the world... is it possible to get a backtrace from these failures?

Do you know where the backtrace is saved? I can try to find it on the server.

@shahmishal
Copy link
Member Author

@swift-ci test Linux

@shahmishal
Copy link
Member Author

18:05:01 Failed Tests (3):
18:05:01   swift-package-tests :: swift-build.txt
18:05:01   swift-package-tests :: swift-package-with-spaces.txt
18:05:01   swift-package-tests :: test-foundation-package/test-foundation-xml.txt

These failure does not look related to the changes, may be the foundation test is.

18:04:55 ******************** TEST 'swift-package-tests :: test-foundation-package/test-foundation-xml.txt' FAILED ********************
18:04:55 Script:
18:04:55 --
18:04:55 : 'RUN: at line 2';   rm -rf /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp
18:04:55 : 'RUN: at line 3';   mkdir -p /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp
18:04:55 : 'RUN: at line 4';   /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swiftc  -o /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp/test-foundation-xml /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/swift-integration-tests/test-foundation-package/test-foundation-xml.swift
18:04:55 : 'RUN: at line 5';   /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp/test-foundation-xml | /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/llvm-linux-x86_64/bin/FileCheck /home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/swift-integration-tests/test-foundation-package/test-foundation-xml.txt
18:04:55 --
18:04:55 Exit Code: 1
18:04:55 
18:04:55 Command Output (stdout):
18:04:55 --
18:04:55 $ ":" "RUN: at line 2"
18:04:55 $ "rm" "-rf" "/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp"
18:04:55 $ ":" "RUN: at line 3"
18:04:55 $ "mkdir" "-p" "/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp"
18:04:55 $ ":" "RUN: at line 4"
18:04:55 $ "/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/usr/bin/swiftc" "-o" "/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/buildbot_linux/none-swift_package_sandbox_linux-x86_64/tests/test-foundation-package/Output/test-foundation-xml.txt.tmp/test-foundation-xml" "/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/swift-integration-tests/test-foundation-package/test-foundation-xml.swift"
18:04:55 # command stderr:
18:04:55 <unknown>:0: error: module file '/home/buildnode/.cache/clang/ModuleCache/2YQE78DQEDMHK/CoreFoundation-5XW88J8P5C88.pcm' is out of date and needs to be rebuilt: signature mismatch
18:04:55 <unknown>:0: note: imported by module 'CFXMLInterface' in '/home/buildnode/.cache/clang/ModuleCache/2YQE78DQEDMHK/CFXMLInterface-33KQ6TD0RL6QE.pcm'
18:04:55 
/home/buildnode/jenkins/workspace/swift-corelibs-foundation-PR-Linux/swift-integration-tests/test-foundation-package/test-foundation-xml.swift:2:8: error: missing required modules: 'CFXMLInterface', 'CoreFoundation'
18:04:55 import FoundationXML
18:04:55        ^
18:04:55 
18:04:55 error: command failed with exit status: 1
18:04:55 
18:04:55 --

@shahmishal
Copy link
Member Author

Going to merge this PR to unblock CI, and we can look into the swift integration test failures after.

@shahmishal shahmishal merged commit 0e18860 into main Dec 14, 2020
@finagolfin
Copy link
Member

I can confirm that those same three tests failed for me on Android.

@parkera parkera deleted the revert-2940-feature/URLComponents.percentEncodedQueryItems branch July 30, 2024 22:11
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