Skip to content

Commit ee351cf

Browse files
danbevMayaLekova
danbev
authored and
MayaLekova
committed
build: make gyp user defined variables lowercase
I mistakenly introduced user defined variables using uppercase characters, reading the gyp documentation they state: "Predefined variables. By convention, these are named with CAPITAL_LETTERS. Predefined variables are set automatically by GYP" and also "By convention, user-defined variables are named with lowercase_letters." This commit renames the user defined variables to lowercase to follow the above mentioned convention. PR-URL: nodejs#16238 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 55dcdf5 commit ee351cf

File tree

4 files changed

+82
-82
lines changed

4 files changed

+82
-82
lines changed

‎common.gypi‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -44,29 +44,29 @@
4444

4545
'conditions': [
4646
['GENERATOR=="ninja"', {
47-
'OBJ_DIR': '<(PRODUCT_DIR)/obj',
48-
'V8_BASE': '<(PRODUCT_DIR)/obj/deps/v8/src/libv8_base.a',
47+
'obj_dir': '<(PRODUCT_DIR)/obj',
48+
'v8_base': '<(PRODUCT_DIR)/obj/deps/v8/src/libv8_base.a',
4949
}, {
50-
'OBJ_DIR%': '<(PRODUCT_DIR)/obj.target',
51-
'V8_BASE%': '<(PRODUCT_DIR)/obj.target/deps/v8/src/libv8_base.a',
50+
'obj_dir%': '<(PRODUCT_DIR)/obj.target',
51+
'v8_base%': '<(PRODUCT_DIR)/obj.target/deps/v8/src/libv8_base.a',
5252
}],
5353
['OS == "win"', {
5454
'os_posix': 0,
5555
'v8_postmortem_support%': 'false',
56-
'OBJ_DIR': '<(PRODUCT_DIR)/obj',
57-
'V8_BASE': '<(PRODUCT_DIR)/lib/v8_libbase.lib',
56+
'obj_dir': '<(PRODUCT_DIR)/obj',
57+
'v8_base': '<(PRODUCT_DIR)/lib/v8_libbase.lib',
5858
}, {
5959
'os_posix': 1,
6060
'v8_postmortem_support%': 'true',
6161
}],
6262
['OS== "mac"', {
63-
'OBJ_DIR%': '<(PRODUCT_DIR)/obj.target',
64-
'V8_BASE': '<(PRODUCT_DIR)/libv8_base.a',
63+
'obj_dir%': '<(PRODUCT_DIR)/obj.target',
64+
'v8_base': '<(PRODUCT_DIR)/libv8_base.a',
6565
}],
6666
['openssl_fips != ""', {
67-
'OPENSSL_PRODUCT': '<(STATIC_LIB_PREFIX)crypto<(STATIC_LIB_SUFFIX)',
67+
'openssl_product': '<(STATIC_LIB_PREFIX)crypto<(STATIC_LIB_SUFFIX)',
6868
}, {
69-
'OPENSSL_PRODUCT': '<(STATIC_LIB_PREFIX)openssl<(STATIC_LIB_SUFFIX)',
69+
'openssl_product': '<(STATIC_LIB_PREFIX)openssl<(STATIC_LIB_SUFFIX)',
7070
}],
7171
['OS=="mac"', {
7272
'clang%': 1,

‎node.gyp‎

Lines changed: 64 additions & 64 deletions
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@
234234
'conditions': [
235235
['OS in "linux freebsd openbsd solaris android"', {
236236
'ldflags': [
237-
'-Wl,--whole-archive,<(OBJ_DIR)/<(STATIC_LIB_PREFIX)'
237+
'-Wl,--whole-archive,<(obj_dir)/<(STATIC_LIB_PREFIX)'
238238
'<(node_core_target_name)<(STATIC_LIB_SUFFIX)',
239239
'-Wl,--no-whole-archive',
240240
],
@@ -772,10 +772,10 @@
772772
{
773773
'action_name': 'node_dtrace_provider_o',
774774
'inputs': [
775-
'<(OBJ_DIR)/<(node_lib_target_name)/src/node_dtrace.o',
775+
'<(obj_dir)/<(node_lib_target_name)/src/node_dtrace.o',
776776
],
777777
'outputs': [
778-
'<(OBJ_DIR)/<(node_lib_target_name)/src/node_dtrace_provider.o'
778+
'<(obj_dir)/<(node_lib_target_name)/src/node_dtrace_provider.o'
779779
],
780780
'action': [ 'dtrace', '-G', '-xnolibs', '-s', 'src/node_provider.d',
781781
'<@(_inputs)', '-o', '<@(_outputs)' ]
@@ -807,7 +807,7 @@
807807
{
808808
'action_name': 'node_dtrace_ustack_constants',
809809
'inputs': [
810-
'<(V8_BASE)'
810+
'<(v8_base)'
811811
],
812812
'outputs': [
813813
'<(SHARED_INTERMEDIATE_DIR)/v8constants.h'
@@ -825,7 +825,7 @@
825825
'<(SHARED_INTERMEDIATE_DIR)/v8constants.h'
826826
],
827827
'outputs': [
828-
'<(OBJ_DIR)/<(node_lib_target_name)/src/node_dtrace_ustack.o'
828+
'<(obj_dir)/<(node_lib_target_name)/src/node_dtrace_ustack.o'
829829
],
830830
'conditions': [
831831
[ 'target_arch=="ia32" or target_arch=="arm"', {
@@ -915,32 +915,32 @@
915915
],
916916

917917
'variables': {
918-
'OBJ_PATH': '<(OBJ_DIR)/<(node_lib_target_name)/src',
919-
'OBJ_GEN_PATH': '<(OBJ_DIR)/<(node_lib_target_name)/gen',
920-
'OBJ_TRACING_PATH': '<(OBJ_DIR)/<(node_lib_target_name)/src/tracing',
921-
'OBJ_SUFFIX': 'o',
922-
'OBJ_SEPARATOR': '/',
918+
'obj_path': '<(obj_dir)/<(node_lib_target_name)/src',
919+
'obj_gen_path': '<(obj_dir)/<(node_lib_target_name)/gen',
920+
'obj_tracing_path': '<(obj_dir)/<(node_lib_target_name)/src/tracing',
921+
'obj_suffix': 'o',
922+
'obj_separator': '/',
923923
'conditions': [
924924
['OS=="win"', {
925-
'OBJ_SUFFIX': 'obj',
925+
'obj_suffix': 'obj',
926926
}],
927927
['GENERATOR=="ninja"', {
928-
'OBJ_PATH': '<(OBJ_DIR)/src',
929-
'OBJ_GEN_PATH': '<(OBJ_DIR)/gen',
930-
'OBJ_TRACING_PATH': '<(OBJ_DIR)/src/tracing',
931-
'OBJ_SEPARATOR': '/<(node_lib_target_name).',
928+
'obj_path': '<(obj_dir)/src',
929+
'obj_gen_path': '<(obj_dir)/gen',
930+
'obj_tracing_path': '<(obj_dir)/src/tracing',
931+
'obj_separator': '/<(node_lib_target_name).',
932932
}, {
933933
'conditions': [
934934
['OS=="win"', {
935-
'OBJ_PATH': '<(OBJ_DIR)/<(node_lib_target_name)',
936-
'OBJ_GEN_PATH': '<(OBJ_DIR)/<(node_lib_target_name)',
937-
'OBJ_TRACING_PATH': '<(OBJ_DIR)/<(node_lib_target_name)',
935+
'obj_path': '<(obj_dir)/<(node_lib_target_name)',
936+
'obj_gen_path': '<(obj_dir)/<(node_lib_target_name)',
937+
'obj_tracing_path': '<(obj_dir)/<(node_lib_target_name)',
938938
}],
939939
['OS=="aix"', {
940-
'OBJ_PATH': '<(OBJ_DIR)/<(node_lib_target_name)/src',
941-
'OBJ_GEN_PATH': '<(OBJ_DIR)/<(node_lib_target_name)/gen',
942-
'OBJ_TRACING_PATH':
943-
'<(OBJ_DIR)/<(node_lib_target_name)/src/tracing',
940+
'obj_path': '<(obj_dir)/<(node_lib_target_name)/src',
941+
'obj_gen_path': '<(obj_dir)/<(node_lib_target_name)/gen',
942+
'obj_tracing_path':
943+
'<(obj_dir)/<(node_lib_target_name)/src/tracing',
944944
}],
945945
]}
946946
]
@@ -974,37 +974,37 @@
974974
],
975975

976976
'libraries': [
977-
'<(OBJ_PATH)<(OBJ_SEPARATOR)async_wrap.<(OBJ_SUFFIX)',
978-
'<(OBJ_PATH)<(OBJ_SEPARATOR)handle_wrap.<(OBJ_SUFFIX)',
979-
'<(OBJ_PATH)<(OBJ_SEPARATOR)env.<(OBJ_SUFFIX)',
980-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node.<(OBJ_SUFFIX)',
981-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_buffer.<(OBJ_SUFFIX)',
982-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_debug_options.<(OBJ_SUFFIX)',
983-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_i18n.<(OBJ_SUFFIX)',
984-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_perf.<(OBJ_SUFFIX)',
985-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_platform.<(OBJ_SUFFIX)',
986-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_url.<(OBJ_SUFFIX)',
987-
'<(OBJ_PATH)<(OBJ_SEPARATOR)util.<(OBJ_SUFFIX)',
988-
'<(OBJ_PATH)<(OBJ_SEPARATOR)string_bytes.<(OBJ_SUFFIX)',
989-
'<(OBJ_PATH)<(OBJ_SEPARATOR)string_search.<(OBJ_SUFFIX)',
990-
'<(OBJ_PATH)<(OBJ_SEPARATOR)stream_base.<(OBJ_SUFFIX)',
991-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_constants.<(OBJ_SUFFIX)',
992-
'<(OBJ_TRACING_PATH)<(OBJ_SEPARATOR)agent.<(OBJ_SUFFIX)',
993-
'<(OBJ_TRACING_PATH)<(OBJ_SEPARATOR)node_trace_buffer.<(OBJ_SUFFIX)',
994-
'<(OBJ_TRACING_PATH)<(OBJ_SEPARATOR)node_trace_writer.<(OBJ_SUFFIX)',
995-
'<(OBJ_TRACING_PATH)<(OBJ_SEPARATOR)trace_event.<(OBJ_SUFFIX)',
996-
'<(OBJ_GEN_PATH)<(OBJ_SEPARATOR)node_javascript.<(OBJ_SUFFIX)',
977+
'<(obj_path)<(obj_separator)async_wrap.<(obj_suffix)',
978+
'<(obj_path)<(obj_separator)handle_wrap.<(obj_suffix)',
979+
'<(obj_path)<(obj_separator)env.<(obj_suffix)',
980+
'<(obj_path)<(obj_separator)node.<(obj_suffix)',
981+
'<(obj_path)<(obj_separator)node_buffer.<(obj_suffix)',
982+
'<(obj_path)<(obj_separator)node_debug_options.<(obj_suffix)',
983+
'<(obj_path)<(obj_separator)node_i18n.<(obj_suffix)',
984+
'<(obj_path)<(obj_separator)node_perf.<(obj_suffix)',
985+
'<(obj_path)<(obj_separator)node_platform.<(obj_suffix)',
986+
'<(obj_path)<(obj_separator)node_url.<(obj_suffix)',
987+
'<(obj_path)<(obj_separator)util.<(obj_suffix)',
988+
'<(obj_path)<(obj_separator)string_bytes.<(obj_suffix)',
989+
'<(obj_path)<(obj_separator)string_search.<(obj_suffix)',
990+
'<(obj_path)<(obj_separator)stream_base.<(obj_suffix)',
991+
'<(obj_path)<(obj_separator)node_constants.<(obj_suffix)',
992+
'<(obj_tracing_path)<(obj_separator)agent.<(obj_suffix)',
993+
'<(obj_tracing_path)<(obj_separator)node_trace_buffer.<(obj_suffix)',
994+
'<(obj_tracing_path)<(obj_separator)node_trace_writer.<(obj_suffix)',
995+
'<(obj_tracing_path)<(obj_separator)trace_event.<(obj_suffix)',
996+
'<(obj_gen_path)<(obj_separator)node_javascript.<(obj_suffix)',
997997
],
998998

999999
'conditions': [
10001000
[ 'node_use_openssl=="true"', {
10011001
'conditions': [
10021002
['node_target_type!="static_library"', {
10031003
'libraries': [
1004-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_crypto.<(OBJ_SUFFIX)',
1005-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_crypto_bio.<(OBJ_SUFFIX)',
1006-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_crypto_clienthello.<(OBJ_SUFFIX)',
1007-
'<(OBJ_PATH)<(OBJ_SEPARATOR)tls_wrap.<(OBJ_SUFFIX)',
1004+
'<(obj_path)<(obj_separator)node_crypto.<(obj_suffix)',
1005+
'<(obj_path)<(obj_separator)node_crypto_bio.<(obj_suffix)',
1006+
'<(obj_path)<(obj_separator)node_crypto_clienthello.<(obj_suffix)',
1007+
'<(obj_path)<(obj_separator)tls_wrap.<(obj_suffix)',
10081008
],
10091009
}],
10101010
],
@@ -1015,9 +1015,9 @@
10151015
[ 'node_use_perfctr=="true"', {
10161016
'defines': [ 'HAVE_PERFCTR=1' ],
10171017
'libraries': [
1018-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_counters.<(OBJ_SUFFIX)',
1019-
'<(OBJ_PATH)<(OBJ_SEPARATOR)'
1020-
'node_win32_perfctr_provider.<(OBJ_SUFFIX)',
1018+
'<(obj_path)<(obj_separator)node_counters.<(obj_suffix)',
1019+
'<(obj_path)<(obj_separator)'
1020+
'node_win32_perfctr_provider.<(obj_suffix)',
10211021
],
10221022
}],
10231023
['v8_enable_inspector==1', {
@@ -1028,11 +1028,11 @@
10281028
'conditions': [
10291029
['node_target_type!="static_library"', {
10301030
'libraries': [
1031-
'<(OBJ_PATH)<(OBJ_SEPARATOR)inspector_agent.<(OBJ_SUFFIX)',
1032-
'<(OBJ_PATH)<(OBJ_SEPARATOR)inspector_io.<(OBJ_SUFFIX)',
1033-
'<(OBJ_PATH)<(OBJ_SEPARATOR)inspector_js_api.<(OBJ_SUFFIX)',
1034-
'<(OBJ_PATH)<(OBJ_SEPARATOR)inspector_socket.<(OBJ_SUFFIX)',
1035-
'<(OBJ_PATH)<(OBJ_SEPARATOR)inspector_socket_server.<(OBJ_SUFFIX)',
1031+
'<(obj_path)<(obj_separator)inspector_agent.<(obj_suffix)',
1032+
'<(obj_path)<(obj_separator)inspector_io.<(obj_suffix)',
1033+
'<(obj_path)<(obj_separator)inspector_js_api.<(obj_suffix)',
1034+
'<(obj_path)<(obj_separator)inspector_socket.<(obj_suffix)',
1035+
'<(obj_path)<(obj_separator)inspector_socket_server.<(obj_suffix)',
10361036
],
10371037
}],
10381038
],
@@ -1042,42 +1042,42 @@
10421042
}],
10431043
[ 'node_use_dtrace=="true" and node_target_type!="static_library"', {
10441044
'libraries': [
1045-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_dtrace.<(OBJ_SUFFIX)',
1045+
'<(obj_path)<(obj_separator)node_dtrace.<(obj_suffix)',
10461046
],
10471047
'conditions': [
10481048
['OS!="mac" and OS!="linux"', {
10491049
'libraries': [
1050-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_dtrace_provider.<(OBJ_SUFFIX)',
1051-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_dtrace_ustack.<(OBJ_SUFFIX)',
1050+
'<(obj_path)<(obj_separator)node_dtrace_provider.<(obj_suffix)',
1051+
'<(obj_path)<(obj_separator)node_dtrace_ustack.<(obj_suffix)',
10521052
]
10531053
}],
10541054
['OS=="linux"', {
10551055
'libraries': [
1056-
'<(SHARED_INTERMEDIATE_DIR)<(OBJ_SEPARATOR)'
1057-
'node_dtrace_provider.<(OBJ_SUFFIX)',
1056+
'<(SHARED_INTERMEDIATE_DIR)<(obj_separator)'
1057+
'node_dtrace_provider.<(obj_suffix)',
10581058
]
10591059
}],
10601060
],
10611061
}, {
10621062
'conditions': [
10631063
[ 'node_use_etw=="true" and OS=="win"', {
10641064
'libraries': [
1065-
'<(OBJ_PATH)<(OBJ_SEPARATOR)node_dtrace.<(OBJ_SUFFIX)',
1066-
'<(OBJ_PATH)<(OBJ_SEPARATOR)'
1067-
'node_win32_etw_provider.<(OBJ_SUFFIX)',
1065+
'<(obj_path)<(obj_separator)node_dtrace.<(obj_suffix)',
1066+
'<(obj_path)<(obj_separator)'
1067+
'node_win32_etw_provider.<(obj_suffix)',
10681068
],
10691069
}]
10701070
]
10711071
}],
10721072
[ 'OS=="win" and node_target_type!="static_library"', {
10731073
'libraries': [
1074-
'<(OBJ_PATH)<(OBJ_SEPARATOR)backtrace_win32.<(OBJ_SUFFIX)',
1074+
'<(obj_path)<(obj_separator)backtrace_win32.<(obj_suffix)',
10751075
],
10761076
}, {
10771077
'conditions': [
10781078
['node_target_type!="static_library"', {
10791079
'libraries': [
1080-
'<(OBJ_PATH)<(OBJ_SEPARATOR)backtrace_posix.<(OBJ_SUFFIX)',
1080+
'<(obj_path)<(obj_separator)backtrace_posix.<(obj_suffix)',
10811081
],
10821082
}],
10831083
],

‎node.gypi‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@
9999
[ 'force_load=="true"', {
100100
'xcode_settings': {
101101
'OTHER_LDFLAGS': [
102-
'-Wl,-force_load,<(V8_BASE)',
102+
'-Wl,-force_load,<(v8_base)',
103103
],
104104
},
105105
}],
@@ -154,7 +154,7 @@
154154
{
155155
'action_name': 'expfile',
156156
'inputs': [
157-
'<(OBJ_DIR)'
157+
'<(obj_dir)'
158158
],
159159
'outputs': [
160160
'<(PRODUCT_DIR)/node.exp'
@@ -186,13 +186,13 @@
186186
[ '(OS=="freebsd" or OS=="linux") and node_shared=="false"'
187187
' and coverage=="false" and force_load=="true"', {
188188
'ldflags': [ '-Wl,-z,noexecstack',
189-
'-Wl,--whole-archive <(V8_BASE)',
189+
'-Wl,--whole-archive <(v8_base)',
190190
'-Wl,--no-whole-archive' ]
191191
}],
192192
[ '(OS=="freebsd" or OS=="linux") and node_shared=="false"'
193193
' and coverage=="true" and force_load=="true"', {
194194
'ldflags': [ '-Wl,-z,noexecstack',
195-
'-Wl,--whole-archive <(V8_BASE)',
195+
'-Wl,--whole-archive <(v8_base)',
196196
'-Wl,--no-whole-archive',
197197
'--coverage',
198198
'-g',
@@ -237,15 +237,15 @@
237237
[ 'force_load=="true"', {
238238
'xcode_settings': {
239239
'OTHER_LDFLAGS': [
240-
'-Wl,-force_load,<(PRODUCT_DIR)/<(OPENSSL_PRODUCT)',
240+
'-Wl,-force_load,<(PRODUCT_DIR)/<(openssl_product)',
241241
],
242242
},
243243
'conditions': [
244244
['OS in "linux freebsd" and node_shared=="false"', {
245245
'ldflags': [
246246
'-Wl,--whole-archive,'
247-
'<(OBJ_DIR)/deps/openssl/'
248-
'<(OPENSSL_PRODUCT)',
247+
'<(obj_dir)/deps/openssl/'
248+
'<(openssl_product)',
249249
'-Wl,--no-whole-archive',
250250
],
251251
}],

‎test/addons/openssl-client-cert-engine/binding.gyp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
'include_dirs': ['../../../deps/openssl/openssl/include'],
1515
'link_settings': {
1616
'libraries': [
17-
'../../../../out/<(PRODUCT_DIR)/<(OPENSSL_PRODUCT)'
17+
'../../../../out/<(PRODUCT_DIR)/<(openssl_product)'
1818
]
1919
},
2020
}]

0 commit comments

Comments
 (0)