whipper-team / whipper

Python CD-DA ripper preferring accuracy over speed
GNU General Public License v3.0
1.11k stars 88 forks source link

yaml.dump has been removed #606

Closed the-confessor closed 2 months ago

the-confessor commented 6 months ago

Lately result/logger.py's has been failing creating the rip log. Looks like there has been a breaking API change in ruamel.yaml.

 File "/usr/lib/python3.11/site-packages/whipper/result/logger.py", line 152, in logRip
    riplog = yaml.dump(
             ^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/ruamel/yaml/main.py", line 1251, in dump
    error_deprecation('dump', 'dump', arg="typ='unsafe', pure=True")
  File "/usr/lib/python3.11/site-packages/ruamel/yaml/main.py", line 1039, in error_deprecation
    raise AttributeError(s, name=None)
AttributeError: 
"dump()" has been removed, use

  yaml = YAML(typ='unsafe', pure=True)
  yaml.dump(...)

instead of file "/usr/lib/python3.11/site-packages/whipper/result/logger.py", line 152
MerlijnWajer commented 2 months ago

See https://github.com/whipper-team/whipper/issues/605