Re: [PATCH v4 1/7] install_scripts: Add get_supported_injection_method API

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

 



Hi,

Apart from some nitpicks below, looks good. ACK with or without
suggested changes.

On Wed, Mar 5, 2014 at 3:01 PM, Giuseppe Scrivano <gscrivan@xxxxxxxxxx> wrote:
> The API allows to query how it is possible to use the script from the
> installation process.

Sounds a bit vague. "use the script from the installation process" ->
"inject the script into the installation process"? 'it is possible'
part is redundant IMO.

> @@ -1309,6 +1309,24 @@ int osinfo_install_script_get_post_install_drivers_signing_req(OsinfoInstallScri
>           OSINFO_DEVICE_DRIVER_SIGNING_REQ_NONE);
>  }
>
> +/**
> + * osinfo_install_script_get_supported_injection_method:
> + * @script: the install script
> + *
> + * Retrieve how it is possible to inject the script in the installation process.

"is is possible" part redundant here too.

_______________________________________________
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