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

Upgrade mozlog package from 3.0 to 3.3. #12510

Merged
merged 1 commit into from Jul 20, 2016
Merged

Conversation

@chenpighead
Copy link
Contributor

chenpighead commented Jul 19, 2016

Update mozlog to 3.3 so we can have correct reftest screenshots displayed with --log-html.
The original mozlog update bugs are listed as follows:
https://bugzilla.mozilla.org/show_bug.cgi?id=1287019
https://bugzilla.mozilla.org/show_bug.cgi?id=1287480


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #12401 and #12403 (github issue number if applicable).
  • There are tests for these changes OR
  • These changes do not require tests because this is just a PyPI package version bumping

Update mozlog to 3.3.


This change is Reviewable

With this upgrade, we are able to run wpt test with "--log-html" to see visual
results on screen.
@highfive
Copy link

highfive commented Jul 19, 2016

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @metajack (or someone else) soon.

@highfive
Copy link

highfive commented Jul 19, 2016

Heads up! This PR modifies the following files:

@jgraham
Copy link
Contributor

jgraham commented Jul 19, 2016

The wptrunner part of this needs to be fixed upstream (i.e. in https://github.com/w3c/wptrunner)

@chenpighead
Copy link
Contributor Author

chenpighead commented Jul 19, 2016

@jgraham PR: w3c/wptrunner#198
I suppose this is what you mean. It's my first time contributing to both servo and wptrunner, so please correct me if I've done something wrong.

@metajack
Copy link
Contributor

metajack commented Jul 19, 2016

r=me assuming this doesn't break our wpt upgrade syncing stuff. @Ms2ger, @jgraham?

@jgraham
Copy link
Contributor

jgraham commented Jul 19, 2016

This also landed upstream now so it should be fine. It would be better if the whole of wptrunner was updated ofc (I don't know if there are any other relevant changes), but it's not essential.

@metajack
Copy link
Contributor

metajack commented Jul 19, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Jul 19, 2016

📌 Commit aec2e4f has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Jul 20, 2016

Testing commit aec2e4f with merge 73cfab0...

bors-servo added a commit that referenced this pull request Jul 20, 2016
Upgrade mozlog package from 3.0 to 3.3.

<!-- Please describe your changes on the following line: -->
Update mozlog to 3.3 so we can have correct reftest screenshots displayed with --log-html.
The original mozlog update bugs are listed as follows:
https://bugzilla.mozilla.org/show_bug.cgi?id=1287019
https://bugzilla.mozilla.org/show_bug.cgi?id=1287480

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #12401 and #12403 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because this is just a PyPI package version bumping

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->

Update mozlog to 3.3.

<!-- 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/12510)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jul 20, 2016

@bors-servo bors-servo merged commit aec2e4f into servo:master Jul 20, 2016
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@shinglyu shinglyu mentioned this pull request Jul 21, 2016
4 of 4 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants
You can’t perform that action at this time.