On 20/04/15 12:57, Jan Beulich wrote: >>>> On 20.04.15 at 12:49, <andrew.coop...@citrix.com> wrote: >> @@ -103,12 +88,20 @@ install: install-man-pages install-html >> >> # Individual file build targets >> man1/%.1: man/%.pod.1 Makefile >> +ifdef POD2MAN > Perhaps better to use ifneq($(POD2MAN),) in such cases?
I was following the prevailing style, but can update all instances. FWIW, it is not currently an issue. > >> @$(INSTALL_DIR) $(@D) >> $(POD2MAN) --release=$(VERSION) --name=$* -s 1 -c "Xen" $< $@ >> +else >> + @echo "pod2man not installed; skipping $(@F)" > Why do you strip off the directory part? Leaving it in place would > make the output even less ambiguous (and hence more helpful). Can do. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
