On 02/06/2014 08:49 AM, Lucas Meneghel Rodrigues wrote:
> The deceptively generic function name load_python_module
> hid the fact that this was an API with an application
> way too narrow to be in virttest.utils_test. So move it
> back to the virttest.libvirt_xml module where it belongs.
> 
> Also, change the API name slightly to load_xml_module.
> This is not a generic python module loader function,
> it has a very specific application.
> 
> CC: Wayne Sun <[email protected]>
> Signed-off-by: Lucas Meneghel Rodrigues <[email protected]>

This is https://github.com/autotest/virt-test/pull/1428 no?

I reviewed your changes, they all LGTM.

-- 
Chris Evich, RHCA, RHCE, RHCDS, RHCSS
Quality Assurance Engineer
e-mail: cevich + `@' + redhat.com o: 1-888-RED-HAT1 x44214

_______________________________________________
Virt-test-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-test-devel

Reply via email to