Skip to content

Use POSIX uname function instead of QSysInfo for BSD Unix#159

Open
Andreyogld3d wants to merge 1 commit intoSaschaWillems:masterfrom
Andreyogld3d:patch-1
Open

Use POSIX uname function instead of QSysInfo for BSD Unix#159
Andreyogld3d wants to merge 1 commit intoSaschaWillems:masterfrom
Andreyogld3d:patch-1

Conversation

@Andreyogld3d
Copy link
Copy Markdown

For GhostBSD we have OS "unknown" for Report.

https://vulkan.gpuinfo.org/displayreport.php?id=11527

We should use POSIX uname function instead of QSysInfo for BSD Unix

After fixing we have correct result:
https://vulkan.gpuinfo.org/displayreport.php?id=11577

Also, see #124

For GhostBSD we have OS "unknown" for Report.

https://vulkan.gpuinfo.org/displayreport.php?id=11527

We should use POSIX uname function instead of QSysInfo for BSD Unix

After fixing we have correct result:
https://vulkan.gpuinfo.org/displayreport.php?id=11577

Also, see SaschaWillems#124
@Andreyogld3d
Copy link
Copy Markdown
Author

@SaschaWillems could you appove this request ?

@SaschaWillems
Copy link
Copy Markdown
Owner

Can you provide two reports for the same driver/gpu combo? One before and one after the change?

Merging this might cause the database logic to flag existing reports as not yet present in the database, so I need to make sure that this won't have a negative effect.

@Andreyogld3d
Copy link
Copy Markdown
Author

Can you provide two reports for the same driver/gpu combo? One before and one after the change?

Hi, I have no BSD Unix right now, may be AMD RX 7600 doesn't support yet. Later, I will provide you report

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.

2 participants