New issue
Advanced search Search tips

Issue 625791 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Sep 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Blocking:
issue 634246
issue 631440



Sign in to add a comment

Design and implement a new V8 developer workflow post GN-migration

Project Member Reported by machenb...@chromium.org, Jul 5 2016

Issue description

Summary: Design and implement a new V8 developer workflow post GN-migration (was: Design and implement a new developer workflow post GN-migration)
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jul 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/7f07809c2c91d7000a4f957df5e804611444def3

commit 7f07809c2c91d7000a4f957df5e804611444def3
Author: machenbach <machenbach@chromium.org>
Date: Thu Jul 21 13:20:15 2016

[tools] Build generator script

Helper script to generate gn arguments based on common
developer defaults or builder configurations.

BUG= chromium:625791 
NOTRY=true

Review-Url: https://codereview.chromium.org/2138693002
Cr-Commit-Position: refs/heads/master@{#37947}

[modify] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/infra/mb/mb_config.pyl
[add] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/tools/dev/v8gen.py

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/7f07809c2c91d7000a4f957df5e804611444def3

commit 7f07809c2c91d7000a4f957df5e804611444def3
Author: machenbach <machenbach@chromium.org>
Date: Thu Jul 21 13:20:15 2016

[tools] Build generator script

Helper script to generate gn arguments based on common
developer defaults or builder configurations.

BUG= chromium:625791 
NOTRY=true

Review-Url: https://codereview.chromium.org/2138693002
Cr-Commit-Position: refs/heads/master@{#37947}

[modify] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/infra/mb/mb_config.pyl
[add] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/tools/dev/v8gen.py

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/7f07809c2c91d7000a4f957df5e804611444def3

commit 7f07809c2c91d7000a4f957df5e804611444def3
Author: machenbach <machenbach@chromium.org>
Date: Thu Jul 21 13:20:15 2016

[tools] Build generator script

Helper script to generate gn arguments based on common
developer defaults or builder configurations.

BUG= chromium:625791 
NOTRY=true

Review-Url: https://codereview.chromium.org/2138693002
Cr-Commit-Position: refs/heads/master@{#37947}

[modify] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/infra/mb/mb_config.pyl
[add] https://crrev.com/7f07809c2c91d7000a4f957df5e804611444def3/tools/dev/v8gen.py

Blocking: 631440
Blocking: -474921
Blockedon: 635902
Blocking: 634246
Project Member

Comment 10 by bugdroid1@chromium.org, Sep 1 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/903172e4fc44a768fc660244aad3b6a1d28c60cf

commit 903172e4fc44a768fc660244aad3b6a1d28c60cf
Author: machenbach <machenbach@chromium.org>
Date: Thu Sep 01 18:05:35 2016

[mb] Add more developer defaults to mb config.

BUG= chromium:625791 

Review-Url: https://codereview.chromium.org/2304623002
Cr-Commit-Position: refs/heads/master@{#39100}

[modify] https://crrev.com/903172e4fc44a768fc660244aad3b6a1d28c60cf/infra/mb/mb_config.pyl

Project Member

Comment 11 by bugdroid1@chromium.org, Sep 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/6c51c7d3a1fab2e918d8ead4f8b76be523bef79e

commit 6c51c7d3a1fab2e918d8ead4f8b76be523bef79e
Author: machenbach <machenbach@chromium.org>
Date: Fri Sep 02 13:07:19 2016

[gn] Add generic subcommands and provide list command

This prepares using mb as a library and not as a sub
process. It is used like that in the new list command that
shows available configurations.

Also added more robust checks of user inputs, better hints
and better docu.

BUG= chromium:625791 
NOTRY=true

Review-Url: https://codereview.chromium.org/2300333002
Cr-Commit-Position: refs/heads/master@{#39136}

[modify] https://crrev.com/6c51c7d3a1fab2e918d8ead4f8b76be523bef79e/infra/mb/mb_config.pyl
[modify] https://crrev.com/6c51c7d3a1fab2e918d8ead4f8b76be523bef79e/tools/dev/v8gen.py

Status: Verified (was: Started)
Blockedon: -635902

Sign in to add a comment