Re: [PATCHv4 08/11] Use OS-specific config in OsinfoInstallScript

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Dec 18, 2012 at 03:35:55AM +0200, Zeeshan Ali (Khattak) wrote:
> On Mon, Dec 17, 2012 at 11:07 PM, Christophe Fergeau
> <cfergeau@xxxxxxxxxx> wrote:
> > When generating the unattended installation script, we can now
> > use osinfo_install_config_get_param_list() to get OS-specific
> > values when available. This will only work when an
> > OsinfoInstallConfigParamList is associated with the
> > OsinfoInstallConfig being processed.
> > ---
> >  osinfo/osinfo_install_script.c | 15 +++++++++++++--
> >  1 file changed, 13 insertions(+), 2 deletions(-)
> >
> > diff --git a/osinfo/osinfo_install_script.c b/osinfo/osinfo_install_script.c
> > index d72c394..8dbef9e 100644
> > --- a/osinfo/osinfo_install_script.c
> > +++ b/osinfo/osinfo_install_script.c
> > @@ -30,6 +30,7 @@
> >  #include <libxslt/xsltutils.h>
> >  #include <libxslt/xsltInternals.h>
> >  #include <glib/gi18n-lib.h>
> > +#include "osinfo_install_config_private.h"
> >  #include "osinfo_install_script_private.h"
> >
> >  G_DEFINE_TYPE (OsinfoInstallScript, osinfo_install_script, OSINFO_TYPE_ENTITY);
> > @@ -604,6 +605,7 @@ static xsltStylesheetPtr osinfo_install_script_load_template(const gchar *uri,
> >
> >  static xmlNodePtr osinfo_install_script_generate_entity_config(OsinfoInstallConfig *config,
> >                                                                 OsinfoEntity *entity,
> > +                                                               gboolean is_install_config,
> >                                                                 const gchar *name,
> >                                                                 GError **error)
> 
> We don't need a boolean parameter for finding out the type of entity.
> :) There is OSINFO_IS_INSTALL_CONFIG() or you can check if
> name=="config".

Yup, I've thought of both, we can also do if (config == entity) {}, but I
preferred to go the explicit way since the caller knows what it wants us to
do.

Christophe

Attachment: pgpMlUDh77C4M.pgp
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux