1249421 - IJ / gradle build fails with missing AndroidManifest.xml

Status: RESOLVED FIXED

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

Description

[Andrzej Hunt :ahunt]

A fresh checkout from now, results in the following Gradle error:

1) ./mach clobber && ./mach build && ./mach artifact install && ./mach package
2) open project in IJ
3) Possibly start a gradle sync

I see the following error in my Messages window:

Gradle 'fx2' project refresh failed:

Error:Cause: java.io.FileNotFoundException: /Users/andrzejhunt/ff/fx2/objdir-frontend/gradle/build/mobile/android/app/generated/source/preprocessed_manifest/AndroidManifest.xml (No such file or directory)
/Users/andrzejhunt/ff/fx2/objdir-frontend/gradle/build/mobile/android/app/generated/source/preprocessed_manifest/AndroidManifest.xml (No such file or directory)

My tip:
changeset:   284701:f9e4a56281d8

Comment 1

[Andrzej Hunt :ahunt]

Replacing "1.5.0" with "1.3.0" in build.gradle, followed by running "./mach gradle app:assembleLocalDebug" results in a successful build

Comment 2

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1249421 - Generate gradle_AndroidManifest.xml like AndroidManifest.xml. r=me,ahunt

Review commit: https://reviewboard.mozilla.org/r/39331/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/39331/

Comment 3

[Andrzej Hunt :ahunt]

Comment on attachment 8729272 [details]
MozReview Request: Bug 1249421 - Generate gradle_AndroidManifest.xml like AndroidManifest.xml. r=me,ahunt

https://reviewboard.mozilla.org/r/39331/#review36165

Clearing review for now, based on discussion on IRC!

Comment 4

[Nick Alexander :nalexander [he/him]]

Attachment: Bug 1249421 - Include stub AndroidManifest.xml.

This should avoid Bug 1249421, but it will also address an
Android-Gradle plugin 3.0 issue, where the plugin requires the
manifest at Gradle configuration time.  Currently we don't provide it
until Gradle evaluation time, leading to this error, and to the plugin
failing.

Review commit: https://reviewboard.mozilla.org/r/193486/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/193486/

Comment 5

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8922439 [details]
Bug 1249421 - Include stub AndroidManifest.xml.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/193486/diff/1-2/

Comment 6

[Max Liu [:maliu]]

Comment on attachment 8922439 [details]
Bug 1249421 - Include stub AndroidManifest.xml.

https://reviewboard.mozilla.org/r/193486/#review199346

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6a3ca81dc56a
Include stub AndroidManifest.xml. r=maliu

Comment 8

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/6a3ca81dc56a