roll gyp 1501:1507

roll gyp 1501:1507

r1503 Ensure that all entries in .sln files are sorted.
r1504 Adding a test to cover XCode UID collisions.
r1505 Allow library entries of the form '-lfoo' on MSVS.
r1506 Correct dump_dependency_json on windows.
r1507 make: drop all system tests

R=thakis@chromium.org
BUG=


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=158862

[Torne, 2012-09-26 14:20:37]

Running trys now to see if anything explodes.

[Nico, 2012-09-26 14:50:45]

You're missing r1502 in the CL description.

On Wed, Sep 26, 2012 at 11:20 PM,  <torne@chromium.org> wrote:
> Reviewers: Nico,
>
> Message:
> Running trys now to see if anything explodes.
>
> Description:
> roll gyp 1501:1507
>
> r1503 Ensure that all entries in .sln files are sorted.
> r1504 Adding a test to cover XCode UID collisions.
> r1505 Allow library entries of the form '-lfoo' on MSVS.
> r1506 Correct dump_dependency_json on windows.
> r1507 make: drop all system tests
>
> R=thakis@chromium.org
> BUG=
>
>
> Please review this at https://codereview.chromium.org/10987051/
>
> SVN Base: svn://svn.chromium.org/chrome/trunk/src
>
> Affected files:
>   M DEPS
>
>
> Index: DEPS
> diff --git a/DEPS b/DEPS
> index
>
f4a403c994e323e09d3feac005136666ed0c644a..ea2e614f90c6cb6cedd7198cd8dc4f6fba716b18
> 100644
> --- a/DEPS
> +++ b/DEPS
> @@ -103,7 +103,7 @@ deps = {
>      (Var("googlecode_url") % "grit-i18n") + "/trunk@67",
>
>    "src/tools/gyp":
> -    (Var("googlecode_url") % "gyp") + "/trunk@1501",
> +    (Var("googlecode_url") % "gyp") + "/trunk@1507",
>
>    "src/v8":
>      (Var("googlecode_url") % "v8") + "/trunk@" + Var("v8_revision"),
>
>

[Torne, 2012-09-26 14:53:40]

There isn't an r1502 in /trunk, so it didn't come up when I did svn log. Looks
like it was a commit to the project wiki.

On 2012/09/26 14:50:45, Nico wrote:
> You're missing r1502 in the CL description.
> 
> On Wed, Sep 26, 2012 at 11:20 PM,  <mailto:torne@chromium.org> wrote:
> > Reviewers: Nico,
> >
> > Message:
> > Running trys now to see if anything explodes.
> >
> > Description:
> > roll gyp 1501:1507
> >
> > r1503 Ensure that all entries in .sln files are sorted.
> > r1504 Adding a test to cover XCode UID collisions.
> > r1505 Allow library entries of the form '-lfoo' on MSVS.
> > r1506 Correct dump_dependency_json on windows.
> > r1507 make: drop all system tests
> >
> > mailto:R=thakis@chromium.org
> > BUG=
> >
> >
> > Please review this at https://codereview.chromium.org/10987051/
> >
> > SVN Base: svn://svn.chromium.org/chrome/trunk/src
> >
> > Affected files:
> >   M DEPS
> >
> >
> > Index: DEPS
> > diff --git a/DEPS b/DEPS
> > index
> >
>
f4a403c994e323e09d3feac005136666ed0c644a..ea2e614f90c6cb6cedd7198cd8dc4f6fba716b18
> > 100644
> > --- a/DEPS
> > +++ b/DEPS
> > @@ -103,7 +103,7 @@ deps = {
> >      (Var("googlecode_url") % "grit-i18n") + "/trunk@67",
> >
> >    "src/tools/gyp":
> > -    (Var("googlecode_url") % "gyp") + "/trunk@1501",
> > +    (Var("googlecode_url") % "gyp") + "/trunk@1507",
> >
> >    "src/v8":
> >      (Var("googlecode_url") % "v8") + "/trunk@" + Var("v8_revision"),
> >
> >

[Nico, 2012-09-26 15:02:05]

lgtm

On Wed, Sep 26, 2012 at 11:53 PM,  <torne@chromium.org> wrote:
> There isn't an r1502 in /trunk, so it didn't come up when I did svn log.
> Looks
> like it was a commit to the project wiki.
>
>
> On 2012/09/26 14:50:45, Nico wrote:
>>
>> You're missing r1502 in the CL description.
>
>
>> On Wed, Sep 26, 2012 at 11:20 PM,  <mailto:torne@chromium.org> wrote:
>> > Reviewers: Nico,
>> >
>> > Message:
>> > Running trys now to see if anything explodes.
>> >
>> > Description:
>> > roll gyp 1501:1507
>> >
>> > r1503 Ensure that all entries in .sln files are sorted.
>> > r1504 Adding a test to cover XCode UID collisions.
>> > r1505 Allow library entries of the form '-lfoo' on MSVS.
>> > r1506 Correct dump_dependency_json on windows.
>> > r1507 make: drop all system tests
>> >
>> > mailto:R=thakis@chromium.org
>>
>> > BUG=
>> >
>> >
>> > Please review this at https://codereview.chromium.org/10987051/
>> >
>> > SVN Base: svn://svn.chromium.org/chrome/trunk/src
>> >
>> > Affected files:
>> >   M DEPS
>> >
>> >
>> > Index: DEPS
>> > diff --git a/DEPS b/DEPS
>> > index
>> >
>
>
>
f4a403c994e323e09d3feac005136666ed0c644a..ea2e614f90c6cb6cedd7198cd8dc4f6fba716b18
>>
>> > 100644
>> > --- a/DEPS
>> > +++ b/DEPS
>> > @@ -103,7 +103,7 @@ deps = {
>> >      (Var("googlecode_url") % "grit-i18n") + "/trunk@67",
>> >
>> >    "src/tools/gyp":
>> > -    (Var("googlecode_url") % "gyp") + "/trunk@1501",
>> > +    (Var("googlecode_url") % "gyp") + "/trunk@1507",
>> >
>> >    "src/v8":
>> >      (Var("googlecode_url") % "v8") + "/trunk@" + Var("v8_revision"),
>> >
>> >
>
>
>
>
> https://codereview.chromium.org/10987051/

[commit-bot: I haz the power, 2012-09-26 16:04:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/torne@chromium.org/10987051/1

[commit-bot: I haz the power, 2012-09-26 19:59:03]

Change committed as 158862