Merge ~osomon/ubuntu-app-platform:add-locales-all into ubuntu-app-platform:master
Proposed by
Olivier Tilloy
on 2016-11-28
| Status: | Merged |
|---|---|
| Approved by: | Christian Dywan on 2017-02-08 |
| Approved revision: | 8676bf0ce8fc24317aa71bd0d2f9710eb17e8bcf |
| Merged at revision: | 94ecd86d28e7b116d44db13150be0178afb85d6c |
| Proposed branch: | ~osomon/ubuntu-app-platform:add-locales-all |
| Merge into: | ubuntu-app-platform:master |
| Diff against target: |
13 lines (+2/-0) 1 file modified
snapcraft.yaml (+2/-0) |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Christian Dywan | 2016-11-28 | Approve on 2017-02-08 | |
|
Review via email:
|
|||
Commit Message
Add locales-all to the stage packages to work around LP: #1576282.
To post a comment you must log in.
| Olivier Tilloy (osomon) wrote : | # |
I agree. Note that this is only a (hopefully) temporary workaround to bug #1576282. This is not really about language packs, it’s about the lack of support for locales definitions in the core image. So really, this is a bug that needs to be fixed in snapd.
| Christian Dywan (kalikiana) wrote : | # |
Let's get this in as per the verdict of the discussion: the increase in size is negligible and so is the benefit of a separate language pack for that reason.
review:
Approve

This weighs in at 127M! I feel we need to have the discussion about language packs at this point, because effectively all the individual platform snaps would need that. Imagine having Qt, Qt 5.7, KDE and GTK+ with each of them containing this snap. That would defeat the point of saving space.