Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

build: fix gn build on windows #49

Merged
merged 1 commit into from
Aug 16, 2018
Merged

Conversation

nornagon
Copy link
Contributor

No description provided.

@nornagon nornagon force-pushed the fix-win-gn branch 10 times, most recently from a656949 to bf99a05 Compare August 15, 2018 05:50
@codebytere codebytere force-pushed the electron-node-v10.2.0 branch from c86e3d3 to c3c62e0 Compare August 15, 2018 17:56
@jkleinsc jkleinsc merged commit ece0a06 into electron-node-v10.2.0 Aug 16, 2018
@jkleinsc jkleinsc deleted the fix-win-gn branch August 16, 2018 20:53
deepak1556 pushed a commit that referenced this pull request Dec 10, 2018
Reverting this enables us to provide slower, but longer-lasting
replacements for the deprecated APIs.

Original commit message:

    Put back deleted V8_DEPRECATE_SOON methods

    This partially reverts
    https://chromium-review.googlesource.com/c/v8/v8/+/1177861,
    which deleted many V8_DEPRECATE_SOON methods rather than moving them to
    V8_DEPRECATED first. This puts them back and marks them V8_DEPRECATED.

    Note V8_DEPRECATED that were deleted in the same CL stay deleted.

    NOTRY=true
    NOPRESUBMIT=true
    NOTREECHECKS=true

    Bug: v8:7786, v8:8240
    Cq-Include-Trybots: luci.chromium.try:linux_chromium_rel_ng
    Change-Id: I00330036d957f98dab403465b25e30d8382aac22
    Reviewed-on: https://chromium-review.googlesource.com/1251422
    Commit-Queue: Dan Elphick <delphick@chromium.org>
    Reviewed-by: Yang Guo <yangguo@chromium.org>
    Reviewed-by: Michael Hablich <hablich@chromium.org>
    Cr-Commit-Position: refs/branch-heads/7.0@{#49}
    Cr-Branched-From: 6e2adae6f7f8e891cfd01f3280482b20590427a6-refs/heads/7.0.276@{#1}
    Cr-Branched-From: bc08a8624cbbea7a2d30071472bc73ad9544eadf-refs/heads/master@{#55424}

Refs: v8/v8@9136dd8
Refs: nodejs/node#23122

PR-URL: nodejs/node#23158
Reviewed-By: Yang Guo <yangguo@chromium.org>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants