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 07:21:30PM +0200, Zeeshan Ali (Khattak) wrote:
> OK but don't think of it as something I want you to implement. :) The
> idea was simply to move this function to Entity class as virtual (and
> internal) and then have Config override it.. but now that I said it, I
> just realized that would be impossible to implement without breaking
> ABI. :(

Well, been there (without breaking the ABI, thanks to Marc-André), and went
back to this approach. You need to be able to get both the transformed and
untransformed value depending on the situation (transformed when generating
the OsinfoInstallScript, untransformed for example when cloning the
OsinfoInstallConfig object).
Given that we need 2 methods anyway, it seemed better not be too smart, and
to keep OsinfoEntity behaviour consistent
(osinfo_entity_set(entity, "A", "B"); followed by osinfo_entity_get(entity,
"A") will always return "B").

Christophe

Attachment: pgpxWTmsl4FmO.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