Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Umbra failure on macos-12 Github runners #2511

Open
derekbruening opened this issue Sep 18, 2024 · 1 comment
Open

Umbra failure on macos-12 Github runners #2511

derekbruening opened this issue Sep 18, 2024 · 1 comment
Labels
Bug-Assert Bug-ToolFail OpSys-OSX

Comments

@derekbruening
Copy link
Contributor

Github deprecated the macos-11 runners (xref DynamoRIO/dynamorio#6864) but when we try to update to macos-12 umbra fails:

ASSERT FAILURE (thread 40924): /Users/runner/work/drmemory/drmemory/umbra/umbra_64.c:518: seg->shadow_end[seg_map_idx] > seg->shadow_base[seg_map_idx] (wrong shadow segment range)

We may have to disable Mac from the package builds until this is fixed.
@derekbruening
Copy link
Contributor Author

PR #2509 is where this is observed.

derekbruening added a commit that referenced this issue Sep 18, 2024
@derekbruening
i#2511 macos-12: Disable Mac package and tests until fixed
20ae647 
Until we can fix the Umbra failure on macos-12, we have to disable Mac
as macos-11 is no longer supported by Github.

Issue: #2511
This was referenced Sep 18, 2024
Merged
Open
derekbruening added a commit that referenced this issue Sep 18, 2024
@derekbruening
i#2511 macos-12: Disable Mac package and tests until fixed (#2512)
bbb7913 
Until we can fix the Umbra failure on macos-12, we have to disable Mac
as macos-11 is no longer supported by Github.

Issue: #2511
derekbruening added a commit that referenced this issue Sep 18, 2024
@derekbruening
i#2511 macos-12: Remove mac fully from package script
43dc37e 
Removes other pieces of the Mac build that were not fully removed from
PR #2512.

Issue: #2511
@derekbruening derekbruening mentioned this issue Sep 18, 2024
Merged
derekbruening added a commit that referenced this issue Sep 18, 2024
@derekbruening
i#2511 macos-12: Remove mac fully from package script (#2513)
24c3c06 
Removes other pieces of the Mac build that were not fully removed from
PR #2512.

Updates download-artifact and upload-artifact to v4.x.x as the old version
won't run anymore (and has a security vulnerability).

Issue: #2511
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug-Assert Bug-ToolFail OpSys-OSX
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@derekbruening