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

Qt5 fixed #101

Open
wants to merge 11 commits into
base: version27
Choose a base branch
from
Open

Qt5 fixed #101

wants to merge 11 commits into from

Conversation

Zacharyprime
Copy link
Member

This might not be ready yet, I'm mainly doing this to keep track of the differences of the branches while I prepare to merge

Zachary Sasser and others added 11 commits July 20, 2021 14:09
Fixing compatability issues with qt5
This may break other things, if it doesn't, this commit will be squashed later
Still needs to be tested extensively before merging
There are still errors to resolve regarding opencv4, but this gets us started
Fixed the Opencv related errors.
I'm not sure why this was working before, I'm guessing at some point the old image manually set the include paths for botui.

This means that OpenCV4 support is now working as far as I can tell.

This now means that we can build everything but harrogate on a fresh version of raspian.
The harrogate errors might be easy to solve, we are planning to move to Ivygate or something similar soon.
@Zacharyprime
Copy link
Member Author

This is complete.
However we need people to test it.
Please post here if you managed to break this version of botui.

I've done some testing but I'm sure I missed something or there is some weird edge case that will pop up later.
If you test it and can't find anything, please also report that here.

@Zacharyprime
Copy link
Member Author

A dependency that was accidentally left out in the build instructions for the experimental OS:
sudo apt-get install qtdeclarative5-dev

If this is not installed it won't be able to find Qt5Quick

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.

2 participants