On Mon, Dec 31, 2012 at 2:20 PM, Zeeshan Ali (Khattak) <zeeshanak@xxxxxxxxx> wrote: > On Thu, Dec 20, 2012 at 6:45 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote: >> This property lists the parameters that can be set for a given >> OsinfoInstallConfig. This is not enforced, it's only there for >> informative purpose. This will also be used in later commits >> in order to automatically apply transformations on values >> for parameters which have an associated OsinfoDatamap. > > Sorry to revive this thread again but I had an idea that I wanted to > discuss before this API goes into a release: How about we have a > OsinfoInstallConfig:install-script (of type OsinfoInstallScript) > rather than having OsinfoInstallConfig:config-params (and > 'path-format', 'avatar-format' etc) duplicated on both objects? IMO > that makes a lot more sense since that not only avoids duplication of > API (and some strings) but also makes things more clear: Whether a > config is associated/specific to a script or not? If it is, which > install script is exactly its tied to? After a long read of the topic, personally, I agree with Zeeshan's suggestion, basically because it contributes for clearness. However, I think we should wait Christophe's opinion (he is on vacations, IIRC). > > > > -- > Regards, > > Zeeshan Ali (Khattak) > FSF member#5124 > > _______________________________________________ > Libosinfo mailing list > Libosinfo@xxxxxxxxxx > https://www.redhat.com/mailman/listinfo/libosinfo Best Regards, -- Fabiano Fidêncio _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo
