medley icon indicating copy to clipboard operation
medley copied to clipboard

Hyphenate file names in library/lafite/

Open rmkaplan opened this issue 2 years ago • 3 comments

Except for LAFITE-INDENT, none of the filenames in the lafite subdirectory conform to the hyphenated filename convention uses by TEDIT, TMAX, and now SKETCH.

It appears that, as a consequence, the lafite files are not analyzed when the whole-system masterscope database is constructed. And functions on them are not found by WHEREIS unless connected to that subdirectory.

The hyphen convention has proven to be useful and robust. I propose renaming the Lafite files according to this convention.

rmkaplan avatar Jan 16 '24 03:01 rmkaplan

i'm uneasy about this special rule ... but can we write it up in the README for the medley repo?

masinter avatar Jan 16 '24 04:01 masinter

Unless we come up with a better solution for organizing all the subfiles of a larger application, I think this is effective and good enough. An additional important advantage of having all such files in a separate subdirectory is that you have a natural way of focusing the targets of FB, FINDCALLERS, EDITCALLERS.

rmkaplan avatar Jan 16 '24 18:01 rmkaplan

The file LAFITE has a variable LAFITEFILES that includes OLDNSMAIL as well as NSMAIL and NEWNSMAIL.

The files that Lafite actually loads are specified elsewhere, so this list may just be away of setting up a Lafite development environment.

But OLDNSMAIL doesn't appear to exist. Is that just a hangover from an early phase?

rmkaplan avatar Feb 24 '24 06:02 rmkaplan

This looks like it has been done. (hyphenation of file names) If so, please close as complete.

MattHeffron avatar Jun 03 '24 22:06 MattHeffron

Completed and closed

rmkaplan avatar Jun 04 '24 04:06 rmkaplan

lispusers/UNDIGESTIFY was not updated to refer to LAFITE-DECLS instead of LAFITEDECLS.

nbriggs avatar Jun 04 '24 05:06 nbriggs

Oops, new pr makes the change.

On Jun 3, 2024, at 10:19 PM, Nick Briggs @.***> wrote:

lispusers/UNDIGESTIFY was not updated to refer to LAFITE-DECLS instead of LAFITEDECLS.

— Reply to this email directly, view it on GitHub https://github.com/Interlisp/medley/issues/1509#issuecomment-2146631642, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQSTUJLL2TNHCTC7YWJLEUDZFVE6DAVCNFSM6AAAAABB4EJ3Y6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGYZTCNRUGI. You are receiving this because you modified the open/close state.

rmkaplan avatar Jun 04 '24 06:06 rmkaplan