Neofetch on brew still requires Imagemagick and screenresolution

[yochem]

Neofetch on brew still requires imagemagick and screenresolution, even though it works fine without them (when running neofetch from source). The dependency on imagemagick was also removed in #956. So are these dependencies still required or are they optional? If you state that this dependency is not required I can make the PR on homebrew to remove the dependencies.

$ brew info neofetch
neofetch: stable 7.1.0 (bottled), HEAD
Fast, highly customisable system info script
https://github.com/dylanaraps/neofetch
/usr/local/Cellar/neofetch/7.1.0 (6 files, 351.5KB) *
  Poured from bottle on 2020-08-13 at 13:57:47
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/neofetch.rb
License: MIT
==> Dependencies
Required: imagemagick ✔, screenresolution ✘
==> Options
--HEAD
        Install HEAD version
==> Analytics
install: 4,986 (30 days), 15,486 (90 days), 76,797 (365 days)
install-on-request: 4,986 (30 days), 15,483 (90 days), 76,699 (365 days)
build-error: 0 (30 days)

By removing the imagemagick and screenresolution as dependency, the following packages don't have to be installed:

$ brew deps neofetch
docbook
docbook-xsl
freetype
gdbm
gettext
ghostscript
glib
gnu-getopt
ilmbase
imagemagick
jpeg
libde265
libffi
libheif
liblqr
libomp
libpng
libtiff
libtool
little-cms2
openexr
openjpeg
[email protected]
pcre
[email protected]
readline
screenresolution
shared-mime-info
sqlite
webp
x265
xmlto
xz

[kbd]

I recently got a conflict with ilmbase when brew upgraded ffmpeg. Turns out ilmbase is deprecated and the only reason it's installed is because of Neofetch:

~$ brew info ilmbase
ilmbase: stable 2.5.5 (bottled) [keg-only]
OpenEXR ILM Base libraries (high dynamic-range image file format)
https://www.openexr.com/
Deprecated because it is not supported upstream!
/usr/local/Cellar/ilmbase/2.5.5 (87 files, 1.8MB) *
  Poured from bottle on 2021-02-23 at 03:46:47
From: https://github.com/Homebrew/homebrew-core/blob/HEAD/Formula/ilmbase.rb
License: BSD-3-Clause
==> Dependencies
Build: cmake ✔
==> Caveats
`ilmbase` has been replaced by `imath`. You may want to `brew uninstall ilmbase`
or `brew unlink ilmbase` to prevent conflicts.

ilmbase is keg-only, which means it was not symlinked into /usr/local,
because ilmbase conflicts with `openexr` and `imath`.

~$ brew uninstall ilmbase          
Error: Refusing to uninstall /usr/local/Cellar/ilmbase/2.5.5
because it is required by neofetch, which is currently installed.
You can override this and force removal with:
  brew uninstall --ignore-dependencies ilmbase

It'd be nice if neofetch fixed this dependency.

[dylanaraps]

This is a homebrew issue. I don't maintain the neofetch packages in distributions.

[yochem]

I was just seeking the confirmation that neofetch does not require imagemagick and screenresolution

[dylanaraps]

Neofetch at a minimum requires only bash + coreutils. Features are used at runtime based on what is available. Take this up with the homebrew maintainer.