[bisq-network/bisq] Fix Windows installer issues (#2103)

[Devin Bileck]

devinbileck requested changes on this pull request.

Tor cleanup works, but might want to remove the backup once done with it. And this time I don't see the app folder being purged.

> +    finally
+      FindClose(FindRec);
+    end;
+  end
+    else
+  begin
+    Log(Format('Failed to list %s', [SourcePath]));
+  end;
+end;
+
+//Delete old app directory to prevent issues during update
+procedure DeleteOldAppDataDirectory;
+var
+  entry: String;
+begin
+  entry := ExpandConstant('{localappdata}') + '\Bisq\';

Is {localappdata} correct? I don't see it purging the app folder anymore with the latest exe, like it did in the previous exe. Did you use {app} previously?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/2103#pullrequestreview-183901915
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20181211/1db9cc18/attachment.html>