Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spike / Bug: Some update-desktop-database binaries might not have a --version switch. #2093

Open
Sewer56 opened this issue Sep 26, 2024 · 2 comments
Labels
os-linux This affects Linux related code.

Comments

@Sewer56
Copy link
Member

Sewer56 commented Sep 26, 2024

Bug Report

Summary

In some distros it may be possible that the version of update-desktop-database shipped does not have a --version flag in the CLI.

A User on Steam Deck is experiencing an error in log for update-desktop-database on every boot with the following contents:

00:00:02.257 [ERROR] (NexusMods.CrossPlatform.Process.RuntimeDependencyChecker) update-desktop-database: exception while querying information|CliWrap.Exceptions.CommandExecutionException: Command execution failed because the underlying process (update-desktop-database#40434) returned a non-zero exit code (1).

Command:
update-desktop-database --version

You can suppress this validation by calling `WithValidation(CommandResultValidation.None)` on the command.
   at CliWrap.Command.ExecuteAsync(ProcessEx process, CancellationToken forcefulCancellationToken, CancellationToken gracefulCancellationToken) in /_/CliWrap/Command.Execution.cs:line 271
   at CliWrap.Command.ExecuteAsync(ProcessEx process, CancellationToken forcefulCancellationToken, CancellationToken gracefulCancellationToken) in /_/CliWrap/Command.Execution.cs:line 287
   at CliWrap.Command.ExecuteAsync(ProcessEx process, CancellationToken forcefulCancellationToken, CancellationToken gracefulCancellationToken) in /_/CliWrap/Command.Execution.cs:line 287
   at CliWrap.Command.ExecuteAsync(ProcessEx process, CancellationToken forcefulCancellationToken, CancellationToken gracefulCancellationToken) in /_/CliWrap/Command.Execution.cs:line 287
   at NexusMods.CrossPlatform.Process.ProcessFactory.ExecuteAsync(Command command, CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/ProcessFactory.cs:line 118
   at NexusMods.CrossPlatform.Process.ProcessFactory.ExecuteAsync(Command command, Boolean logProcessOutput, CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/ProcessFactory.cs:line 98
   at NexusMods.CrossPlatform.Process.ProcessFactory.ExecuteAsync(Command command, Boolean logProcessOutput, CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/ProcessFactory.cs:line 101
   at NexusMods.CrossPlatform.Process.ProcessFactory.ExecuteAsync(Command command, Boolean logProcessOutput, CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/ProcessFactory.cs:line 101
   at NexusMods.CrossPlatform.Process.ExecutableRuntimeDependency.QueryInstallationInformationImpl(CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/ExecutableRuntimeDependency.cs:line 56
   at NexusMods.CrossPlatform.Process.RuntimeDependencyChecker.StartAsync(CancellationToken cancellationToken) in /_/src/NexusMods.CrossPlatform/Process/RuntimeDependencyChecker.cs:line 27

And their process stdout is:

Unknown option --version
Run "update-desktop-database --help" to see a full list of available command line options.

Other information

Chat from Discord.

This is relatively low priority; it's not breaking, but it pollutes logs. Users and devs may get wrong impressions if there are errors related to URL handlers.

@Sewer56 Sewer56 added the os-linux This affects Linux related code. label Sep 26, 2024
@erri120
Copy link
Member

erri120 commented Sep 26, 2024

Do you know what OS the user had installed? The man page of the tool should also show the version.

@Sewer56
Copy link
Member Author

Sewer56 commented Sep 26, 2024

Do you know what OS the user had installed?

SteamOS. As mentioned, they're running on Deck. The top of their logs was saying SteamOS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
os-linux This affects Linux related code.
Projects
Status: No status
Development

No branches or pull requests

2 participants