diff options
author | Alexander Larsson <alexl@redhat.com> | 2018-02-15 10:36:06 +0100 |
---|---|---|
committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-02-15 13:02:02 +0000 |
commit | 195976a1b8307d57519f63827fc491edd9d0a99d (patch) | |
tree | b37fb42b7de63ba0109c34b94b57a4b755984c06 | |
parent | 4b754f38514c3f66350283c081d7cb29a820800b (diff) | |
download | flatpak-195976a1b8307d57519f63827fc491edd9d0a99d.tar.gz |
Fix persisted directories on atomic
On atomic /home is a symlink to /var/home, so when we bind-mount
the persistent directories we need to early-resolve the symlinks
to avoid running into issues with /newroot.
In most cases we do this already by calling flatpak_bwrap_add_bind_arg,
but the persistent dir case did not, because that function required
the target to exist, and the persistent directoried might not.
However, these days flatpak_bwrap_add_bind_arg is fine if the base
dir doesn't exists but the target does, which is the case here,
so we can use it now.
This fixes e.g. steam: https://github.com/flatpak/flatpak/issues/1278
Closes: #1422
Approved by: cgwalters
-rw-r--r-- | common/flatpak-context.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/common/flatpak-context.c b/common/flatpak-context.c index 1aa327d9..25d1dbb9 100644 --- a/common/flatpak-context.c +++ b/common/flatpak-context.c @@ -1918,10 +1918,7 @@ flatpak_context_append_bwrap_filesystem (FlatpakContext *context, g_mkdir_with_parents (src, 0755); - /* We stick to flatpak_bwrap_add_args instead of flatpak_bwrap_add_bind_arg because persisted - * folders don't need to exist outside the chroot. - */ - flatpak_bwrap_add_args (bwrap, "--bind", src, dest, NULL); + flatpak_bwrap_add_bind_arg (bwrap, "--bind", src, dest); } } |