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

Make shared boost the default, simpler VERSION_STRING, fix Doxygen #158

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

vespakoen
Copy link
Collaborator

Let's see what CI says ;)

@petterreinholdtsen
Copy link

Well, what was learned from this?

@vespakoen
Copy link
Collaborator Author

There is still some work to do on this. I am working on it in my free time sporadically, might take a while ;)

@petterreinholdtsen
Copy link

petterreinholdtsen commented Sep 4, 2023 via email

@vespakoen vespakoen force-pushed the some-fixes branch 2 times, most recently from 52afd29 to 7dc0155 Compare September 8, 2023 07:50
@petterreinholdtsen
Copy link

petterreinholdtsen commented Sep 8, 2023 via email

@petterreinholdtsen
Copy link

petterreinholdtsen commented Sep 23, 2023 via email

@vespakoen
Copy link
Collaborator Author

I am having issues with the Windows version, using boost python precompiled from boost.org.

See: https://github.com/aewallin/opencamlib/actions/runs/6157111279/job/16707179864

It seems to be linking to dll's that are later not found anymore.

I setup a Windows VM on my own machine and got it to compile with no problems, I am not sure what is going wrong in CI and it is hard to figure out what it has linked to, since the VM doesn't have "dumpbin" and I am not sure how to install it.

I think after all this hassle I gave up on this for a bit.

So in short, it looks like everything is working (CD workflow is fine, using statically compiled boost)
Shared boost (in Test workflow) works on macOS and Linux, and also on my local Windows VM, just not in Github CI.

I would like to figure out why first...

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

2 participants