markcarver/Sparkle — comment on commit 3f1d59343168237e760951cb6026b26b8a6f9b34
File:  line 0
By andymatuschak:

I do appreciate your attempts to create consistency within the delegate methods!

> `mayUpdateAndRestart` calls the delegate method `updaterShouldRelaunchApplication:`. If the delegate returns `NO` then `installWithToolAndRelaunch:` will abort the update.

Hm. I’m not sure the semantics are that weird. I can imagine an app not wanting to be relaunched due to some long-running, non-abortable operation. The delegate’s really preventing the relaunch there, not the actual installation of the update.

> Rright now, with `SUAutomaticUpdateDriver`, Sparkle will wait for the 7 days before it will eventually show the alert dialog and prompt the user to install and restart. This delegate method allows the delegate to bypass this alert altogether and invoke `installWithToolAndRelaunch:`

But that would lead to the app being terminated without any user input!