Skip to content

Add patch to fix permission denied issue during PyTables installation. #22975

Closed
dagonzalezfo wants to merge 3 commits intoeasybuilders:developfrom
dagonzalezfo:fix-PyTables-permission-issue
Closed

Add patch to fix permission denied issue during PyTables installation. #22975
dagonzalezfo wants to merge 3 commits intoeasybuilders:developfrom
dagonzalezfo:fix-PyTables-permission-issue

Conversation

@dagonzalezfo
Copy link
Copy Markdown
Contributor

During the PyTable package installation, you will get:

PermissionError: [Errno 13] Permission denied: '<buildpath>/tables/libblosc2.so'

This is trigger by using shutil.copy, that create a copy of from $EBROOTLIBLOSC2/lib/libblosc2.so on <buildpath>/tables/libblosc2.so. However, shutil.copy enforces to have the same permissions as the origin, causing the issue.

This can be solved by changing shutil.copy by shutil.copyfile.

@Crivella
Copy link
Copy Markdown
Contributor

Crivella commented Jun 4, 2025

Tried to install the same package without this patch, but it still works, which is weird considering the behavior of shutil.copy and shutil.copyfile should be the same (I mean across our 2 different tests) as they came from the EB-intalled python.

Are you doing your test on some more problematic filesystem (eg NFS or Lustre)?

@dagonzalezfo
Copy link
Copy Markdown
Contributor Author

This issue is related with installing this package using the EESSI machinery. I manage to reproduce this issue on using a local storage. Without this patch, I get the reported error. In any case, I will do some testing outside of EESSI and update this PR accordingly.

@casparvl
Copy link
Copy Markdown
Contributor

Maybe I'm not fully understanding, but would this still be needed if we have #23042 ? I think that makes the current PR obsolete, no?

@dagonzalezfo
Copy link
Copy Markdown
Contributor Author

As Caspar comment, this is covered by #23042

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 this pull request may close these issues.

4 participants