-
Notifications
You must be signed in to change notification settings - Fork 294
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
Add bal:DisplayFilesInUseDialogCondition attribute to disable "Files In Use" #573
base: main
Are you sure you want to change the base?
Add bal:DisplayFilesInUseDialogCondition attribute to disable "Files In Use" #573
Conversation
Disabling display skips showing the "Files In Use" dialog and returning a result as if the user had chosen to ignore the dialog and reboot in the case of files that were unable to be replaced.
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request |
@@ -1231,34 +1231,52 @@ class CWixStandardBootstrapperApplication : public CBootstrapperApplicationBase | |||
__inout int* pResult | |||
) | |||
{ | |||
HRESULT hr = S_OK; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wondered if the DisplayFilesInUseDialogCondition
should be evaluated in OnExecutePackageBegin
or OnPlanPackageBegin
instead. I left it here because that's closest to its only use and that seemed clearer. Happy to move it if either of those are more appropriate places.
I have read the CLA Document and I hereby sign the CLA. |
recheck. |
@charles-juicelabs The CLA bot is very particular about the message. I believe the |
We develop an application with a DLL that is loaded into many processes, largely outside of our control. It is confusing for our users when they are prompted to close these seemingly unrelated applications when we update the DLL. Updating the DLL requires a reboot independently of any files being in use anyway, so hiding the "Files In Use" dialog is a nice solution for us.
There are other examples of people wanting to disable the "Files In Use" dialog:
Add
bal:DisplayFilesInUseDialogCondition
, similar tobal::DisplayInternalUICondition
, to be added toMsiPackage
elements to control the display of "Files In Use" inWixStandardBoostrapperApplication
.For example:
Fixes wixtoolset/issues#8807.