Skip to content
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

Force state pickling to use a specific protocol #15611

Merged
merged 1 commit into from
Mar 1, 2019

Conversation

jdm
Copy link
Contributor

@jdm jdm commented Mar 1, 2019

Per https://stackoverflow.com/a/2204702/3830 this avoids an exception that occurs when pickling test manifest items and breaks the nightly Servo sync.

@wpt-pr-bot wpt-pr-bot added infra wptrunner The automated test runner, commonly called through ./wpt run labels Mar 1, 2019
@jgraham jgraham merged commit 107b55b into web-platform-tests:master Mar 1, 2019
bors-servo pushed a commit to servo/servo that referenced this pull request Mar 3, 2019
Force a pickling protocol that will accept certain awkward classes.

This cherry-picks the important part of web-platform-tests/wpt#15611 to unbreak the nightly WPT sync.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/22961)
<!-- Reviewable:end -->
bors-servo pushed a commit to servo/servo that referenced this pull request Mar 3, 2019
Force a pickling protocol that will accept certain awkward classes.

This cherry-picks the important part of web-platform-tests/wpt#15611 to unbreak the nightly WPT sync.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/22961)
<!-- Reviewable:end -->
marcoscaceres pushed a commit that referenced this pull request Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra wptrunner The automated test runner, commonly called through ./wpt run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants