Skip to content

Commit 282ee2c

Browse files
committed
issue #43: add info messages concerning install scripts purpose
1 parent ad55554 commit 282ee2c

File tree

2 files changed

+28
-0
lines changed

2 files changed

+28
-0
lines changed

Diff for: src/dotnet-install.ps1

+18
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,18 @@ function Say-Verbose($str) {
145145
}
146146
}
147147

148+
function Say-Text-Separator() {
149+
try
150+
{
151+
Write-Host ""
152+
}
153+
catch
154+
{
155+
# Some platforms cannot utilize Write-Host (Azure Functions, for instance). Fall back to Write-Output
156+
Write-Output ""
157+
}
158+
}
159+
148160
function Say-Invocation($Invocation) {
149161
$command = $Invocation.MyCommand;
150162
$args = (($Invocation.BoundParameters.Keys | foreach { "-$_ `"$($Invocation.BoundParameters[$_])`"" }) -join " ")
@@ -635,6 +647,12 @@ function Prepend-Sdk-InstallRoot-To-Path([string]$InstallRoot, [string]$BinFolde
635647
}
636648
}
637649

650+
Say "Please note that the intended use of this script is for Continuous Integration (CI) scenarios, where:"
651+
Say "- The SDK needs to be installed without user interaction and without admin rights."
652+
Say "- The SDK installation doesn't need to persist across multiple CI runs."
653+
Say "To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer."
654+
Say-Text-Separator
655+
638656
$CLIArchitecture = Get-CLIArchitecture-From-Architecture $Architecture
639657
$SpecificVersion = Get-Specific-Version-From-Version -AzureFeed $AzureFeed -Channel $Channel -Version $Version -JSonFile $JSonFile
640658
$DownloadLink, $EffectiveVersion = Get-Download-Link -AzureFeed $AzureFeed -SpecificVersion $SpecificVersion -CLIArchitecture $CLIArchitecture

Diff for: src/dotnet-install.sh

+10
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,10 @@ say_verbose() {
5959
fi
6060
}
6161

62+
say_text_separator() {
63+
printf "\n" >&3
64+
}
65+
6266
# This platform list is finite - if the SDK/Runtime has supported Linux distribution-specific assets,
6367
# then and only then should the Linux distribution appear in this list.
6468
# Adding a Linux distribution to this list does not imply distribution-specific support.
@@ -1061,6 +1065,12 @@ if [ "$no_cdn" = true ]; then
10611065
azure_feed="$uncached_feed"
10621066
fi
10631067
1068+
say "Please note that the intended use of this script is for Continuous Integration (CI) scenarios, where:"
1069+
say "- The SDK needs to be installed without user interaction and without admin rights."
1070+
say "- The SDK installation doesn't need to persist across multiple CI runs."
1071+
say "To set up a development environment or to run apps, use installers rather than this script. Visit https://dotnet.microsoft.com/download to get the installer."
1072+
say_text_separator
1073+
10641074
check_min_reqs
10651075
calculate_vars
10661076
script_name=$(basename "$0")

0 commit comments

Comments
 (0)