rougier / freetype-py

Python binding for the freetype library
Other
298 stars 88 forks source link

Facing issues in triggering travis build #120

Open odidev opened 4 years ago

odidev commented 4 years ago

Hi @rougier ,

Travis has added support for ARM64. I have added arm64 jobs and was facing issues with multibuild docker test image. It has been resolved by multibuild as they have added architecture check in install_run function. I have rebased and tried to trigger the build but it is giving error "Build config did not create any jobs."

Requests link: https://travis-ci.org/github/odidev/freetype-py/requests?requestId=206745125

Can you suggest me on what needs to be done for triggering a travis build?

rougier commented 4 years ago

Hi @odidev, and sorry for the delay. Did you make a PR or do you test from your own repository ?

odidev commented 4 years ago

Hi @rougier ,

Thanks for the reply.

I have tested in my own repository. It is working after I have updated global python version from 3.5 to 3.7 and disabled deployment phase.

I want to add a PR as well for adding arm64 jobs in travis. I have one more question regarding cmake installation in multibuild_config.sh, Is it required for building and running freetype_py tests?

anthrotype commented 4 years ago

cmake is used to orchestrate the build of the freetype library that is embedded in the wheel packages https://github.com/rougier/freetype-py/blob/master/setup-build-freetype.py

anthrotype commented 4 years ago

unless FREETYPEPY_BUNDLE_FT=1 is set in the environment when building from source, cmake+ninja should not be needed

https://github.com/rougier/freetype-py/blob/c578643b5736130c4341584655cb9813c32b5482/setup.py#L20-L21

odidev commented 4 years ago

@anthrotype ,

Thanks for the info.

While building freetype wheel, I am facing below error in travis: ERROR: Command errored out with exit status 1: command: /opt/python/cp36-cp36m/bin/python -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-qeir3nui/setup.py'"'"'; file='"'"'/tmp/pip-req-build-qeir3nui/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-8ao_6qrq cwd: /tmp/pip-req-build-qeir3nui/

Report link: https://travis-ci.org/github/ossdev07/freetype-py/jobs/675620306 Travis link: https://github.com/ossdev07/freetype-py/blob/master/.travis.yml

Could you please have a look and suggest me if I am missing anything. Thanks.

anthrotype commented 4 years ago

you want to build freetype for arm64? I actually don't know if freetype cmake setup works with that, sorry. Maybe you can ask on the freetype mailing list. Honestly I'm not even sure if multibuild (which we use to set up the CI for building linux wheels) supports that architecture. I think you may have more luck by not building an embedded freetype library from source (not setting FREETYPEPY_BUNDLE_FT) and instead let freetype-py use the system freetype library.

anthrotype commented 4 years ago

oh sorry, I just read your comment above about multibuild

It has been resolved by multibuild as they have added architecture check in install_run function

well, that's good to hear. I don't have experience building for arm64, so I can't be of help. Maybe others reading this will.

odidev commented 4 years ago

@anthrotype ,

Thanks for the help. I will check.