★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/23961Nouvelle recherche | Portail wanayoo
Skip to content
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.

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

build: add -Werror=undefined-inline to clang builds #23961

Merged
merged 1 commit into from Oct 31, 2018

Conversation

Projects
None yet
8 participants
@refack
Copy link
Member

refack commented Oct 29, 2018 •

Hopefully the start of a trend to move more cases to -Werror

Refs: #23954
Refs: #23910
Refs: #23880

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@refack refack requested review from danbev, bnoordhuis and cjihrig Oct 29, 2018

@refack refack requested a review from targos Oct 29, 2018

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 29, 2018 •

@refack refack force-pushed the refack:Werror-undefined-inline branch from fa54260 to ad46b32 Oct 29, 2018

@refack refack changed the title build: add -Werror="undefined-inline" to clang builds build: add -Werror=undefined-inline to clang builds Oct 29, 2018

@refack refack added the blocked label Oct 29, 2018

@targos

This comment has been minimized.

Copy link
Member

targos commented Oct 29, 2018

I guess we should block on #23954?

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 29, 2018

Fails as expected on macOS (with clang)

15:06:27   /usr/local/bin/ccache c++ -o /Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj.target/node_lib/src/inspector/worker_inspector.o ../src/inspector/worker_inspector.cc '-D_DARWIN_USE_64_BIT_INODE=1' '-DNODE_ARCH="x64"' '-DNODE_WANT_INTERNALS=1' '-DV8_DEPRECATION_WARNINGS=1' '-DNODE_OPENSSL_SYSTEM_CERT_PATH=""' '-DHAVE_INSPECTOR=1' '-DHAVE_DTRACE=1' '-D__POSIX__' '-DNODE_USE_V8_PLATFORM=1' '-DNODE_HAVE_I18N_SUPPORT=1' '-DNODE_HAVE_SMALL_ICU=1' '-DNODE_PLATFORM="darwin"' '-DHAVE_OPENSSL=1' '-DUCONFIG_NO_SERVICE=1' '-DU_ENABLE_DYLOAD=0' '-DU_STATIC_IMPLEMENTATION=1' '-DU_HAVE_STD_STRING=1' '-DUCONFIG_NO_BREAK_ITERATION=0' '-DHTTP_PARSER_STRICT=0' '-D_LARGEFILE_SOURCE' '-D_FILE_OFFSET_BITS=64' '-DNGHTTP2_STATICLIB' -I../src -I/Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj/gen -I/Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj/gen/include -I/Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj/gen/src -I../deps/v8/include -I../deps/icu-small/source/i18n -I../deps/icu-small/source/common -I../deps/zlib -I../deps/http_parser -I../deps/cares/include -I../deps/uv/include -I../deps/nghttp2/lib/includes -I../deps/openssl/openssl/include  -Os -gdwarf-2 -mmacosx-version-min=10.7 -arch x86_64 -Wall -Wendif-labels -W -Wno-unused-parameter -Wall -Wendif-labels -W -Wno-unused-parameter -Werror=undefined-inline -std=gnu++1y -stdlib=libc++ -fno-rtti -fno-exceptions -fno-strict-aliasing -MMD -MF /Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/.deps//Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj.target/node_lib/src/inspector/worker_inspector.o.d.raw   -c
15:06:28 In file included from ../src/inspector/worker_inspector.cc:3:
15:06:28 In file included from ../src/inspector/main_thread_interface.h:8:
15:06:28 In file included from ../src/env.h:27:
15:06:28 In file included from ../src/aliased_buffer.h:7:
15:06:28 ../src/util.h:65:11: error: inline function 'node::Calloc<unsigned char>' is not defined [-Werror,-Wundefined-inline]
15:06:28 inline T* Calloc(size_t n);
15:06:28           ^
15:06:28 ../src/aliased_buffer.h:41:15: note: used here
15:06:28     buffer_ = Calloc<NativeT>(count);
15:06:28               ^
15:06:28 1 error generated.
15:06:28 make[2]: *** [/Users/iojs/build/workspace/node-test-commit-osx/nodes/osx1011/out/Release/obj.target/node_lib/src/inspector/worker_inspector.o] Error 1

Waiting for #23954

@targos

This comment has been minimized.

Copy link
Member

targos commented Oct 29, 2018

Why didn't it fail on FreeBSD?

@targos

targos approved these changes Oct 29, 2018

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 29, 2018

Why didn't it fail on FreeBSD?

Probably because the clang GYP variable is not set. I'll fix that in a separate PR.

@refack refack self-assigned this Oct 30, 2018

@refack refack removed the blocked label Oct 30, 2018

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 30, 2018 •

@bnoordhuis

This comment has been minimized.

Copy link
Member

bnoordhuis commented Oct 31, 2018

@refack Did you forget to rebase? It's still failing with the same build error.

@refack refack force-pushed the refack:Werror-undefined-inline branch from ad46b32 to 9304541 Oct 31, 2018

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 31, 2018

Did you forget to rebase?

pushed without force, and didn't look at results...

new resume: https://ci.nodejs.org/job/node-test-pull-request/18260/

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

@refack refack force-pushed the refack:Werror-undefined-inline branch from 9304541 to c515e5c Oct 31, 2018

@refack

This comment has been minimized.

Copy link
Member Author

refack commented Oct 31, 2018

@refack refack removed request for bnoordhuis and danbev Oct 31, 2018

@refack refack merged commit c515e5c into nodejs:master Oct 31, 2018

13 checks passed

node-test-commit tests passed
Details
node-test-commit-aix tests passed
Details
node-test-commit-linuxone tests passed
Details
node-test-commit-osx tests passed
Details
node-test-commit-plinux tests passed
Details
node-test-commit-smartos tests passed
Details
node-test-commit-windows-fanned tests passed
Details
node-test-linter tests passed
Details
node-test-linux-linked-debug tests passed
Details
node-test-linux-linked-openssl110 tests passed
Details
node-test-linux-linked-shared tests passed
Details
node-test-linux-linked-withoutintl tests passed
Details
node-test-linux-linked-zlib tests passed
Details

@refack refack deleted the refack:Werror-undefined-inline branch Oct 31, 2018

targos added a commit that referenced this pull request Nov 2, 2018

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

@refack refack removed their assignment Nov 6, 2018

@BridgeAR BridgeAR referenced this pull request Nov 14, 2018

Merged

Release proposal: v11.2 #24350

@MylesBorins

This comment has been minimized.

Copy link
Member

MylesBorins commented Nov 27, 2018

opted to land on 10.x but not 8.x

MylesBorins added a commit that referenced this pull request Nov 27, 2018

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

@codebytere codebytere referenced this pull request Nov 27, 2018

Closed

v10.13.1 proposal #24675

rvagg added a commit that referenced this pull request Nov 28, 2018

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

MylesBorins added a commit that referenced this pull request Nov 29, 2018

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>

@codebytere codebytere referenced this pull request Nov 29, 2018

Merged

v10.14.2 proposal #24727

MylesBorins added a commit that referenced this pull request Dec 3, 2018

build: add -Werror=undefined-inline to clang builds
PR-URL: #23961
Refs: #23954
Refs: #23910
Refs: #23880
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.