HubTurbo / addressbook

A small app for training and experimentation
Other
1 stars 3 forks source link

Upgrader should keep retrying when trying to replace updater #536 #537

Closed m133225 closed 8 years ago

m133225 commented 8 years ago

Fixes #536

m133225 commented 8 years ago

I don't think it's possible to upgrade the updater jar. 1) If updater jar is used as a run-time dependency: we cannot modify the updater jar 2) If updater jar is used as a separate application: we cannot pass it the notifier object for it to report its status

I think we require 2 components, each with its own update functionalities, in order to able to update any component as desired.

m133225 commented 8 years ago

Will try the following:

EDIT: Wait, isn't this just the same as the old architecture :confused: EDIT2: I think the old architecture would be the correct way to deal with this. Kind of a waste of time but at least refactoring (both code and dependencies) & extraction of launcher jar have been done. Also, we might want to put the updater jar back into the main application. @damithc Require your input. Will attempt to clear/resolve all updater stuff by tomorrow, even if it requires a large revert change.

damithc commented 8 years ago

The rule we did not consider in the new architecture is that a jar cannot modify another jar on which it has a runtime dependency?

m133225 commented 8 years ago

Yes, that's probably the case.

damithc commented 8 years ago

The solution is for the updatee to launch a separate updator app that will do the update when the updatee is no longer active?

m133225 commented 8 years ago

Yes, and the updater will be left alone after the update.