Opened on 01/24/2019 at 11:21:48 AM

Closed on 02/11/2019 at 11:50:36 AM

Last modified on 03/18/2019 at 07:20:11 PM

#7238 closed change (fixed)

Fix multi-locale builds

Reported by: diegocarloslima Assignee: diegocarloslima
Priority: P2 Milestone: Adblock-Browser-for-Android-1.5.0
Module: Adblock-Browser-for-Android Keywords:
Cc: Blocked By:
Blocking: Platform: Adblock Browser for Android
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29989567/

Description (last modified by diegocarloslima)

Background

Currently, we use mozharness to generate our multi-locale builds. It seems that Mozilla doesn't use it to generate Android multi-locale builds for a long time. With the last merge, the multi-locale build now throws an error and fails when trying to build the project.

What to change

We should find an alternative way to generate multi-locale builds. As reference, I could find some articles/issues related to this issue:
Simple Firefox for Android build
Building and testing multi-locale Firefox for Android
Instructions for multi-locale builds
Bug 1362496 - Add chrome-% for Firefox

For this ticket, we should also consider skipping the toolkit/mozapps/installer/find-dupes.py script. We already had to add in the past some exceptions to mobile/android/installer/allowed-dupes.mn and it looks like we would need to add a considerable amount of new rules for this issue. Also, it seems that the find-dupes script can sometimes lead to Resource temporarily unavailable error, since it seems to try to open lots of files at the same time.

Attachments (0)

Change History (5)

comment:1 Changed on 01/24/2019 at 11:32:31 AM by diegocarloslima

  • Description modified (diff)

comment:2 Changed on 01/24/2019 at 04:23:56 PM by diegocarloslima

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:3 Changed on 02/11/2019 at 11:49:15 AM by abpbot

A commit referencing this issue has landed:
Issue 7238 - Fix multi-locale builds

comment:4 Changed on 02/11/2019 at 11:50:36 AM by diegocarloslima

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 03/18/2019 at 07:20:11 PM by aboegler

QA LGTM, changing language in Android OS settings and browser settings working as expected.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from diegocarloslima.
 
Note: See TracTickets for help on using tickets.