Roundup fails on Python packages whose dependencies include C extensions

NASA-PDS / roundup-action

Do a "roundup", a/k/a PDS-style continuous integration and delivery

Apache License 2.0

1 stars 4 forks source link

Roundup fails on Python packages whose dependencies include C extensions #3

Closed nutjob4life closed 3 years ago

nutjob4life commented 3 years ago

🐛 Describe the Bug

When the Roundup is used on a Python package that includes a dependency like numpy, i.e.,

setup(…, install_requires=['numpy'], …)

and that dependency has C extensions, the assembly fails because the package setup needs gcc and other build tools at setup time.

📑 To Reproduce

Run the Roundup on pds-doi-service.

Or more simply:

docker container run --interactive --tty --rm --entrypoint /usr/local/bin/pip nasapds/pds-github-actions-base:latest install numpy

🔍 Expected Behavior

For the assembly to complete.

nutjob4life commented 3 years ago

Fixed in dde8c8223c3efa3682b40760d27a14a11e227c17

tloubrieu-jpl commented 3 years ago

Hi @nutjob4life , the action still fails on the pandas install for the pds-doi-service action: ERROR: Failed building wheel for pandas\nERROR: Failed to build one or more wheels\nerror: Command '['/usr/local/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/tmp/tmpzkqdb4t6', '--quiet', 'pandas>=1.0']' returned non-zero exit status 1.\n"» Traceback (most recent call last):

See the full action log at https://github.com/NASA-PDS/pds-doi-service/actions/runs/338169191

Should I update something so that the new docker image is used ?

nutjob4life commented 3 years ago

@tloubrieu-jpl I pushed another new base image to Docker Hub. Try it again!