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

Send error to stderr #19

Merged
merged 1 commit into from
Mar 19, 2017
Merged

Send error to stderr #19

merged 1 commit into from
Mar 19, 2017

Conversation

mmuman
Copy link
Member

@mmuman mmuman commented Mar 19, 2017

No description provided.

@M5oul
Copy link
Member

M5oul commented Mar 19, 2017

Why do you want to send an error when falling back to default node address?
It works, why returning an error?

@mmuman
Copy link
Member Author

mmuman commented Mar 19, 2017

It's not part of the proper output, it's a warning. If you want to parse the output it's really annoying.

@M5oul
Copy link
Member

M5oul commented Mar 19, 2017

./sikaj diffi
and
./silkaj diffi -p duniter.org:10901
give 0 to echo $? output.

I don't understand the usage of this modification.

@mmuman
Copy link
Member Author

mmuman commented Mar 19, 2017

I meant parse stdout. From a script or something else.
Like I want to use it with Argos but it will use the warning as menu title instead of the intended name...

@mmuman
Copy link
Member Author

mmuman commented Mar 19, 2017

Sending stuff to stderr has nothing to do with returning an error code to the shell, btw.

@M5oul
Copy link
Member

M5oul commented Mar 19, 2017

Ok. Let's merge.

@M5oul M5oul merged commit 3373597 into duniter:master Mar 19, 2017
@M5oul M5oul mentioned this pull request Mar 21, 2017
@M5oul M5oul added this to the 0.2.0 milestone Mar 28, 2017
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