Skip to content

Commit f2d7b80

Browse files
danbevaddaleax
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 005e343 commit f2d7b80

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
@@ -984,7 +984,10 @@ def configure_openssl(o):
984984
]
985985
else:
986986
o['variables']['openssl_fips'] = ''
987-
987+
try:
988+
os.remove('config_fips.gypi')
989+
except OSError:
990+
pass
988991

989992
if options.without_ssl:
990993
def without_ssl_error(option):

0 commit comments

Comments
 (0)