keremet/rpm-ostree
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

upgrade: Print changed packages again

Browse Source 
First I noticed we needed to query the new default *after* the
transaction had run.  Then, I noticed the obvious embarassing logic
error.

Fixes: 3f1e7c85a18182669899f4065d23fec6ac0962c
Closes: #396

Closes: #397
Approved by: jlebon
This commit is contained in:
Colin Walters 2016-07-17 16:43:20 -04:00 committed by Atomic Bot
parent 64c6f8d906
commit 454139d41b
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/app/rpmostree-builtin-upgrade.c
View File

@ -129,14 +129,14 @@ rpmostree_builtin_upgrade (int argc,
goto out; goto out;
} }
new_default_deployment = rpmostree_os_dup_default_deployment (os_proxy);
if (!rpmostree_transaction_get_response_sync (sysroot_proxy, if (!rpmostree_transaction_get_response_sync (sysroot_proxy,
transaction_address, transaction_address,
cancellable, cancellable,
error)) error))
goto out; goto out;
new_default_deployment = rpmostree_os_dup_default_deployment (os_proxy);
if (opt_preview || opt_check) if (opt_preview || opt_check)
{ {
g_autoptr(GVariant) result = NULL; g_autoptr(GVariant) result = NULL;
@ -169,7 +169,7 @@ rpmostree_builtin_upgrade (int argc,
if (previous_default_deployment == new_default_deployment) if (previous_default_deployment == new_default_deployment)
changed = FALSE; changed = FALSE;
else else
changed = g_variant_equal (previous_default_deployment, new_default_deployment); changed = !g_variant_equal (previous_default_deployment, new_default_deployment);
if (!changed) if (!changed)
{ {