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

All tests fail due to bad file permissions #4375

Closed
u55 opened this Issue Jan 7, 2015 · 7 comments

Comments

Projects
None yet
3 participants
@u55
Copy link

commented Jan 7, 2015

Hi scipy developers,

I noticed that the file permissions have changed in the 0.14.1 tarball release. Regular users no longer have read permission for many of the files, which means that when I build scipy from source and install it myself, by default some of the files are unreadable without elevating my privledges. For instance, after a fresh scipy install, every single one of the tests fail with a warning similar to:

ERROR: Failure: PermissionError ([Errno 13] Permission denied: '/usr/lib/python3.4/site-packages/scipy/cluster/tests/test_hierarchy.py')

Compare the file permissions in the source tarball of version 0.14.1 versus 0.14.0 (here I am showing one of the aforementioned test files):

$ ls -lh scipy-0.14.1/scipy/cluster/tests/test_hierarchy.py
-rw-r----- 1 user user 57K Dec 30 11:46 scipy-0.14.1/scipy/cluster/tests/test_hierarchy.py

$ ls -lh scipy-0.14.0/scipy/cluster/tests/test_hierarchy.py
-rw-r--r-- 1 user user 57K May 3 2014 scipy-0.14.0/scipy/cluster/tests/test_hierarchy.py

Was this change to the file permissions intentional? If so, then is it documented somewhere? It seems like a bug to me since I have to manually change the file permissions when I build scipy.

Thanks.

@rgommers

This comment has been minimized.

Copy link
Member

commented Jan 7, 2015

Likely unintentional. @pv?

@u55

This comment has been minimized.

Copy link
Author

commented Jan 7, 2015

Note, the file permissions are also problematic in the source tarball of release-candidate version 0.15.0rc1:

$ ls -lh scipy-0.15.0rc1/scipy/cluster/tests/test_hierarchy.py
-rw-r----- 1 user user 35K Dec 15 09:48 scipy-0.15.0rc1/scipy/cluster/tests/test_hierarchy.py

@u55

This comment has been minimized.

Copy link
Author

commented Jan 12, 2015

Well, I never heard anything on this issue from Pauli (@pv), but I see that he released version 0.15.0 yesterday and it has the correct file permissions.

For example:
$ ls -lh scipy-0.15.0/scipy/cluster/tests/test_hierarchy.py
-rw-r--r-- 1 user user 35K Jan 11 08:27 scipy-0.15.0/scipy/cluster/tests/test_hierarchy.py

As far as I am concerned, this issue can be closed.
Thanks.

@pv

This comment has been minimized.

Copy link
Member

commented Jan 16, 2015

Fixed in 8e5c9d7 (in maintenance/0.15.x; pavement.py in master isn't fixed yet)

@rgommers

This comment has been minimized.

Copy link
Member

commented Oct 25, 2015

Hmm, just coming across this. This fix is missing in master, but we haven't had any complaints about this for 0.16.x. @pv do you think we need to push that commit to master?

@rgommers rgommers added this to the 0.17.0 milestone Oct 25, 2015

@pv

This comment has been minimized.

Copy link
Member

commented Oct 25, 2015

Probably best to push it in master, I don't think the permissions come right otherwise (then it probably depends on what default settings you have active for shell).

@rgommers rgommers closed this in e33cd1b Oct 25, 2015

@rgommers

This comment has been minimized.

Copy link
Member

commented Oct 25, 2015

OK thanks, done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.