Re: help needed: Windows CI broken

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
> 

thanks,

-jeff


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