If we can get no resolutions (or no current resolution) we fall back on
1280x720 so we have something.
if this happens, write the entire xrandr output to the (debug) log.
This is a bit crude, and re-runs xrandr to log the output. But that is only in the failure case.
Using .seconds can't return more than a days worth of seconds, so
downloads stop working if you don't get them after a day of inactivity.
So, total_seconds() for the win
This way if a shortcuts.vdf file cannot be read or interpreted, we hide
relevant commands (but non-Steam commands should remain).
Added a comment to all_shortcuts_set() explaining what I think it is meant to
do. If I'm right, it's still slightly buggy.
I provide no error handling in has_steamtype_runner() or vdf_file_exists()
as I think these are not as likely to have errors (though anything can
raise exceptions of course).
This apparently harmless method pops a lot of dialogs, but some are
warnings.
This removes the runner-not-installed error dialog, which appears after a
warning to the same effect. The user just said no to this, he knows about it.
This makes the launch method also signal game-error, without which these
exceptions are not reported at all.
Also, improves the game-error handling to handle an entire exception,
and logger.exception().
- Relax condition for finding an amd vulkan loader to the presence
of the "amd" keyword
- If "pro" is also contained in the same loader file, then recognise
as proprietary amdgpu-pro driver
- Otherwise cannot separate between them and offer the old naming scheme
Signed-off-by: Adel KARA SLIMANE <adel.ks@zegrapher.com>
I think that sometimes the appname is listed as 'appname', in
lower-case.
This checks for that, and also avoids the crash if there is no app name
anyway.
I don't understand what this code was ever meant to do, but the list of
paths is not a dict and does not support lookups like this.
It seems to me that listing all the steam paths is probably reasonable.
The minimum version in practice used to be 0.50.0, however that can be
lowered by removing unneeded kwargs from a couple of functions.
- i18n.gettext() `install_dir` defaults to the localedir option, and
doesn't need to be manually overridden to that location
- configure_file() `install` defaults to true if `install_dir` is set,
and before 0.50 that was also the *only* way to mark it as installed
Removing both these kwargs means that there is more leeway with the minimum
required version.
