[Development] Reporting errors for powershell scripts executed by Coin

[Konstantin Tokarev]

> Hi
> 
> We ignore the results of PS1 scripts because A) We have bad scripts that try to remove folders that don't exists anymore (should be cleaned) and B) as your example said, if we enabled enforcing of them now, things would break apart (fix needed as well 😝)

B is fixed already, however one VM snapshot had to be manually removed for re-provisioning

> 
> -T
> 
> -----Original Message-----
> From: Development [mailto:development-bounces+tony.sarajarvi=qt.io at qt-project.org] On Behalf Of Konstantin Tokarev
> Sent: torstai 21. joulukuuta 2017 20.37
> To: development at qt-project.org
> Subject: [Development] Reporting errors for powershell scripts executed by Coin
> 
> Hi all,
> 
> AFAIU, some time ago Coin considered ps1 script to be failed its exit code was non-null (e.g, Exit(1) certainly aborted further provisioning process).
> 
> Now I see that Exit(1) is ignored (see [1]). What is the correct way now, throw exception?
> 
> [1] https://testresults.qt.io/coin/api/results/provisioning/qtci-windows-10-x86_64-10-f8e8db/provision_1513869138/log.txt.gz
> 
> Words "conan exited with code 1" are printed before script does Exit(1)
> 
> --
> Regards,
> Konstantin
> _______________________________________________
> Development mailing list
> Development at qt-project.org
> http://lists.qt-project.org/mailman/listinfo/development
-- 
Regards,
Konstantin