On 2/28/19 4:17 PM, Fabiano Fidêncio wrote: > Some new attributes have been added to OsinfoMedia, as: > - the install-script list (during this cycle); > - installer-script (1.3.0); > > However, none of those attributes have been filled in in the media > accordingly to matched media. > > Signed-off-by: Fabiano Fidêncio <[email protected]> > --- > osinfo/osinfo_db.c | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/osinfo/osinfo_db.c b/osinfo/osinfo_db.c > index 5696279..f59c804 100644 > --- a/osinfo/osinfo_db.c > +++ b/osinfo/osinfo_db.c > @@ -642,6 +642,9 @@ static void fill_media(OsinfoDb *db, OsinfoMedia *media, > const gchar *initrd_path; > const gchar *arch; > const gchar *url; > + gint i; > + gboolean installer_script; > + OsinfoInstallScriptList *install_script_list; > GList *variants, *node; > > set_languages_for_media(db, media, matched_media); > @@ -685,6 +688,23 @@ static void fill_media(OsinfoDb *db, OsinfoMedia *media, > "eject-after-install", eject_after_install, > NULL); > } > + installer_script = > osinfo_entity_get_param_value_boolean_with_default(OSINFO_ENTITY(matched_media), > + > OSINFO_MEDIA_PROP_INSTALLER_SCRIPT, > + > TRUE); > + g_object_set(G_OBJECT(media), > + "installer-script", installer_script, > + NULL); > + install_script_list = > osinfo_media_get_install_script_list(matched_media); > + if (install_script_list != NULL && > + osinfo_list_get_length(OSINFO_LIST(install_script_list)) > 0) { > + for (i = 0; i < > osinfo_list_get_length(OSINFO_LIST(install_script_list)); i++) { > + OsinfoInstallScript *script; > + > + script = > OSINFO_INSTALL_SCRIPT(osinfo_list_get_nth(OSINFO_LIST(install_script_list), > i)); > + osinfo_media_add_install_script(media, script); > + } > + } > + > if (os != NULL) > osinfo_media_set_os(media, os); > } >
I'm not really familiar with this code but AFAICT it looks okay Reviewed-by: Cole Robinson <[email protected]> - Cole _______________________________________________ Libosinfo mailing list [email protected] https://www.redhat.com/mailman/listinfo/libosinfo
