FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

CI: add -v to appveyor pytest incantation by tacaswell · Pull Request #9229 · matplotlib/matplotlib · GitHub

Repository navigation

CI: add -v to appveyor pytest incantation - #9229

Closed
tacaswell wants to merge 3 commits into
matplotlib:masterfrom
tacaswell:tst_appveyor_verbose
Closed

tacaswell wants to merge 3 commits into
matplotlib:masterfrom
tacaswell:tst_appveyor_verbose

Conversation

Copy link
Copy Markdown
Member

To check which test it is failing on

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

anntzer commented Sep 26, 2017

Copy link
Copy Markdown
Contributor

On a tangent, I just discovered you can remote-desktop into the appveyor VM during the build... https://www.appveyor.com/docs/how-to/rdp-to-build-worker/ handy.

Copy link
Copy Markdown
Member

xref #9176

QuLogic commented Sep 26, 2017

Copy link
Copy Markdown
Member

-1073740791 == c0000409 == STATUS_STACK_BUFFER_OVERRUN; I've not had much luck triggering any error in valgrind on Linux, though it's a bit hard to sort through. I added pytest-faulthandler to AppVeyor here, but I'm not sure if it'll be effective.

QuLogic commented Sep 27, 2017 •
edited
Loading

Copy link
Copy Markdown
Member

According to UBsan, there is a memcpy on NULL in FreeType. I am not sure if that is the cause of the problem here, but a long time ago Windows printf didn't like NULL strings so it might be the reason. (It's also triggered by a few other tests, but test_cull_markers is the first one.)

It is fixed in FreeType 2.6.3, but using that version causes 930 failures. Also, SF is not redirecting properly so we can't download it onto builders right now (it's working from cache, mostly). I'm not sure how we want to go about fixing it.

anntzer commented Sep 27, 2017

Copy link
Copy Markdown
Contributor

Change the download logic (setupext.py line 1181 etc.) so that on a hash mismatch the second url gets tried? Or simpler, just prefer downloading from savannah...
Although interestingly https://www.freetype.org/download.html is down too.

QuLogic mentioned this pull request Sep 27, 2017
2 tasks done

Copy link
Copy Markdown
Member Author

QuLogic commented Sep 27, 2017

Copy link
Copy Markdown
Member

There's a table here for that.

Copy link
Copy Markdown
Member Author

The two appevyor fails on this PR are in a different place than the rest (after all of the tests passed) or the freetype issue.

Copy link
Copy Markdown
Member Author

tacaswell mentioned this pull request Oct 5, 2017
dstansby added this to the 2.2 (next feature release) milestone Oct 8, 2017
Kojoley closed this Oct 14, 2017
Kojoley removed this from the 2.2 milestone Oct 14, 2017
tacaswell deleted the tst_appveyor_verbose branch October 15, 2017 03:10
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL