inveniosoftware / invenio-cli

CLI module for Invenio
https://invenio-cli.readthedocs.io
MIT License
10 stars 43 forks source link

If `ImageMagick` isn't installed, `check-requirements` will report both failure and success #291

Closed max-moser closed 2 years ago

max-moser commented 2 years ago

Package version (if known): 1.0.4

Describe the bug

The output of invenio-cli check-requirements is confusing, because it mentions both success as well as failure.

Steps to Reproduce

  1. Install Ubuntu (e.g. on a VM)
  2. Install most basic dependencies (but not ImageMagick... which is listed as an "extended" dependency in the docs: https://inveniordm.docs.cern.ch/install/requirements/)
  3. Execute invenio-cli check-requirements
  4. See error

Expected behavior

When one of the dependencies isn't found, no success message should be displayed.

Also, should ImageMagick even be checked without --development flag?

Screenshots (if applicable)

image