Skip to content
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

[Errno 13] Permission denied when executing with bdist_pex #570

Closed
fhoering opened this issue Oct 2, 2018 · 1 comment · Fixed by #575
Closed

[Errno 13] Permission denied when executing with bdist_pex #570

fhoering opened this issue Oct 2, 2018 · 1 comment · Fixed by #575
Assignees
Labels

Comments

@fhoering
Copy link
Contributor

fhoering commented Oct 2, 2018

It seems that this review released in version 1.4.6 has introduced a regression when executing pex with the bdist_pex target

running bdist_pex
Writing environment pex into dist/test-0.1.pex
error: [Errno 13] Permission denied: '/tmp/tmpaumuj9wb/.deps/tensorflow-1.10.1-cp36-cp36m-manylinux1_x86_64.whl/tensorflow/libtensorflow_framework.so'

The package that fails is tensorflow.
Minimal setup.py:

#!/usr/bin/env python3

import setuptools
import os

setuptools.setup(
    name='test',
    version='0.1',
    install_requires=["tensorflow==1.10.1"],
    packages=setuptools.find_packages())

It works when just calling the command line tool pex:
pex tensorflow==1.10.1

Can you have a look ?

@jsirois
Copy link
Member

jsirois commented Oct 2, 2018

Hrm. Will do.

@jsirois jsirois self-assigned this Oct 2, 2018
@jsirois jsirois mentioned this issue Oct 2, 2018
jsirois added a commit to jsirois/pex that referenced this issue Oct 4, 2018
Previously clone copied distributions twice. One copy was made by the
underlying Chroot, and then another was manually performed. The manual
copy was only needed to update the internal _distributions set, so just
do that.

A test that failed before the fix is added to ensure distributions with
unwriteable files can be cloned.

Fixes pex-tool#570
jsirois added a commit that referenced this issue Oct 4, 2018
Previously clone copied distributions twice. One copy was made by the
underlying `Chroot`, and then another was manually performed. The
manual copy was only needed to update the internal `_distributions`
set, so just do that.

A test that failed before the fix is added to ensure distributions with
unwriteable files can be cloned.

Fixes #570
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants