-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
libwebp-base should add a run_constrained on libwebp version #26
Comments
@isuruf I'm having a hard time trying to fix this. I believe that this part in the recipe is wrong: https://github.com/conda-forge/libwebp-feedstock/blob/master/recipe/meta.yaml#L56-L58 It declares a run_constrained:
- libwebp =={{ version }} in the
|
This recipe is a mess and I'd consider it a bad hack. I can have a look at fixing it, but not for a while. |
This might be because there is a circular dependency with
|
One of these libraries must not strictly depend on the other. Which is it? They can't both require the other to be compiled. Otherwise, they should be the same package? |
I think it would be best if |
as an intermediate fix for conda-forge/libwebp-feedstock#26
@dschreij could we speak as to why we need a base and a non-base package? There might be an other way for us to do this without split packages (which are really annoying to deal with for me). |
@hmaarrfk, in #28 we discussed that the best way is to change libtiff to depend only on libwebp-base (conda-forge/libtiff-feedstock#58 (comment)) and then make a new feedstock for libwebp-base instead of having one split feedstock (todo). Do you have time to set that up today? Otherwise I volunteer to do it this weekend. |
This also has surfaced over at gdal-feedstock, see conda-forge/gdal-feedstock#370 Anything we can do to fix gdal? |
@kmuehlbauer Please wait a few days for this libwebp to be rebuilt. |
Because some libwebp features depend on libtiff and libtiff is built against the libwebp libraries, we must split the libwebp feedstock into parts. One part depends on libtiff and the other does not. Related to: conda-forge/libwebp-feedstock#26 conda-forge/libwebp-feedstock#27 conda-forge/libwebp-feedstock#28 conda-forge/libwebp-feedstock#29 conda-forge/libtiff-feedstock#48 conda-forge/libtiff-feedstock#58
Otherwise, the following environment is possible,
The text was updated successfully, but these errors were encountered: