Skip to content

ci: Windows latest boost build fix#734

Merged
eyal0 merged 2 commits intopcb2gcode:masterfrom
mar0x:ci-windows-boost-fix
Dec 15, 2025
Merged

ci: Windows latest boost build fix#734
eyal0 merged 2 commits intopcb2gcode:masterfrom
mar0x:ci-windows-boost-fix

Conversation

@mar0x
Copy link
Copy Markdown
Contributor

@mar0x mar0x commented Dec 14, 2025

Windows is a very specific platform for build. This time boost lookup script was failed to find headers installed from msys2 repository. There were two possible solutions: use '--with-boost /mingw64' or add one more 'standard' location to find. The last was implemented.

Next thing is a replacement of -I to -isystem here and there to avoid false positive warnings inside 3rd party libraries. Unfortunately this replacement affects the way how #include_next directive works (again, on Windows):

D:/a/_temp/msys64/mingw64/include/c++/15.2.0/cstdlib:83:15: fatal error: stdlib.h: No such file or directory
   83 | #include_next <stdlib.h>

Preprocessor failed to find /mingw64/include/stdlib.h. I've found the only way to fix this: do not replace -I to -isystem for 'standard' boost path.

@mar0x mar0x changed the title CI: Windows latest boost build fix ci: Windows latest boost build fix Dec 15, 2025
@mar0x mar0x mentioned this pull request Dec 15, 2025
@eyal0
Copy link
Copy Markdown
Contributor

eyal0 commented Dec 15, 2025

The -I versus -isystem thing has been a long time annoyance and now the compiler doesn't even support it. We'll give up on it as necessary.

@eyal0 eyal0 merged commit 6c8e590 into pcb2gcode:master Dec 15, 2025
13 checks passed
timaydin pushed a commit to timaydin/pcb2gcode that referenced this pull request Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants