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

Support D8 for dex file compilation #1469

Closed
cobey opened this issue Mar 23, 2018 · 7 comments
Closed

Support D8 for dex file compilation #1469

cobey opened this issue Mar 23, 2018 · 7 comments
Labels
enhancement Proposed change to current functionality. vs-sync For internal use only; creates a VSTS "mirror" issue.
Milestone

Comments

@cobey
Copy link

cobey commented Mar 23, 2018

D8 is now suggested dex compiler for Android development, with enhanced speed and reduced file sizes. We should aim to support D8 as the dex compiler.

VS bug #660386

@cobey cobey added enhancement Proposed change to current functionality. vs-sync For internal use only; creates a VSTS "mirror" issue. labels Mar 23, 2018
@atsushieno
Copy link
Contributor

There is already a solution for d8. https://github.com/atsushieno/xamarin-android-d8-build

@cobey
Copy link
Author

cobey commented Mar 30, 2018

@jonpryor for context

We want this to be a default experience within Xamarin.Android. While the solution presented is a fine interim solution, we want the improvements that d8 offers as default for 15.8

@atsushieno
Copy link
Contributor

I have added D8 support within xamarin-android itself so that users don't have to install my nuget package independently. Instead, <AndroidEnableD8>true</AndroidEnableD8> is required in the app .csproj.

I don't think enabling D8 by default at this stage is appropriate. What is doable instead is to add the property above in the default Android app project templates.

@atsushieno
Copy link
Contributor

(I mean, added to #1489)

@JonDouglas
Copy link
Contributor

@atsushieno When can we bump our templates to include the property by default? #1489 is still outstanding, so I would guess 15.9?

@atsushieno
Copy link
Contributor

Can we please fix any blocking issue first so that I can continue the work?

@xamarin-release-manager xamarin-release-manager added this to the d16-0 milestone Aug 7, 2018
@JonDouglas
Copy link
Contributor

Closing this issue in favor of #2040

@ghost ghost locked as resolved and limited conversation to collaborators Jun 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Proposed change to current functionality. vs-sync For internal use only; creates a VSTS "mirror" issue.
Projects
None yet
Development

No branches or pull requests

4 participants