Skip to content

ci: check out a referenced angr/binaries pull request - #54

Merged
twizmwazin merged 1 commit into
masterfrom
feature/binaries-ref-sync
Aug 24, 2026
Merged

twizmwazin merged 1 commit into
masterfrom
feature/binaries-ref-sync

Conversation

@zardus

@zardus zardus commented Aug 24, 2026

Copy link
Copy Markdown
Member

THIS MESSAGE WAS GENERATED BY AN AUTOMATED PROCESS

The angr/binaries checkout carries no ref, so this workflow always builds against master. A pull request whose new test loads a fixture from an open angr/binaries pull request fails here no matter what its body references, which is the same gap angr/cle closed in angr/cle#738.

This resolves the ref with the shared action from angr/ci-settings and passes it to the existing checkout, so this workflow's own checkout options are untouched.

Blocked on angr/ci-settings#123, which adds the action; this references it at master.

The binaries checkout has no ref, so it always builds against master and a pull
request whose test needs a fixture from an open angr/binaries pull request fails
here regardless of what its body references. Resolve the ref with the shared
angr/ci-settings action and pass it to the existing checkout, which keeps this
workflow's own checkout options.
@zardus

zardus commented Aug 24, 2026

Copy link
Copy Markdown
Member Author

THIS MESSAGE WAS GENERATED BY AN AUTOMATED PROCESS

The failing jobs all stop in Set up job, before any checkout or test runs:

Can't find 'action.yml', 'action.yaml' or 'Dockerfile' for action 'angr/ci-settings/actions/binaries-ref@master'

That is the expected state while angr/ci-settings#123 is open: this references the action at master, where it does not exist yet. The red is not a property of this diff and there is nothing here to repair. Merging angr/ci-settings#123 and re-running the whole run clears it.

The workflow file was checked with yaml.safe_load before pushing, and the resolution logic itself was exercised against five pull-request body forms in angr/ci-settings#123.

@angr-bot

Copy link
Copy Markdown
Member

Corpus decompilation diffs can be found at angr/dec-snapshots@master...angr/pysoot_54

@twizmwazin
twizmwazin merged commit 44e1451 into master Aug 24, 2026
16 of 28 checks passed
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.

3 participants