Closed Simon-Laux closed 9 months ago
Started test build 102538
Started test build 102540
Build 102538 failed
Build 102540 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/85320/chat.delta.desktop.flatpakref
Started test build 104132
Started test build 104133
Build 104132 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/86922/chat.delta.desktop.flatpakref
Build 104133 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/86923/chat.delta.desktop.flatpakref
Started test build 105052
Build 105052 failed
Started test build 105069
Build 105069 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/87874/chat.delta.desktop.flatpakref
Started test build 105085
Build 105085 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/87890/chat.delta.desktop.flatpakref
I mildly object the removal of DRI access, because device=all is an accident rather than a deliberate choice.
I wonder how self-explanatory ELECTRON_OZONE_PLATFORM_HINT is. Neither the code nor the commit reveals why it's necessary so it could be difficult to assess when it's appropriate to remove it in the future.
But I don't block on those for merging, so feel free to merge.
I mildly object the removal of DRI access, because device=all is an accident rather than a deliberate choice.
The flathub validator bot rejects other device rules coexisting with device=all, so the alternative would be removing device=all and breaking qr code scanning in device linking.
I wonder how self-explanatory ELECTRON_OZONE_PLATFORM_HINT is. Neither the code nor the commit reveals why it's necessary so it could be difficult to assess when it's appropriate to remove it in the future.
I guess a comment linking to electron environment variables documentation and saying the app won't launch under wayland otherwise would be trivial to add.
Started test build 105192
Started test build 105193
I mildly object the removal of DRI access, because device=all is an accident rather than a deliberate choice
It is unfortunate that there seems not to be a more specific way to access webcams, like a device=webcams/cameras
or a portal that is so mature that it was added to Electron.
Build 105192 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/87998/chat.delta.desktop.flatpakref
Build 105193 failed
Started test build 105196
Build 105196 was cancelled
Started test build 105197
Build 105197 successful To test this build, install it from the testing repository:
flatpak install --user https://dl.flathub.org/build-repo/88003/chat.delta.desktop.flatpakref
Observations:
124 works (it is already merged into this pr), but has blurriness issues with systemwide display scaling in gnome.
Update: I solved both observations/issues
TODO
closes https://github.com/flathub/chat.delta.desktop/issues/118 closes https://github.com/deltachat/deltachat-desktop/issues/2800