-
Notifications
You must be signed in to change notification settings - Fork 136
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 version functions to LFE #299
base: develop
Are you sure you want to change the base?
Conversation
I've updated the docs with formatting fixes. |
@rvirding Is there any chance we could get this into the 1.2.1 release? I've been dying for this version function in LFE for ages ... it would be nice to clean up all the duplicate version-getting code in all the different LFE libraries ... |
👍 I'd like to see (or write myself) a patch for #286 in 1.2.1 as well. |
I have a few comments:
|
If you were to add some docs in lfe_shell and the lfe.1.md file I would not
complain. ;-)
|
@rvirding regarding I'd like to be able to call |
Will rebase and address the review feedback ... |
d23a153
to
60efed3
Compare
60efed3
to
756a723
Compare
756a723
to
45be999
Compare
45be999
to
e630358
Compare
I've updated to rebase off of the LFE 2.0 code base in |
e630358
to
93ff287
Compare
First iteration has version info and supporting functions. Top-leve `version` function uses process table to memoize expensive `os:cmd` calls to tools and `uname`.
93ff287
to
a01c765
Compare
Fixes #297.
Add support for a
version
function as well as other "system info" functions. Tasks:(version)
and(version 'app-nam)
)Bonus tasks: