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

system diagnostics command #1838

Merged
merged 2 commits into from
Oct 19, 2015
Merged

system diagnostics command #1838

merged 2 commits into from
Oct 19, 2015

Conversation

whyrusleeping
Copy link
Member

Provide a command to easily gather system information to streamline the issue reporting process.

Saves us from having to ask for a laundry list of information about the users environment whenever they report a bug.

License: MIT
Signed-off-by: Jeromy jeromyj@gmail.com

@jbenet jbenet added the status/in-progress In progress label Oct 13, 2015
@whyrusleeping whyrusleeping force-pushed the feat/sysdiag branch 2 times, most recently from 0bb88a6 to 85b5b23 Compare October 13, 2015 21:19
@jbenet
Copy link
Member

jbenet commented Oct 14, 2015

  • needs a sharness test to check it works, even just comparing it against a grep for uname or whatever.

@whyrusleeping
Copy link
Member Author

just a note, uname is not cross platform, so those tests are likely to pass on most systems. probably not reliable on windows (because uname has to be installed by some other tool).

@jbenet
Copy link
Member

jbenet commented Oct 19, 2015

Ok we can probably address it by doing it right later; portable way to check arch. LGTM

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
@whyrusleeping whyrusleeping merged commit 1cef170 into master Oct 19, 2015
@jbenet jbenet removed the status/in-progress In progress label Oct 19, 2015
@whyrusleeping whyrusleeping deleted the feat/sysdiag branch October 19, 2015 02:54
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

Successfully merging this pull request may close these issues.

2 participants