<div id="__MailbirdStyleContent" style="font-size: 10pt;font-family: Arial;color: #000000;text-align: left" dir="ltr">
                                        Hi Dave --<div><br></div><div>> <span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">So I think that disabling the preference should simply go back to the</span></div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">> traditional way of reading the sources file, nothing more.</span><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">And that's not very user friendly. Users that just play around with the settings might disable the cache and then save the image. Boom! They get errors and have no idea where to get the .sources file. Posts on squeak-dev will appear. How remembers the "traditional way" just in time? Not many, I suppose. That issue can easily be avoided. I guess we have all the information. All preferences should work in both states, enabled and disabled. The system should take care of this.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Anyway, you may push the current version into Trunk. :-) But I would really want to tweak the user experience a little bit more to avoid unnecessary surprises. But we can do that later on.</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px"><br></span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Best,</span></div><div><span style="font-family: Arial, Helvetica, sans-serif;font-size: 13px">Marcel</span></div><div class="mb_sig"></div>
                                        <blockquote class="history_container" type="cite" style="border-left-style: solid;border-width: 1px;margin-top: 20px;margin-left: 0px;padding-left: 10px;min-width: 500px">
                        <p style="color: #AAAAAA; margin-top: 10px;">Am 24.01.2022 23:56:04 schrieb David T. Lewis <lewis@mail.msen.com>:</p><div style="font-family:Arial,Helvetica,sans-serif">On Mon, Jan 24, 2022 at 03:46:06PM +0100, Marcel Taeumel wrote:<br>><br>> - In CompressedSources class >> #initializeSource:, you forgot a #closeSourceFiles before that #openSourceFiles<br>> - In CompressedSources class >> #initializeSource:, you should support a "nil" argument so that all pragma preferences can be reset; in this case a "nil" argument should just leave the current preference value as is<br>> <br><br>Thanks, I updated these in DTL-internal-sources-dtl.10 in the inbox.<br><br>><br>> - If you disable that preference, it should write the source to the file system again if it cannot find a file already.<br>><br><br>I would prefer not to write anything to the file system because the sources<br>file normally comes with the Squeak distribution. It might be located on<br>various places, but it is basically a read-only thing that normally should<br>not be modified or replaced by something that we do in the image.<br><br>So I think that disabling the preference should simply go back to the<br>traditional way of reading the sources file, nothing more.<br><br>><br>> Yet, I like it that you do not delete the .source file after you enabled that preference bc. more than one .image might already on that file.<br>><br><br>Agreed, I do not think that any existing source files should be altered<br>or replaced.<br><br>><br>> ...or maybe disabling the preference MUST always write out that .source file and complain if there is already such a file. Otherwise you cannot guarantee that you have some .source that match your .image? Hmm...<br>><br><br>I do think that it might make sense, when disabling the preference, to<br>first check if a file-based sources file can be located. If no file is<br>found, then do not allow the preference to be disabled. And as a separate<br>matter, I think that we have some utility methods for removing "developer<br>tools" (I don't recall the details, as I never do that) and those methods<br>would need to remove the cached sources without a check.<br><br>If we want to do this, then I think I would like to first merge System-dtl.1277<br>from the inbox, which will make it easier to check if a sources file can<br>be located on the file system.<br> <br>> There seems to be another bug. After switching to the cached .sources file, I cannot "recover changes" anymore, which is strange since the .changes file is not touched.<br><br>I am not seeing this, but I suspect it is a side effect of the first<br>issue that you spotted. I was forgetting to close the sources files<br>before reopening them, and perhaps on Windows this is preventing access<br>to the already open changes file for "recover changes".<br><br>Thanks a lot for the review :-)<br><br>Dave<br><br><br></div></blockquote></div>