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? -- Regards, Zeeshan Ali (Khattak) FSF member#5124 _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo
