Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29947559: Issue 7114 - Remove az and untranslated resources from Windows packages (Closed)

Created:
Nov. 20, 2018, 1:50 a.m. by Sebastian Noack
Modified:
Nov. 21, 2018, 6:38 p.m.
Reviewers:
tlucas, geo
Visibility:
Public.

Description

Issue 7114 - Remove az and untranslated resources from Windows packages

Patch Set 1 #

Patch Set 2 : Different approach #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -4 lines) Patch
M packagerEdge.py View 1 1 chunk +14 lines, -0 lines 0 comments Download
M tests/expecteddata/AppxManifest_edge_development_build.xml View 1 1 chunk +0 lines, -2 lines 0 comments Download
M tests/expecteddata/AppxManifest_edge_release_build.xml View 1 1 chunk +0 lines, -2 lines 0 comments Download

Messages

Total messages: 5
Sebastian Noack
Geo, can you try the following? 1. Apply the patch to adblockpluschrome/buildtools 2. Run "build.py ...
Nov. 20, 2018, 1:55 a.m. (2018-11-20 01:55:46 UTC) #1
Sebastian Noack
On 2018/11/20 01:55:46, Sebastian Noack wrote: > 2. Run "build.py build -t edge" For got ...
Nov. 20, 2018, 3:07 a.m. (2018-11-20 03:07:42 UTC) #2
geo
On 2018/11/20 03:07:42, Sebastian Noack wrote: > On 2018/11/20 01:55:46, Sebastian Noack wrote: > > ...
Nov. 20, 2018, 7:50 p.m. (2018-11-20 19:50:41 UTC) #3
Sebastian Noack
As per the discussion on IRC, it seems that the package doesn't install either if ...
Nov. 20, 2018, 9:35 p.m. (2018-11-20 21:35:53 UTC) #4
geo
Nov. 21, 2018, 6:29 p.m. (2018-11-21 18:29:02 UTC) #5
On 2018/11/20 21:35:53, Sebastian Noack wrote:
> As per the discussion on IRC, it seems that the package doesn't install either
> if it has any localized resources for the 'az' locale (renaming it to az-latn
> works but then the translation isn't picked up either, so we can as well just
> remove it), or if the manifest lists the 'uz' locale without providing a
> translation for the description.
> 
> In order to keep things simple and as consistent as possible, I uploaded a new
> patch that removes the "Resources/az" folder, and then removes every
<Resource>
> element from appxmanifest.xml that doesn't have a corresponding subfolder
under
> "Resources".
> 
> Can you test again with the new patch? Thanks.

LGTM.

Powered by Google App Engine
This is Rietveld