Project

General

Profile

Bug #1267

wrong dependencies for pcr/mat: mutagen is not optional

GNUtoo - over 5 years ago - . Updated over 5 years ago.

Status:
fixed
Priority:
bug
Assignee:
% Done:

100%


Description

Here's how it ends up without mutagen:

$ mat-gui
Traceback (most recent call last):
  File "/usr/bin/mat-gui", line 18, in <module>
    from libmat import mat
  File "/usr/lib/python2.7/site-packages/libmat/mat.py", line 22, in <module>
    import strippers  # this is loaded here because we need LOGGING_LEVEL
  File "/usr/lib/python2.7/site-packages/libmat/strippers.py", line 5, in <module>
    import mutagenstripper
  File "/usr/lib/python2.7/site-packages/libmat/mutagenstripper.py", line 6, in <module>
    from mutagen.flac import FLAC
ImportError: No module named mutagen.flac
$ mat
Traceback (most recent call last):
  File "/usr/bin/mat", line 10, in <module>
    from libmat import mat
  File "/usr/lib/python2.7/site-packages/libmat/mat.py", line 22, in <module>
    import strippers  # this is loaded here because we need LOGGING_LEVEL
  File "/usr/lib/python2.7/site-packages/libmat/strippers.py", line 5, in <module>
    import mutagenstripper
  File "/usr/lib/python2.7/site-packages/libmat/mutagenstripper.py", line 6, in <module>
    from mutagen.flac import FLAC
ImportError: No module named mutagen.flac

After installing mutagen it launches:

# pacman -S mutagen
[...]
$ mat
ERROR:root:Unable to find exiftool: no images support
usage: mat [-h] [-a] [-b] [-L] [-c] [-d] [-l] [-v] [files [files ...]]
$ mat-gui # no crash

Mutagen is an optional dependency, but given that it crashes without it, it should probably be a required dependency instead:

$ pacman -Q -i mat
[...]
Optional Deps   : perl-image-exiftool: extended image support, [installed]
                  mutagen: extended audio format support [installed]

History

#1

Updated by ovruni over 5 years ago

  • % Done changed from 0 to 100
  • Assignee set to ovruni
  • Status changed from open to fixed
  • Due date set to 2017-03-29
#2

Updated by Anonymous over 5 years ago

Perfecto! 100%

Also available in: Atom PDF