Upgrade AGP to latest version (8.3.0)
Categories
(Firefox for Android :: General, task, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox126 | --- | fixed |
People
(Reporter: mcarare, Assigned: mcarare)
References
Details
(Whiteboard: [fxdroid][group1])
Attachments
(1 file, 1 obsolete file)
This is needed for upgrading Compose to 1.6.0.
At the moment AGP 8.3.0 is not released yet.
Comment 1•1 year ago
|
||
| Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
| Assignee | ||
Updated•1 year ago
|
Updated•1 year ago
|
Comment 3•1 year ago
|
||
Authored by https://github.com/mcarare
https://github.com/mozilla-mobile/firefox-android/commit/9db0adfe4ad016c9aa09cb4c0d11581e4465c3e8
[main] Bug 1879132 - Upgrade AGP to 8.3.0 version.
Authored by https://github.com/mcarare
https://github.com/mozilla-mobile/firefox-android/commit/dfcb84dafee347edd1c351d6651e5fe50e7bf571
[main] Bug 1879132 - Use SdkSuppress instead of RequiresApi in tests.
Comment 4•1 year ago
|
||
Authored by https://github.com/rvandermeulen
https://github.com/mozilla-mobile/firefox-android/commit/6e68351872d08075272be21f3aa397a5b12b989b
[main] Revert "Bug 1879132 - Upgrade AGP to 8.3.0 version." for likely causing bug 1884359
Updated•1 year ago
|
Comment 5•1 year ago
•
|
||
I can reproduce the android-startup-test-fenix-arm-nightly-robo-opt crashes on Try with the AGP bump and can confirm that setting android.enableNewResourceShrinker.preciseShrinking to false fixes them. I guess we still may want to figure out why that particular resource is being optimized away in the first place, but we can at least land the upgrade with that change as a workaround for the time-being and unblock the Compose update.
https://treeherder.mozilla.org/jobs?repo=try&revision=7fec6e76a2993549c2126ecb8e640f6f0e727631&group_state=expanded
Comment 6•1 year ago
|
||
Updated•1 year ago
|
Comment 8•1 year ago
|
||
Should we file a follow-up for investigating why that new optimization broke Fenix builds? I guess the longer-term concern is if they eventually remove the ability to disable it in some future release.
Comment 9•1 year ago
|
||
| bugherder | ||
| Assignee | ||
Comment 10•1 year ago
|
||
I think we should. I am also considering making a minimal reproducible example to open an issue for Google to investigate.
Description
•