Skip to content

Commit a04d4ea

Browse files
danbevMylesBorins
authored andcommitted
build: clean up config_fips.gypi
Currently when configuring the project using --openssl-fips a gyp include file name config_fips.gypi will be created. If the project is later configured but without the --openssl-fips flag an error will occur. For example: $ ./configure --openssl-fips=bogus $ ./configure && make -j8 ... /node/deps/openssl/fips/fipsld: line 8: /bin/fipsld: No such file or directory Error 127 This commit suggests removing the generate config_fips.gypi when the --openssl-fips flag is not give on the command line. PR-URL: #13837 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 8ba7843 commit a04d4ea

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

configure

+4-1
Original file line numberDiff line numberDiff line change
@@ -973,7 +973,10 @@ def configure_openssl(o):
973973
]
974974
else:
975975
o['variables']['openssl_fips'] = ''
976-
976+
try:
977+
os.remove('config_fips.gypi')
978+
except OSError:
979+
pass
977980

978981
if options.without_ssl:
979982
def without_ssl_error(option):

0 commit comments

Comments
 (0)