search

LLNL / lmt

Lustre Monitoring Tools
GNU General Public License v2.0
67 stars 21 forks source link

Added lmtsh man page #16

Closed haasken closed 11 years ago

haasken commented 11 years ago

Added an lmtsh man page consisting of the usage information that is printed by "lmtsh -h". The man page is in section 8 because it can only be run by root, and, as a result, it fits under the category of System adminstration commands. Added the new file to the configure.ac and Makefile.am files so that it will be included in the build. Also added the generated man page to scripts/.gitignore.

Closes #15

haasken commented 11 years ago

@morrone I'm wondering why this pull request says it is closed with unmerged commits. Where normally I should see an option to safely delete my branch where this commit came from, instead I see "Closed with unmerged commits: This pull request is closed, but the haasken:lmtsh-man-page branch has unmerged commits."

See https://github.com/blog/1335-tidying-up-after-pull-requests for a reference. I see something like the second image there when I would expect to see something like the first. Also, I'm not listed as a contributor on this project. Do you have any idea about either of these issues? Thanks!

morrone commented 11 years ago

@hassken, please see https://github.com/chaos/lmt/commits/master. Your commit is there, and you are listed as the author.

I suspect the confusion comes from github being very push-oriented, whereas we tend to prefer to cherry-pick small independant commits to keep the master branch's history clean and linear.

I can't really comment on the github "contributor" thing. I don't know what they are using to make that determination.