Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Windows CI build likely does not work #21

Closed
CouleeApps opened this issue Jan 30, 2024 · 3 comments
Closed

Windows CI build likely does not work #21

CouleeApps opened this issue Jan 30, 2024 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@CouleeApps
Copy link
Contributor

I see you have based the github actions build for this off of https://github.com/CouleeApps/binja-ci-tests/blob/master/binja.patch and I just wanted to let you know that the Windows build this patch produces does not actually work... https://github.com/CouleeApps/binja-ci-tests/issues/1

I haven't had time to go figure out what the correct solution is, but just a heads up in advance that this is a known issue.

@CouleeApps CouleeApps changed the title Windows build likely does not work Windows CI build likely does not work Jan 30, 2024
@emesare
Copy link
Owner

emesare commented Jan 30, 2024

Thanks for opening the issue, my temporary solution has been manually building it and replacing the broken one.

Hopefully I can figure out a real solution soon because if I forget to replace the broken one I am sure someone is going to be really mad when their binary ninja doesn't open on windows 😃

@emesare emesare added bug Something isn't working help wanted Extra attention is needed labels Jan 30, 2024
@CouleeApps
Copy link
Contributor Author

CouleeApps commented Apr 5, 2024

Latest updates to the base repo should support building on windows. I migrated it to https://github.com/Vector35/sample_plugin_cpp (which apparently breaks github actions, thanks github). Let me know if those changes get this repo to work, and if you run into any issues along the way :)

@emesare
Copy link
Owner

emesare commented Apr 10, 2024

Latest updates to the base repo should support building on windows. I migrated it to https://github.com/Vector35/sample_plugin_cpp (which apparently breaks github actions, thanks github). Let me know if those changes get this repo to work, and if you run into any issues along the way :)

Fixed! Thanks again :)

@emesare emesare closed this as completed Apr 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants