search

sonic2kk / steamtinkerlaunch

Linux wrapper tool for use with the Steam client for custom launch options and 3rd party programs
GNU General Public License v3.0
2.04k stars 70 forks source link

Check if steam is installed #1058

Closed Mte90 closed 3 months ago

Mte90 commented 3 months ago

So right now executing STL without steam you get this:

ven 15 mar 2024, 11:56:37, CET WARN - setSteamPaths - Steam 'userdata' directory not found, other variables depend on it - Expect problems
grep: : File o directory non esistente
grep: : File o directory non esistente
ven 15 mar 2024, 11:56:38, CET ERROR - checkIntDeps - '' was not found! Check 'steamtinkerlaunch --help' for alternatives and/or read 'https://github.com/sonic2kk/steamtinkerlaunch/wiki/Yad'
ven 15 mar 2024, 11:56:38, CET ERROR - checkIntDeps - Yad version '0' is too old. You need to update to at least '7.2'

So I think that the tool should check if the steam binary exists, also in my case I don't have yad but is still reporting a version.

sonic2kk commented 3 months ago

I noted this on the PR but just for continuity of anyone reading this, we will only do this if we're not running Steam Flatpak (INFLATPAK -eq 0).

also in my case I don't have yad but is still reporting a version.

It's giving 0 which is invalid, so this is fine imo. But also as noted on #1059, we could have improved logging for missing Yad.