-
Notifications
You must be signed in to change notification settings - Fork 824
Added mrbgems/mruby-bin-utils
#6057
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: master
Are you sure you want to change the base?
Conversation
e8d5b7a to
6db0af1
Compare
mrbgems/mruby-bin-utils/README.md
Outdated
| end | ||
| ``` | ||
|
|
||
| Then compile and link your code in the generated command line program. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this line needs to be updated. You need to describe clearly what we get from these command-lines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modified.
mrbgems/mruby-bin-utils/README.md
Outdated
| Then compile and link your code in the generated command line program. | ||
|
|
||
| ```console | ||
| $ bin/mruby-cc yourcode.c |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some may misunderstand mruby-cc etc. only take one argument.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Modified.
|
According to our current naming convention, we can expect the executable binary In addition, maybe we can have |
The document was converted from Japanese by https://www.deepl.com/.
6db0af1 to
2c54c70
Compare
|
Thank you for the review. I can't think of anything else appropriate for the name. That said, it wouldn't be a good idea to separate them individually either. As for mruby-build, I will explore the feasibility a bit more. |
If this gem isn't accepted by the mruby project, I will release it in my personal repository.
Current problem:
bin/mruby-ccdepend on each compiler.bin/mruby-cc -c, I get a "'linker' input unused" warning; I checked with clang, but gcc does not.😸 Perhaps some disgruntled person will rewrite it in C.