Re: [libosinfo PATCHv3 10/12] Add osinfo_install_config_new_for_script

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

 



On Wed, Dec 12, 2012 at 5:18 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> In order to be able to automatically transform configuration parameters set
> on OsinfoInstallConfig instances (ie translate from generic value to
> OS-specific value),

Hmm.. this seemingly small change is rather a big change in terms of
API design as so far we have kept install config separate from install
script. They only meet each other when generating scripts from
template.

Since the OS-specific values are actually needed when transforming the
template to script (i-e in osinfo_install_script_generate_config_xml),
 I wonder if there is another less intrusive (in terms of API) way to
achieve the goal. One idea that comes to mind is to create a
OS-specific temporary config (internally) based on the pass (to
_install_script_generate*) config and script.

Also how would the app know which _new() to use? If we accept this
change, I reluctantly suggest we deprecate the other _new().

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list


[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux