Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-14 Thread Petr Viktorin
On 06/13/2012 11:40 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 06/12/2012 02:38 PM, Simo Sorce wrote: On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-14 Thread Martin Kosek
On Thu, 2012-06-14 at 10:51 +0200, Petr Viktorin wrote: On 06/13/2012 11:40 PM, Rob Crittenden wrote: [snip] 1. We could detect and supress unexpected output by default and include a note at the end, something like: Unexpected output suppressed, use --all to show. That would work with

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-13 Thread Rob Crittenden
Petr Viktorin wrote: On 06/12/2012 02:38 PM, Simo Sorce wrote: On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-12 Thread Simo Sorce
On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket asks. Instead it's printed, but not formatted nicely as

Re: [Freeipa-devel] [PATCH] 0062 Don't crash when server returns extra output

2012-06-12 Thread Petr Viktorin
On 06/12/2012 02:38 PM, Simo Sorce wrote: On Tue, 2012-06-12 at 13:12 +0200, Petr Viktorin wrote: This will make older clients usable if new output items get added to commands. Since there might be important information in the extra output, it's not ignored as the ticket asks. Instead it's