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
flatpak: 1.0.5 -> 1.1.2 #52754
flatpak: 1.0.5 -> 1.1.2 #52754
Conversation
@GrahamcOfBorg build flatpak |
This is a dev version but other distros package it as well https://repology.org/metapackage/flatpak/versions so perhaps it is fine. We should also update xdg-desktop-portal and xdg-desktop-portal-gtk. |
}; | ||
|
||
patches = [ | ||
./respect-path-env-var.patch | ||
./0001-test-doc-portal-Skip-all-tests-if-FUSE-isn-t-support.patch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a link to the source.
@@ -1,15 +1,15 @@ | |||
{ stdenv, fetchFromGitHub, autoreconfHook, pkgconfig, libxml2, xdg-desktop-portal, gtk3, glib }: | |||
|
|||
let | |||
version = "1.0.2"; | |||
version = "1.1.0"; | |||
in stdenv.mkDerivation rec { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The lockdown will require gsettings-desktop-schemas
.
For an example how to do it see https://github.com/NixOS/nixpkgs/pull/42177/files#diff-81007e7b56315f92406c1cf3d7ee1d47R16
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is tackling the lockdown aspect a blocker? I can look into a solution similar to what you've linked, but might take a bit (may need to start testing using a proper gnome session so I can access these things?).
On the other hand if the "fail" mode here isn't really "fail-safe" (such that user requests permission be denied but it's ignored since flatpak can't find the setting or something) that's no good and certainly should be addressed!
FWIW upstream has more changes re:lockdown (flatpak/xdg-desktop-portal#267), not sure if that makes integration easier or not :).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If schemas are not available, the program will crash.
0f46672
to
5099098
Compare
When running
That does not seem to happen when running the tests outside of VM using |
7d74bb0
to
ce51941
Compare
Do not add geoclue dependency yet: Complains it's not new enough, apparently wants an untagged version (2.5.1 is latest tag) checking for libgeoclue-2.0 >= 2.5.2... no configure: error: Package requirements (libgeoclue-2.0 >= 2.5.2) were not met: Requested 'libgeoclue-2.0 >= 2.5.2' but version of Geoclue Client Library is 2.5.1
Lockdown requires gsettings-desktop-schemas
ce51941
to
ac91818
Compare
I tested successfully installing flatpak app and running it. |
Yay!! Thanks for your work on this!
…On Thu, 03 Jan 2019 16:23:20 +0000 (UTC), Jan Tojnar ***@***.***> wrote:
Merged #52754 into master.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#52754 (comment) part: text/html
|
Motivation for this change
WIP-ish, at least it builds now :).
IIRC there are compelling reasons to upgrade
(UI and maybe sandbox improvements) but will
need to look through release notes again.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)