Open Bug 1894737 Opened 5 months ago Updated 3 days ago

Building geckoview spends time evaluating unrelated gradle projects (fenix, focus, ...)

Categories

(Firefox Build System :: Android Studio and Gradle Integration, defect, P1)

defect

Tracking

(Not tracked)

ASSIGNED

People

(Reporter: florian, Assigned: adhingra)

References

(Blocks 1 open bug)

Details

(Whiteboard: [avocado sprint])

Attachments

(1 file)

Whenever gradle is called (4 times for a full build, 2 for a no-op mach build), 122 gradle:evaluatate-project markers are reported.
Try build profile: https://share.firefox.dev/3QuolhD
Local no-op build: https://share.firefox.dev/3UnNA6q

The projects taking the longer to evaluate are: Fenix (calling hg id 3 or 4 times), focus-android and support-base. Fixing bug 1894702 might make these have a duration closer to the other projects (a few ms), but the total time spent to evaluate these 122 projects would still be about 2s locally and 9s on CI.

Here's the list of the 122 projects:

android-components
annotations
browser-domains
browser-engine-gecko
browser-engine-system
browser-errorpages
browser-icons
browser-menu
browser-menu2
browser-session-storage
browser-state
browser-storage-sync
browser-tabstray
browser-thumbnails
browser-toolbar
browser-toolbar2
compose-awesomebar
compose-browser-toolbar
compose-cfr
compose-engine
compose-tabstray
concept-awesomebar
concept-base
concept-engine
concept-fetch
concept-menu
concept-push
concept-storage
concept-sync
concept-tabstray
concept-toolbar
exoplayer2
feature-accounts
feature-accounts-push
feature-addons
feature-app-links
feature-autofill
feature-awesomebar
feature-containers
feature-contextmenu
feature-customtabs
feature-downloads
feature-findinpage
feature-fxsuggest
feature-intent
feature-logins
feature-media
feature-privatemode
feature-prompts
feature-push
feature-pwa
feature-qr
feature-readerview
feature-recentlyclosed
feature-search
feature-serviceworker
feature-session
feature-share
feature-sitepermissions
feature-syncedtabs
feature-tab-collections
feature-tabs
feature-toolbar
feature-top-sites
feature-webauthn
feature-webcompat
feature-webcompat-reporter
feature-webnotifications
fenix
focus-android
geckoview
geckoview_example
lib-auth
lib-crash
lib-crash-sentry
lib-dataprotect
lib-fetch-httpurlconnection
lib-fetch-okhttp
lib-jexl
lib-publicsuffixlist
lib-push-firebase
lib-state
messaging_example
mozilla-lint-rules
port_messaging_example
samples-browser
service-contile
service-digitalassetlinks
service-firefox-accounts
service-glean
service-location
service-nimbus
service-pocket
service-sync-autofill
service-sync-logins
service-telemetry
support-android-test
support-base
support-images
support-ktx
support-license
support-locale
support-remotesettings
support-rusterrors
support-rusthttp
support-rustlog
support-test
support-test-appservices
support-test-fakes
support-test-libstate
support-utils
support-webextensions
test_runner
tooling-detekt
tooling-fetch-tests
tooling-lint
ui-autocomplete
ui-colors
ui-fonts
ui-icons
ui-tabcounter
ui-widgets

Could this list be made more specific when we are building geckoview, Fenix or Focus?

Marking bug as P1 since bug 1887646 is also marked as such and this bug is an action towards solving bug 1887646.

Priority: -- → P1

The severity field is not set for this bug.
:rsainani, could you have a look please?

For more information, please visit BugBot documentation.

Flags: needinfo?(rsainani)

Add option to just run fenix/focus like ./mach build --subproject fenix.

Whiteboard: [avocado sprint]
Assignee: nobody → adhingra
Attachment #9417776 - Attachment description: WIP: Bug 1894737 - Building geckoview spends time evaluating unrelated gradle projects. → Bug 1894737 - Building geckoview spends time evaluating unrelated gradle projects.
Attachment #9417776 - Attachment description: Bug 1894737 - Building geckoview spends time evaluating unrelated gradle projects. → Bug 1894737 - Building geckoview spends time evaluating unrelated gradle projects. r=nalexander
Flags: needinfo?(rsainani)
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: