Skip to content

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
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.

4 participants