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

Cygwin docbuild fails #27514

Closed
vbraun opened this issue Mar 19, 2019 · 10 comments
Closed

Cygwin docbuild fails #27514

vbraun opened this issue Mar 19, 2019 · 10 comments

Comments

@vbraun
Copy link
Member

vbraun commented Mar 19, 2019

This is on the cygwin buildbot:

**********************************************************************
File "src/sage_setup/docbuild/__init__.py", line 105, in sage_setup.docbuild.builder_helper
Failed example:
    build_many(build_ref_doc, [("docname", "en", "html", {})])
Expected:
    Traceback (most recent call last):
    ...
    Exception: ('Non-exception during docbuild: abort pool operation', BaseException('abort pool operation',))
Got:
    Process Process-3733:1:
    Traceback (most recent call last):
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/multiprocessing/process.py", line 267, in _bootstrap
        self.run()
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/multiprocessing/process.py", line 114, in run
        self._target(*self._args, **self._kwargs)
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage_setup/docbuild/__init__.py", line 80, in build_ref_doc
        getattr(ReferenceSubBuilder(doc, lang), format)(*args, **kwds)
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage_setup/docbuild/__init__.py", line 763, in _wrapper
        getattr(DocBuilder, build_type)(self, *args, **kwds)
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage_setup/docbuild/__init__.py", line 145, in f
        raise Exception("Non-exception during docbuild: %s"%(e,), e)
    Exception: ('Non-exception during docbuild: abort pool operation', BaseException('abort pool operation',))
    Traceback (most recent call last):
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 671, in _run
        self.compile_and_execute(example, compiler, test.globs)
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage/doctest/forker.py", line 1095, in compile_and_execute
        exec(compiled, globs)
      File "<doctest sage_setup.docbuild.builder_helper[7]>", line 1, in <module>
        build_many(build_ref_doc, [("docname", "en", "html", {})])
      File "/home/Admin/sage-buildbot-worker/sage_git/build/local/lib/python2.7/site-packages/sage_setup/docbuild/utils.py", line 214, in _build_many
        raise worker_exc
    WorkerDiedException: worker for ('docname', 'en', 'html', {}) died with non-zero exit code 1
**********************************************************************
1 item had failures:
   1 of  10 in sage_setup.docbuild.builder_helper
    [36 tests, 1 failure, 2.33 s]
----------------------------------------------------------------------

CC: @embray @jdemeyer

Component: build

Author: Erik Bray

Branch/Commit: a7d28a1

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/27514

@vbraun vbraun added this to the sage-8.8 milestone Mar 19, 2019
@embray
Copy link
Contributor

embray commented Mar 20, 2019

comment:1

I'm not sure what to even do about this test since it's not applicable to the new build_many code. This test is a regression test for behavior specific to multiprocessing.Pool.map.

@embray
Copy link
Contributor

embray commented Apr 17, 2019

comment:2

Fixes the issue with a small bit of refactoring to ensure that the test in question is always actually testing the build_many implementation to which it applies.


New commits:

a7d28a1Ticket #27514: Rename the default implementation of build_many to

@embray
Copy link
Contributor

embray commented Apr 17, 2019

Commit: a7d28a1

@embray
Copy link
Contributor

embray commented Apr 17, 2019

Branch: u/embray/ticket-27514

@embray
Copy link
Contributor

embray commented Apr 17, 2019

Author: Erik Bray

@embray
Copy link
Contributor

embray commented Jun 12, 2019

comment:3

Ping?

@vbraun
Copy link
Member Author

vbraun commented Jun 12, 2019

Reviewer: Volker Braun

@embray
Copy link
Contributor

embray commented Jun 13, 2019

comment:5

Thank you!

@vbraun
Copy link
Member Author

vbraun commented Jun 27, 2019

Changed branch from u/embray/ticket-27514 to a7d28a1

@embray
Copy link
Contributor

embray commented Jul 3, 2019

comment:7

Not in Sage 8.8. Let's please to try keep tickets' milestones related to the release in which we actually intend to include them, and in particular the release in which they were actually included, especially when closing tickets.

@embray embray modified the milestones: sage-8.8, sage-8.9 Jul 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants