Skip to content

Conversation

fujii
Copy link
Contributor

@fujii fujii commented Nov 6, 2023

824e7a8

Rewrite Python CGI under http/tests/xmlhttprequest to use binary mode for stdout for Windows Python
https://bugs.webkit.org/show_bug.cgi?id=264238

Reviewed by Ross Kirsling.

Windows Python automatically converts '\n' to '\r\n' in text mode.
Use sys.stdout.buffer.write instead of sys.stdout.write.

* LayoutTests/http/tests/xmlhttprequest/resources/access-control-allow-lists.py:
* LayoutTests/http/tests/xmlhttprequest/resources/access-control-allow-with-body.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir1/basic-auth.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir2/basic-auth.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir2/catch.py:
* LayoutTests/http/tests/xmlhttprequest/resources/chunked-transfer.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-authorization.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-check-cookies.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-no-authorization.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-preflight-get.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-set-cookies.py:
* LayoutTests/http/tests/xmlhttprequest/resources/logout/resource.py:
* LayoutTests/http/tests/xmlhttprequest/resources/re-login/resource.py:
* LayoutTests/platform/wincairo/TestExpectations:

Canonical link: https://commits.webkit.org/270258@main

1247821

Misc iOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug 🧪 wpe-wk2
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🛠 gtk
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🧪 gtk-wk2
✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🧪 api-gtk
✅ 🛠 tv ✅ 🧪 mac-AS-debug-wk2
✅ 🛠 tv-sim
✅ 🛠 🧪 merge ✅ 🛠 watch
✅ 🛠 watch-sim

@fujii fujii self-assigned this Nov 6, 2023
@fujii fujii added the Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases label Nov 6, 2023
@fujii fujii requested review from donny-dont and rkirsling November 6, 2023 07:49
Copy link
Member

@rkirsling rkirsling left a comment

Choose a reason for hiding this comment

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

rs=me

@fujii fujii added the safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks label Nov 6, 2023
@webkit-ews-buildbot webkit-ews-buildbot added merge-queue Applied to send a pull request to merge-queue and removed safe-merge-queue Applied to automatically send a pull-request to merge-queue after passing EWS checks labels Nov 6, 2023
@webkit-ews-buildbot
Copy link
Collaborator

Safe-Merge-Queue: Build #2525.

… for stdout for Windows Python

https://bugs.webkit.org/show_bug.cgi?id=264238

Reviewed by Ross Kirsling.

Windows Python automatically converts '\n' to '\r\n' in text mode.
Use sys.stdout.buffer.write instead of sys.stdout.write.

* LayoutTests/http/tests/xmlhttprequest/resources/access-control-allow-lists.py:
* LayoutTests/http/tests/xmlhttprequest/resources/access-control-allow-with-body.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir1/basic-auth.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir2/basic-auth.py:
* LayoutTests/http/tests/xmlhttprequest/resources/basic-auth-default/dir2/catch.py:
* LayoutTests/http/tests/xmlhttprequest/resources/chunked-transfer.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-authorization.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-check-cookies.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-no-authorization.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-preflight-get.py:
* LayoutTests/http/tests/xmlhttprequest/resources/cross-origin-set-cookies.py:
* LayoutTests/http/tests/xmlhttprequest/resources/logout/resource.py:
* LayoutTests/http/tests/xmlhttprequest/resources/re-login/resource.py:
* LayoutTests/platform/wincairo/TestExpectations:

Canonical link: https://commits.webkit.org/270258@main
@webkit-commit-queue webkit-commit-queue force-pushed the eng/Rewrite-Python-CGI-under-http-tests-xmlhttprequest-for-Windows-Python branch from 1247821 to 824e7a8 Compare November 6, 2023 09:51
@webkit-commit-queue
Copy link
Collaborator

Committed 270258@main (824e7a8): https://commits.webkit.org/270258@main

Reviewed commits have been landed. Closing PR #20023 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 824e7a8 into WebKit:main Nov 6, 2023
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Nov 6, 2023
@fujii fujii deleted the eng/Rewrite-Python-CGI-under-http-tests-xmlhttprequest-for-Windows-Python branch November 6, 2023 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tools / Tests Tools in the Tools directory, build issues, test infrastructure, and bugs in test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants