Skip to content

WIP: CI - #37

Open
illwieckz wants to merge 11 commits into
masterfrom
illwieckz/ci
Open

illwieckz wants to merge 11 commits into
masterfrom
illwieckz/ci

Conversation

@illwieckz

Copy link
Copy Markdown
Member

Work in progress GitHub Actions CI.

@illwieckz
illwieckz force-pushed the illwieckz/ci branch 4 times, most recently from 8fd1166 to 47f0b13 Compare October 10, 2026 21:40
@illwieckz

Copy link
Copy Markdown
Member Author

This is built atop #35 to properly build and test dump_syms_dwarf:

@illwieckz

illwieckz commented Oct 11, 2026 •

Copy link
Copy Markdown
Member Author

I added more tests to test for the presence of dump_syms, dump_syms_dwarf and dump_syms_mac, then actually test symbolizing a bunch of binaries.

@illwieckz

illwieckz commented Oct 11, 2026 •

Copy link
Copy Markdown
Member Author

So I have a working CI for:

  • Linux amd64 GCC
  • Linux arm64 GCC
  • Linux amd64 Clang
  • Windows amd64 MinGW
  • macOS amd64 AppleClang
  • macOS arm64 AppleClang

They all build the tools and run the tests successfully (arm64 builds are done on arm64 machines). The Windows job builds and runs on Linux (MinGW and Wine).

@illwieckz

Copy link
Copy Markdown
Member Author

So I have a working CI for:

  • Linux amd64 GCC
  • Linux arm64 GCC
  • Linux amd64 Clang
  • Windows amd64 MinGW (running on Linux)
  • Windows arm64 MinGW (running on Windows)
  • macOS amd64 AppleClang
  • macOS arm64 AppleClang

This covers all the variations I wanted to see. Especially the difference between MinGW on Linux and Wine and MinGW on Windows is useful to have. Making one of them testing another architecture at the same time is the cherry on the top.

@illwieckz
illwieckz force-pushed the illwieckz/ci branch 4 times, most recently from c6e5e97 to d17a4d4 Compare October 12, 2026 06:25
Align PageHeader to std::max_align_t to ensure the memory
following the header is properly aligned on 32-bit targets.

This fixes that static assertion failure when building with
i686-linux-gnu-gcc:

error: static assertion failed:
 The first byte after `PageHeader` must be properly aligned.
@illwieckz
illwieckz force-pushed the illwieckz/ci branch 2 times, most recently from 0bd5f3a to c0f1bdb Compare October 15, 2026 03:17
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.

1 participant