Skip to content

Define {MEDLEY} as MEDLEYDIR, function MEDLEYDIR produces {MEDLEY} names#2566

Open
rmkaplan wants to merge 6 commits intomasterfrom
rmk177--MEDLEY-INIT-VARS-on-pseudohost-MEDLEY-MEDLEYDIR
Open

Define {MEDLEY} as MEDLEYDIR, function MEDLEYDIR produces {MEDLEY} names#2566
rmkaplan wants to merge 6 commits intomasterfrom
rmk177--MEDLEY-INIT-VARS-on-pseudohost-MEDLEY-MEDLEYDIR

Conversation

@rmkaplan
Copy link
Copy Markdown
Contributor

This is a stab at defining the pseudohost {MEDLEY} as the (possibly new) value of MEDLEYDIR when a sysout or makesys starts up, as discussed in #2563.

It also wraps PSEUDOFILENAME around the body of MEDLEYDIR, so that (MEDLEYDIR ...) produces a {MEDLEY} file name. The variable DISPLAYFONTDIRECTORIES is thus returned as {MEDLEY}/fonts/medleydisplayfonts, and so also for other MEDLEY-INIT-VARS (LISPUSERSDIRECTORIES etc.)

Marked as draft for now

@rmkaplan rmkaplan marked this pull request as draft April 16, 2026 00:51
@pamoroso
Copy link
Copy Markdown
Member

I tested the PR on Linux Mint 22.1 Cinnamon and it works as expected.

@rmkaplan
Copy link
Copy Markdown
Contributor Author

I worked a little more on this to get the {MEDLEY} pseudohost available early in the loadup, in particular before any fonts are loaded. I moved PSEUDOHOSTS into 1LISPSET, so it can create {MEDLEY} from the variable MEDLEYDIR, and then MEDLEYDIR in front of that so that MEDLEYDIR is defined in the loadup.

The function MEDLEYDIR then systematically returns the PSEUDOFILENAME of its arguments, so that all of the MEDLEY-INIT-VARS are now {MEDLEY} paths.

Seems to work.

@pamoroso
Copy link
Copy Markdown
Member

Doing the apps loadup at commit 193e85e fails with an error and drops to URAID:

loadup-error
paolo@lispmachine:~/medley/medley$ ./scripts/loadup-all.sh -apps
>>>>> START loadup-init
"/home/paolo/bin/lde" "/home/paolo/medley/medley/internal/loadups/starter.sysout" -id "loadup_init_1" -title "Medley::loadup_init_1" -g 1024x768 -sc 1024x768 -noscroll
MEDLEYDIR: "/home/paolo/medley/medley"
LOGINDIR: "/home/paolo/medley/medley/loadups/build/logindir"
GREET FILE: "/home/paolo/medley/medley/loadups/build/loadup-init.init"
REM.CM FILE: ""
VMEM FILE: "/home/paolo/medley/medley/loadups/build/logindir/vmem/lisp_loadup_init_1.virtualmem"
+++++ SUCCESS +++++
[...]
>>>>> START loadup-lisp-from-mid
"/home/paolo/bin/lde" "/home/paolo/medley/medley/loadups/build/init-mid.sysout" -id "loadup_lisp_from_mid_1" -title "Medley::loadup_lisp_from_mid_1" -g 1024x768 -sc 1024x768 -noscroll
MEDLEYDIR: "/home/paolo/medley/medley"
LOGINDIR: "/home/paolo/medley/medley/loadups/build/logindir"
GREET FILE: "/home/paolo/medley/medley/greetfiles/NOGREET"
REM.CM FILE: "/home/paolo/medley/medley/loadups/build/loadup-lisp-from-mid.cm"
VMEM FILE: "/home/paolo/medley/medley/loadups/build/logindir/vmem/lisp_loadup_lisp_from_mid_1.virtualmem"

*Error* URAID Called:
Enter the URaid
IL:FIND-DEBUGGER-ENTRY-FRAME

< lu
  0 :    0x12db2 : IL:RAID
  1 :    0x12d9c : IL:FAULTAPPLY
  2 :    0x12d50 : IL:\INTERPRETER
  3 :    0x12d2e : SI:*UNWIND-PROTECT*
  4 :    0x12cf6 : CL:ERROR
  5 :    0x12cda : CL:CERRORA0002
  6 :    0x12ca2 : CL:CERROR
  7 :    0x12c62 : IL:DOFILESLOAD
  8 :    0x12c48 : IL:LOADUP
  9 :    0x12c18 : IL:LOADUP-LISP
 10 :    0x12bf6 : IL:\EVALFORM
 11 :    0x12bdc : CL:PROGN
 12 :    0x12bbc : IL:\EVALFORM
 13 :    0x12ba8 : IL:EVAL
 14 :    0x12b8e : IL:EVALQT
 15 :    0x12b78 : IL:\REPEATEDLYEVALQT
 16 :    0x12b58 : IL:\EVALFORM
 17 :    0x12b0a : IL:\MAKE.PROCESS0
 18 :    0x11802 : CL:T

@pamoroso
Copy link
Copy Markdown
Member

This is the loadups directory of the failed loadup: loadups.zip

@rmkaplan
Copy link
Copy Markdown
Contributor Author

rmkaplan commented Apr 16, 2026 via email

@rmkaplan
Copy link
Copy Markdown
Contributor Author

rmkaplan commented Apr 16, 2026 via email

@pamoroso
Copy link
Copy Markdown
Member

Commit c29c3f0 fixes the loadup, which now completes successfully, and the pseudohost works as expected.

@pamoroso
Copy link
Copy Markdown
Member

Still working fine at commit c0bbc3d.

@rmkaplan rmkaplan marked this pull request as ready for review April 17, 2026 15:23
@rmkaplan
Copy link
Copy Markdown
Contributor Author

I upgraded this from Draft.

@rmkaplan
Copy link
Copy Markdown
Contributor Author

If/when this is merged, the next step would be to remove all of the entries in MEDLEY-INIT-VARS that call the function MEDLEYDIR, and to put those variables (with {MEDLEY} attached) back in their logical places (POSTSCRIPTFONTDIRECTORIES would appear only on POSTSCRIPTSTREAM, for example).

@pamoroso
Copy link
Copy Markdown
Member

Still no issues at commit 8c8b9f7.

@MattHeffron MattHeffron linked an issue Apr 19, 2026 that may be closed by this pull request
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.

Should PSEUDOHOSTS automatically define {MEDLEY}?

2 participants