Bug 1735337 - python-nbconvert fails to build with Python 3.8
Summary: python-nbconvert fails to build with Python 3.8
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: python-nbconvert
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
Assignee: Mukundan Ragavan
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: PYTHON38
TreeView+ depends on / blocked
 
Reported: 2019-07-31 19:35 UTC by Miro Hrončok
Modified: 2019-08-19 21:59 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-08-19 18:07:34 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Miro Hrončok 2019-07-31 19:35:12 UTC
python-nbconvert fails to build with Python 3.8.0b3.

=================================== FAILURES ===================================
______________________ TestNbConvertApp.test_allow_errors ______________________

self = <nbconvert.tests.test_nbconvertapp.TestNbConvertApp testMethod=test_allow_errors>

    def test_allow_errors(self):
        """
        Verify that conversion is aborted with '--execute' if an error is
        encountered, but that conversion continues if '--allow-errors' is
        used in addition.
        """
        with self.create_temp_cwd(['notebook3*.ipynb']):
            # Convert notebook containing a cell that raises an error,
            # both without and with cell execution enabled.
            output1, _ = self.nbconvert('--to markdown --stdout notebook3*.ipynb')  # no cell execution
            output2, _ = self.nbconvert('--to markdown --allow-errors --stdout notebook3*.ipynb')  # no cell execution; --allow-errors should have no effect
            output3, _ = self.nbconvert('--execute --allow-errors --to markdown --stdout notebook3*.ipynb')  # with cell execution; errors are allowed
    
            # Un-executed outputs should not contain either
            # of the two numbers computed in the notebook.
            assert '23' not in output1
            assert '42' not in output1
            assert '23' not in output2
            assert '42' not in output2
    
            # Executed output should contain both numbers.
            assert '23' in output3
>           assert '42' in output3
E           AssertionError: assert '42' in '\n# Notebook with errors\n\nThis notebook contains a cell which deliberately throws an exception. This is to test if ...n\n```python\nprint("The answer to the question about life, the universe and everything is: {}".format(43 - 1))\n```\n'

nbconvert/tests/test_nbconvertapp.py:382: AssertionError

For the build logs, see:
https://copr-be.cloud.fedoraproject.org/results/@python/python3.8/fedora-rawhide-x86_64/00989882-python-nbconvert/

For all our attempts to build python-nbconvert with Python 3.8, see:
https://copr.fedorainfracloud.org/coprs/g/python/python3.8/package/python-nbconvert/

Testing and mass rebuild of packages is happening in copr. You can follow these instructions to test locally in mock if your package builds with Python 3.8:
https://copr.fedorainfracloud.org/coprs/g/python/python3.8/

Let us know here if you have any questions.

Comment 1 Ben Cotton 2019-08-13 17:06:12 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 31 development cycle.
Changing version to '31'.

Comment 2 Ben Cotton 2019-08-13 18:51:37 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 31 development cycle.
Changing version to 31.

Comment 3 Miro Hrončok 2019-08-14 22:24:06 UTC
The coordinated rebuild of Python 3.8 has started in the `f32-python` side tag.

If you figure out how to rebuild this package, please don't rebuild it in regular rawhide, but use the side tag instead:

    on branch master:
    $ fedpkg build --target=f32-python

To wait for a build to show up in the side tag, do:

    $ koji wait-repo f32-python --build=<nvr>

Where <nvr> is name-version-release of the source package, e.g. python-foo-1.1-2.fc32.

An updated mock config is posted at:
http://copr.fedorainfracloud.org/coprs/g/python/python3.8/

Note that it will take a while before the essential packages are rebuilt, so don't expect all your dependencies to be available right away.

Thanks. Let us know if you need up to date info, or if you have any questions.



PS this message is mass posted to all the bugs that block the PYTHON38 bug. If this is also a Fedora 31 FTBFS bug and you manage to fix it, you can do a f31 build as usual:

    on branch f31:
    $ fedpkg build

Comment 4 Mukundan Ragavan 2019-08-16 21:32:00 UTC
Sorry for the lack of response.

At the moment, I do not know how to fix this at all. 

I was trying to update to 5.6.0 and get this - https://koji.fedoraproject.org/koji/taskinfo?taskID=37085705

Any help is appreciated.

Comment 5 Miro Hrončok 2019-08-16 21:55:34 UTC
I will try to investigate next week.

Comment 6 Miro Hrončok 2019-08-19 18:07:34 UTC
It actually built ¯\_(ツ)_/¯

Comment 7 Mukundan Ragavan 2019-08-19 21:59:38 UTC
(In reply to Miro Hrončok from comment #6)
> It actually built ¯\_(ツ)_/¯

Saw that! :)


Note You need to log in before you can comment on or make changes to this bug.