[RFC PATCH 0/2] add JSON output to net ads
jra at samba.org
Fri Jul 6 17:37:43 UTC 2018
On Fri, Jul 06, 2018 at 04:48:51PM +0200, Philipp Gesang via samba-technical wrote:
> Hey guys,
> the informational commands of the ``net ads'' family format their output in an
> ad-hoc manner that is not specified. For automated processing it would be
> useful to optionally encode the output as JSON.
> These two patches against master add such a JSON output to ``net ads info'' and
> ``net ads lookup''. They have been forward-ported from the 4.7-ish Samba that
> we run. Compilation succeeds with master but I'm still in the process of getting
> working test environment, so they're virtually untested.
> I'd appreciate your feedback.
Sorry, I'm going to have to do a NAK on this work
until the json_XXX() code is fixed to stop returning
(void) on errors.
The longer this sore remains in the code, the more
code we'll have to fix until it's done correctly.
I'm going to NAK anything adding more (void)json_XXX()
calls until this gets fixed.
More information about the samba-technical