Skip to content

Implement the CI in GitHub Actions - #37

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

Implement the CI in GitHub Actions#37
illwieckz wants to merge 14 commits into
masterfrom
illwieckz/ci

Conversation

@illwieckz

@illwieckz illwieckz commented Oct 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

GitHub Actions CI.

@illwieckz
illwieckz force-pushed the illwieckz/ci branch 5 times, most recently from a8354c0 to 89e4361 Compare October 11, 2026 01:00
@illwieckz

illwieckz commented Oct 11, 2026

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 •
edited
Loading

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 force-pushed the illwieckz/ci branch 8 times, most recently from c6ede68 to ca62c1a Compare October 11, 2026 02:24
@illwieckz illwieckz mentioned this pull request Oct 11, 2026
Closed
@illwieckz

illwieckz commented Oct 11, 2026 •
edited
Loading

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
illwieckz force-pushed the illwieckz/ci branch 7 times, most recently from 58c391f to 37b3394 Compare October 11, 2026 07:51
This was referenced Oct 11, 2026
Closed
Merged
Merged
@illwieckz
src/common/memory_allocator: preserve 64-bit alignment masks
41e22ce 
Use the value type for the alignment mask to prevent
truncation of 64-bit values on 32-bit targets.
@illwieckz
src/client/linux/minidump_writer: make minidump stack-skipping test m…
96e1c90 
…ore deterministic

- Check pthread attribute initialization, configuration, and thread
  creation errors.
- Give the helper thread a dedicated and zero-initialized stack, to make
  the test less dependent on residual stack contents when checking whether
  stacks referencing the executable are skipped.

The test checks that a helper thread that runs in libc and doesn't
reference the main executable is excluded from the minidump.
Initializing the dedicated stack should reduce the chance that stale
pointers into the executable mapping cause the stack to be included.

This changes a bit the original conditions of the test, since residual
stack contents that could affect the result are eliminated. But it
should make the test more deterministic.

This test has been observed to sometimes fail in CI, reporting
more threads with dumped stacks than expected.
@illwieckz
configure: detect macOS build machine
62f76d1
@illwieckz
illwieckz force-pushed the illwieckz/ci branch 3 times, most recently from 63d0075 to 2f6ae6f Compare October 15, 2026 20:32
@illwieckz illwieckz mentioned this pull request Oct 15, 2026
Merged
This was referenced Oct 15, 2026
Open
Open
@illwieckz illwieckz changed the title WIP: CI Implement the CI in GitHub Actions Oct 16, 2026
@illwieckz

illwieckz commented Oct 16, 2026

Copy link
Copy Markdown
Member Author

This looks ready to me.

Here are some things that can be merged before hand:

@illwieckz
illwieckz force-pushed the illwieckz/ci branch 2 times, most recently from f9ce546 to e9f2afe Compare October 20, 2026 03:52
@illwieckz illwieckz mentioned this pull request Oct 20, 2026
Merged
@illwieckz illwieckz mentioned this pull request Oct 20, 2026
Open
@illwieckz
illwieckz force-pushed the illwieckz/ci branch 3 times, most recently from 3f7e0e2 to 98f6b27 Compare October 21, 2026 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@illwieckz