Skip to content

Commit

Permalink
[mono] Workaround for the assumption that dotnet is installed (dotn…
Browse files Browse the repository at this point in the history
…et#138)

* [mono] Workaround for the assumption that `dotnet` is installed

We moved to building purely with mono, earlier and thus skip the dotnet
installation in the scripts. This breaks that. Fixing it up in-place for
now.

Commit 2: (Patch on behalf of Alexander Köplinger)
Also, fix the command line, as the `dotnet nuget` uses a different
command line syntax than the standalone `nuget` that mono bundles.

    `nuget locals http-cache -c` -> `nuget locals http-cache -Clear`

* [mono] Workaround: Clear nuget http-cache, if we can

To work around dotnet/arcade#3932 , `nuget
locals http-cache -Clear` is invoked, but in our CI, `nuget` doesn't
seem to be installed in bockbuild's staged prefix.

So, instead we do that only if `dotnet` is available.
  • Loading branch information
radical authored Sep 17, 2019
1 parent d3e6fe8 commit 8cfbe97
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions eng/common/tools.sh
Original file line number Diff line number Diff line change
Expand Up @@ -340,8 +340,8 @@ function MSBuild {

# Work around issues with Azure Artifacts credential provider
# https://github.com/dotnet/arcade/issues/3932
if [[ "$ci" == true ]]; then
dotnet nuget locals http-cache -c
if [[ "$ci" == true && `which dotnet` ]]; then
dotnet nuget locals http-cache -c
fi

local toolset_dir="${_InitializeToolset%/*}"
Expand Down

0 comments on commit 8cfbe97

Please sign in to comment.