From d03b8bbad9f40024b994c03960758446c614f7a6 Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Thu, 19 Sep 2013 22:26:03 -0400 Subject: [PATCH] deploy: Do a full system sync after writing bootloader config This is just something I noticed on inspection; we should catch any changes to /boot in the sync(), even though theoretically gio should have done fdatasync(). --- src/libostree/ostree-sysroot-deploy.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/libostree/ostree-sysroot-deploy.c b/src/libostree/ostree-sysroot-deploy.c index 4639eadd..8a6fd4ea 100644 --- a/src/libostree/ostree-sysroot-deploy.c +++ b/src/libostree/ostree-sysroot-deploy.c @@ -1096,12 +1096,6 @@ ostree_sysroot_write_deployments (OstreeSysroot *self, goto out; } - if (!full_system_sync (cancellable, error)) - { - g_prefix_error (error, "Full sync: "); - goto out; - } - if (bootloader && !_ostree_bootloader_write_config (bootloader, new_bootversion, cancellable, error)) { @@ -1109,6 +1103,12 @@ ostree_sysroot_write_deployments (OstreeSysroot *self, goto out; } + if (!full_system_sync (cancellable, error)) + { + g_prefix_error (error, "Full sync: "); + goto out; + } + if (!swap_bootloader (self, self->bootversion, new_bootversion, cancellable, error)) {