Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(103)

Issue 29727633: Issue 6502 - Add Chinese build variant (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 8 months ago by diegocarloslima
Modified:
1 year, 7 months ago
Reviewers:
jens, anton
CC:
René Jeschke
Visibility:
Public.

Description

Issue 6502 - Add Chinese build variant

Patch Set 1 #

Total comments: 2

Patch Set 2 : Reverting changes in package name due to an error #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -4 lines) Patch
M adblockplussbrowser/AndroidManifest.xml View 1 1 chunk +1 line, -1 line 0 comments Download
M adblockplussbrowser/build.gradle View 2 chunks +20 lines, -3 lines 0 comments Download

Messages

Total messages: 5
diegocarloslima
1 year, 8 months ago (2018-03-19 23:37:59 UTC) #1
diegocarloslima
https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml File adblockplussbrowser/AndroidManifest.xml (right): https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml#newcode3 adblockplussbrowser/AndroidManifest.xml:3: package="${applicationId}" This is just to keep consistency. We shall ...
1 year, 8 months ago (2018-03-19 23:39:49 UTC) #2
anton
On 2018/03/19 23:39:49, diegocarloslima wrote: > https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml > File adblockplussbrowser/AndroidManifest.xml (right): > > https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml#newcode3 > ...
1 year, 8 months ago (2018-03-20 06:06:56 UTC) #3
diegocarloslima
https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml File adblockplussbrowser/AndroidManifest.xml (right): https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser/AndroidManifest.xml#newcode3 adblockplussbrowser/AndroidManifest.xml:3: package="${applicationId}" On 2018/03/19 23:39:49, diegocarloslima wrote: > This is ...
1 year, 8 months ago (2018-03-20 12:12:18 UTC) #4
jens
1 year, 7 months ago (2018-03-27 09:44:58 UTC) #5
On 2018/03/20 12:12:18, diegocarloslima wrote:
>
https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser...
> File adblockplussbrowser/AndroidManifest.xml (right):
> 
>
https://codereview.adblockplus.org/29727633/diff/29727634/adblockplussbrowser...
> adblockplussbrowser/AndroidManifest.xml:3: package="${applicationId}"
> On 2018/03/19 23:39:49, diegocarloslima wrote:
> > This is just to keep consistency. We shall remove those redundant fields in
> > manifest in a follow up issue
> 
> That change would produce a 'Cannot read packageName' error while performing
> Gradle sync, so I had to revert it. I will create a follow up issue to remove
> unused stuff for Gradle builds

LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 87257f5