<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 19, 2018 at 8:34 AM, Bert Freudenberg <span dir="ltr"><<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""><div style="font-family:arial,helvetica,sans-serif;font-size:small;color:#000000"><span style="font-family:arial,sans-serif;color:rgb(34,34,34)">On 18 February 2018 at 23:30, Chris Cunningham </span><span dir="ltr" style="font-family:arial,sans-serif;color:rgb(34,34,34)"><<a href="mailto:cunningham.cb@gmail.com" target="_blank">cunningham.cb@gmail.com</a>></span><span style="font-family:arial,sans-serif;color:rgb(34,34,34)"> wrote:</span><br></div></span><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Based on feedback from last round of asking for feedback (and a few spelling fixes, too).  I have removed discussion about the method movement, since it *should* just work; and I've put pointers in for new topic on renaming classes.<div></div></div></blockquote></span></div></div></div></blockquote><div>... </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>* When you need to ensure a preamble or postscript script is run for a package, include the version in a new configuration (otherwise it could be skipped - trunk only loads the latest version of a package if specific versions are not called out in the update configuration)<br></div></div></blockquote><div><br></div></span><div><div style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">​Not quite. If you need a preamble/postscript to run again, it needs to be modified (just add a new line or space). </div><div style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)"><br></div><div style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">These scripts should be coded to be safely run multiple times. That means, each part of the script should check if it is necessary to run, then execute the needed system massaging. Newer additions should go on the bottom. It's not usually needed to remove older parts of the script when they are properly protected by these guards. But after some time passes (let's say after a new system release) then cleaning up these scripts is a good idea.</div><br></div><div><div style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">A modified script *​will* be run at the next update even if not mentioned explicitly in a config map. That is because as the last step of updating, the updater builds an implicit config map from all the latest packages in the default repository, and loads it.</div></div><span class=""><div><br></div></span></div></div></div></blockquote><div>I didn't get across what I wanted here.  The idea/concern was that if the script was added since the last update configuration, and is being changed again without the package being mentioned in the next update configuration, then one of the previous versions with the previous script needs to be specifically mentioned in an update configuration.  Otherwise folks updating from an older version of the image will skip all versions of the package with the older script.</div><div><br></div><div>Of course, that isn't even close to what I wrote, was it?</div><div><br></div><div>So, how about that mention of the scripts be removed from here.  Somehow the usage (what you said above) is added to the script editor for clarity of the user (and maybe documented in the Wiki for good measure as well).  And the release instructions mentions building a specific config map for the release (?) that ensures all past scripts have been run, so it is safe to clear them out if desired.</div><div><br></div><div>-cbc</div></div></div></div>