Mybash deps check is bugged

[cartercanedy]

Describe the bug

mybash-setup.sh uses command_exists "bash" "bash-completion" ... as if it fails if one of the dependencies does not exist, but this won't work as written. Also, bash-completion and fontconfig aren't executables, so the check would always fail if it did work as intended...

To Reproduce

1. run the bash prompt script
2. install fails if deps aren't installed, most likely to happen on fresh distro installs

Screenshots

If applicable, add screenshots to help explain your problem.

Checklist

• I checked for duplicate issues.
• I checked already existing discussions.
• This issue is not included in the roadmap.
• This issue is present on both stable and development branches.

[cartercanedy]

/label 'script'

you're wrong, the ! means if even all are installed it will still install, same for if none are found or if only 1 / 2 are found

can not reproduce this issue as well. there is really no issue here

[cartercanedy]

It is an issue, it's reproducible using both openSUSE and Kali

[cartercanedy]

same for if none are found or if only 1 / 2 are found

command -v <arg1> <arg2>... will only evaluate <arg1> without examining the rest of the arguments, which is problematic for several reason, namely fc-cache from fontconfig, which is not installed on headless distros like kali-headless and others

same for if none are found or if only 1 / 2 are found

command -v <arg1> <arg2>... will only evaluate <arg1> without examining the rest of the arguments, which is problematic for several reason, namely fc-cache from fontconfig, which is not installed on headless distros like kali-headless and others

read what i said please

[cartercanedy]

@nnyyxxxx I don't understand why all of my contributions have been met with such fierce criticism on your part.

You're free to say that I'm wrong, but the fundamental issue still remains, regardless.

@nnyyxxxx I don't understand why all of my contributions have been met with such fierce criticism on your part.

You're free to say that I'm wrong, but the fundamental issue still remains, regardless.

i cant reproduce the issue you're having man... all that PR is doing is bloating up the func, if you want a clear-cut solution to your nonexistent problem then remove every other dep from the command_exists var

[cartercanedy]

@nnyyxxxx I don't understand why all of my contributions have been met with such fierce criticism on your part.

You're free to say that I'm wrong, but the fundamental issue still remains, regardless.

i cant reproduce the issue you're having man... all that PR is doing is bloating up the func, if you want a clear-cut solution to your nonexistent problem then remove every other dep from the command_exists var

Ok, close the issue then, if you're insisting that there's no issue

[cartercanedy]

These are the exact steps to reproduce:

1. Using WSL, install openSUSE Leap 15.6 viawsl --install -d openSUSE-Leap-15.6 --web-download and complete out-of-box setup wizard
2. Install cargo + git + gcc with zypper via sudo zypper install cargo gcc git
3. clone main branch of LinUtil via git clone --depth 1 https://github.com/christitustech/linutil.git
4. Run with cargo r
5. Attempt to run Bash prompt script, which fails when attempting to invoke fc-cache