Tomasz Chmielewski wrote: > Tomasz Chmielewski wrote: >> [EMAIL PROTECTED] wrote: >>> [this time with the actual patch :-)] >>> >>> here is a patch against 0.9.9.test3 >>> >>> it handles: >>> * replace "if (debug)" with "dinfo" on several occasions (for readability) >>> * there is an error: lines #1734 and line #1740 are mixed up(1), >>> * in "reboot()" the fso "filesystemobject" is to be used by "default" only. >>> >>> (1) these lines ["requires an immediate reboot."] and ["is not an error."] >>> are exchanged within every version i can remember. >>> >>> so no important changes, just cosmetic ones. >> Hmm, when is "requires an immediate reboot" used anyway (some examples)? > > OK, I see, but there seems to be a bug in "upgarde" and "remove" with > regard to reboot="true" for a given exit code?
I have it, it was this code in upgrade and remove functions, and also existed in every version I can remember: :) // if found, command was successful if (exitNode != null) { info("Command in removal of " + packageName + " returned " + "non-zero exit code [" + result + "]. This exit code " + "is not an error."); continue; } -- Tomasz Chmielewski http://wpkg.org Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ wpkg-users mailing list wpkg-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wpkg-users