1189459 - Allow generating geckoview AAR files when MOZ_APP_NAME is not 'fennec'

Status: RESOLVED FIXED

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

Description

[Nick Alexander :nalexander [he/him]]

There are a few places in https://dxr.mozilla.org/mozilla-central/source/python/mozbuild/mozbuild/action/package_geckolibs_aar.py where we hard-code 'fennec'.  It's not a big deal, but it does mean that fabrice's patch to add a second Fennec-like app doesn't automatically create functional geckoview and geckolibs AAR files.  Might as well fix it.

Comment 1

[Nick Alexander :nalexander [he/him]]

Attachment: MozReview Request: Bug 1189459 - Allow generating geckoview AAR files when MOZ_APP_NAME is not 'fennec'. r?gps

Bug 1189459 - Allow generating geckoview AAR files when MOZ_APP_NAME is not 'fennec'. r?gps

This just allows a little versatility for consumers such as b2gdroid,
which are Fennec-like but don't have MOZ_APP_NAME=fennec.

I elected to pass appname as a parameter rather than modify the
existing distdir because I expect to want to differentiate, in some
way, the output AAR files based on the underlying name.  That is, in
future we might generate geckoview-fennec-VERSION.aar and
geckolibs-b2gdroid-VERSION.aar, or stuff the name into the Ivy version
information, or...

This also fixes a typo in one of the JarFinder instantiations.

Comment 2

[Nick Alexander :nalexander [he/him]]

gps: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b892ecb65f3 successfully builds geckoview AAR for b2gdroid, so I'm pretty confident this is doing good things.

Comment 3

[Gregory Szorc [:gps]]

Comment on attachment 8641224 [details]
MozReview Request: Bug 1189459 - Allow generating geckoview AAR files when MOZ_APP_NAME is not 'fennec'. r?gps

https://reviewboard.mozilla.org/r/14447/#review13211

Ship It!

Comment 4

[Nick Alexander :nalexander [he/him]]

url:        https://hg.mozilla.org/integration/fx-team/rev/fffe3ac33207713a9d71fcddadeeb938e6221a2a
changeset:  fffe3ac33207713a9d71fcddadeeb938e6221a2a
user:       Nick Alexander <[e-mail]>
date:       Thu Jul 30 11:50:44 2015 -0700
description:
Bug 1189459 - Allow generating geckoview AAR files when MOZ_APP_NAME is not 'fennec'. r=gps

This just allows a little versatility for consumers such as b2gdroid,
which are Fennec-like but don't have MOZ_APP_NAME=fennec.

I elected to pass appname as a parameter rather than modify the
existing distdir because I expect to want to differentiate, in some
way, the output AAR files based on the underlying name.  That is, in
future we might generate geckoview-fennec-VERSION.aar and
geckolibs-b2gdroid-VERSION.aar, or stuff the name into the Ivy version
information, or...

This also fixes a typo in one of the JarFinder instantiations.

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/fffe3ac33207