[wpkg-users] WPKG 1.1.0 reboot loop during upgrade

[Mark Cooke]

On Tue, 2009-07-07 at 21:34 +0200, Rainer Meier wrote:
> Hi Mark,
> 
> Mark Cooke wrote:
> > I updated the packages.xml wiki with some information about the options for
> > 'exit code reboot' and 'package wide reboot'.
> 
> Thanks for the contribution!


> > I've realised that my confusion comes from the two different meanings for
> > 'reboot=true'.
> > 
> > Reboot at the package level:
> > 
> > True      :  Reboot after all processing is complete
> 
> Actually this means reboot after all processing on the _current_ package is
> done. If there are multiple packages to be processed the reboot will happen
> right after the package which specified reboot=true (no matter if more packages
> are scheduled to be run).

Yes.  Sloppy wording in my email.  The wiki text I inserted was 'true -
always reboot after processing this package'.

> > False     :  Never reboot for this package
> 
> Correct.
> 
> 
> > Delayed   :  Equivalent to never (as it is 'not true')
> 
> There is no such thing like a delayed reboot on package level - it's ignored and
> therefore equal to false. So your statement above is true.

I was double checking :)

> > Postponed :  Equivalent to never (as it is 'not true')
> 
> No.
> If you specify reboot=postponed at package level the system will reboot after
> WPKG termination (all packages processed) in case the package checks yield true
> (successfully installed). If the checks are wrong instead WPKG will regard
> failed checks as a warning instead of an error because it might be that the
> software package is installed successfully but checks are false until reboot.

*red face*

I managed to read an old v0.9 wpkg.js when I checked the package level
flag processing, where the code is just 'true' or 'anything else'.
Having read the actual 1.1 source, then all this interaction is nicely
commented there...

I've fixed the wording on the wiki.

> > Reboot at the exit-code level:
> > 
> > True      :  Reboot immediately this command is done
> 
> Correct.
> 
> 
> > Delayed   :  Reboot at end of this package processing
> 
> Correct.
> 
> 
> > Postponed :  Reboot after all packages are processed
> 
> Correct.
> 
> 
> br,
> Rainer