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

Warn me if uninstallation isn't 100% successful #45

Open
igitur opened this issue Nov 18, 2019 · 7 comments
Open

Warn me if uninstallation isn't 100% successful #45

igitur opened this issue Nov 18, 2019 · 7 comments
Labels

Comments

@igitur
Copy link

@igitur igitur commented Nov 18, 2019

Hi,

Often during uninstallation or upgrade, the package directory isn't removed fully, due to various reasons (e.g. the shell is holding on to a .dll file).

During upgrade, I would then end up with a duplicate directory, e.g this:
image

I'd like the option to try forcefully delete the original directory first so that the new package can be installed in the original directory instead of defaulting to a _2 version.

For pure uninstallation, I think a normal message to inform the user is sufficient.

For upgrade, I think the process should be paused and the user should have a choice of either manually deleting the original directory, or there should be an option for NPackd to proceed and attempt to install the newer package in the original directory anywa.

@tim-lebedkov

This comment has been minimized.

Copy link
Owner

@tim-lebedkov tim-lebedkov commented Nov 18, 2019

Npackd already tries different options to delete a directory completely. Do you have these options enabled?

Bildschirmfoto vom 2019-11-18 19-30-42

@igitur

This comment has been minimized.

Copy link
Author

@igitur igitur commented Nov 18, 2019

Yes, I do. But there are still some cases where it would fail. I don't expect NPackd to fix that. But I would like to be warned when that happens so that I don't end up with duplicate directories.

@tim-lebedkov

This comment has been minimized.

Copy link
Owner

@tim-lebedkov tim-lebedkov commented Nov 18, 2019

Sorry, didn't want to close the issue.

@tim-lebedkov tim-lebedkov reopened this Nov 18, 2019
@tim-lebedkov tim-lebedkov transferred this issue from tim-lebedkov/npackd Nov 23, 2019
tim-lebedkov added a commit that referenced this issue Dec 1, 2019
@tim-lebedkov

This comment has been minimized.

Copy link
Owner

@tim-lebedkov tim-lebedkov commented Dec 1, 2019

Here is how it looks in the console:

Bildschirmfoto vom 2019-12-01 12-35-48

@tim-lebedkov

This comment has been minimized.

Copy link
Owner

@tim-lebedkov tim-lebedkov commented Dec 1, 2019

Here is how it looks in the event log:

Bildschirmfoto vom 2019-12-01 12-37-34

tim-lebedkov added a commit that referenced this issue Dec 2, 2019
@tim-lebedkov

This comment has been minimized.

Copy link
Owner

@tim-lebedkov tim-lebedkov commented Dec 2, 2019

Here is how it looks in the UI:

Bildschirmfoto vom 2019-12-02 19-58-55

tim-lebedkov added a commit that referenced this issue Dec 3, 2019
@tim-lebedkov

This comment has been minimized.

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