FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects
Commit c8fb58fa authored by Dr Adam Thorn's avatar Dr Adam Thorn
Browse files

irfanview: add "if exist" check before running rmdir on removal

I was getting:

Exit code returned non-successful value: 2
Package: Irfanview.
Command:
cmd /c rmdir /s /q "%programfiles(x86)%\irfanview"
Removal of Irfanview successful.

i.e. the rmdir was returning 2 ("directory not found") but because that
was the final remove cmd and the package check condition was no longer
satisfied, wpkg was happy that the removal had completed.
parent ddc5767a
No related branches found
No related tags found
1 merge request!77Irfanview.xml
Pipeline #28251 passed
......@@ -26,7 +26,7 @@
<remove cmd='"%programfiles(x86)%\irfanview\iv_uninstall.exe" /silent' />
<remove cmd='cmd /c %WPKGSOFTWARE%\wpkg\tools\waitforprocess.cmd iv_uninstall.exe' />
<!-- the uninstaller leaves itself behind, so remove it -->
<remove cmd='cmd /c rmdir /s /q "%programfiles(x86)%\irfanview"' />
<remove cmd='cmd /c if exist "%programfiles(x86)%irfanview" rmdir /s /q "%programfiles(x86)%\irfanview"' />
</package>
</packages:packages>
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment