-
-
Notifications
You must be signed in to change notification settings - Fork 59
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
Add headless options #396
Add headless options #396
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
@h-vetinari i'm not sure if you have the bandwidth to attempt: #365 again It seems that there is some use of having multiple qt builds with opencv, and at least a headless build. |
7be1461
to
77dc196
Compare
…nda-forge-pinning 2024.03.12.21.52.52
56ad651
to
4d7491a
Compare
…nda-forge-pinning 2024.03.12.21.52.52
92 builds is somewhat riddiculous. we need a way to cut these down.... |
…nda-forge-pinning 2024.03.13.03.14.31
While I am not super excited about the 56 hour build time total build time (it should be closer to 6 if we could share the outputs well), it does address many grievances. It does also give us the flexibility to remove the qt5 build readily when we feel we are ready for it. A review would be greatly appreciated. |
Hm. Perhaps I can find some time to retry. But the multi-stage build process of opencv makes that really much much harder than I've seen for basically any other library. 🥲 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks very nice IMO, just a few questions/suggestions.
…nda-forge-pinning 2024.03.16.19.50.45
Would benefit from: conda-forge/staged-recipes#25688
Closes #337
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)