-
Notifications
You must be signed in to change notification settings - Fork 20
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 repo #38
Comments
Comment by carljv Did you get dabble to compile? Relying on the lexer/parser in libdparse feels better to me, since it gives you consistency with the whole DScanner, DCD, dfmt, ecosystem. Is there any reason to prefer the pegged approach? I haven't dug really deeply into this, but it seems like both of these require making/using shared libs. Is that a blocker for OS X? Thanks for picking this up, I think it will be really useful, as well as a nice component of "the first five minutes." |
Comment by wilzbach
Nope :/
I think both REPLs use libdparse, but dabble uses pegged for it's own Meta command https://github.com/callumenator/dabble/blob/master/src/dabble/grammars.d which makes it more powerful (in theory - as said I couldn't compile).
Nope, but we should be aware that compiling the external lib can require more effort.
Of course a good Makefile can deal with all this, but currently
Great to hear that you share my opinion :) |
Comment by wilzbach Pinging @John-Colvin - the author of PydMagic. |
Comment by carljv Ah. Yes, meta commands are nice. It's not clear to me that an actual grammar w/ parser is necessary for them. But I'd have to think about it harder. |
Meta commands could simply use Also see #6 |
Issue by wilzbach
Wednesday May 25, 2016 at 19:15 GMT
Originally opened as https://github.com/drepl/drepl-old/issues/1
2) dabble
dabble's approach looks generally good and more scalable, but I have no idea how one can import custom packages in there
The text was updated successfully, but these errors were encountered: