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

Extracting files on Linux doesn't preserve file permissions #132

Closed
MattesMrzik opened this issue Nov 7, 2023 · 1 comment · Fixed by #185
Closed

Extracting files on Linux doesn't preserve file permissions #132

MattesMrzik opened this issue Nov 7, 2023 · 1 comment · Fixed by #185
Assignees
Labels
bug Something isn't working install installation process of IDE + tools and install commandlet

Comments

@MattesMrzik
Copy link
Contributor

Expected behavior

On Linux: using the methods untar and unzip in ToolCommandlet to extract files preserves file permissions.

Actual behavior

These methods do not preserve the file permissions. In particular, the tool binaries are not executable.

Steps to reproduce

Install a tool like helm or aws on a Linux machine.

Affected version:

  • OS: Linux
@MattesMrzik MattesMrzik added bug Something isn't working install installation process of IDE + tools and install commandlet labels Nov 7, 2023
@github-project-automation github-project-automation bot moved this to 🆕 New in IDEasy board Nov 8, 2023
@MattesMrzik MattesMrzik changed the title Extracting Files on Linux doesn't preserve file permissions Extracting files on Linux doesn't preserve file permissions Nov 9, 2023
@MattesMrzik MattesMrzik self-assigned this Nov 9, 2023
@hohwille hohwille moved this from 🆕 New to 📋 Backlog in IDEasy board Jan 16, 2024
@MattesMrzik MattesMrzik moved this from 📋 Backlog to 🏗 In progress in IDEasy board Jan 16, 2024
MattesMrzik added a commit to MattesMrzik/IDEasy that referenced this issue Jan 17, 2024
also wrote a test for untar.
Unzip still does not preserve file permissions
MattesMrzik added a commit to MattesMrzik/IDEasy that referenced this issue Jan 18, 2024
- removed the code I experimented with regarding the Zip extraction, which will probably get addressed in a different PR.
- the tar extraction tests now test NONE, GZ, and BZIP2 compression
@MattesMrzik
Copy link
Contributor Author

For the zip extraction, see hint: #185 (comment).

hohwille added a commit to MattesMrzik/IDEasy that referenced this issue Jan 26, 2024
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in IDEasy board Jan 26, 2024
@hohwille hohwille added this to the release:2024.02.001 milestone Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working install installation process of IDE + tools and install commandlet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants