mock-webserver icon indicating copy to clipboard operation
mock-webserver copied to clipboard

Windows support

Open jonasraoni opened this issue 2 years ago • 3 comments

Hey @donatj,

I just got your v3wip branch, rebased, added minor updates and it's passing through the tests.

jonasraoni avatar Jul 30 '22 15:07 jonasraoni

Very cool!

I'll need to poke and prod this later, I'm AFK for a couple days, but I'm excited to this.

donatj avatar Jul 30 '22 16:07 donatj

No problem, I'm just helping another open source project that would like to run their tests under Windows 😁

jonasraoni avatar Jul 30 '22 17:07 jonasraoni

I've just checked and I can confirm these updates worked for my use case.

jonasraoni avatar Jul 31 '22 12:07 jonasraoni

Sorry for the delay in getting this merged. I really wanted to get PHP running on a Windows system and give it a try before hand.

After hamfisting my way through getting PHP going on Windows (you guys really configure PATH with a GUI?!) everything appears to work as expected.

donatj avatar Aug 16 '22 01:08 donatj

Also, I just want to be sassy and add "why you wouldn't just use WSL is a mystery to me" 😆

donatj avatar Aug 16 '22 01:08 donatj

Hmmph. Once I merged it, I'm getting a semi-consistent CI failure on PHP 5.5... Well that's no fun

donatj avatar Aug 16 '22 01:08 donatj

you guys really configure PATH with a GUI?!

@donatj Windows got some new things, there's a Windows Server Core, which has no GUI, and the PowerShell. Before the introduction of PowerShell, I think you'd have to use the setx :)

why you wouldn't just use WSL is a mystery to me

I'm just helping another repo, but I think it makes sense to have a test against Windows, considering minor differences might break something (e.g. PHP_EOL) 🤔

CI failure on PHP 5.5

Let me know if you need a hand 😅

jonasraoni avatar Aug 16 '22 06:08 jonasraoni

Also, if you're ok to release a "broken" package (considering that it wasn't working in Windows anyway, it's already an improvement), I'll use it in the other repository and clear one debt 😅

jonasraoni avatar Aug 16 '22 07:08 jonasraoni