mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
Remove unneeded code, document creates param and ensure creates only affects state=present
This commit is contained in:
parent
5d7f67569f
commit
069fa71d87
2 changed files with 7 additions and 10 deletions
|
@ -27,7 +27,8 @@ module: win_msi
|
||||||
version_added: "1.7"
|
version_added: "1.7"
|
||||||
short_description: Installs and uninstalls Windows MSI files
|
short_description: Installs and uninstalls Windows MSI files
|
||||||
description:
|
description:
|
||||||
- Installs or uninstalls a Windows MSI file that is already located on the target server
|
- Installs or uninstalls a Windows MSI file that is already located on the
|
||||||
|
target server
|
||||||
options:
|
options:
|
||||||
path:
|
path:
|
||||||
description:
|
description:
|
||||||
|
@ -40,6 +41,10 @@ options:
|
||||||
- present
|
- present
|
||||||
- absent
|
- absent
|
||||||
default: present
|
default: present
|
||||||
|
creates:
|
||||||
|
description:
|
||||||
|
- Path to a file created by installing the MSI to prevent from
|
||||||
|
attempting to reinstall the package on every run
|
||||||
author: Matt Martz
|
author: Matt Martz
|
||||||
'''
|
'''
|
||||||
|
|
||||||
|
|
|
@ -35,7 +35,7 @@ If ($params.extra_args.GetType)
|
||||||
$extra_args = $params.extra_args;
|
$extra_args = $params.extra_args;
|
||||||
}
|
}
|
||||||
|
|
||||||
If ($params.creates.GetType)
|
If ($params.creates.GetType -and $params.state.GetType -and $params.state -ne "absent")
|
||||||
{
|
{
|
||||||
If (Test-File $creates)
|
If (Test-File $creates)
|
||||||
{
|
{
|
||||||
|
@ -44,8 +44,6 @@ If ($params.creates.GetType)
|
||||||
}
|
}
|
||||||
|
|
||||||
$logfile = [IO.Path]::GetTempFileName();
|
$logfile = [IO.Path]::GetTempFileName();
|
||||||
$stdoutfile = [IO.Path]::GetTempFileName();
|
|
||||||
$stderrfile = [IO.Path]::GetTempFileName();
|
|
||||||
if ($params.state.GetType -and $params.state -eq "absent")
|
if ($params.state.GetType -and $params.state -eq "absent")
|
||||||
{
|
{
|
||||||
msiexec.exe /x $params.path /qb /l $logfile $extra_args;
|
msiexec.exe /x $params.path /qb /l $logfile $extra_args;
|
||||||
|
@ -59,13 +57,7 @@ Set-Attr $result "changed" $true;
|
||||||
|
|
||||||
$logcontents = Get-Content $logfile;
|
$logcontents = Get-Content $logfile;
|
||||||
Remove-Item $logfile;
|
Remove-Item $logfile;
|
||||||
$stdoutcontents = Get-Content $stdoutfile;
|
|
||||||
Remove-Item $stdoutfile;
|
|
||||||
$stderrcontents = Get-Content $stderrfile;
|
|
||||||
Remove-Item $stderrfile;
|
|
||||||
|
|
||||||
Set-Attr $result "log" $logcontents;
|
Set-Attr $result "log" $logcontents;
|
||||||
Set-Attr $result "stdout" $stdoutcontents;
|
|
||||||
Set-Attr $result "stderr" $stderrcontents;
|
|
||||||
|
|
||||||
Exit-Json $result;
|
Exit-Json $result;
|
||||||
|
|
Loading…
Reference in a new issue