Skip to content

Fix Issue #1848: Medley script (and hence loadup script) now checks for lde (ldeinit) on PATH #2129

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 28, 2025

Conversation

fghalasz
Copy link
Member

This PR modifies the medley script (and hence the loadup script) to check for and use lde (ldeinit) on PATH.

The following is now how medley script looks for the lde (ldeinit) executable:

  1. If the --maikodir argument is given, use the lde in OSVERSION.MACHINETYPE subdir of the value of the argument; if not found ->error
  2. If the MAIKODIR env variable is set, use the lde in the OSVERSION.MACHINETYPE subdir of the value of $MAIKODIR; if not found -> error.
  3. If neither --maikodir nor MAIKODIR, then look for lde in the following order:
    a. On the PATH
    b. In the OSVERSION.MACHINETYPE subdir of MEDLEYDIR/maiko
    c. In the OSVERSION.MACHINETYPE subdir of MEDLEYDIR/../maiko
    If lde not found in any of these -> error.

…ts on the PATH before looking for it in MEDLEYDIR/maiko or MEDLEYDIR/../maiko.
@fghalasz fghalasz requested a review from nbriggs April 28, 2025 06:08
@fghalasz fghalasz self-assigned this Apr 28, 2025
@pamoroso
Copy link
Contributor

I tested this PR on Linux Mint 22.1 Cinnamon and my usual apps loadup built successfully. I have nothing unusual to report.

Copy link
Contributor

@MattHeffron MattHeffron left a comment

Choose a reason for hiding this comment

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

Approved per discussion in meeting 4/28.

@masinter masinter merged commit f53da75 into master Apr 28, 2025
@fghalasz fghalasz deleted the fgh_maiko-PATH branch June 17, 2025 05:32
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