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.

Sign up for GitHub

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

Jump to bottom

tclsh not installed into right location #24

Closed
flokli opened this issue Feb 6, 2020 · 2 comments
Closed

tclsh not installed into right location #24

flokli opened this issue Feb 6, 2020 · 2 comments

Comments

@flokli
Copy link
Contributor

flokli commented Feb 6, 2020

tclIndex.sh tries to invoke tclsh, however, the tclsh binary gets installed into a wrong location,

source/src/tcltk/tcltk8.5.4/tcl8.5.4/unix/../../../inst/bin/tclsh8.5, which means source/src/tcltk/inst/bin/tclsh8.5.

I assume this wasn't caught, because there's another tclsh in $PATH.
@flokli
Copy link
Contributor Author

flokli commented Feb 6, 2020

Except for the HTcl part, we probably could also simply use tcl from $PATH.

@thoughtpolice
Copy link
Contributor

See also #28.

thoughtpolice added a commit to thoughtpolice/bsc that referenced this issue Feb 7, 2020
@thoughtpolice
fix(bluetcl): re-use already-built tclsh
c3f621b 
There was an implicit, lingering dependency on tclsh being available in
the ambient user `$PATH`, even though we build a copy of it! So just
re-use the existing built copy of tclsh in the 8.5 distribution to make
things simpler.

Closes B-Lang-org#24.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
thoughtpolice added a commit to thoughtpolice/bsc that referenced this issue Feb 7, 2020
@thoughtpolice
fix(bluetcl): re-use already-built tclsh
6ab54c9 
There was an implicit, lingering dependency on tclsh being available in
the ambient user `$PATH`, even though we build a copy of it! So just
re-use the existing built copy of tclsh in the 8.5 distribution to make
things simpler.

Closes B-Lang-org#24.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
thoughtpolice added a commit to thoughtpolice/bsc that referenced this issue Feb 7, 2020
@thoughtpolice
fix(bluetcl): re-use already-built tclsh
94d1c4a 
There was an implicit, lingering dependency on tclsh being available in
the ambient user `$PATH`, even though we build a copy of it! So just
re-use the existing built copy of tclsh in the 8.5 distribution to make
things simpler.

Closes B-Lang-org#24.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
@quark17 quark17 closed this as completed in 5d54dcf Feb 7, 2020
ncihnegn pushed a commit to ncihnegn/bsc that referenced this issue Mar 4, 2020
@thoughtpolice
fix(bluetcl): re-use already-built tclsh
875a60f 
There was an implicit, lingering dependency on tclsh being available in
the ambient user `$PATH`, even though we build a copy of it! So just
re-use the existing built copy of tclsh in the 8.5 distribution to make
things simpler.

Closes B-Lang-org#24.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@thoughtpolice @flokli