diff options
author | Jasper St. Pierre <jstpierre@mecheye.net> | 2013-09-06 19:35:23 -0400 |
---|---|---|
committer | Jasper St. Pierre <jstpierre@mecheye.net> | 2013-09-08 11:50:51 -0400 |
commit | f49ed9e74d8cf7b6d900be9e40cf1daf5dde11d0 (patch) | |
tree | ce55da0159382813443e85a3aa044cabf5c44421 /src | |
parent | 8ac0f99ed6ac873ee80f04c1b72de73bbd32dba1 (diff) | |
download | ostree-f49ed9e74d8cf7b6d900be9e40cf1daf5dde11d0.tar.gz |
builtin-commit: Don't parse the parent's GVariant by hand
Instead, use OstreeRepoFile as a handle for the parent commit.
We need to add an accessor for the metadata checksum, as that
hasn't been exposed before.
https://bugzilla.gnome.org/show_bug.cgi?id=707727
Diffstat (limited to 'src')
-rw-r--r-- | src/libostree/ostree-repo-file.c | 6 | ||||
-rw-r--r-- | src/libostree/ostree-repo-file.h | 1 | ||||
-rw-r--r-- | src/ostree/ot-builtin-commit.c | 29 |
3 files changed, 16 insertions, 20 deletions
diff --git a/src/libostree/ostree-repo-file.c b/src/libostree/ostree-repo-file.c index aff60371..994d554c 100644 --- a/src/libostree/ostree-repo-file.c +++ b/src/libostree/ostree-repo-file.c @@ -362,6 +362,12 @@ ostree_repo_file_tree_get_contents_checksum (OstreeRepoFile *self) return self->tree_contents_checksum; } +const char * +ostree_repo_file_tree_get_metadata_checksum (OstreeRepoFile *self) +{ + return self->tree_metadata_checksum; +} + /** * ostree_repo_file_nontree_get_local: * diff --git a/src/libostree/ostree-repo-file.h b/src/libostree/ostree-repo-file.h index 4243b80b..f554be1f 100644 --- a/src/libostree/ostree-repo-file.h +++ b/src/libostree/ostree-repo-file.h @@ -66,6 +66,7 @@ void ostree_repo_file_tree_set_metadata (OstreeRepoFile *self, GVariant *metadata); const char *ostree_repo_file_tree_get_contents_checksum (OstreeRepoFile *self); +const char *ostree_repo_file_tree_get_metadata_checksum (OstreeRepoFile *self); gboolean ostree_repo_file_is_tree (OstreeRepoFile *self); diff --git a/src/ostree/ot-builtin-commit.c b/src/ostree/ot-builtin-commit.c index 20ab457d..329cd532 100644 --- a/src/ostree/ot-builtin-commit.c +++ b/src/ostree/ot-builtin-commit.c @@ -227,15 +227,10 @@ ostree_builtin_commit (int argc, char **argv, OstreeRepo *repo, GCancellable *ca gs_unref_object GFile *arg = NULL; gs_free char *parent = NULL; gs_free char *commit_checksum = NULL; - gs_unref_variant GVariant *parent_commit = NULL; gs_free char *contents_checksum = NULL; gs_unref_object OstreeMutableTree *mtree = NULL; gs_free char *tree_type = NULL; gs_unref_hashtable GHashTable *mode_adds = NULL; - gs_unref_variant GVariant *parent_content_csum_v = NULL; - gs_unref_variant GVariant *parent_metadata_csum_v = NULL; - gs_free char *parent_content_checksum = NULL; - gs_free char *parent_metadata_checksum = NULL; OstreeRepoCommitModifier *modifier = NULL; OstreeRepoTransactionStats stats; @@ -270,13 +265,6 @@ ostree_builtin_commit (int argc, char **argv, OstreeRepo *repo, GCancellable *ca if (!ostree_repo_resolve_rev (repo, opt_branch, TRUE, &parent, error)) goto out; - if (opt_skip_if_unchanged && parent) - { - if (!ostree_repo_load_variant (repo, OSTREE_OBJECT_TYPE_COMMIT, - parent, &parent_commit, error)) - goto out; - } - if (!opt_subject && !opt_body) { if (!commit_editor (repo, opt_branch, &opt_subject, &opt_body, cancellable, error)) @@ -390,17 +378,18 @@ ostree_builtin_commit (int argc, char **argv, OstreeRepo *repo, GCancellable *ca if (!ostree_repo_write_mtree (repo, mtree, &contents_checksum, cancellable, error)) goto out; - if (opt_skip_if_unchanged && parent_commit) + if (opt_skip_if_unchanged && parent) { - g_variant_get_child (parent_commit, 6, "@ay", &parent_content_csum_v); - g_variant_get_child (parent_commit, 7, "@ay", &parent_metadata_csum_v); + const char *metadata_checksum; + gs_unref_object OstreeRepoFile *parent_root; + + if (!ostree_repo_read_commit (repo, parent, (GFile **) &parent_root, cancellable, error)) + goto out; - parent_content_checksum = ostree_checksum_from_bytes_v (parent_content_csum_v); - parent_metadata_checksum = ostree_checksum_from_bytes_v (parent_metadata_csum_v); + metadata_checksum = ostree_mutable_tree_get_metadata_checksum (mtree); - if (strcmp (contents_checksum, parent_content_checksum) == 0 - && strcmp (ostree_mutable_tree_get_metadata_checksum (mtree), - parent_metadata_checksum) == 0) + if (strcmp (contents_checksum, ostree_repo_file_tree_get_contents_checksum (parent_root)) == 0 && + strcmp (metadata_checksum, ostree_repo_file_tree_get_metadata_checksum (parent_root)) == 0) skip_commit = TRUE; } |