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.

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

build-windows: include 5 needed DLLs in the package #275

Merged
merged 2 commits into from
Aug 2, 2022

Conversation

stoffu
Copy link

@stoffu stoffu commented Jul 5, 2022

No description provided.

@stoffu stoffu changed the title Aeon ci workaround build-windows: include 5 needed DLLs in the package build-windows: include 5 needed DLLs in the package Jul 5, 2022
@stoffu
Copy link
Author

stoffu commented Jul 6, 2022

@aeonix
This fixes #274.
Please create another release v0.14.2.2-aeon after merging this PR.

@iamsmooth
Copy link

Why is the macOS CI failing?

@stoffu
Copy link
Author

stoffu commented Jul 10, 2022

It's getting a segmentation fault when finishing the IBD test mode. It used to work fine previously, and I have no idea why. Should this be fixed before releasing?

@stoffu stoffu changed the title build-windows: include 5 needed DLLs in the package [WIP] build-windows: include 5 needed DLLs in the package Jul 10, 2022
@iamsmooth
Copy link

Maybe try to assess why it is happening then decide?

@stoffu
Copy link
Author

stoffu commented Jul 16, 2022

While I was investigating, the error disappeared somehow without me making any changes to the code. Very weird, but I guess that’s how it is with GitHub CI.

@stoffu stoffu changed the title [WIP] build-windows: include 5 needed DLLs in the package build-windows: include 5 needed DLLs in the package Jul 16, 2022
@BigslimVdub
Copy link

So are unit tests the only update for .2?

@iamsmooth
Copy link

@BigslimVdub The important update is including the DLLs so the windows build will actually work. The unit tests failed for some unknown reason and then started working again for some unknown reason.

@aeonix aeonix merged commit c1ebcc6 into aeonix:master Aug 2, 2022
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.

None yet

4 participants