On 29.12.2012 01:30, Zeeshan Ali (Khattak) wrote: > From: "Zeeshan Ali (Khattak)" <zeeshanak@xxxxxxxxx> > > For the new _identify_media() method to be able to easily replace > _db_guess_os_from_media() usage in apps as it intends to, it must > provide the ID of the media in the DB. > > This fixes a recent regression in Boxes caused by switching to > _identify_media(): > > https://bugzilla.gnome.org/show_bug.cgi?id=685826#c35 (and following > comments) > --- > osinfo/osinfo_db.c | 10 +++++++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/osinfo/osinfo_db.c b/osinfo/osinfo_db.c > index e3f6f45..adf2737 100644 > --- a/osinfo/osinfo_db.c > +++ b/osinfo/osinfo_db.c > @@ -624,6 +624,7 @@ static void fill_media (OsinfoDb *db, OsinfoMedia *media, > gboolean is_installer; > gboolean is_live; > gint reboots; > + const gchar *id; > const gchar *kernel_path; > const gchar *initrd_path; > const gchar *arch; > @@ -634,6 +635,9 @@ static void fill_media (OsinfoDb *db, OsinfoMedia *media, > osinfo_media_set_languages(media, languages); > g_list_free(languages); > > + id = osinfo_entity_get_id(OSINFO_ENTITY(matched_media)); > + g_object_set(G_OBJECT(media), "id", id, NULL); > + > arch = osinfo_media_get_architecture(matched_media); > if (arch != NULL) > g_object_set(G_OBJECT(media), "architecture", arch, NULL); > @@ -670,9 +674,9 @@ static void fill_media (OsinfoDb *db, OsinfoMedia *media, > * > * Try to match a newly created @media with a media description from @db. > * If found, @media will be filled with the corresponding information > - * stored in @db. In particular, after a call to > - * osinfo_db_identify_media(), if the media could be identified, its > - * OsinfoMedia::os property will be set. > + * stored in @db. In particular, after a call to osinfo_db_identify_media(), if > + * the media could be identified, its OsinfoEntify::id and OsinfoMedia::os > + * properties will be set. > * > * Returns: TRUE if @media was found in @db, FALSE otherwise > */ > ACK series Michal _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo