Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1692172 - python-markdown FTBFS with PyYAML 5
Summary: python-markdown FTBFS with PyYAML 5
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-markdown
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Thomas Moschny
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: PYTHON38
TreeView+ depends on / blocked
 
Reported: 2019-03-24 20:39 UTC by Miro Hrončok
Modified: 2020-02-08 13:53 UTC (History)
2 users (show)

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


Attachments (Terms of Use)
Rebased upstream patch (2.36 KB, patch)
2019-03-25 06:59 UTC, Miro Hrončok
no flags Details | Diff

Description Miro Hrončok 2019-03-24 20:39:42 UTC
There is a consistent %check failure in python-markdown:

======================================================================
ERROR: Failure: YAMLLoadWarning (calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/nose/loader.py", line 251, in generate
    for test in g():
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/__init__.py", line 141, in TestSyntax
    config = get_config(dir_name)
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/__init__.py", line 75, in get_config
    config = YamlConfig(defaults, os.path.join(dir_name, 'test.cfg'))
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/__init__.py", line 39, in __init__
    self._config = yaml.load(f)
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 109, in load
    load_warning('load')
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 55, in load_warning
    warnings.warn(message, YAMLLoadWarning, stacklevel=3)
YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.

======================================================================
ERROR: testExtensionConfigOption (tests.test_apis.TestCliOptionParsing)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/test_apis.py", line 717, in testExtensionConfigOption
    options, logging_level = parse_options(['-c', self.tempfile])
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/markdown/__main__.py", line 88, in parse_options
    extension_configs = yaml.load(fp)
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 109, in load
    load_warning('load')
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 55, in load_warning
    warnings.warn(message, YAMLLoadWarning, stacklevel=3)
YAMLLoadWarning: Failed parsing extension config file: /tmp/tmpZpOzSE.yml

======================================================================
ERROR: testExtensonConfigOptionAsJSON (tests.test_apis.TestCliOptionParsing)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/test_apis.py", line 747, in testExtensonConfigOptionAsJSON
    options, logging_level = parse_options(['-c', self.tempfile])
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/markdown/__main__.py", line 88, in parse_options
    extension_configs = yaml.load(fp)
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 109, in load
    load_warning('load')
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 55, in load_warning
    warnings.warn(message, YAMLLoadWarning, stacklevel=3)
YAMLLoadWarning: Failed parsing extension config file: /tmp/tmpUaj1Qq.yml

======================================================================
ERROR: testExtensonConfigOptionBadFormat (tests.test_apis.TestCliOptionParsing)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/tests/test_apis.py", line 760, in testExtensonConfigOptionBadFormat
    self.assertRaises(yaml.YAMLError, parse_options, ['-c', self.tempfile])
  File "/usr/lib64/python2.7/unittest/case.py", line 511, in assertRaises
    callableObj(*args, **kwargs)
  File "/builddir/build/BUILD/Markdown-2.6.11/python2/markdown/__main__.py", line 88, in parse_options
    extension_configs = yaml.load(fp)
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 109, in load
    load_warning('load')
  File "/usr/lib64/python2.7/site-packages/yaml/__init__.py", line 55, in load_warning
    warnings.warn(message, YAMLLoadWarning, stacklevel=3)
YAMLLoadWarning: Failed parsing extension config file: /tmp/tmpoBfUjw.yml

----------------------------------------------------------------------
Ran 135 tests in 0.275s


This is because updated PyYAML now does those new warnings.
I think the test suite should be adapted to not treat them as errors (as a workaround before markdown adapts to the new API).

This FTBFS is blocking the bootstrap of Python 3.8.

Comment 1 Miro Hrončok 2019-03-24 20:40:00 UTC
See also: https://apps.fedoraproject.org/koschei/build/6172904

Comment 3 Miro Hrončok 2019-03-25 06:59:58 UTC
Created attachment 1547602 [details]
Rebased upstream patch

Comment 4 Thomas Moschny 2019-03-25 08:34:25 UTC
Updated to 3.0.1 and cherry-picked upstream commit. Thanks for the heads-up.

Comment 5 Miro Hrončok 2020-02-08 13:53:28 UTC
Thank you.


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