On 04/06/2017 12:16 AM, Ben Pfaff wrote:
> On Wed, Mar 29, 2017 at 01:49:42PM +0200, Timothy Redaelli wrote:
>> Since Python 2.7 and python-six are needed to build Open vSwitch,
>> ./configure should return an error if they are missing or if they are too old
>>
>> Signed-off-by: Timothy Redaelli <tredae...@redhat.com>
> 
> Thanks for the patch.
> 
> I think that if we do it this way, then eventually we'll have to debug
> the problem.  I suggest that we use the following instead, which does a
> better job of dumping problems into config.log for troubleshooting.
> 
> What do you think?

LGTM

Tested cases:
 - missing python
 - python version < 2.7
 - python version >= 2.7, without python-six
 - python version >= 2.7, with old python-six

 - python version >= 2.7, with correct python-six

Thank you

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to