gxensemble repl seems to be broken in master #1258

Open
opened 2024-07-26 00:55:09 +00:00 by teggotic · 1 comment
teggotic commented 2024-07-26 00:55:09 +00:00 (Migrated from github.com)

In current master branch (github.com/mighty-gerbils/gerbil@8e999aa1ee) if you try to run ensemble repl command, it would fail due to undefined variable.

However, if you compile the latest release branch v0.18.1, there would be no such issue.

image
image

I'm very new to gerbil (and to scheme languages as a whole), however, I assume that the problem appeared after github.com/mighty-gerbils/gerbil@341bc7699a commit, which removed the definition. Which means that there are only usages of the current-module-registry variable, both are in the src/tools/gxensemble.ss file, like github.com/mighty-gerbils/gerbil@8e999aa1ee/src/tools/gxensemble.ss (L700)

On that note, there is another problem, although it does not affect anything I've tried. The warning you can see on the screenshots above. It happens when running an ensemble tool.

*** WARNING -- Variable "find-library-module" used in module "gxensemble~0" is undefined

It seems like, the find-library-module one appeared after this commit github.com/mighty-gerbils/gerbil@d23b9aa070.

In current master branch (https://github.com/mighty-gerbils/gerbil/commit/8e999aa1ee2eeb4cb5402ef5a66e2fffca53ce24) if you try to run `ensemble repl` command, it would fail due to undefined variable. However, if you compile the latest release branch `v0.18.1`, there would be no such issue. ![image](https://github.com/user-attachments/assets/1fb22b2e-1814-4c6b-a38c-5e908dce38a4) ![image](https://github.com/user-attachments/assets/604b4614-0703-4a55-8642-c00ecc9a060f) I'm very new to gerbil (and to scheme languages as a whole), however, I assume that the problem appeared after https://github.com/mighty-gerbils/gerbil/commit/341bc7699abe4f9697a5f083b29fadf74215270f commit, which removed the definition. Which means that there are only usages of the `current-module-registry` variable, both are in the `src/tools/gxensemble.ss` file, like https://github.com/mighty-gerbils/gerbil/blob/8e999aa1ee2eeb4cb5402ef5a66e2fffca53ce24/src/tools/gxensemble.ss#L700 On that note, there is another problem, although it does not affect anything I've tried. The warning you can see on the screenshots above. It happens when running an ensemble tool. ``` *** WARNING -- Variable "find-library-module" used in module "gxensemble~0" is undefined ``` It seems like, the `find-library-module` one appeared after this commit https://github.com/mighty-gerbils/gerbil/commit/d23b9aa07005614db82d002d0398a24b0ad48b90.
vyzo commented 2024-07-26 05:18:56 +00:00 (Migrated from github.com)

Thanks for reporting!

Yes indeed, the repl has broken with the recent changes.

Thanks for reporting! Yes indeed, the repl has broken with the recent changes.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mighty-gerbils/gerbil#1258
No description provided.