Re: help needed: Windows CI broken

On 2026-06-01 12:53 p.m., Jeff McKenna wrote:
> On 2026-06-01 12:39 p.m., Joe Orton wrote:
>> On Mon, Jun 01, 2026 at 12:24:14PM -0300, Jeff McKenna wrote:
>>> On 2026-06-01 9:54 a.m., Joe Orton wrote:
>>>> Windows builds have been failing in CI for a while, I think because of
>>>> some changes in the GHA images.
>>>>
>>>> I burnt some Claude tokens trying to fix the trunk to no avail [1], but
>>>> I have zero interest in this. It will be ... frustrating... if people
>>>> start voting against future 2.4.x rcs because Windows builds are broken
>>>> in some way which was trivial to catch in CI and fix ahead of time.
>>>>
>>>> Example trunk failure: https://github.com/apache/httpd/actions/ 
>>>> runs/25912053842/job/76159381743
>>>>
>>>> 2.4.x failure is "simpler" because of the cmake version mismatch:
>>>> https://github.com/apache/httpd/actions/runs/25488877019/ 
>>>> job/74791446628
>>>> but working around that as suggested did not help.
>>>>
>>>> Regards, Joe
>>>>
>>>> [1] https://github.com/notroj/httpd/tree/refs/heads/maybe-fix- 
>>>> windows-ci
>>>
>>> Branch 2.4.x needs this change to be backported: https://github.com/ 
>>> apache/httpd/commit/08138ac986851217ff8f03b495d12ceca9051e24
>>
>> Does that definitely work? I tried -DCMAKE_POLICY_VERSION_MINIMUM=3.5
>> and it failed:
>> https://github.com/apache/httpd/actions/runs/26754488163/job/78850661003
> 
> Ah!  I didn't notice that attempt.  To answer your question: yes that 
> CMake change that I made works.  The run that you pointed to you here 
> makes it past that (thanks to that fix) and now breaks on a different 
> issue.
> 
> I had made a change locally to avoid that error: Unexpected enablement 
> value "i" for mod_authnz_ldap
> 
> But I wasn't proud of my workaround, and now I realize that I should 
> create a pull request for my workaround for that "i" error (if I 
> remember, I had to "unset(i)" outside of a for loop, to avoid that 
> error.  Will file a new pull request now (to the 2.4.x branch, if you 
> already applied the other fix that I mentioned).
> 
>>
>>> PS. Suggest to use the Backport action in the httpd repo (we use this 
>>> and it
>>> works beautifully https://github.com/MapServer/MapServer/blob/ 
>>> main/.github/workflows/backport.yml
>>> )
>>
>> Sounds useful.... with writeable git ;)
>>
>>> Regarding trunk, there seems to be many changes in trunk that break the
>>> Windows MSVC compiler, it would take me a while to go through all of 
>>> those
>>> changes (hmm I wonder why those changes made it through your CI 
>>> tests / why
>>> they were merged before fixing the tests in Windows - as now it seems 
>>> like a
>>> mountain of fixes are needed)
>>
>> I don't know or care about either cmake or Windows, so I don't have much
>> invested in this. The Windows CI started failing in early May and I'm
>> pretty sure it was not the relevant commit to trunk which triggered
>> that, but an image change:
>>
>> https://github.com/apache/httpd/actions/workflows/windows.yml? 
>> query=branch%3Atrunk
>>
>> Regards, Joe
>>

New pull request (into branch 2.4.x) : 
https://github.com/apache/httpd/pull/657

(backports earlier change, and also tackles the error :
     Unexpected enablement value "i" for mod_authnz_ldap


thanks,

-jeff



-- 
Jeff McKenna
GatewayGeo: Developers of MS4W, & offering MapServer Consulting/Dev
co-founder of FOSS4G
http://gatewaygeo.com/