Skip to content
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

Tcl 9 support #524

Open
garfieldnate opened this issue Dec 4, 2024 · 1 comment
Open

Tcl 9 support #524

garfieldnate opened this issue Dec 4, 2024 · 1 comment

Comments

@garfieldnate
Copy link
Collaborator

The default brew-installed version of Tcl is now 9.0. The release page lists a few incompatibilities: https://www.tcl-lang.org/software/tcltk/9.0.html. For now, I am pinning the build to Tcl 8, but it would be good to know if our tiny bit of Tcl code is compatible with 9.0, and if/when it will be desirable to update the Soar build to use 9.0.

garfieldnate added a commit that referenced this issue Dec 4, 2024
Revert the last commit, which didn't help find the correct Tcl installation.

Since the default brew-installed version of Tcl is now 9.0, the runner image
installs Tcl 8 using the prefix `tcl-tk@8`. Try both prefixes and use the one
that works.

Opened #524, as well, to consider whether we need to support Tcl 9 in the
future.
@garfieldnate
Copy link
Collaborator Author

Note that running Tcl 9's tclsh on our test Tcl script, after compiling Soar against Tcl 8, gives a segfault.

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

No branches or pull requests

1 participant