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

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

 



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".

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124


[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