Give the VM more heap in pub (on Mac/Linux).

Give the VM more heap in pub (on Mac/Linux).

BUG=https://code.google.com/p/dart/issues/detail?id=14730
R=ahe@google.com, nweiz@google.com

Committed: https://code.google.com/p/dart/source/detail?r=32803

[Bob Nystrom, 2014-02-15 00:56:28]

I'll do Windows in a separate patch.

[nweiz, 2014-02-15 01:00:51]

lgtm

https://codereview.chromium.org/168013003/diff/1/sdk/bin/pub
File sdk/bin/pub (right):

https://codereview.chromium.org/168013003/diff/1/sdk/bin/pub#newcode39
sdk/bin/pub:39: fi
I hope you've manually tested this thoroughly, because I don't really know bash
array stuff well enough to be confident that it works.

[Bob Nystrom, 2014-02-18 17:36:26]

Hey, Peter. Do you mind taking a quick look at this too? Nathan and I aren't
bash experts and I'd like to make sure I'm not doing something dumb.

Thanks!

https://codereview.chromium.org/168013003/diff/1/sdk/bin/pub
File sdk/bin/pub (right):

https://codereview.chromium.org/168013003/diff/1/sdk/bin/pub#newcode39
sdk/bin/pub:39: fi
On 2014/02/15 01:00:52, nweiz wrote:
> I hope you've manually tested this thoroughly, because I don't really know
bash
> array stuff well enough to be confident that it works.

It's copied verbatim from the dart2js script, modulo a name change.

[ahe, 2014-02-19 10:25:38]

LGTM, I see the batch file changes in a separate CL.

[Bob Nystrom, 2014-02-19 17:27:45]

On 2014/02/19 10:25:38, ahe wrote:
> LGTM, I see the batch file changes in a separate CL.

Great, thank you!

- bob

[Bob Nystrom, 2014-02-19 18:36:16]

Committed patchset #1 manually as r32803 (presubmit successful).