[cffi] Updated recipe to last cffi version (1.11.5)

[goffi-contrib]

No description provided.

[AndreMiras]

Why chaning the url string?

[AndreMiras]

Thanks for updating the changes.
The conditional build seems to be failing and the error is still unclear to me.
I've started it again, let's see. If it's still failing I'd like to investigate a bit more. Maybe it would be another excuse to improve our CI tools.
In the meantime, do you know you can "override" the version without touching the recipe? All you need to do is specify it with ==version just like when you pip install a package.

[goffi-contrib]

@AndreMiras no, I didn't know. Thanks for the tip.
About the error, it smells like an incomplete docker image, autotools seem to be missing.

[AndreMiras]

Good catch for the autotools, the error was in the middle of nowhere 👍
So note for me later.

android/build/other_builds/libffi/armeabi/libffi/autogen.sh: 2: exec: autoreconf: not found
...
Traceback (most recent call last):

  File "setup_testapp_python2.py", line 30, in <module>

    package_data={'testapp': ['*.py', '*.png']}

  File "/usr/lib/python2.7/distutils/core.py", line 151, in setup

    dist.run_commands()

  File "/usr/lib/python2.7/distutils/dist.py", line 953, in run_commands

    self.run_command(cmd)

  File "/usr/lib/python2.7/distutils/dist.py", line 972, in run_command

    cmd_obj.run()

  File "/home/user/pythonforandroid/bdistapk.py", line 81, in run

    main()

  File "/home/user/pythonforandroid/toolchain.py", line 999, in main

    ToolchainCL()

  File "/home/user/pythonforandroid/toolchain.py", line 532, in __init__

    getattr(self, args.subparser_name.replace('-', '_'))(args)

  File "/home/user/pythonforandroid/toolchain.py", line 145, in wrapper_func

    build_dist_from_args(ctx, dist, args)

  File "/home/user/pythonforandroid/toolchain.py", line 189, in build_dist_from_args

    build_recipes(build_order, python_modules, ctx)

  File "/home/user/pythonforandroid/build.py", line 580, in build_recipes

    recipe.build_arch(arch)

  File "/home/user/pythonforandroid/recipes/libffi/__init__.py", line 37, in build_arch

    shprint(sh.Command('./autogen.sh'), _env=env)

  File "/home/user/pythonforandroid/logger.py", line 176, in shprint

    for line in output:

  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 863, in next

    self.wait()

  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 792, in wait

    self.handle_command_exit_code(exit_code)

  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 815, in handle_command_exit_code

    raise exc

sh.ErrorReturnCode_127: 



  RAN: /home/user/.local/share/python-for-android/build/other_builds/libffi/armeabi/libffi/autogen.sh



  STDOUT:

/home/user/.local/share/python-for-android/build/other_builds/libffi/armeabi/libffi/autogen.sh: 2: exec: autoreconf: not found





  STDERR:



Traceback (most recent call last):
  File "./ci/rebuild_updated_recipes.py", line 100, in <module>
    main()
  File "./ci/rebuild_updated_recipes.py", line 96, in main
    build(target_python, recipes)
  File "./ci/rebuild_updated_recipes.py", line 72, in build
    requirements, _err_to_out=True, _iter=True):
  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 863, in next
    self.wait()
  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 792, in wait
    self.handle_command_exit_code(exit_code)
  File "/home/user/venv/local/lib/python2.7/site-packages/sh.py", line 815, in handle_command_exit_code
    raise exc
sh.ErrorReturnCode_1: 

  RAN: /home/user/venv/bin/python setup_testapp_python2.py apk --sdk-dir /opt/android/android-sdk --ndk-dir /opt/android/android-ndk --bootstrap sdl2 --requirements cffi,python2
...

Which reminds me, that I wanted to discuss with the team of some optional ways to handle system dependencies.

[goffi-contrib]

@AndreMiras I think this recipe should be added to BROKEN_RECIPES_PYTHON2 (same as #1440 (comment)), but I wait for the other to be merged first, to avoid conflict.

[AndreMiras]

Yes indeed go for it since #1440 was just merged.
Please add a comment just like we did for protobuf_cpp so we know why this is listed as broken.
Thanks for all your contributions 🍻

[goffi-contrib]

@AndreMiras done

[AndreMiras]

Merged! And thanks for always squashing your commits, really appreciated 👍