-
Notifications
You must be signed in to change notification settings - Fork 222
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
CLI #156
Comments
In what ways would this make Krakatau easier to use? |
Improvements by adding a setup.py, uploading to PyPI and having a single command line tool called like the package:
|
This post was very helpful. Perhaps it's worth including it to readme.txt |
I'm currently rewriting Krakatau in Rust, so the PyPI stuff is no longer relevant. But if you have any suggestions on how to best distribute/install it, I'm all ears. |
@Storyyeller Robert, do you have repo with Rust-based version to share ? |
I haven't published the Rust version yet. I was planning to release an initial version for testing once the assembler was done. |
FYI, the Rust version is now live (it is on the "v2" branch). This only includes the assembler and disassembler for now - I'm still trying to figure out what to do for the new decompiler. |
Currently, the scripts are harder to use than necessary. I think the situation would greatly improve by the following:
setup.py
assemble.py
,disassemble.py
anddecompile.py
toKrakatau/cli
so that one can access the code from the package.bin/krakatau
CLI script which hasassmble
,disassemble
, anddecompile
as subcommandsIf you are interested in it, I could do it. This would add the dependency
click
(a commonly used CLI package) in the setup.py and in thebin/krakatau
.What do you think?
The text was updated successfully, but these errors were encountered: