Skip to content

Conversation

@jonathanpeppers
Copy link
Member

We discovered the build fails on Windows if you:

  • wipe out ~\android-toolchain
  • git clean -dxf
  • msbuild Xamarin.Android.sln /t:Prepare
  • Or just have a fresh machine

We need to reorder the steps of what happens in PrepareWindows.targets:

  1. git submodule update
  2. build xa-prep-tasks, which gets .nuget\NuGet.exe
  3. nuget restore
  4. build android-toolchain , which is the missing step that installs the Android SDK
  5. proceed with everything else

Without step no. 3, the missing Android SDK that gets installed to
~\android-toolchain causes the <JdkInfo /> task to fail.

We discovered the build fails on Windows if you:
- wipe out `~\android-toolchain`
- `git clean -dxf`
- `msbuild Xamarin.Android.sln /t:Prepare`
- Or just have a fresh machine

We need to reorder the steps of what happens in `PrepareWindows.targets`:
1. `git submodule update`
2. build `xa-prep-tasks`, which gets `.nuget\NuGet.exe`
2. `nuget restore`
3. build `android-toolchain` , which is the missing step that installs the Android SDK
4. proceed with everything else

Without step no. 3, the missing Android SDK that gets installed to
`~\android-toolchain` causes the `<JdkInfo />` task to fail.
@jonpryor jonpryor merged commit 3bfafea into dotnet:master Dec 21, 2017
@jonathanpeppers jonathanpeppers deleted the windows-fresh-machine branch June 12, 2019 13:15
jonpryor pushed a commit that referenced this pull request Nov 3, 2020
Fixes: xamarin/monodroid#1129
Fixes: xamarin/monodroid#1130
Fixes: xamarin/monodroid#1131

Changes: xamarin/monodroid@04b0423...0eef889

  * xamarin/monodroid@0eef88915: [tools/msbuild] workaround <FastDeploy/> hang in .NET 5 (#1134)
  * xamarin/monodroid@1ff611e08: [tools/msbuild] Terminate running app on FastDeploy. (#1132)
  * xamarin/monodroid@a9e5dd1c2: [tools/msbuild] Improved no device error messages for FastDeploy (#1133)
@github-actions github-actions bot locked and limited conversation to collaborators Feb 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants