Skip to content
This repository was archived by the owner on Jan 13, 2025. It is now read-only.

Commit d2ff5cd

Browse files
authored
Merge pull request #695 from xamarin/dev/build-merger
Make sure the manifest merger builds on mac
2 parents 28e3da2 + e754d1a commit d2ff5cd

File tree

6 files changed

+12
-8
lines changed

6 files changed

+12
-8
lines changed

Util/AndroidManifestMerger/build.cake

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
using System.Xml.Linq;
22

3-
var target = Argument("target", "Default");
3+
var target = Argument("target", "ci");
44

5-
var PACKAGE_VERSION = "1.0.0-preview02";
5+
var PACKAGE_VERSION = "1.0.0-preview03";
66

77
var TOOL_VERSION = "26.5.0";
88

@@ -62,7 +62,8 @@ Task("tests")
6262
// as ensuring that the target is properly skipped on subsequent builds.
6363

6464
var android = (XNamespace)"http://schemas.android.com/apk/res/android";
65-
var xdoc = XDocument.Load("./samples/FancyMergingApp/obj/Release/90/android/AndroidManifest.xml");
65+
var files = GetFiles("./samples/FancyMergingApp/obj/Release/**/android/AndroidManifest.xml");
66+
var xdoc = XDocument.Load(files.Single().FullPath);
6667

6768
var serviceName = "com.google.firebase.components.ComponentDiscoveryService";
6869
var services = xdoc
@@ -78,7 +79,7 @@ Task("tests")
7879
throw new Exception("Manifests were merged, but data was lost.");
7980
});
8081

81-
Task("Default")
82+
Task("ci")
8283
.IsDependentOn("externals")
8384
.IsDependentOn("nuget")
8485
.IsDependentOn("tests");

Util/AndroidManifestMerger/nuget/Xamarin.Android.ManifestMerger.nuspec

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
<files>
1616
<file src="..\externals\merger\*.jar" target="build/monoandroid" />
1717
<file src="..\source\*.targets" target="build/monoandroid" />
18+
<file src="..\source\_._" target="lib/monoandroid/_._" />
1819
<file src="..\External-Dependency-Info.txt" target="THIRD-PARTY-NOTICES.txt" />
1920
</files>
20-
</package>
21+
</package>

Util/AndroidManifestMerger/samples/FancyMergingApp/FancyMergingApp.csproj

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@
100100
<PackageReference Include="Xamarin.Firebase.Perf" Version="71.1620.0" />
101101
</ItemGroup>
102102
<ItemGroup>
103-
<PackageReference Include="Xamarin.Android.ManifestMerger" Version="1.0.0-preview02" />
103+
<PackageReference Include="Xamarin.Android.ManifestMerger" Version="1.0.0-preview03" />
104104
</ItemGroup>
105105
<Import Project="$(MSBuildExtensionsPath)\Xamarin\Android\Xamarin.Android.CSharp.targets" />
106106
</Project>

Util/AndroidManifestMerger/source/Xamarin.Android.ManifestMerger.targets

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
<_MergerJar Include="$(MSBuildThisFileDirectory)\*.jar" />
2626
</ItemGroup>
2727

28-
<Exec Command="&quot;$(_MergerJavaPath)&quot; -cp &quot;@(_MergerJar)&quot; &quot;com.android.manifmerger.Merger&quot; --main &quot;$(IntermediateOutputPath)android\AndroidManifest.xml&quot; --libs &quot;@(_Proxy_ExtractedManifestDocuments)&quot; --out &quot;$(IntermediateOutputPath)android\AndroidManifest.xml&quot;" />
28+
<Exec Command="&quot;$(_MergerJavaPath)&quot; -cp &quot;@(_MergerJar, '$([System.IO.Path]::PathSeparator)')&quot; &quot;com.android.manifmerger.Merger&quot; --main &quot;$(IntermediateOutputPath)android\AndroidManifest.xml&quot; --libs &quot;@(_Proxy_ExtractedManifestDocuments, '$([System.IO.Path]::PathSeparator)')&quot; --out &quot;$(IntermediateOutputPath)android\AndroidManifest.xml&quot;" />
2929

3030
<Touch Files="$(_AndroidStampDirectory)_GenerateJavaStubs.stamp;@(_TypeMapAssemblySource);$(_AndroidStampDirectory)_After_GenerateJavaStubs.stamp" AlwaysCreate="True" />
3131

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+

manifest.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -600,7 +600,8 @@
600600
- Name: AndroidManifestMerger
601601
BuildScript: ./Util/AndroidManifestMerger/build.cake
602602
TriggerPaths: [ Util/AndroidManifestMerger ]
603-
WindowsBuildTargets: [ Default ]
603+
WindowsBuildTargets: [ ci ]
604+
MacBuildTargets: [ ci ]
604605
- Name: Xamarin.AndroidBinderator
605606
BuildScript: ./Util/Xamarin.AndroidBinderator/build.cake
606607
TriggerPaths: [ Util/Xamarin.AndroidBinderator ]

0 commit comments

Comments
 (0)