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

dokan_wix: Fix KB3033929-check #266

Merged
merged 1 commit into from Jun 25, 2016
Merged

Conversation

Rondom
Copy link
Contributor

@Rondom Rondom commented Jun 20, 2016

Not yet tested, don't merge

Due to some oddity, we need to specify a version that is one version
lower than the one we are actually searching for.
References: #260, #244, #187

Quoting WiX-documentation:
Important: When doing a locale-neutral search for a file, you must set
the MinVersion property to one revision number lower than the actual
version you want to search for.

Due to some oddity, we need to specify a version that is one version
lower than the one we are actually searching for.
References: dokan-dev#260, dokan-dev#244, dokan-dev#187

Quoting WiX-documentation:
Important: When doing a locale-neutral search for a file, you must set
the MinVersion property to one revision number lower than the actual
version you want to search for.
@Rondom
Copy link
Contributor Author

Rondom commented Jun 25, 2016

Done tests. Works and detects both missing and installed update correctly.

@Liryna Liryna merged commit d0ceb94 into dokan-dev:master Jun 25, 2016
@Liryna
Copy link
Member

Liryna commented Jun 25, 2016

Ah @Rondom ! Thank you for your time and contribution as always ❤️
very appreciate !

@Rondom Rondom deleted the win7_install branch September 13, 2016 15:55
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