Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Xamarin.Android.Build.Tasks] Fix up lint output resource paths. #3368

Merged
merged 1 commit into from Aug 27, 2019

Conversation

dellis1972
Copy link
Contributor

Fixes #3360

The lint tool is run against our $(IntermediateOutputPath)
directory, as a result we end up with paths which point to
that directory.

We should be fixing those paths up to point to the normal
Resources directory in the project, just like we do
with aapt/aapt2.

@dellis1972 dellis1972 added the do-not-merge PR should not be merged. label Jul 18, 2019
@dellis1972
Copy link
Contributor Author

I might try to unify the Resource path fix up in to a central location.

@dellis1972 dellis1972 removed the do-not-merge PR should not be merged. label Aug 22, 2019
@dellis1972 dellis1972 marked this pull request as ready for review August 22, 2019 15:50
Fixes xamarin#3360

The lint tool is run against our `$(IntermediateOutputPath)`
directory, as a result we end up with paths which point to
that directory.

We should be fixing those paths up to point to the normal
`Resources` directory in the project, just like we do
with `aapt/aapt2`.
@jonpryor jonpryor merged commit 4084dfc into xamarin:master Aug 27, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Jan 29, 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.

Analysis output not useful without option to not minimize Android resources
4 participants