-
Notifications
You must be signed in to change notification settings - Fork 37
linux: Detect unofficial Flatpaks #178
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
base: master
Are you sure you want to change the base?
Conversation
5cbc079 to
9c6a28b
Compare
521ddcf to
e44947d
Compare
|
Since obsproject/obs-studio#11788 landed we might rethink how this could be done the "proper" way. We can already grab "valid" commit hashes fairly easily: # same for `flathub-beta` of course
❯ flatpak remote-info flathub com.obsproject.Studio --log | grep Commit | cut -d: -f2
4249e4245f75bbec5ad89a4bcfb2726ecbe54c7f3a642e6dbe2820ff4ac2dbc9
caf0e8d12e2e072b8955a01815fac498746fd8da8de752f6d38a986fd5b21462
dee53614e7db9461d9e8247db81dfdfda4cdbbdad2d329a522c3946c19cc0386
bda2690638a8f078c50a9f525e5b8a996c9186e71af1d65ff33fa7396c8de5c3
a57498baa7462d859465d5ddedb1918b677f4b1306c7613da73c7c49e327dd19
dc38e67bbca5d6c9c3d005f84bbf006109310de3852a9cd32b1dc19e030b1b0c
68d5e09141770575d2c962c06dea2bde8c8f97f035a75f6a47b0c1dfd39a7749
bf9183bb1b4f291037a55df64402ed2dda0a8236907545c1a699ca1b205396fe
4c452dfbd40bad7156b04449b61eb0bb651456f0848cd7e04c3064aac1ab2e90
71d974e21fd96594d6ce66314962435a46674e1c441abcc9a6d64cbe5a5f7eda
9acb8be364db52dcb4ea8ff0b20d63579ecd18d22bd7419deb20a2245356ffd7
c5bc6eb99d2d638e0892320dd43d0cd5f948d6d90bf786abe2995087a068d131
e641d66f8f509ff18c1e82bfd6e32e0e0021e56cb01d1471b00f56d82ea8bfa1Obviously it doesn't really make sense to grab this on every log, but we could write this to a file that is updated either manually or through some CI, that is then read for Flatpak logs. |
e44947d to
aa110c6
Compare
aa110c6 to
7c0194f
Compare
7c0194f to
f8f2887
Compare
This is a quick and dirty detection that assumes that if a distribution were to package their own Flatpak, they would also use their own custom built runtime (we already have one such example). A "proper" detection would involve changing how and what OBS logs about the Flatpak environment and use that instead. Since that requires some additional considerations, that is out of scope for now.
f8f2887 to
b302e3e
Compare
|
Noticed a small issue with a log that was just posted in linux-support: Was still showing Fixed that up and because the Log-Level needed to be dynamic for this anyway, I also made it dynamic based on the results of the individual system info checks:
Also cleaned up some inconsistencies with inserted |
Description
This is a quick and dirty detection that assumes that if a distribution were to package their own Flatpak, they would also use their own custom built runtime (we already have one such example).
A "proper" detection would involve changing how and what OBS logs about the Flatpak environment and use that instead. Since that requires some additional considerations, that is out of scope for now. See obsproject/obs-studio#11788 for work towards that.
As discussed with tytan in
#documentationon Discord, we're going with this quick detection until such time that this "proper" detection is ready.Motivation and Context
We recently had a user in
#linux-supporthaving issues with both their BlackMagic capture card as well as their Nvidia GPU being detected. Upon inspection of the log, tytan noticed this was not our official Flatpak, and it unfortunately also happens to be broken in other ways.How Has This Been Tested?
The second string was tested with the same log, replacing the runtime identifier.
Types of changes
Checklist: