Skip to content

Commit 932c83b

Browse files
fzipiCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent ef44728 commit 932c83b

File tree

5 files changed

+14
-14
lines changed

5 files changed

+14
-14
lines changed

.github/workflows/test-ci-windows.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ jobs:
235235
236236
# Check VC++ redistributables
237237
$vc142x64 = (Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\VisualStudio\14.0\VC\Runtimes\x64" -Name "Installed" -ErrorAction SilentlyContinue).Installed
238-
$vc142x86 = (Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\VisualStudio\14.0\VC\Runtimes\x86" -Name "Installed" -ErrorAction SilentlyContinue).Installed
238+
$vc142x86 = (Get-ItemProperty -Path "HKLM:\SOFTWARE\WOW6432Node\Microsoft\VisualStudio\14.0\VC\Runtimes\x86" -Name "Installed" -ErrorAction SilentlyContinue).Installed
239239
Write-Host "VC++ 2019 x64 installed: $vc142x64"
240240
Write-Host "VC++ 2019 x86 installed: $vc142x86"
241241

apache2/Makefile.win

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
LIBS = $(APACHE)\lib\libhttpd.lib \
1111
$(APACHE)\lib\libapr-1.lib \
1212
$(APACHE)\lib\libaprutil-1.lib \
13-
$(PCRE)\pcre.lib \
13+
$(PCRE)\pcre2-8.lib \
1414
$(CURL)\libcurl.lib \
1515
$(LIBXML2)\win32\bin.msvc\libxml2.lib \
1616
Ws2_32.lib \

iis/dependencies/build_pcre.bat

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ copy /y "%WORK_DIR%\pcre2\pcre2-8.lib" "%OUTPUT_DIR%"
2323
@if not exist "%WORK_DIR%\pcre2\include" mkdir "%WORK_DIR%\pcre2\include"
2424
xcopy /y "%WORK_DIR%\pcre2\src\pcre2*.h" "%WORK_DIR%\pcre2\include\" >nul 2>&1
2525
copy /y "%WORK_DIR%\pcre2\pcre2.h" "%WORK_DIR%\pcre2\include\" >nul 2>&1
26-
echo "a"
26+
@echo PCRE2 build and deployment completed successfully.
2727
@exit /B 0
2828

2929
:file_not_found_bin

iis/installer.wxs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@
122122
<![CDATA[Installed OR VC142X64INSTALLED]]>
123123
</Condition>
124124
<Condition Message="This application requires Microsoft Visual C++ 2019 Redistributable (x86) for 32-bit support. Please download and install from: https://aka.ms/vs/17/release/vc_redist.x86.exe">
125-
<![CDATA[Installed OR VC142X86INSTALLED]]>
125+
<![CDATA[Installed OR (&ModSec32 <> 3) OR VC142X86INSTALLED]]>
126126
</Condition>
127127
<?else?>
128128
<Condition Message="This application requires Microsoft Visual C++ 2019 Redistributable (x86). Please download and install from: https://aka.ms/vs/17/release/vc_redist.x86.exe">

iis/wix/list_dependencies.bat

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -46,30 +46,30 @@ pause
4646
:LOOP_FILE
4747
SET /a log_file=%RANDOM%+100000
4848
SET log_file=%TEMP%\ModSecurityIIS-dependencies-%log_file:~-5%.TXT
49-
IF EXIST %log_file% GOTO LOOP_FILE
49+
IF EXIST "%log_file%" GOTO LOOP_FILE
5050

5151
echo Saving logs at: %log_file%
5252
echo.
5353

5454
REM Log prerequisite checks to file
55-
echo ================================================ >> %log_file%
56-
echo Visual C++ 2019 Redistributable Check >> %log_file%
57-
echo ================================================ >> %log_file%
55+
echo ================================================ >> "%log_file%"
56+
echo Visual C++ 2019 Redistributable Check >> "%log_file%"
57+
echo ================================================ >> "%log_file%"
5858

5959
reg query "HKLM\SOFTWARE\Microsoft\VisualStudio\14.0\VC\Runtimes\x64" /v Installed 2>nul | find "0x1" >nul
6060
if %errorlevel% equ 0 (
61-
echo [OK] Visual C++ 2019 Redistributable (x64) is installed >> %log_file%
61+
echo [OK] Visual C++ 2019 Redistributable (x64) is installed >> "%log_file%"
6262
) else (
63-
echo [MISSING] Visual C++ 2019 Redistributable (x64) is NOT installed >> %log_file%
64-
echo Download from: https://aka.ms/vs/17/release/vc_redist.x64.exe >> %log_file%
63+
echo [MISSING] Visual C++ 2019 Redistributable (x64) is NOT installed >> "%log_file%"
64+
echo Download from: https://aka.ms/vs/17/release/vc_redist.x64.exe >> "%log_file%"
6565
)
6666

6767
reg query "HKLM\SOFTWARE\Microsoft\VisualStudio\14.0\VC\Runtimes\x86" /v Installed 2>nul | find "0x1" >nul
6868
if %errorlevel% equ 0 (
69-
echo [OK] Visual C++ 2019 Redistributable (x86) is installed >> %log_file%
69+
echo [OK] Visual C++ 2019 Redistributable (x86) is installed >> "%log_file%"
7070
) else (
71-
echo [MISSING] Visual C++ 2019 Redistributable (x86) is NOT installed >> %log_file%
72-
echo Download from: https://aka.ms/vs/17/release/vc_redist.x86.exe >> %log_file%
71+
echo [MISSING] Visual C++ 2019 Redistributable (x86) is NOT installed >> "%log_file%"
72+
echo Download from: https://aka.ms/vs/17/release/vc_redist.x86.exe >> "%log_file%"
7373
)
7474
echo. >> %log_file%
7575

0 commit comments

Comments
 (0)