On Thu, 06 Oct 2016, Markus Heiser <markus.heiser@xxxxxxxxxxx> wrote: > with this series a reST-directive kernel-cmd is introduced. The kernel-cmd > directive includes contend from the stdout of a command-line (@mchehab asked > for). I like the fact that this removes Documentation/media/Makefile, and cleans up the Sphinx build rule in Documentation/Makefile.sphinx. Does this also make the documentation buildable with sphinx-build directly, without the kernel build system? If so, great. However, I would have much preferred the approach I proposed months ago, having the extension itself do specifically what parse-headers.pl does now. While it may seem generic on the surface, I don't think it's a clean or a secure approach to allow running of arbitrary scripts from PATH while building documentation. It's certainly not an approach that should be encouraged. In part, the reason the DocBook build became so unwieldy was the proliferation of arbitrary scripts and tools required to make it happen. I think it would be really sad to let this happen to the Sphinx build. I am *already* sad about how parse-headers.pl was bolted on to the build. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
