On Thu, Mar 09, 2017 at 06:15:13PM +0300, Ilya Maximets wrote: > On 07.03.2017 21:54, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff <b...@ovn.org> > > Acked-by: Stephen Finucane <step...@that.guru> > > --- > > Documentation/automake.mk | 15 ++++++++++++++- > > Documentation/sphinx-version-blacklist | 2 ++ > > 2 files changed, 16 insertions(+), 1 deletion(-) > > create mode 100644 Documentation/sphinx-version-blacklist > > > > diff --git a/Documentation/automake.mk b/Documentation/automake.mk > > index a74807fde532..f7f1fe61d1b7 100644 > > --- a/Documentation/automake.mk > > +++ b/Documentation/automake.mk > > @@ -86,7 +86,8 @@ EXTRA_DIST += \ > > Documentation/internals/contributing/documentation-style.rst \ > > Documentation/internals/contributing/libopenvswitch-abi.rst \ > > Documentation/internals/contributing/submitting-patches.rst \ > > - Documentation/requirements.txt > > + Documentation/requirements.txt \ > > + Documentation/sphinx-version-blacklist > > > > # You can set these variables from the command line. > > SPHINXOPTS = > > @@ -120,3 +121,15 @@ endif > > .PHONY: htmldocs > > .PHONY: check-docs > > .PHONY: clean-docs > > + > > +ALL_LOCAL += sphinx-version-check > > +sphinx-version-check: $(EXTRA_DIST) > > + @if grep -n -f $(srcdir)/Documentation/sphinx-version-blacklist $?; \ > > + then \ > > + echo "See above for list of uses of features that Sphinx 1.1.3"; \ > > + echo "does not support. Please avoid using these features.."; \ > > + exit 1; \ > > + else \ > > + : > $@; \ > > + fi > > +CLEANFILES += sphinx-version-check > > diff --git a/Documentation/sphinx-version-blacklist > > b/Documentation/sphinx-version-blacklist > > new file mode 100644 > > index 000000000000..a67339bf2758 > > --- /dev/null > > +++ b/Documentation/sphinx-version-blacklist > > @@ -0,0 +1,2 @@ > > +code-block:: *ps1con > > +code-block:: *doscon > > I don't feel this patch is fully correct, because it's not the features of > sphinx. And its version not really connected with version of 'pygments' > library.
OK, can you explain the real problem then? We're making changes to the documentation on the basis that old versions of Sphinx does not support features. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev