aardappel / treesheets

TreeSheets : Free Form Data Organizer (see strlen.com/treesheets)
zlib License
2.6k stars 192 forks source link

Warn users when installing into a non-empty directory #256

Open kornelijepetak opened 2 years ago

kornelijepetak commented 2 years ago

I have (or better yet: had) a bunch of software installed at C:\software\development.

I wanted to install your app at C:\software\development\treesheets but I pressed next too early, so it installed all the files into the C:\software\development instead. So, instead of manually deleting all the files and folders that the installer put there, I ran the uninstaller.

Well, the important part is that once the uninstaller removed the files of the application, it started to remove the root folder of the installation. This should not be the case. The uninstaller should NOT remove the root folder if it has not created it or it contains files not created by the installer.

You can't fix my problem now, but maybe put this warning somewhere or replace/fix the un/installer.

aardappel commented 2 years ago

Yup that is lazyness.. here's the offending section of the installer script:

https://github.com/aardappel/treesheets/blob/master/TS_installer.nsi#L104-L112

That would need to be replaced by an exact list of the files it installs.

That said, you're the first to install it into a dir with existing files, it is normally not a problem.

kornelijepetak commented 2 years ago

As I said, I have to take care of the fallout myself 😄 It's my fault to have the \ character immediately next to an Return/Enter. 🤷‍♂️

Ok, after several hours of reinstalling various applications, let me now get this TreeSheets into the right folder 😄

stevehansen commented 2 years ago

Easier "fix" might be to warn when the installation folder isn't empty?

aardappel commented 2 years ago

Well, the folder is never empty because it at least has the TS files in it.

PRs for more precise installer behavior welcome.

stevehansen commented 2 years ago

I see my message wasn't as clear as I intended, warn during installation that the installation folder isn't empty.

kornelijepetak commented 2 years ago

I have changed the title of the issue in order to make this issue more of a real issue than a rant. Somebody more versed in NSIS could maybe know how to do this easily. I never used it so I can't make a PR. But at least now, it's an issue, and not just my rant. 😉