1128362 - Add gecko libraries Gradle project

Status: RESOLVED WONTFIX

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

Description

[Nick Alexander :nalexander [he/him]]

This ticket tracks adding a gecko libraries Gradle project that contains the .so libraries in assets/ and lib/.

The idea is to do two things: first, be more deliberate about copying and packaging compiled libraries from dist/.  In this respect, this new libraries project mirrors the omnijar project.  Second, provide a named dependency that we can download rather than produce locally in the future.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: bz://1128362/nalexander

/r/3241 - Bug 1128362 - Add gecko_libs Gradle project. r=me

Pull down this commit:

hg pull review -r 623ba1bfde62d52d31b858a8a81042e94c91ef93

Comment 2

[Monty the Python Review Bot]

https://reviewboard.mozilla.org/r/3241/#review2655

Always look on the bright side of life.

I analyzed your Python changes and found 2 errors.

The following files were examined:

  mobile/android/mach_commands.py

::: mobile/android/mach_commands.py
(Diff revision 1)
> +        srcdir('gecko_libs/build.gradle', 'mobile/android/gradle/gecko_libs/build.gradle')

E501: line too long (90 > 79 characters)
Limit all lines to a maximum of 79 characters.

There are still many devices around that are limited to 80 character
lines; plus, limiting windows to 80 characters makes it possible to have
several windows side-by-side.  The default wrapping on such devices looks
ugly.  Therefore, please limit all lines to a maximum of 79 characters.
For flowing long blocks of text (docstrings or comments), limiting the
length to 72 characters is recommended.

Reports error E501.

::: mobile/android/mach_commands.py
(Diff revision 1)
> +        srcdir('gecko_libs/src/main/AndroidManifest.xml', 'mobile/android/gradle/gecko_libs/AndroidManifest.xml')

E501: line too long (113 > 79 characters)
Limit all lines to a maximum of 79 characters.

There are still many devices around that are limited to 80 character
lines; plus, limiting windows to 80 characters makes it possible to have
several windows side-by-side.  The default wrapping on such devices looks
ugly.  Therefore, please limit all lines to a maximum of 79 characters.
For flowing long blocks of text (docstrings or comments), limiting the
length to 72 characters is recommended.

Reports error E501.

Comment 3

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8557684 [details]
MozReview Request: bz://1128362/nalexander

/r/3241 - Bug 1128362 - Part 1: Share generateCodeAndResources. r=me
/r/6443 - Bug 1128362 - Part 2: Copy omni.ja from dist/fennec manually. r=me
/r/6445 - Bug 1128362 - Part 3: Make running Robocop from IntelliJ work. r=me
/r/6447 - Bug 1128362 - Part 4: Move some build steps earlier. r=gps
/r/6449 - Bug 1128362 - Part 5: Use pre-built geckolibs dependency in debug configuration. r=me
/r/6451 - Bug 1128362 - Part 6: Make Gradle's release configuration work like the old debug configuration. r=me

Pull down these commits:

hg pull -r 97c890d9a7bbd043d9acfac8bbce92464578d478 https://reviewboard-hg.mozilla.org/gecko/

Comment 4

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8557684 [details]
MozReview Request: bz://1128362/nalexander

/r/3241 - Bug 1128362 - Part 1: Share generateCodeAndResources. r=me
/r/6443 - Bug 1128362 - Part 2: Copy omni.ja from dist/fennec manually. r=me
/r/6445 - Bug 1128362 - Part 3: Make running Robocop from IntelliJ work. r=me
/r/6447 - Bug 1128362 - Part 4: Move some build steps earlier. r=gps
/r/6449 - Bug 1128362 - Part 5: Use pre-built geckolibs dependency in debug configuration. r=me
/r/6451 - Bug 1128362 - Part 6: Make Gradle's release configuration work like the old debug configuration. r=me
/r/6633 - Fix
/r/6635 - Bug 1112673 - Part 1: Refactor shared footer functionality into RemoteTabsBaseFragment. r=margaret
/r/6637 - Bug 1112673 - Part 2: Add hidden devices hint to Remote Tabs panel footer. r=margaret

Pull down these commits:

hg pull -r 36bbe078391d43fbfd3f00e00630764e8228ca18 https://reviewboard-hg.mozilla.org/gecko/

Comment 5

[Nick Alexander :nalexander [he/him]]

psd: thanks for the commits!  I had some issues with yours (not huge) and wanted to rework the visuals a bit, so I took your work and ran with it.  Care to test this and provide feedback?

margaret: could you review the last two commits?  I don't know how to edit the RB request to be more sensible.

Comment 6

[Nick Alexander :nalexander [he/him]]

antlam: here's the visual:

https://people.mozilla.org/~nalexander/screenshots/Hidden.Devices.Tip.png

Comment 7

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Hm, I kind of question whether this bit of information is suitable to be displayed in our snippets. It seems too prominent. But maybe I need more context. 

Some questions:
 - How many devices do most of our users have synced?
 - Why is this ability to "hide" really that important to call out? After all, long-press is common amongst our Panels
 - We already have a snippet overload situation, it might be best to be more conservative when adding another.

From a UX point of view, the snippet isn't really used (anywhere else) to show "tips" on a per Panel basis. Rather, we display "tips" in the empty state of Panels so that's seems more fitting. OR maybe we need to design a non-empty state way to display tips in our Panels.

Comment 8

[Nick Alexander :nalexander [he/him]]

Sorry, this all is in the wrong bug because I follow links from HG and RB and I didn't make the review request correctly.  I'm going to manually move comments and request over.

Comment 10

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 2: Copy omni.ja from dist/fennec manually. r=me

Comment 11

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Fix

Comment 12

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1112673 - Part 1: Refactor shared footer functionality into RemoteTabsBaseFragment. r=margaret

Comment 13

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1112673 - Part 2: Add hidden devices hint to Remote Tabs panel footer. r=margaret

Comment 14

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 4: Move some build steps earlier. r=gps

Comment 15

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 5: Use pre-built geckolibs dependency in debug configuration. r=me

Comment 16

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 6: Make Gradle's release configuration work like the old debug configuration. r=me

Comment 17

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 3: Make running Robocop from IntelliJ work. r=me

Comment 18

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1128362 - Part 1: Share generateCodeAndResources. r=me

Comment 19

[Nick Alexander :nalexander [he/him]]

We're not going to take this approach; we've gone in a different direction with Bug 1233882.