Re: [ovs-dev] [PATCH v2 4/5] make: Ensure flake8, sphinx run when required

2017-01-04 Thread Stephen Finucane
On Wed, 2017-01-04 at 08:34 -0800, Ben Pfaff wrote: > On Wed, Jan 04, 2017 at 04:25:40PM +, Stephen Finucane wrote: > > On Wed, 2017-01-04 at 08:20 -0800, Ben Pfaff wrote: > > > On Mon, Jan 02, 2017 at 11:47:41AM +, Stephen Finucane wrote: > > > > If someone makes changes to documentation o

Re: [ovs-dev] [PATCH v2 4/5] make: Ensure flake8, sphinx run when required

2017-01-04 Thread Ben Pfaff
On Wed, Jan 04, 2017 at 04:25:40PM +, Stephen Finucane wrote: > On Wed, 2017-01-04 at 08:20 -0800, Ben Pfaff wrote: > > On Mon, Jan 02, 2017 at 11:47:41AM +, Stephen Finucane wrote: > > > If someone makes changes to documentation or Python scripts, they > > > should > > > validate these cha

Re: [ovs-dev] [PATCH v2 4/5] make: Ensure flake8, sphinx run when required

2017-01-04 Thread Stephen Finucane
On Wed, 2017-01-04 at 08:20 -0800, Ben Pfaff wrote: > On Mon, Jan 02, 2017 at 11:47:41AM +, Stephen Finucane wrote: > > If someone makes changes to documentation or Python scripts, they > > should > > validate these changes using the relevant targets. However, said > > targets > > use optional

Re: [ovs-dev] [PATCH v2 4/5] make: Ensure flake8, sphinx run when required

2017-01-04 Thread Ben Pfaff
On Mon, Jan 02, 2017 at 11:47:41AM +, Stephen Finucane wrote: > If someone makes changes to documentation or Python scripts, they should > validate these changes using the relevant targets. However, said targets > use optional dependencies and are not guaranteed to be enabled. Enforce > running

[ovs-dev] [PATCH v2 4/5] make: Ensure flake8, sphinx run when required

2017-01-02 Thread Stephen Finucane
If someone makes changes to documentation or Python scripts, they should validate these changes using the relevant targets. However, said targets use optional dependencies and are not guaranteed to be enabled. Enforce running of these checks whenever changes are made to select files, ensuring the u