Skip to content

add BAKTRACE to HCFILES fails. Skip HC of CLTL2 (package errors) and dinfo (not really tedit).#2598

Merged
rmkaplan merged 5 commits into
masterfrom
lmm-hcfiles-improvements
May 11, 2026
Merged

add BAKTRACE to HCFILES fails. Skip HC of CLTL2 (package errors) and dinfo (not really tedit).#2598
rmkaplan merged 5 commits into
masterfrom
lmm-hcfiles-improvements

Conversation

@masinter
Copy link
Copy Markdown
Member

@masinter masinter commented May 8, 2026

No description provided.

@pamoroso
Copy link
Copy Markdown
Member

pamoroso commented May 8, 2026

On Linux Mint 22.1 Cinnamon HCFILES seems to run to completion and skip the CLTL2 directory as expected.

paolo@lispmachine:~/medley/medley/scripts$ ./do_hcfiles.sh 
"/home/paolo/bin/lde" "/home/paolo/medley/medley/loadups/apps.sysout" -id "hcfiles_1" -title "Medley::hcfiles_1" -g 1024x768 -sc 1024x768 -noscroll
MEDLEYDIR: "/home/paolo/medley/medley"
LOGINDIR: "/tmp/hcfiles-75506"
GREET FILE: "/home/paolo/medley/medley/greetfiles/NOGREET"
REM.CM FILE: "/tmp/hcfiles-75506/hcfiles.cm"
VMEM FILE: "/tmp/hcfiles-75506/vmem/lisp_hcfiles_1.virtualmem"
Added hcfiles.dribble to /home/paolo/medley/medley/loadups
Added hcfiles-fails.txt to /home/paolo/medley/medley/loadups

Here are the dribble files:

@masinter masinter requested a review from rmkaplan May 8, 2026 18:15
@rmkaplan
Copy link
Copy Markdown
Contributor

sources/mesatypes is just the txt file. Did you mean a lispsource file?

@pamoroso
Copy link
Copy Markdown
Member

It still works with no issues at commit 5e88d17.

@masinter
Copy link
Copy Markdown
Member Author

@rmkaplan I'm not sure what you're looking at but the result of this PR is that there is a MESATYPES, MESATYPES.TXT, MESATYPES.LCOM in the {MEDLEY}, where MESATYPES is the lisp source and MESATYPES.TXT is the documentation for it, and there are no files MESATYPES* (of either case) in {MEDLEY}.

@rmkaplan
Copy link
Copy Markdown
Contributor

rmkaplan commented May 10, 2026 via email

@rmkaplan
Copy link
Copy Markdown
Contributor

But if MESATYPES is going to be moved from internal/ into sources/ and perhaps become part of a release, the documentation needs to be upgraded. At least with one or two examples of what one of these new declarations might look like. Otherwise it really is internal.

@masinter
Copy link
Copy Markdown
Member Author

masinter commented May 10, 2026 via email

@rmkaplan
Copy link
Copy Markdown
Contributor

If there are examples somewhere, maybe some of them should be copied into the documentation as illustrations of what functionality this provides.

@nbriggs
Copy link
Copy Markdown
Contributor

nbriggs commented May 10, 2026

MESATYPES is used by DOVEDECLS, DOVEETHERDECLS and LOCALFILE to access the mesa records (with mesa type declarations) in interfaces exported by the mesa code running on the I/O processor and the file system structures.

We include LOCALFILE in LOADUP-LISP, so indirectly we use MESATYPES.

There are examples of MESATYPE and MESARECORD declarations in the source files that use it.

@masinter
Copy link
Copy Markdown
Member Author

if we do #2565 then there's a good chance MESATYPES will also become unused and can be deleted or moved.

for now, though, without editing the content, the rename to sources seems best.

@masinter
Copy link
Copy Markdown
Member Author

masinter commented May 11, 2026 via email

@rmkaplan
Copy link
Copy Markdown
Contributor

One of the Tedit ones was a bug in reading Tedit file-format 0 (from early 1985). I'll put the fix in a separate mini-Tedit PR I'm about to push.

A bunch of others were font-not-found for size 13. Those are probably 10/13 swaps. Some are missing image object fns. And then there's the rest

I think the dribble file should be opened as UTF-8. You might be able to specify that by passing an open UTF-8 stream to DRIBBLE:
(DRIBBLE (OPENSTREAM 'foo.dribble 'OUTPUT 'NEW '((FORMAT :UTF-8))))

@pamoroso
Copy link
Copy Markdown
Member

Still looking good at commit a08e4b8.

Copy link
Copy Markdown
Member

@pamoroso pamoroso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@rmkaplan rmkaplan merged commit 01c90e4 into master May 11, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants