Skip to content

Commit b44e5ea

Browse files
committed
Use GetFileHash and replace RemoveDirFixed
1 parent 548e76f commit b44e5ea

File tree

1 file changed

+29
-30
lines changed

1 file changed

+29
-30
lines changed

src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets

Lines changed: 29 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -913,13 +913,12 @@ because xbuild doesn't support framework reference assemblies.
913913
<_TypeMapKind Condition=" '$(AndroidIncludeDebugSymbols)' == 'True' And '$(_InstantRunEnabled)' == 'True' ">strings-files</_TypeMapKind>
914914
<_TypeMapKind Condition=" '$(AndroidIncludeDebugSymbols)' == 'True' And '$(_InstantRunEnabled)' != 'True' ">strings-asm</_TypeMapKind>
915915
</PropertyGroup>
916-
<ComputeHash
916+
<GetFileHash
917917
Condition=" Exists('$(_NuGetAssetsFile)') "
918-
CopyMetadata="False"
919-
Source="$(_NuGetAssetsFile)"
918+
Files="$(_NuGetAssetsFile)"
920919
>
921-
<Output TaskParameter="Output" ItemName="_NuGetAssetsFileHash" />
922-
</ComputeHash>
920+
<Output TaskParameter="Items" ItemName="_NuGetAssetsFileHash" />
921+
</GetFileHash>
923922
<ItemGroup>
924923
<!-- List of items we want to trigger a build if changed -->
925924
<_PropertyCacheItems Include="AotAssemblies=$(AotAssemblies)" />
@@ -948,7 +947,7 @@ because xbuild doesn't support framework reference assemblies.
948947
<_PropertyCacheItems Include="OS=$(OS)" />
949948
<_PropertyCacheItems Include="AndroidIncludeDebugSymbols=$(AndroidIncludeDebugSymbols)" />
950949
<_PropertyCacheItems Include="AndroidPackageNamingPolicy=$(AndroidPackageNamingPolicy)" />
951-
<_PropertyCacheItems Include="_NuGetAssetsFileHash=%(_NuGetAssetsFileHash.Hash)" />
950+
<_PropertyCacheItems Include="_NuGetAssetsFileHash=%(_NuGetAssetsFileHash.FileHash)" />
952951
<_PropertyCacheItems Include="TypeMapKind=$(_TypeMapKind)" />
953952
<_PropertyCacheItems Include="AndroidManifestPlaceholders=$(AndroidManifestPlaceholders)" />
954953
<_PropertyCacheItems Include="ProjectFullPath=$(MSBuildProjectFullPath)" />
@@ -1288,7 +1287,7 @@ because xbuild doesn't support framework reference assemblies.
12881287
</ItemGroup>
12891288

12901289
<!-- Delete our temporary directory -->
1291-
<RemoveDirFixed
1290+
<RemoveDir
12921291
Directories="$(ResgenTemporaryDirectory)"
12931292
ContinueOnError="WarnAndContinue"
12941293
/>
@@ -1817,7 +1816,7 @@ because xbuild doesn't support framework reference assemblies.
18171816
</CopyGeneratedJavaResourceClasses>
18181817

18191818
<!-- Delete our temporary directory -->
1820-
<RemoveDirFixed
1819+
<RemoveDir
18211820
Directories="$(AaptTemporaryDirectory)"
18221821
ContinueOnError="WarnAndContinue"
18231822
/>
@@ -2492,11 +2491,11 @@ because xbuild doesn't support framework reference assemblies.
24922491

24932492
<Target Name="_CleanMsymArchive"
24942493
DependsOnTargets="_GetAndroidPackageName">
2495-
<RemoveDirFixed Directories="$(OutDir)$(_AndroidPackage).$(AndroidPackageFormat).mSYM" Condition=" '$(_AndroidPackage)' != '' " />
2494+
<RemoveDir Directories="$(OutDir)$(_AndroidPackage).$(AndroidPackageFormat).mSYM" Condition=" '$(_AndroidPackage)' != '' " />
24962495
</Target>
24972496

24982497
<Target Name="_CleanDesignTimeIntermediateDir">
2499-
<RemoveDirFixed Directories="$(_AndroidIntermediateDesignTimeBuildDirectory)" Condition="Exists ('$(_AndroidIntermediateDesignTimeBuildDirectory)')" />
2498+
<RemoveDir Directories="$(_AndroidIntermediateDesignTimeBuildDirectory)" Condition="Exists ('$(_AndroidIntermediateDesignTimeBuildDirectory)')" />
25002499
</Target>
25012500

25022501
<Target Name="_CleanGeneratedDeploymentFiles">
@@ -2507,27 +2506,27 @@ because xbuild doesn't support framework reference assemblies.
25072506
</Target>
25082507

25092508
<Target Name="_CleanMonoAndroidIntermediateDir" DependsOnTargets="_CleanGeneratedDeploymentFiles;_CleanMsymArchive">
2510-
<RemoveDirFixed Directories="$(IntermediateOutputPath)android" Condition="Exists ('$(IntermediateOutputPath)android')" />
2509+
<RemoveDir Directories="$(IntermediateOutputPath)android" Condition="Exists ('$(IntermediateOutputPath)android')" />
25112510
<!-- FIXME: remove this extraneous rmdir after a few release cycles since we release the one we killed it. -->
2512-
<RemoveDirFixed Directories="$(IntermediateOutputPath)assemblies" Condition="Exists ('$(IntermediateOutputPath)assemblies')" />
2513-
<RemoveDirFixed Directories="$(IntermediateOutputPath)generated" Condition="Exists ('$(IntermediateOutputPath)generated')" />
2514-
<RemoveDirFixed Directories="$(IntermediateOutputPath)codebehind" Condition="Exists ('$(IntermediateOutputPath)codebehind')" />
2515-
<RemoveDirFixed Directories="$(IntermediateOutputPath)res" Condition="Exists ('$(IntermediateOutputPath)res')" />
2516-
<RemoveDirFixed Directories="$(IntermediateOutputPath)aidl" Condition="Exists ('$(IntermediateOutputPath)aidl')" />
2517-
<RemoveDirFixed Directories="$(IntermediateOutputPath)bundles" Condition="Exists ('$(IntermediateOutputPath)bundles')" />
2518-
<RemoveDirFixed Directories="$(IntermediateOutputPath)__library_projects__" Condition="Exists ('$(IntermediateOutputPath)__library_projects__')" />
2519-
<RemoveDirFixed Directories="$(_AndroidLibrayProjectIntermediatePath)" Condition="Exists ('$(_AndroidLibrayProjectIntermediatePath)')" />
2520-
<RemoveDirFixed Directories="$(IntermediateOutputPath)$(_LibraryProjectImportsDirectoryName)" Condition="Exists ('$(IntermediateOutputPath)$(_LibraryProjectImportsDirectoryName)')" />
2521-
<RemoveDirFixed Directories="$(IntermediateOutputPath)$(_NativeLibraryImportsDirectoryName)" Condition="Exists ('$(IntermediateOutputPath)$(_NativeLibraryImportsDirectoryName)')" />
2522-
<RemoveDirFixed Directories="$(IntermediateOutputPath)assets" Condition="Exists ('$(IntermediateOutputPath)assets')" />
2523-
<RemoveDirFixed Directories="$(IntermediateOutputPath)java" Condition="Exists ('$(IntermediateOutputPath)java')" />
2524-
<RemoveDirFixed Directories="$(IntermediateOutputPath)proguard" Condition="Exists ('$(IntermediateOutputPath)proguard')" />
2525-
<RemoveDirFixed Directories="$(MonoAndroidIntermediateResourceCache)" Condition="Exists ('$(MonoAndroidIntermediateResourceCache)')" />
2526-
<RemoveDirFixed Directories="$(_AndroidAotBinDirectory)" Condition="Exists ('$(_AndroidAotBinDirectory)')" />
2527-
<RemoveDirFixed Directories="$(_AndroidLibraryFlatArchivesDirectory)" Condition="Exists ('$(_AndroidLibraryFlatArchivesDirectory)')" />
2528-
<RemoveDirFixed Directories="$(_AndroidLibraryFlatFilesDirectory)" Condition="Exists ('$(_AndroidLibraryFlatFilesDirectory)')" />
2529-
<RemoveDirFixed Directories="$(_AndroidStampDirectory)" Condition="Exists ('$(_AndroidStampDirectory)')" />
2530-
<RemoveDirFixed Directories="$(_AndroidApplicationSharedLibraryPath)" Condition="Exists ('$(_AndroidApplicationSharedLibraryPath)')" />
2511+
<RemoveDir Directories="$(IntermediateOutputPath)assemblies" Condition="Exists ('$(IntermediateOutputPath)assemblies')" />
2512+
<RemoveDir Directories="$(IntermediateOutputPath)generated" Condition="Exists ('$(IntermediateOutputPath)generated')" />
2513+
<RemoveDir Directories="$(IntermediateOutputPath)codebehind" Condition="Exists ('$(IntermediateOutputPath)codebehind')" />
2514+
<RemoveDir Directories="$(IntermediateOutputPath)res" Condition="Exists ('$(IntermediateOutputPath)res')" />
2515+
<RemoveDir Directories="$(IntermediateOutputPath)aidl" Condition="Exists ('$(IntermediateOutputPath)aidl')" />
2516+
<RemoveDir Directories="$(IntermediateOutputPath)bundles" Condition="Exists ('$(IntermediateOutputPath)bundles')" />
2517+
<RemoveDir Directories="$(IntermediateOutputPath)__library_projects__" Condition="Exists ('$(IntermediateOutputPath)__library_projects__')" />
2518+
<RemoveDir Directories="$(_AndroidLibrayProjectIntermediatePath)" Condition="Exists ('$(_AndroidLibrayProjectIntermediatePath)')" />
2519+
<RemoveDir Directories="$(IntermediateOutputPath)$(_LibraryProjectImportsDirectoryName)" Condition="Exists ('$(IntermediateOutputPath)$(_LibraryProjectImportsDirectoryName)')" />
2520+
<RemoveDir Directories="$(IntermediateOutputPath)$(_NativeLibraryImportsDirectoryName)" Condition="Exists ('$(IntermediateOutputPath)$(_NativeLibraryImportsDirectoryName)')" />
2521+
<RemoveDir Directories="$(IntermediateOutputPath)assets" Condition="Exists ('$(IntermediateOutputPath)assets')" />
2522+
<RemoveDir Directories="$(IntermediateOutputPath)java" Condition="Exists ('$(IntermediateOutputPath)java')" />
2523+
<RemoveDir Directories="$(IntermediateOutputPath)proguard" Condition="Exists ('$(IntermediateOutputPath)proguard')" />
2524+
<RemoveDir Directories="$(MonoAndroidIntermediateResourceCache)" Condition="Exists ('$(MonoAndroidIntermediateResourceCache)')" />
2525+
<RemoveDir Directories="$(_AndroidAotBinDirectory)" Condition="Exists ('$(_AndroidAotBinDirectory)')" />
2526+
<RemoveDir Directories="$(_AndroidLibraryFlatArchivesDirectory)" Condition="Exists ('$(_AndroidLibraryFlatArchivesDirectory)')" />
2527+
<RemoveDir Directories="$(_AndroidLibraryFlatFilesDirectory)" Condition="Exists ('$(_AndroidLibraryFlatFilesDirectory)')" />
2528+
<RemoveDir Directories="$(_AndroidStampDirectory)" Condition="Exists ('$(_AndroidStampDirectory)')" />
2529+
<RemoveDir Directories="$(_AndroidApplicationSharedLibraryPath)" Condition="Exists ('$(_AndroidApplicationSharedLibraryPath)')" />
25312530
<Delete Files="$(IntermediateOutputPath)R.cs.flag" />
25322531
<Delete Files="$(IntermediateOutputPath)acw-map.txt" />
25332532
<Delete Files="$(IntermediateOutputPath)customview-map.txt" />

0 commit comments

Comments
 (0)