Skip to content

[v12.x backport] deps: update to cjs-module-lexer@1.2.2 #39422

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
guybedford wants to merge 15 commits into from
Closed

[v12.x backport] deps: update to cjs-module-lexer@1.2.2 #39422

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
f186283
[v12.x] deps: update to cjs-module-lexer@1.1.1
guybedford Mar 30, 2021
daab721
build: label PRs with GitHub Action instead of nodejs-github-bot
phillipj Mar 6, 2021
be1fbc2
build: fix label-pr workflow
targos Apr 25, 2021
ca90848
deps: V8: cherry-pick 530080c44af2
May 3, 2021
f1f3949
deps: V8: backport 1b1eda0876aa
targos Jul 3, 2021
e60c91c
deps: V8: cherry-pick 92e6d3317082
targos Jul 3, 2021
1f5f768
deps: V8: cherry-pick e6f62a41f5ee
targos Jul 3, 2021
4a678dd
deps: V8: cherry-pick 7b3332844212
targos Jul 3, 2021
a223078
deps: V8: cherry-pick cc641f6be756
targos Jul 3, 2021
366db63
deps: V8: cherry-pick 7c182bd65f42
targos Jul 3, 2021
c683f6e
deps: V8: cherry-pick 0b3a4ecf7083
targos Jul 7, 2021
5627e37
deps: V8: backport 895949419186
targos Jul 7, 2021
61c274b
deps: V8: update build dependencies
targos Jul 3, 2021
da140d3
deps: update to cjs-module-lexer@1.2.1
guybedford Apr 28, 2021
295ec0d
deps: update to cjs-module-lexer@1.2.2
guybedford Jul 15, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
deps: V8: backport 895949419186 
Original commit message:

    Add -Wno-string-concatenation to test/cctest:cctest_sources

    v8/test/cctest/interpreter/test-bytecode-generator.cc contains lots of string arrays with intentional concatenation.

    Bug: chromium:1114873
    Change-Id: Ie9d35c3849b5b0a6d1d01b6ce21fb80a320d8736
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2366829
    Commit-Queue: Arthur Eubanks <aeubanks@google.com>
    Reviewed-by: Sathya Gunasekaran  <gsathya@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#69625}

Refs: v8/v8@8959494

PR-URL: #39245
Refs: nodejs/build#2696
Reviewed-By: Richard Lau <rlau@redhat.com>
  • Loading branch information
@targos
targos committed Jul 11, 2021
commit 5627e37375813e08bc2f2ac4d420451c29255ce8
2 changes: 1 addition & 1 deletion common.gypi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@

# Reset this number to 0 on major V8 upgrades.
# Increment by one for each non-official patch applied to deps/v8.
'v8_embedder_string': '-node.54',
'v8_embedder_string': '-node.55',

##### V8 defaults for Node.js #####

Expand Down
7 changes: 7 additions & 0 deletions deps/v8/test/cctest/BUILD.gn
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,12 @@ v8_header_set("cctest_headers") {
]
}

config("cctest_sources_config") {
if (is_clang) {
cflags = [ "-Wno-string-concatenation" ]
}
}

v8_source_set("cctest_sources") {
testonly = true

Expand Down Expand Up @@ -376,6 +382,7 @@ v8_source_set("cctest_sources") {
configs = [
"../..:external_config",
"../..:internal_config_base",
":cctest_sources_config",
]

public_deps = [
Expand Down