Hi Andrew,

This is the only place the exit code is needed. Without it the script would show up as failed in Deployit. Agree that for consistency we should add the exit statements everywhere. But again, this is the only place where I needed it to get the correct outcome for Deployit.
Gr,

Coert


Op 18 jan. 2014, om 07:29 heeft Andrew Phillips <notifications@github.com<mailto:notifications@github.com>> het volgende geschreven:


In deployit-udm-plugins/middleware-plugins/ssis-plugin/ssis/destroy-ispackage.ps1:

> +     $SqlVersion = Get-SqlVersion -ServerInstance $ServerInstance
> +
> +     # Set Dtutil Path based on Sql Version
> +     Set-DtutilPath -SqlVersion $SqlVersion
> +
> +    Write-Host "Removing package [$PackageFullName] on [$ServerInstance]."
> +    remove-package  -ServerInstance $ServerInstance -PackageFullName $PackageFullName
> +
> +    #Verify Package
> +    if(test-packagepath -ServerInstance $ServerInstance -PackageFullName $PackageFullName){
> +     Write-Host "Package [$PackageFullName] still exists on [$ServerInstance]."
> +     Exit 1
> +    }
> +    else{
> +     Write-Host "Package [$PackageFullName] no longer found on [$ServerInstance]."
> +     Exit 0


Not needed? Or, if we prefer this, add an Exit 0 in the previous script?

—
Reply to this email directly or view it on GitHub<https://github.com/xebialabs/community-plugins/pull/48/files#r8986697>.