mirror of
https://github.com/nodejs/node.git
synced 2025-08-15 13:48:44 +02:00
deps: use proper C standard when building libuv
Upstream libuv commits:bb706f5fe7
018363a163
libuv was updated to 1.51.0 in0315283cbd
. v1.51.0 was the release updating from c90 to c11. The standard was back then also updated from c89 to c90. and C89 was never an official standard to begin with. So, I wonder how this managed to stay without breaking anyone's builds for this long. This atleast breaks the builds for Android using NDK r28b and r28a, as LLONG_MAX is not defined by the Clang compiler for older C standards. Fixes: ../../deps/uv/src/unix/linux.c:2331:36: error: use of undeclared identifier 'LLONG_MAX' 2331 | constraint->quota_per_period = LLONG_MAX; | ^ 1 error generated. PR-URL: https://github.com/nodejs/node/pull/58587 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
This commit is contained in:
parent
3c351c272f
commit
d79d550657
1 changed files with 2 additions and 2 deletions
4
deps/uv/uv.gyp
vendored
4
deps/uv/uv.gyp
vendored
|
@ -190,7 +190,7 @@
|
|||
'-Wno-unused-parameter',
|
||||
'-Wstrict-prototypes',
|
||||
],
|
||||
'OTHER_CFLAGS': [ '-g', '--std=gnu89' ],
|
||||
'OTHER_CFLAGS': [ '-g', '--std=gnu11' ],
|
||||
},
|
||||
'conditions': [
|
||||
[ 'OS=="win"', {
|
||||
|
@ -262,7 +262,7 @@
|
|||
'cflags': [
|
||||
'-fvisibility=hidden',
|
||||
'-g',
|
||||
'--std=gnu89',
|
||||
'--std=gnu11',
|
||||
'-Wall',
|
||||
'-Wextra',
|
||||
'-Wno-unused-parameter',
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue