[squeak-dev] Why not just get rid of the sources file entirely (was: The Inbox: System-dtl.1277.mcz)

Jakob Reschke jakres+squeak at gmail.com
Mon Jan 17 19:47:03 UTC 2022


Hi Dave,

What about the case when Squeak must be stopped without the possibility to
save the image (destroyed state, Vm crash, ...)? Would we still be able to
recover changes in another way?

Kind regards,
Jakob

David T. Lewis <lewis at mail.msen.com> schrieb am Mo., 17. Jan. 2022, 19:17:

> Rather that putting a lot of effort into figuring out how to package
> the sources file and how to locate sources files in various places
> for various different operating systems, why don't we just get rid
> of the file entirely?
>
> Traditionally the sources file is delivered as a disk file separate
> from the images file. That is good if you are dealing with small images
> and VMs with limited address space, but nowadays our 64-bit release
> image is over 50MB in size, and typical working images get much larger
> than that. Meanwhile, a compressed sources file is only(!) 6.7MB on disk.
>
> So rather than use files, just keep the sources in the image. It would
> work like this:
>
> - Load the compressed sources file into an object, hold it in a class var
> - If the in-image sources exist, use them, otherwise use file based sources
> - Allow the in-image sources to be cleared so that small images or images
>   transmitted over the network do not need to carry the sources along
> - Existing disk-based sources lookup would work as before
>
> I actually have this working now, and I am not seeing any problems when
> using the in-image sources. The sources file never needs to be reopened
> or located from the file system.
>
> The attached png shows an explorer on the sources file array in my
> image after changing over to the in-image compressed sources. The raw
> compressed data is highlighted in the explorer.
>
> If this sounds worthwhile, I'll clean it up a bit and post a change set
> or inbox submission.
>
> Dave
>
>
> On Mon, Jan 17, 2022 at 12:56:01PM +0100, Marcel Taeumel wrote:
> > Hi Dave --
> >
> > +1
> >
> > Did you double-check whether it is safe to remove that interface on
> FileDirectory class???
> >
> > Best,
> > Marcel
> > Am 02.01.2022 09:42:09 schrieb David T. Lewis <lewis at mail.msen.com>:
> > See discussion at:
> >
> >
> http://lists.squeakfoundation.org/pipermail/squeak-dev/2022-January/218008.html
> >
> > Dave
> >
> > On Sat, Jan 01, 2022 at 08:47:18PM +0000, commits at source.squeak.org
> wrote:
> > > A new version of System was added to project The Inbox:
> > > http://source.squeak.org/inbox/System-dtl.1277.mcz
> > >
> > > ==================== Summary ====================
> > >
> > > Name: System-dtl.1277
> > > Author: dtl
> > > Time: 1 January 2022, 2:57:48.389886 pm
> > > UUID: 4d975f20-d62d-4b7e-8abb-bc511d976834
> > > Ancestors: System-mt.1276
> > >
> > > Look for the sources file in well-known locations on some platforms.
> For Unix, if not found in the usual locations then look in
> /usr/share/squeak and /usr/local/share/squeak. Hooks for other platforms
> may be added to SmalltalkImage>>sourcesFilePaths.
> > >
> > > Refactor to reduce duplication of logic in locateSourcesEntry and
> openSources:forImage: and to ensure that the full set of file paths is
> searched for stc compressed sources files prior to searching for regular
> sources files.
> > >
> > > =============== Diff against System-mt.1276 ===============
> > >
> >
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20220117/4ad14aab/attachment.html>


More information about the Squeak-dev mailing list