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

Linux .deb packaging - postrm is removing user configuration from home directory!? #470

Closed
frederickjh opened this issue Nov 6, 2017 · 3 comments
Assignees
Labels
bug Something aint working right!
Milestone

Comments

@frederickjh
Copy link
Contributor

Hi!
In examining the .deb of the newest version of Lando I found that on line 13 of postrm the user configuration for Lando is being removed from their home directory.

rm -rf "$HOME/.lando"

This does not seem like it is correct. Programs should not be deleting a user's configuration for them that is in their home directory.

If a user installs Lando then changes their personal configuration then uninstalls it to install a new version they would lose all their configuration changes. This is not nice for the user.

Please consider removing this from the postrm script.

Thanks for your consideration of this!
Frederick

@pirog
Copy link
Member

pirog commented Nov 6, 2017

@frederickjh, im guessing this is a hold over from our older kalabox project which sadly required this for upgrading.

Feel free to submit a PR that removes the line and note that it looks like we also do this on OSX
https://github.com/lando/lando/blob/master/installer/osx/uninstall.sh#L26

@frederickjh
Copy link
Contributor Author

@pirog I created pull request #475 to fix this issue for Linux and also for OSX.

@pirog
Copy link
Member

pirog commented Nov 10, 2017

Awesome! Merged this in and will roll it out in our next release.

@pirog pirog added the bug Something aint working right! label Nov 10, 2017
@pirog pirog added this to the 3.0.0 milestone Nov 10, 2017
@pirog pirog closed this as completed Nov 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something aint working right!
Projects
None yet
Development

No branches or pull requests

2 participants