Skip to content

Conversation

@DanisDGK
Copy link

Closes #2744

@jamesbt365
Copy link
Member

When this is merged the changes should probably be backported before the next 0.6.x release unless anything is stopping/not ideal with this pull.

@gonX
Copy link
Member

gonX commented Sep 30, 2023

This doesn't close the linked issue (which wants it in diagnostics), but it does log to the console which Distro is being used, which is useful.

Diagnostics excerpt:

{
  "OperatingSystem": {
    "Platform": 4,
    "ServicePack": "",
    "Version": "6.5.5.273",
    "VersionString": "Unix 6.5.5.273"
  }
}

Daemon output:

[RootHub:Debug]	Initialized internal child hubs: HidSharpDeviceRootHub
[Detect:Debug]	Configuration overrides exist: '/home/gonx/git/opentabletdriver/bin/Configurations'
[Platform:Info]	Linux distribution: arch
[Detect:Info]	Searching for tablets...
[Detect:Info]	No tablets were detected.

I personally think it's more interesting to have the diagnostics filled out correctly, but it is a good start.

@X9VoiD
Copy link
Member

X9VoiD commented Oct 17, 2023

Superseded by #2944

@X9VoiD X9VoiD closed this Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Linux distribution info to exported diagnostics

4 participants