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

install.sh doesn't clean up after installing or updating #6067

Open
rvdm opened this issue Jul 3, 2024 · 0 comments · May be fixed by #6068
Open

install.sh doesn't clean up after installing or updating #6067

rvdm opened this issue Jul 3, 2024 · 0 comments · May be fixed by #6068
Labels
🐛 bug Something isn't working as expected.

Comments

@rvdm
Copy link

rvdm commented Jul 3, 2024

Current Behavior

When running install.sh, it creates a temporary archive as a target for the download. The target archive gets installed, but does not get removed afterwards.

Expected Behavior

temporary files get cleaned after installation was done

Additional context/Screenshots

n/a

Possible Solution

I think the fix is to do a check of the existence of the archive (and/or any result from unpacking) and to remove that file before returning. This can either be done in the install() function or in the main loop.

Environment

  • Starship version: 1.19.0
  • zsh version: zsh 5.8 (x86_64-debian-linux-gnu)
  • Operating system: Debian 11.0.0
  • Terminal emulator: iTerm2 3.4.19
  • Git Commit Hash:
  • Branch/Tag: master
  • Rust Version: rustc 1.78.0 (9b00956e5 2024-04-29)
  • Rust channel: release
  • Build Time: 2024-05-15 17:09:04 +00:00

All of the above is one of my starship installs. I reproduced this on all of my linux machines.

Relevant Shell Configuration

n/a

@rvdm rvdm added the 🐛 bug Something isn't working as expected. label Jul 3, 2024
@cmd-devel cmd-devel linked a pull request Jul 3, 2024 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working as expected.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant