Skip to content

[Xamarin.Android.Build.Tasks] Add CopyResource items to FileWrites. - #2288

Merged
jonpryor merged 7 commits into
dotnet:masterfrom
dellis1972:filewritescopyresource
Oct 13, 2018
Merged

jonpryor merged 7 commits into
dotnet:masterfrom
dellis1972:filewritescopyresource

Conversation

@dellis1972

Copy link
Copy Markdown
Contributor

Not quite sure how we ever got away with this. As part
of the build process we extract from EmbeddedResource
a bunch of .java files into the intermediate directory.

These are used for a number of things like multidex and
bootstrapping mono. For some reason we NEVER added those
to the FileWrites group. So there is a good chance these
files will get deleted by an incremental clean.

This was picked up by a unit test in monodroid which
deals with fast deployment.

Not quite sure how we ever got away with this. As part
of the build process we extract from `EmbeddedResource`
a bunch of `.java` files into the intermediate directory.

These are used for a number of things like multidex and
bootstrapping mono. For some reason we NEVER added those
to the `FileWrites` group. So there is a good chance these
files will get deleted by an incremental clean.

This was picked up by a unit test in monodroid which
deals with fast deployment.
@dellis1972
dellis1972 force-pushed the filewritescopyresource branch from 4ffe636 to 6d45402 Compare October 11, 2018 14:34
Comment thread src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets
Comment thread src/Xamarin.Android.Build.Tasks/Xamarin.Android.Common.targets
// change one of the properties (say AotAssemblies)
// do another build. it should NOT hose the resource directory.
var path = Path.Combine ( "temp", TestName );
var proj = new XamarinAndroidApplicationProject () {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops I think this one should be the new XamarinFormsAndroidApplicationProject.

@jonpryor
jonpryor merged commit 5e47af3 into dotnet:master Oct 13, 2018
@dellis1972
dellis1972 deleted the filewritescopyresource branch October 15, 2018 09:35
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 2, 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.

3 participants