Skip to content

Bump GitHub Actions versions to fix artifact upload #151

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.

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

peace-maker
Copy link

The cmake workflow building the C++ project fails to upload artifacts for a while due to the upload-artifact Action changing the backend in v4 to be incompatible with v3 forcing everyone to update.

The macos-12 runner image was removed on 12/3/24 so update to the next oldest available one. I'm not sure the build dependencies are meant to be old to allow running on older versions?

Build Ghidra plugins for all newer Ghidra versions too for easy installation of whatever version you're running.

I couldn't test the workflow in my fork due to the encrypted idasdk requirement.

Bump upload-artifact to v4 for the cmake workflow since v3 was deprecated and disabled.
The macos-12 image was removed on 12/3/24.
The `runner` object isn't documented to have a `workspace` member. Switch to using the documented `github.workspace` variable.

https://docs.github.com/de/actions/writing-workflows/choosing-what-your-workflow-does/accessing-contextual-information-about-workflow-runs#runner-context
Copy link

google-cla bot commented May 31, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@peace-maker
Copy link
Author

I just realized this is partly covered by #147 too.

colinmkinsella added a commit to colinmkinsella/binexport that referenced this pull request Jun 8, 2025
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