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

opencv: remove gtk dependency #7589

Merged
merged 1 commit into from
Dec 13, 2018
Merged

Conversation

cotequeiroz
Copy link
Member

Maintainer: WRTnode Team [email protected], @noello (a member of the WRTnode Team that I could find) / @champtar (even though you're not listed as the maintainer, you have the first and last commits here)
Compile tested: ramips, openwrt master
Run tested: none

Description:
Avoids this:

Package opencv is missing dependencies for the following libraries:
libglib-2.0.so.0
libgthread-2.0.so.0

Signed-off-by: Eneas U de Queiroz [email protected]

@cotequeiroz
Copy link
Member Author

Another one that needs #7588

@thess
Copy link
Member

thess commented Dec 6, 2018

CircleCI updated - rebase PR and see if OK now.

@cotequeiroz
Copy link
Member Author

bash: unzip: command not found

Not ready yet?

@thess
Copy link
Member

thess commented Dec 7, 2018

You are still using the previous Docker image - You didn't rebase your PR to pickup the newer image and script updates.

@cotequeiroz
Copy link
Member Author

cotequeiroz commented Dec 7, 2018

I guess I had not fetched the latest commits. It works now.

@hnyman
Copy link
Contributor

hnyman commented Dec 13, 2018

You should increase the PKG_RELEASE, just in case, I think.

In general, this is one of the last few packages failing in buildbot for any of selected major targets (like the the package archs for ar71xx, mvebu, ipq806x and x86). I have never seen the failure list for ar71xx, ipq806x and mvebu to be only approx. 5 packages at the same time, like now, so really a spectacular situation at the moment. So, big thanks for @cotequeiroz and @neheb for fixing a bunch of packages that were failing for a long time.

@cotequeiroz
Copy link
Member Author

The PKG_RELEASE was an oversight, sorry. It does change configure parameters, which may reflect in the final binary. I usually don't increase if it's a host-only fix.

As for making packages work, it was a pleasure helping you out.

Cheers

Signed-off-by: Eneas U de Queiroz <[email protected]>
@hnyman hnyman merged commit 12dc221 into openwrt:master Dec 13, 2018
@neheb
Copy link
Contributor

neheb commented Dec 13, 2018 via email

@cotequeiroz cotequeiroz deleted the opencv_no-gtk branch March 11, 2019 11:31
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.

4 participants