Merge pull request #2805 from dotnet/piotrpMSFT/AzurePublisher
Piotrp msft/azure publisher
This commit is contained in:
commit
ac5432bff2
2 changed files with 7 additions and 4 deletions
|
@ -69,13 +69,16 @@ namespace Microsoft.DotNet.Cli.Build
|
|||
// at the same time and someone already took care of publishing and we have no work to do.
|
||||
if (AzurePublisherTool.IsLatestSpecifiedVersion(targetVersionFile))
|
||||
{
|
||||
AzurePublisherTool.ReleaseLeaseOnBlob(targetContainer, leaseId);
|
||||
AzurePublisherTool.ReleaseLeaseOnBlob(semaphoreBlob, leaseId);
|
||||
return c.Success();
|
||||
}
|
||||
else
|
||||
{
|
||||
// This is an old drop of latest so remove all old files to ensure a clean state
|
||||
AzurePublisherTool.ListBlobs($"{targetContainer}").ToList().ForEach(f => AzurePublisherTool.TryDeleteBlob(f));
|
||||
AzurePublisherTool.ListBlobs($"{targetContainer}")
|
||||
.Select(s => s.Replace("/dotnet/", ""))
|
||||
.ToList()
|
||||
.ForEach(f => AzurePublisherTool.TryDeleteBlob(f));
|
||||
|
||||
// Drop the version file signaling such for any race-condition builds (see above comment).
|
||||
AzurePublisherTool.DropLatestSpecifiedVersion(targetVersionFile);
|
||||
|
|
4
scripts/obtain/dotnet-install.sh
vendored
4
scripts/obtain/dotnet-install.sh
vendored
|
@ -245,7 +245,7 @@ is_dotnet_package_installed() {
|
|||
|
||||
local install_root=$1
|
||||
local relative_path_to_package=$2
|
||||
local specific_version=$3
|
||||
local specific_version=${3//[$'\t\r\n']}
|
||||
|
||||
local dotnet_package_path=$(combine_paths $(combine_paths $install_root $relative_path_to_package) $specific_version)
|
||||
say_verbose "is_dotnet_package_installed: dotnet_package_path=$dotnet_package_path"
|
||||
|
@ -343,7 +343,7 @@ construct_download_link() {
|
|||
local azure_feed=$1
|
||||
local azure_channel=$2
|
||||
local normalized_architecture=$3
|
||||
local specific_version=$4
|
||||
local specific_version=${4//[$'\t\r\n']}
|
||||
|
||||
local osname=$(get_current_os_name)
|
||||
|
||||
|
|
Loading…
Reference in a new issue