Closed Bug 1881215 Opened 2 years ago Closed 1 year ago

rename mobile/android/focus-android to just mobile/android/focus (post move to central)

Categories

(Focus :: General, task, P3)

All
Android
task

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: jcristau, Assigned: adhingra)

References

(Blocks 1 open bug)

Details

Attachments

(1 obsolete file)

It feels weird/unnecessary to have that "focus-android" path in an android directory.

Product: Fenix → Focus
Severity: -- → N/A
Priority: -- → P3
No longer blocks: monorepo-cleanup
Severity: N/A → S4

I believe the blocking meta bug was removed in error.

Assignee: nobody → adhingra
Status: NEW → ASSIGNED

I would really suggest not to do this.

It might seem like a simple change, but I don't think the downstream consequences are easy to identify.

  • There is RelEng automation used to import strings from the android-l10n repository.
  • There is automation in GitHub to expose the strings from the mozilla-firefox/firefox repository into android-l10n.
  • There is Pontoon and other automation expecting to find strings in the current path (example for Italian, notice the URL).

And there is probably more relying on the existing path.

If you think this work is required, this might be large enough to require planning and time on my team's roadmap.
In any case, it needs investigation before any change lands, and that's not something I have bandwidth to do in the coming weeks.

Attachment #9494504 - Attachment description: WIP: Bug 1881215 - Rename mobile/android/focus-android to just mobile/android/focus → Bug 1881215 - Rename mobile/android/focus-android to just mobile/android/focus
Attachment #9494504 - Attachment is obsolete: true

Based on comment 3, let's close this. Re-open if you feels strongly about it.

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: