openvpn-gui icon indicating copy to clipboard operation
openvpn-gui copied to clipboard

Installation fails with "incorrect" installer name

Open Nightmaster opened this issue 5 years ago • 4 comments

Hello.

I found a small but annoying problem with the installer. If the installer itself is name "openvpn.exe" (case insensitive, I think, but not sure), the installation process fails cause it tries to kill all "openvpn.exe" processes, and in this case itself...

Solution came from an old post on the forum. If this problem could be solved it would be great!

Thanks in advance

Nightmaster avatar Nov 27 '19 10:11 Nightmaster

Hi,

On Wed, Nov 27, 2019 at 02:51:40AM -0800, BLAISE wrote:

I found a small but annoying problem with the installer. If the installer itself is name "openvpn.exe" (case insensitive, I think, but not sure), the installation process fails cause it tries to kill all "openvpn.exe" processes, and in this case itself...

Solution came from an old post on the forum. If this problem could be solved it would be great!

This is very much a case of "pilot error". Don't rename installers to the same name of the program they will install. Nothing good will ever come out of it.

@mattock: can we make the installer abort if called as openvpn.exe?

gert

-- "If was one thing all people took for granted, was conviction that if you feed honest figures into a computer, honest figures come out. Never doubted it myself till I met a computer with a sense of humor." Robert A. Heinlein, The Moon is a Harsh Mistress

Gert Doering - Munich, Germany [email protected]

cron2 avatar Nov 27 '19 20:11 cron2

an idea to stop processes is to unlock executable files. however, it might be TOO offensive. killing openvpn.exe does not mean it is exactly the binary we are trying to override.

maybe we should find more intelligent way ? @rozmansi , @mattock as for MSI installer, how close are we to it ? does MSI provide such intelligent way to override locked files ?

chipitsine avatar Nov 27 '19 20:11 chipitsine

and answering to "can we make the installer abort..." - it seems to make sense. I'll send patch soon

chipitsine avatar Nov 27 '19 20:11 chipitsine

@chipitsine thanks! That would definitely useful.

mattock avatar Nov 29 '19 11:11 mattock