msi: Implement the InstallServices action

[Robert Shearman]

James Hawkins wrote:

> +static UINT ITERATE_InstallService(MSIRECORD *rec, LPVOID param)
> +{
> ...
> +    service = CreateServiceW(hscm, name, disp, SERVICE_START | 
> GENERIC_ALL,
> +                             serv_type, start_type, err_control, 
> file->TargetPath,
> +                             load_order, NULL, depends, serv_name, pass);
> +    if (!service)
> +        ERR("Failed to create service %s: %ld\n", debugstr_w(disp), 
> GetLastError());
> +
> +    if (!StartServiceW(service, 0, NULL))
> +        ERR("Failed to start service %s: %ld\n", debugstr_w(disp), 
> GetLastError());


Is this function really supposed to start the service as well? Shouldn't 
this be done by the StartServices action instead 
(http://msdn.microsoft.com/library/default.asp?url=/library/en-us/msi/setup/startservices_action.asp)?

-- 
Rob Shearman