-
-
Notifications
You must be signed in to change notification settings - Fork 13k
bsc: uses_from_macos "tcl-tk", backport TCL 9 support #236366
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
base: main
Are you sure you want to change the base?
Conversation
|
||
uses_from_macos "bison" => :build | ||
uses_from_macos "flex" => :build | ||
uses_from_macos "libffi" | ||
uses_from_macos "perl" | ||
uses_from_macos "tcl-tk" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macOS tcl-tk
is old and deprecated; we probably shouldn't be using it anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would need upstream changes as they specifically set to system copy - https://github.com/B-Lang-org/bsc/blob/main/platform.sh#L81-L86
if [ ${OSTYPE} = "Darwin" ] ; then
# Have Makefile avoid Homebrew's install of tcl on Mac
TCLSH=/usr/bin/tclsh
else
TCLSH=`which tclsh`
fi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably worth clarifying with upstream if this change, introduced over 4 years ago, still makes sense, especially since:-
sonoma$ /usr/bin/tclsh
WARNING: This version of tcl is included in macOS for compatibility with legacy software.
In future versions of macOS the tcl runtime will not be available by
default, and may require you to install an additional package.
%
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe I'll try make TCLSH=...
to see if it overrides lookup.
EDIT: insufficient as still used to find tclversion and tclinc:
Using tclsh: /opt/homebrew/opt/tcl-tk/bin/tclsh
Tcl version: 8.5
Using Tcl defines: TCL85
Using tcl include flags:
Using tcl library flags: -ltcl8.5
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Upstream should be aware that system Tcl is outdated and has mentioned interest in moving away from it back in 2022:
Someone in issue discussion patched out the system Tcl logic like: kammoh/bsc@ded6d77
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?Looks like this uses system TCL on macOS - https://github.com/B-Lang-org/bsc/blob/main/platform.sh#L81-L86