Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upbuild: add -Werror=undefined-inline to clang builds #23961
Conversation
This comment has been minimized.
This comment has been minimized.
nodejs-github-bot
added
the
build
label
Oct 29, 2018
refack
requested review from
danbev,
bnoordhuis and
cjihrig
Oct 29, 2018
refack
added
C++
meta
lib / src
labels
Oct 29, 2018
refack
requested a review
from
targos
Oct 29, 2018
This comment has been minimized.
This comment has been minimized.
refack
force-pushed the
refack:Werror-undefined-inline
branch
from
fa54260
to
ad46b32
Oct 29, 2018
refack
changed the title
build: add -Werror="undefined-inline" to clang builds
build: add -Werror=undefined-inline to clang builds
Oct 29, 2018
refack
added
the
blocked
label
Oct 29, 2018
This comment has been minimized.
This comment has been minimized.
|
I guess we should block on #23954? |
This comment has been minimized.
This comment has been minimized.
|
Fails as expected on macOS (with clang)
Waiting for #23954 |
This comment has been minimized.
This comment has been minimized.
|
Why didn't it fail on FreeBSD? |
targos
approved these changes
Oct 29, 2018
This comment has been minimized.
This comment has been minimized.
Probably because the |
cjihrig
approved these changes
Oct 29, 2018
richardlau
approved these changes
Oct 29, 2018
refack
self-assigned this
Oct 30, 2018
joyeecheung
approved these changes
Oct 30, 2018
refack
removed
the
blocked
label
Oct 30, 2018
This comment has been minimized.
This comment has been minimized.
|
#23954 landed so unblocked. |
This comment has been minimized.
This comment has been minimized.
|
@refack Did you forget to rebase? It's still failing with the same build error. |
refack
force-pushed the
refack:Werror-undefined-inline
branch
from
ad46b32
to
9304541
Oct 31, 2018
This comment has been minimized.
This comment has been minimized.
pushed without force, and didn't look at results... new resume: https://ci.nodejs.org/job/node-test-pull-request/18260/ |
refack
force-pushed the
refack:Werror-undefined-inline
branch
from
9304541
to
c515e5c
Oct 31, 2018
This comment has been minimized.
This comment has been minimized.
|
Two more hours, so lets CI again: https://ci.nodejs.org/job/node-test-pull-request/18262/ |
refack
removed request for
bnoordhuis and
danbev
Oct 31, 2018
refack
merged commit c515e5c
into
nodejs:master
Oct 31, 2018
13 checks passed
refack
deleted the
refack:Werror-undefined-inline
branch
Oct 31, 2018
targos
added a commit
that referenced
this pull request
Nov 2, 2018
refack
removed their assignment
Nov 6, 2018
This was referenced Nov 15, 2018
MylesBorins
added
dont-land-on-v8.x
land-on-v10.x
labels
Nov 27, 2018
This comment has been minimized.
This comment has been minimized.
|
opted to land on 10.x but not 8.x |
refack commentedOct 29, 2018
•
edited
Hopefully the start of a trend to move more cases to
-WerrorRefs: #23954
Refs: #23910
Refs: #23880
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes