Hi On Tue, Jul 14, 2015 at 8:24 PM, Jim Davis <jim.epost@xxxxxxxxx> wrote: > On Mon, Jul 13, 2015 at 7:00 PM, Ulf Magnusson <ulfalizer.lkml@xxxxxxxxx> wrote: >> On Mon, Jul 13, 2015 at 05:39:25PM -0700, Greg KH wrote: >>> On Sun, Jul 12, 2015 at 08:08:46PM +0200, Ulf Magnusson wrote: >>> > Hello, >>> > >>> > Tested with next-20150709. >>> > >>> > Documentation/kdbus/Makefile lacks an 'installmandocs' target, causing >>> > problems for >>> > >>> > %docs: scripts_basic FORCE >>> > $(Q)$(MAKE) $(build)=scripts build_docproc >>> > $(Q)$(MAKE) $(build)=Documentation/DocBook $@ >>> > $(Q)$(MAKE) $(build)=Documentation/kdbus $@ >>> > >>> > from the top-level Makefile. It still works, but no KDBUS-related man >>> > pages are installed, and the final error isn't reassuring. >>> >>> What's the final error? >>> >> >> $ sudo make installmandocs >> GEN Documentation/DocBook//v4l2.xml >> find Documentation/DocBook/man -name '*.9' | xargs gzip -f >> mkdir -p /usr/local/man/man9/ >> install Documentation/DocBook/man/*.9.gz /usr/local/man/man9/ >> make[1]: *** No rule to make target 'installmandocs'. Stop. >> Makefile:1359: recipe for target 'installmandocs' failed >> make: *** [installmandocs] Error 2 >> >> >> The "*** No rule to make target 'installmandocs'. Stop." error is >> generated for Documentation/kdbus/Makefile. The other man pages still >> install fine. > > There's a similar problem with 'make cleandocs'. Adding noop rules > for installdocs and cleandocs to Documentation/kdbus/Makefile at least > gets rid of the error messages. So the Docbook targets support much more %docs targets than just cleandocs and installdocs. I don't think it's reasonable to guarantee that all those targets are supported for all documentation subdirs. Hence, I'd propose the attached patch. This should at least make sure the targets won't fail. But I think we should look into cleandocs and installmandocs. I'll have a look into that later. This patch should just fix all the other targets right now. Thanks David diff --git a/Documentation/kdbus/Makefile b/Documentation/kdbus/Makefile index af87641..8caffe5 100644 --- a/Documentation/kdbus/Makefile +++ b/Documentation/kdbus/Makefile @@ -38,3 +38,7 @@ mandocs: $(MANFILES) htmldocs: $(HTMLFILES) clean-files := $(MANFILES) $(HTMLFILES) + +# we don't support other %docs targets right now +%docs: + @true -- To unsubscribe from this list: send the line "unsubscribe linux-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
