[RFC PATCH v2 2/3] s3: net: implement json output for ads info

[Andrew Bartlett]

On Thu, 2018-07-12 at 08:31 +0200, Philipp Gesang wrote:
> -<| Quoting Philipp Gesang <philipp.gesang at intra2net.com>, on Tuesday, 2018-07-10 09:20:24 AM |>-
> > > > Attached is my first attempt, please build on that.
> > > > 
> > > > CI: https://gitlab.com/catalyst-samba/samba/pipelines/25428061
> 
> See here: https://gitlab.com/phgsng/samba/commits/i2n-net-ads-json
> 
> Your branch diverged from the error code one so I had to
> reconcile the two first (commit 7dc72313b9).
> 
> > > Also, please sign up to gitlab and do CI via that.  It will ensure you
> > > don't break supported combinations while adding this and so make review
> > > easier. 
> > > 
> > > https://wiki.samba.org/index.php/Samba_CI_on_gitlab
> 
> Miraculously, the CI worked as soon as I forked you repo. It
> passes now: https://gitlab.com/phgsng/samba/pipelines/25588751
> 
> How do you wish me to proceed: send a merge request against your
> repo on Gitlab or resubmit it here as patches?

G'Day Philipp,

Yes, you get a partial CI on the shared (free) runners.  Once I connect
you with the shared repo you can run the full testsuite if you push
into that repo (start branches with your username). 

For now, patches here are still the best way to get good attention.

Andrew Bartlett

-- 
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team         https://samba.org
Samba Development and Support, Catalyst IT   
https://catalyst.net.nz/services/samba