TalaatHarb opened a new pull request, #4050:
URL: https://github.com/apache/iggy/pull/4050

   ## Which issue does this PR address?
   
   Closes #4037 
   
   ## Rationale
   
   Provide the ability to build the system natively on windows os
   
   ## What changed?
   
   `iggy-server` could not be built or run natively on Windows. The failure was 
not a
   single blocker but a chain of them: build/runtime dependency issues and also 
Unix only syscalls in the storage and transport layers. This PR adds windows 
alternative for the Unix only syscalls and adds documentation of how to build 
iggy-server on windows.
   
   ## Local Execution
   
   - Passed on Windows
   - Pre-commit hooks that have windows setup ran. (helmfmt for example isn't 
available through winget so I didn't run it)
   due to the nature of this PR focusing on windows setup the behavior of the 
pre-commit hooks might not cover everything
   
   ## AI Usage
   
   Tools used: GitHub Copilo was used
   Sope of usage: generated functions as alternatives for the Unix only issues
   Verification: Build, Tests and basic runtime behavior verified after the 
modifications (This PR makes `cargo build -p server --bin iggy-server` produce 
a working Windows binary that boots, serves traffic, shuts down gracefully, and 
recovers
   its data on restart which wasn't available before). Other OS targets 
behavior should be intact and verified through CI.
   I can explain the logic behind every line of the code if asked.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to