David Caro has posted comments on this change. Change subject: Added possible json option value encoding ......................................................................
Patch Set 1: (1 comment) http://gerrit.ovirt.org/#/c/35238/1/mock_configs/mock_genconfig File mock_configs/mock_genconfig: Line 57: value > Because mock expects a python file, not a json one Until now with simple types, the json strings matched the python strings, so no problem, but now that we want more complex options like arrays json output is no longer valid -- To view, visit http://gerrit.ovirt.org/35238 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I66feb645fab0308b7fea9e759cdb5d88b2278a41 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Barak Korren <bkor...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl> Gerrit-Reviewer: Eyal Edri <ee...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches