From e17656f9c66c7ead389cbebfcaba67fc206ed966 Mon Sep 17 00:00:00 2001 From: RafaelGSS Date: Fri, 15 Jul 2022 15:54:15 -0300 Subject: [PATCH] doc: mention Win 32-bit openssl build issue --- doc/contributing/maintaining-openssl.md | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/doc/contributing/maintaining-openssl.md b/doc/contributing/maintaining-openssl.md index 526bbc50354634..d8e93b89a48f10 100644 --- a/doc/contributing/maintaining-openssl.md +++ b/doc/contributing/maintaining-openssl.md @@ -97,6 +97,16 @@ Use `make` to regenerate all platform dependent files in % make -C deps/openssl/config ``` +**Note**: If the 32-bit Windows is failing to compile run this workflow instead: + +```console +% make -C deps/openssl/config clean +# Edit deps/openssl/openssl/crypto/perlasm/x86asm.pl changing +# #ifdef to %ifdef to make it compatible to nasm on 32-bit Windows. +# See: https://github.com/nodejs/node/pull/43603#issuecomment-1170670844 +# Reference: https://github.com/openssl/openssl/issues/18459 +``` + ## 3. Check diffs Check diffs to ensure updates are right. Even if there are no updates in openssl