Commit 22fa6bd
build: ibmi follow aix visibility
PR-URL: #60360
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Abdirahim Musse <abdirahim.musse@ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 2d48d17 commit 22fa6bd
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -50,7 +50,7 @@ | |||
| 50 | 50 | # including unnecessary internal symbols, which may lead to run-time fixups. | |
| 51 | 51 | # This is not done on AIX where symbols are exported by tools/create_expfile.sh | |
| 52 | 52 | # see https://github.com/nodejs/node/pull/56290#issuecomment-2582703109 | |
| 53 | - ['OS!="aix"', { | ||
| 53 | + ['OS!="aix" and OS!="os400"', { | ||
| 54 | 54 | 'defines': [ | |
| 55 | 55 | 'BUILDING_V8_SHARED', # Make V8_EXPORT visible. | |
| 56 | 56 | ] | |
@@ -65,7 +65,7 @@ | |||
| 65 | 65 | 'GCC_SYMBOLS_PRIVATE_EXTERN': 'YES', # -fvisibility=hidden | |
| 66 | 66 | 'GCC_INLINES_ARE_PRIVATE_EXTERN': 'YES' # -fvisibility-inlines-hidden | |
| 67 | 67 | }, | |
| 68 | - }, 'OS!="aix" and (OS!="win" or clang==1)', { | ||
| 68 | + }, '(OS!="aix" and OS!="os400") and (OS!="win" or clang==1)', { | ||
| 69 | 69 | 'cflags': [ | |
| 70 | 70 | '-fvisibility=hidden', | |
| 71 | 71 | '-fvisibility-inlines-hidden' | |
0 commit comments