search

fonttools / skia-pathops

Python bindings for the Skia library's Path Ops
https://skia.org/docs/dev/present/pathops/
BSD 3-Clause "New" or "Revised" License
47 stars 14 forks source link

Conda build failed #56

Closed gauranshkumar closed 2 years ago

gauranshkumar commented 2 years ago

Hey team, I am trying to make a conda build for this package here https://github.com/conda-forge/staged-recipes/pull/19143, but it dosen't seems to be build successfully. I have tried configuring the recipe for conda build as per the requirements but it fails at this error https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=516887&view=logs&j=6f142865-96c3-535c-b7ea-873d86b887bd&t=22b0682d-ab9e-55d7-9c79-49f3c3ba4823&l=1344 I appreciate your support in this. Thanks!

gauranshkumar commented 2 years ago

@davelab6 and other maintainer can you please help in this regard. Thanks!

pavelzw commented 2 years ago

It seems like a nightmare to fix this... The whole build process uses custom virtual envs that install ninja separately from everything else and uses all different kinds of hacks that are not really compatible with conda.

anthrotype commented 2 years ago

I'm not familiar with conda, but if you manage to build your own standalone skia library by other means, then you can point skia-pathops to the directory where the compiled library is so it can link to it. You need to export environment variable BUILD_SKIA_FROM_SOURCE=0 before building from source to prevent setup.py from trying to build skia from source, then you export a SKIA_LIBRARY_DIR with the path to the directory containing the skia library (this will be passed on to setuptools' Extension.library_dirs and the linker will search in there when linking the extension module). Someone also contributed support for finding skia using pkg-config (see https://github.com/fonttools/skia-pathops/pull/54), not sure if that's of interest for conda. I hope that helps..

pavelzw commented 2 years ago

@naveen521kk added skia-pathops to conda-forge in conda-forge/staged-recipes #19570 :rocket:. So this issue can be closed.