1. 02 Jul, 2021 2 commits
    • Zhi An Ng's avatar
      Revert "[build] Separate out inspector as a shared library" · 50fb0a2f
      Zhi An Ng authored
      This reverts commit 92bfb63c.
      
      Reason for revert: Broke build https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux%20-%20shared/43249/overview
      
      Original change's description:
      > [build] Separate out inspector as a shared library
      >
      > This makes src/inspector:inspector into a v8_component producing a
      > shared library in component builds. To enable this, all of its exported
      > are now marked with V8_INSPECTOR_EXPORT.
      >
      > This also inverts the dependency between src/inspector:inspector and
      > :v8_base_without_compiler, and instead makes d8 and some tests depend on
      > inspector rather than getting it via v8.
      >
      > As a result, the no_check_targets exclusions list in .gn is reduced.
      >
      > Ultimately embedders like chromium should depend on :v8 and optionally
      > src/inspector:inspector, but to allow that transition to occur, this
      > renames :v8 to :v8_lib and introduces a new :v8 which depends on v8 and
      > inspector. Once all embedders have changed to reflect the new structure,
      > this part can be reverted.
      >
      > Bug: v8:11917
      > Change-Id: Ia8b15f07fb15acc5e1f111b1a80248def4285fd0
      > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2999088
      > Reviewed-by: Clemens Backes <clemensb@chromium.org>
      > Reviewed-by: Michael Achenbach <machenbach@chromium.org>
      > Reviewed-by: Yang Guo <yangguo@chromium.org>
      > Commit-Queue: Dan Elphick <delphick@chromium.org>
      > Cr-Commit-Position: refs/heads/master@{#75532}
      
      Bug: v8:11917
      Change-Id: I0ed27ed95211d13b8b3438a8c0a42d577806c475
      No-Presubmit: true
      No-Tree-Checks: true
      No-Try: true
      Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3003452
      Auto-Submit: Zhi An Ng <zhin@chromium.org>
      Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
      Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
      Cr-Commit-Position: refs/heads/master@{#75533}
      50fb0a2f
    • Dan Elphick's avatar
      [build] Separate out inspector as a shared library · 92bfb63c
      Dan Elphick authored
      This makes src/inspector:inspector into a v8_component producing a
      shared library in component builds. To enable this, all of its exported
      are now marked with V8_INSPECTOR_EXPORT.
      
      This also inverts the dependency between src/inspector:inspector and
      :v8_base_without_compiler, and instead makes d8 and some tests depend on
      inspector rather than getting it via v8.
      
      As a result, the no_check_targets exclusions list in .gn is reduced.
      
      Ultimately embedders like chromium should depend on :v8 and optionally
      src/inspector:inspector, but to allow that transition to occur, this
      renames :v8 to :v8_lib and introduces a new :v8 which depends on v8 and
      inspector. Once all embedders have changed to reflect the new structure,
      this part can be reverted.
      
      Bug: v8:11917
      Change-Id: Ia8b15f07fb15acc5e1f111b1a80248def4285fd0
      Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2999088Reviewed-by: 's avatarClemens Backes <clemensb@chromium.org>
      Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
      Reviewed-by: 's avatarYang Guo <yangguo@chromium.org>
      Commit-Queue: Dan Elphick <delphick@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#75532}
      92bfb63c
  2. 11 May, 2021 1 commit
  3. 30 Apr, 2021 3 commits
  4. 29 Apr, 2021 1 commit
    • Dan Elphick's avatar
      Reland "[build] Enable gn check for most of v8" · 55ae1a52
      Dan Elphick authored
      This is a reland of dbd5ae52
      
      Less common configs were fixed in other CLs.
      
      Original change's description:
      > [build] Enable gn check for most of v8
      >
      > Currently there are 32 gn check failures across v8 (not including
      > third_party). These are due to issues in cppgc and inspector, which are
      > not trivially resolveable, so for now 4 targets (+ third_party) are
      > being excluded from the checks.
      >
      > Bug: v8:7330
      > Change-Id: I506abe8e452c26239262499fc1fe2a6347a4c84b
      > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2851886
      > Reviewed-by: Michael Achenbach <machenbach@chromium.org>
      > Commit-Queue: Dan Elphick <delphick@chromium.org>
      > Cr-Commit-Position: refs/heads/master@{#74255}
      
      Bug: v8:7330
      Change-Id: I82047cfa7f8213312158123885808c2a4ad02160
      Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2859842
      Commit-Queue: Dan Elphick <delphick@chromium.org>
      Commit-Queue: Michael Achenbach <machenbach@chromium.org>
      Auto-Submit: Dan Elphick <delphick@chromium.org>
      Reviewed-by: 's avatarMichael Achenbach <machenbach@chromium.org>
      Cr-Commit-Position: refs/heads/master@{#74278}
      55ae1a52
  5. 28 Apr, 2021 2 commits
  6. 29 Jul, 2019 1 commit
  7. 13 May, 2019 1 commit
  8. 28 Mar, 2018 1 commit
  9. 12 May, 2017 1 commit
  10. 13 Mar, 2017 1 commit
  11. 29 Nov, 2016 1 commit
  12. 14 Sep, 2016 1 commit
  13. 27 Jul, 2016 1 commit
  14. 28 Jun, 2016 1 commit
  15. 07 Jun, 2016 1 commit
  16. 06 Jun, 2016 1 commit
    • machenbach's avatar
      [gn] Port test262 archiving to gn · 0d65554c
      machenbach authored
      This calls the action that archives test262 in gn. In gn
      we can't specify an action output outside the product
      directory. This works around it with an extra action
      stamp file in the product directory, while the archive
      remains in the test directory.
      
      We don't want to generate the archive in the product
      directory, as some legacy archiving scripts might include
      it and it's too large. It should only be included in the
      swarming tasks that are going to use it for testing.
      
      BUG=chromium:474921
      
      Review-Url: https://codereview.chromium.org/2034713005
      Cr-Commit-Position: refs/heads/master@{#36731}
      0d65554c
  17. 18 May, 2016 1 commit
  18. 02 May, 2016 1 commit