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

MSI User Interface #625

Merged
merged 14 commits into from Jul 13, 2020
Merged

Conversation

directionless
Copy link
Contributor

@directionless directionless commented Jul 6, 2020

Add a basic user interface to the MSI. This is based on wix's WixUI_Minimal, but removes some of the screens we don't need. This is not currently exposed through package-builder

An additional notable change is to the MSI's ProductCode. This introduces a random factor, ensuring that each MSI has a unique ProductCode. Which means that any MSI will be able to install over other versions. This is an unusual choice from an MSI perspective, but it fit with out minimal usage of it

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Jul 6, 2020

This pull request introduces 1 alert when merging cc63b83 into 8743fe2 - view on LGTM.com

new alerts:

  • 1 for Useless assignment to local variable

@directionless directionless marked this pull request as ready for review July 6, 2020 21:53
@directionless directionless requested review from blaedj and jnog July 9, 2020 13:30
Copy link
Contributor

@blaedj blaedj left a comment

Choose a reason for hiding this comment

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

lgtm, leaning on the assumption that this has been tested by @FritzX6 and @terracatta as well.

pkg/packagekit/wix/doc.go Outdated Show resolved Hide resolved
pkg/packagekit/wix/wix.go Outdated Show resolved Hide resolved
directionless and others added 2 commits July 13, 2020 13:15
Co-authored-by: blaedj <blaedj@users.noreply.github.com>
@directionless directionless merged commit d4a5205 into kolide:master Jul 13, 2020
@directionless directionless deleted the seph/msi-confirmation branch July 13, 2020 18:05
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.

None yet

2 participants