Skip to content

Commit

Permalink
build: fix GN build for sqlite and nghttp2
Browse files Browse the repository at this point in the history
PR-URL: nodejs#55529
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Cheng Zhao <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
  • Loading branch information
codebytere authored and louwers committed Nov 2, 2024
1 parent 23c99a1 commit cb20aa5
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 1 deletion.
3 changes: 3 additions & 0 deletions deps/nghttp2/unofficial.gni
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,9 @@ template("nghttp2_gn_build") {
if (is_clang || !is_win) {
cflags_c = [
"-Wno-implicit-fallthrough",
# Ref https://github.com/nghttp2/nghttp2/pull/2258
# This can be removed when the above PR is ingested.
"-Wno-extra-semi",
]
}
}
Expand Down
8 changes: 7 additions & 1 deletion deps/sqlite/unofficial.gni
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,14 @@ template("sqlite_gn_build") {
forward_variables_from(invoker, "*")
public_configs = [ ":sqlite_config" ]
sources = gypi_values.sqlite_sources
cflags_c = [
"-Wno-implicit-fallthrough",
"-Wno-unreachable-code-return",
"-Wno-unreachable-code-break",
"-Wno-unreachable-code",
]
if (is_win) {
cflags_c = [
cflags_c += [
"-Wno-sign-compare",
"-Wno-unused-but-set-variable",
"-Wno-unused-function",
Expand Down

0 comments on commit cb20aa5

Please sign in to comment.