Building bsc on MacOS works!

[b-lang-org]

Building bsc on MacOS works!

[mtikekar]

Btw, make check runs until it needs bluetcl

[bpfoley]

That's great news! I haven't been actively working on a macOS build, but I'd certainly like to have one. Thanks for exploring this. At least we know we're moving in the right direction.

[thoughtpolice]

The libbdpi.so issue is due to a bug in the build system when using the STP no-op stub. I have a fix for it.

I think having macOS (and eventually: windows!) builds is a noble goal. It's probably worth having bluetcl in working order. Are you using tcl from something like homebrew? I'm not sure the best way to install tcl/tk/itcl/itk is, but that's the most important step at this point...

[mtikekar]

Yes, I am using the tcl-tk package from Homebrew. It includes itcl but not itk. I tried compiling itk from source but failed at ./configure.

…

On Fri, Feb 28, 2020, 11:30 PM Austin Seipp ***@***.***> wrote: The libbdpi.so issue is due to a bug in the build system when using the STP no-op stub. I have a fix for it. I think having macOS (and eventually: windows!) builds is a noble goal. It's probably worth having bluetcl in working order. Are you using tcl from something like homebrew? I'm not sure the best way to install tcl/tk/itcl/itk is, but that's the most important step at this point... — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#81?email_source=notifications&email_token=ABCYIGZ3X6S35MKAK3PC3UDRFHQEZA5CNFSM4K5MFWMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLGNKY#issuecomment-592864939>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABCYIG5AIEOV3DG4OL2ELFDRFHQEZANCNFSM4K5MFWMA> .

[bpfoley]

I'll try packaging itk/itcl for homebrew.

[mtikekar]

Turns out that MacOS's builtin tcl already has itk/itcl at /System/Library/Tcl/8.5. Perhaps it is best to try to use them instead of the homebrew versions.

[ncihnegn]

I completely finished make STP_STUB=1 with system tcl/tk plus built-from-source itcl/itk.
System itcl/itk won't work because they don't come with stublib.

When make check, run into this error:

Checking bsc compiles
bsc -sim FibOne.bsv
Elaborated module file created: mkFibOne.ba

Checking compile for Bluespec simulator
bsc -sim -o mkFibOne.bexe -e mkFibOne mkFibOne.ba
Bluesim object created: mkFibOne.{h,o}
Bluesim object created: model_mkFibOne.{h,o}
Simulation shared library created: mkFibOne.bexe.so
Simulation executable created: mkFibOne.bexe
./mkFibOne.bexe > smoke_test_bluesim.out
called Tcl_FindHashEntry on deleted table
called Tcl_FindHashEntry on deleted table
/bin/sh: line 1:  4762 Abort trap: 6           ./mkFibOne.bexe > smoke_test_bluesim.out
make[1]: *** [smoke_test_bluesim] Error 134
make: *** [check] Error 2

[mtikekar]

I removed the stublibs from the Makefile to use the system itcl/itk and it compiled fine. I did run into the same error you see. But when I looked at examples/smoke_test/smoke_test_bluesim.out, the output is all there, so mkFibOne.bexe crashed only at the end. The error message seems quite common and people are blaming Tcl 8.5 and recommend trying 8.6 (https://groups.google.com/forum/#!topic/comp.lang.tcl/BKexjImWklo)

[ncihnegn]

Now I can complete make, with some small patches on Makefiles and stp.
It seems the only major issue is that bluetcl will report an error on exit.

[bpfoley]

Just for reference, the Tcl_Exit bug in question is this one: https://sourceforge.net/p/tcl/bugs/4043/

[mtikekar]

I got a Github action to build on MacOS 10.15: https://github.com/mtikekar/bsc/tree/mtikekar-macos. You can download the inst directory from https://github.com/mtikekar/bsc/actions/runs/49104659 and try it out. You'll need brew install tcl-tk (version 8.6.10) to run.

[jameyhicks]

Nice progress!

[quark17]

PR #98 gets BSC (with real STP) to compile on MacOS and adds it to CI. I'd appreciate if folks could review that PR and add any comments. I'd like to merge that PR, if there's consensus.

[bpfoley]

This is working on HEAD now, so I'll close the issue. Thanks everyone!