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

Add a 'clashi' executable #208

Closed
thoughtpolice opened this issue Mar 24, 2017 · 0 comments
Closed

Add a 'clashi' executable #208

thoughtpolice opened this issue Mar 24, 2017 · 0 comments

Comments

@thoughtpolice
Copy link
Contributor

thoughtpolice commented Mar 24, 2017

Currently, clash works by default in batch mode, and can be invoked in interactive mode with clash --interactive. This is a wonderful feature that puts it leagues ahead of most traditional HDLs, of course.

However, this is extremely tedious to type and I can feel my fingers crying every time I do it. A clashi executable that mimics ghci would be fantastic.

Pros:

  • Incredible bloat decrease: only %20 more effort than clash, as opposed to 280% for clash --interactive. Amazing.

Cons:

  • Kind of stupid to implement. I suppose A) Setup.hs for clash-ghc could install a script to do this for us; B) the clash-ghc package could have a new exe clashi that uses process to invoke clash --interactive <remaining args>, or C) somehow thread --interactive through the current Main.hs for clash-ghc, but this is complicated because doing that is GHC's responsibility when parsing arguments to create DynFlags.

I think approach 2) (simply execve(2) the clash binary) is the right approach. (This is actually what GHC for Windows does, since it cannot use a shell script.)

thoughtpolice added a commit to thoughtpolice/clash-compiler that referenced this issue Mar 25, 2017
Fixes clash-lang#208.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
thoughtpolice added a commit to thoughtpolice/clash-compiler that referenced this issue Mar 25, 2017
Fixes clash-lang#208.

Signed-off-by: Austin Seipp <aseipp@pobox.com>
christiaanb pushed a commit that referenced this issue Mar 26, 2017
Fixes #208.

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant