Skip to content

GPII-3092: Removed installer#179

Merged
amb26 merged 7 commits intoGPII:masterfrom
stegru:GPII-3092
Jul 25, 2018
Merged

GPII-3092: Removed installer#179
amb26 merged 7 commits intoGPII:masterfrom
stegru:GPII-3092

Conversation

@stegru
Copy link
Member

@stegru stegru commented Jun 1, 2018

Removed the installer - only the installer from gpii-app is required.

@gpii-bot
Copy link

gpii-bot commented Jun 1, 2018

CI job failed: https://ci.gpii.net/job/windows-tests/510/

@gpii-bot
Copy link

gpii-bot commented Jun 1, 2018

CI job failed: https://ci.gpii.net/job/windows-tests/511/

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/546/

jqUnit.assertTrue("Testing a registry key that always exists.",
gpii.deviceReporter.registryKeyExists("HKEY_CURRENT_USER",
"Software\\Microsoft\\Command Processor", "CompletionChar", "REG_DWORD"));
gpii.deviceReporter.registryKeyExists("HKEY_LOCAL_MACHINE",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Has this picked up something from some ancient/stray branch? The change looks reasonable, but why has it showed up in this diff?

Copy link
Member Author

@stegru stegru Jul 11, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removed key was added by the Wix installer compiler when it's installer, but now that wix has been removed, this key is no longer being created.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did the Wix installer compiler tinker with the command processor completion char?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know.

There is the same key in HKEY_LOCAL_MACHINE. I wonder if the installer started cmd, which caused the value to be read from HKLM then written back to HKCU.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh I'm talking about the installer for WiX - the installation process of the Wix toolset.

@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/584/

# Conflicts:
#	Vagrantfile
#	provisioning/Installer.ps1
@gpii-bot
Copy link

CI job passed: https://ci.gpii.net/job/windows-tests/585/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants