Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixes #61: More output control #62

Merged
merged 1 commit into from
Apr 8, 2016
Merged

Fixes #61: More output control #62

merged 1 commit into from
Apr 8, 2016

Conversation

zbeekman
Copy link
Contributor

@zbeekman zbeekman commented Apr 8, 2016

Allow user to redirect default output to other opened units.

Fixes #61

This can allow MPI programs that won't flood the terminal if
--version or --help are passed.

 - Add optional arguments to `init` method
   - `usage_lun` specify optional alternate output unit other than
     `stderr`
   - `version_lun` specify optional alternate version/license output
     unit other than `stdout`
   - `error_lun` specify optional alternate error info/message output
     unit other than `stderr`
 - Fixes #61
 - Will allow users to use MPI without `--help` or `--version` exploding
   with output when running on many MPI ranks
@CLAassistant
Copy link

CLA assistant check
All committers have accepted the CLA.

@codecov-io
Copy link

Current coverage is 75.73%

Merging #62 into master will increase coverage by +0.04% as of c3e37cd

@@            master     #62   diff @@
======================================
  Files           11      11       
  Stmts         1843    1846     +3
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit           1395    1398     +3
  Partial          0       0       
  Missed         448     448       

Review entire Coverage Diff as of c3e37cd

Powered by Codecov. Updated on successful CI builds.

@szaghi
Copy link
Owner

szaghi commented Apr 8, 2016

@zbeekman wonderful! I am now quite busy (at CNR now some stable positions are being assigned by a puhlic, statal call... quite a nightmare to prepare the submisiin), your help is very appreciated!

@szaghi szaghi merged commit 414e265 into szaghi:master Apr 8, 2016
@zbeekman
Copy link
Contributor Author

zbeekman commented Apr 8, 2016

@szaghi Good luck!

@zbeekman zbeekman deleted the error-help-unit-specify branch April 8, 2016 19:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants