1220887 - Don't require |mach gradle-install| for srcdir Gradle builds

Status: RESOLVED FIXED

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

Description

[Nick Alexander :nalexander [he/him]]

Bug 1176133 tracks removing |mach gradle-install| entirely.  This tracks folding a minimal version of the symlinking into the srcdir settings.gradle file itself.

This is possible 'cuz settings.gradle executes very early in the IDE import project sequence: before Gradle project evaluation time, and thus before any Gradle task is executed.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1220887 - Make 'base' Gradle project in the srcdir. r?sebastian

Bug 1220887 - Make 'base' Gradle project in the srcdir. r?sebastian

This is the last Gradle project that isn't in the srcdir.  Since base/
doesn't have the correct package prefix directory structure, we still
need to symlink, but we only need one link.  This effectively
deprecates |mach gradle-install|.

This should improve the robustness of our Gradle configuration,
ensuring that we always have projects to import.  Since
settings.gradle executes very early in the IDE import project
sequence: before Gradle project evaluation time, and thus before any
Gradle task is executed, we should always see a complete project.  (It
was possible to see incomplete Gradle configurations if |mach
gradle-install| hadn't been run at just the right time.)

Comment 2

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8682639 [details]
MozReview Request: Bug 1220887 - Make 'base' Gradle project in the srcdir. r?sebastian

https://reviewboard.mozilla.org/r/24119/#review21835

I still needed to run |mach gradle-install| because I saw the following error without. Is this expected at this stage?

> FAILURE: Build failed with an exception.
> 
> * What went wrong:
> A problem occurred configuring project ':app'.
> > Failed to notify project evaluation listener.
>    > Cannot evaluate module base : Configuration with name 'default' not found.
>    > Task with name 'spoonDebugAndroidTest' not found in project ':app'.
> 
> * Try:
> Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Comment 3

[Nick Alexander :nalexander [he/him]]

https://hg.mozilla.org/integration/fx-team/rev/319e09c9668b8d2937c49dabdfd84a30d8771ac1
Bug 1220887 - Make 'base' Gradle project in the srcdir. r=sebastian

Comment 4

[Nick Alexander :nalexander [he/him]]

(In reply to Sebastian Kaspari (:sebastian) from comment #2)
> Comment on attachment 8682639 [details]
> MozReview Request: Bug 1220887 - Make 'base' Gradle project in the srcdir.
> r?sebastian
> 
> https://reviewboard.mozilla.org/r/24119/#review21835
> 
> I still needed to run |mach gradle-install| because I saw the following
> error without. Is this expected at this stage?
> 
> > FAILURE: Build failed with an exception.
> > 
> > * What went wrong:
> > A problem occurred configuring project ':app'.
> > > Failed to notify project evaluation listener.
> >    > Cannot evaluate module base : Configuration with name 'default' not found.
> >    > Task with name 'spoonDebugAndroidTest' not found in project ':app'.
> > 
> > * Try:
> > Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

This is not expected.  I clobbered and after |mach build && mach artifact-install && mach package|, I could |./gradlew app:assembleDebug| without issues.  I could also import the topsrcdir into IJ no trouble.

I think you're seeing remnants of the old configuration and symlinks.  We'll see!

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/319e09c9668b