move zip template to a non-android-specific .gni file

move zip template to a non-android-specific .gni file

The zip template really isn't android-specific, let's move it to its
own .gni file which doesn't assert(is_android).

Committed: https://crrev.com/3dba3175c9f137742ddb10dbe9a10a5417186402
Cr-Commit-Position: refs/heads/master@{#349016}

[Mostyn Bramley-Moore, 2015-09-15 14:57:08]

mostynb@opera.com changed reviewers:
+ agrieve@chromium.org

[Mostyn Bramley-Moore, 2015-09-15 14:57:08]

@agrieve: PTAL at this minor adjustment.

[Mostyn Bramley-Moore, 2015-09-15 17:06:43]

The CQ bit was checked by mostynb@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-09-15 17:07:40]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1345733002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1345733002/1

[agrieve, 2015-09-15 17:20:16]

agrieve@chromium.org changed reviewers:
+ dpranke@google.com
- agrieve@chromium.org

[agrieve, 2015-09-15 17:20:33]

On 2015/09/15 17:07:40, commit-bot: I haz the power wrote:
> Dry run: CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1345733002/1
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1345733002/1

Punting to dpranke - as I'm not sure if there's a better place for
non-android-specific rules (e.g. move this and its script into //build?)

[commit-bot: I haz the power, 2015-09-15 18:14:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-15 18:14:58]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2015-09-15 20:18:29]

Is there a plan to use the zip template from any other files? If not, I'm not
sure that there's much of an advantage to moving it into its own file.

[Mostyn Bramley-Moore, 2015-09-15 20:26:06]

On 2015/09/15 20:18:29, Dirk Pranke wrote:
> Is there a plan to use the zip template from any other files? If not, I'm not
> sure that there's much of an advantage to moving it into its own file.

I'm using it from a downstream linux project.  It might be useful in chromium
itself in the future, but if not I don't think this patch does any harm in the
meantime.

[Dirk Pranke, 2015-09-15 20:48:24]

On 2015/09/15 20:26:06, Mostyn Bramley-Moore wrote:
> On 2015/09/15 20:18:29, Dirk Pranke wrote:
> > Is there a plan to use the zip template from any other files? If not, I'm
not
> > sure that there's much of an advantage to moving it into its own file.
> 
> I'm using it from a downstream linux project.  It might be useful in chromium
> itself in the future, but if not I don't think this patch does any harm in the
> meantime.

Okay, if there's another use for it.

We don't really have a place for generic templates yet, but build/config/android
is probably not right, so maybe move it to build/config ?

[Mostyn Bramley-Moore, 2015-09-15 21:07:48]

The CQ bit was checked by mostynb@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-09-15 21:08:45]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1345733002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1345733002/20001

[Mostyn Bramley-Moore, 2015-09-15 21:08:45]

On 2015/09/15 20:48:24, Dirk Pranke wrote:
> Okay, if there's another use for it.
> 
> We don't really have a place for generic templates yet, but
build/config/android
> is probably not right, so maybe move it to build/config ?

Done in patch set 2.

[Dirk Pranke, 2015-09-15 21:18:02]

dpranke@chromium.org changed reviewers:
+ dpranke@chromium.org

[Dirk Pranke, 2015-09-15 21:18:03]

lgtm

[commit-bot: I haz the power, 2015-09-15 22:07:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-09-15 22:08:00]

Dry run: This issue passed the CQ dry run.

[Mostyn Bramley-Moore, 2015-09-15 22:08:27]

The CQ bit was checked by mostynb@opera.com

[commit-bot: I haz the power, 2015-09-15 22:08:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1345733002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1345733002/20001

[commit-bot: I haz the power, 2015-09-15 22:16:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-15 22:16:44]

Patchset 2 (id:??) landed as
https://crrev.com/3dba3175c9f137742ddb10dbe9a10a5417186402
Cr-Commit-Position: refs/heads/master@{#349016}

[commit-bot: I haz the power, 2015-09-23 12:50:36]

Patchset 2 (id:??) landed as
https://crrev.com/3dba3175c9f137742ddb10dbe9a10a5417186402
Cr-Commit-Position: refs/heads/master@{#349016}