Blacksmith-shop / pdsh

Automatically exported from code.google.com/p/pdsh
GNU General Public License v2.0
0 stars 0 forks source link

Output all exit codes to stdout #58

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Hi,

I am sending you a patch for pdsh-2.29 that implements an option (-A) to 
display all return values from all hosts on stdout instead of just displaying 
non zero return values on stderr.

The display on stdout is done because a race condition can occur between stderr 
and stdout under specific conditions, trashing some lines.

Regards,

-- 
Clément Bourgeois
HP EMEA

Original issue reported on code.google.com by mark.gro...@gmail.com on 24 Oct 2013 at 4:03

GoogleCodeExporter commented 8 years ago

Original comment by clement9...@gmail.com on 25 Oct 2013 at 8:09

Attachments:

GoogleCodeExporter commented 8 years ago
I just realized that -A will conflict with the genders module. I think this one 
might have to wait until pdsh supports long options, or we could trigger off an 
environment variable for now. How does that sound?

Original comment by mark.gro...@gmail.com on 8 Nov 2013 at 12:15

GoogleCodeExporter commented 8 years ago
[deleted comment]
GoogleCodeExporter commented 8 years ago
Sounds great, I let you judge what name suits you best.

I just realized there was a typo missing \ in the patch I attached, here is the 
correct one.

Original comment by clement9...@gmail.com on 17 Dec 2013 at 3:06

Attachments: