Hmm. That target usually generated PDF versions of Texinfo documents, not PDF versions of manpages. It might still be appropriate.
I'm not sure that /usr/share/doc is a good place to install manpages. I'm going to think about this for a while. On Tue, Jan 10, 2012 at 10:44:24PM -0800, Romain Lenglet wrote: > Why do you use a custom target instead of the standard "pdf" target? > http://www.gnu.org/prep/standards/standards.html#Standard-Targets > > The standard place to install PDF documentation (only by the > standard "install-pdf" target, not the general "install") is > $(pdfdir), which by default corresponds to > $(prefix)/share/doc/<packagename>. > -- > Romain Lenglet > > On 2012-01-10 17:24, Ethan Jackson wrote: > >Looks good. > > > >Ethan > > > >On Mon, Nov 28, 2011 at 15:26, Ben Pfaff<b...@nicira.com> wrote: > >>It might be nice to actually install these, but there's no standard place > >>to install them. Nevertheless, it's nice to have the target around > >>because it shows how to generate PDFs. It should make it easier for me to > >>regenerate the PDF for ovs-vswitchd.conf.db(5) on openvswitch.org, for > >>example. > >> > >>Reported-by: Alan Shieh<ash...@nicira.com> > >>Bug #8153. > >>--- > >> AUTHORS | 1 + > >> Makefile.am | 19 +++++++++++++++++++ > >> NEWS | 3 ++- > >> configure.ac | 1 + > >> m4/openvswitch.m4 | 12 ++++++++++++ > >> 5 files changed, 35 insertions(+), 1 deletions(-) > >> > >>diff --git a/AUTHORS b/AUTHORS > >>index 00aa15e..3e136d9 100644 > >>--- a/AUTHORS > >>+++ b/AUTHORS > >>@@ -54,6 +54,7 @@ provided helpful bug reports or suggestions. > >> Aaron M. Ucko u...@debian.org > >> Aaron Rosen aro...@clemson.edu > >> Ahmed Bilal numan...@gmail.com > >>+Alan Shieh ash...@nicira.com > >> Alex Yip a...@nicira.com > >> Alexey I. Froloff ra...@altlinux.org > >> Bob Ball bob.b...@citrix.com > >>diff --git a/Makefile.am b/Makefile.am > >>index c0a7ade..60b2698 100644 > >>--- a/Makefile.am > >>+++ b/Makefile.am > >>@@ -165,6 +165,25 @@ manpage-check: $(man_MANS) $(dist_man_MANS) > >>$(noinst_man_MANS) > >> CLEANFILES += manpage-check > >> endif > >> > >>+if HAVE_GROFF > >>+if HAVE_PS2PDF > >>+# The 'man-pdf' target doesn't get built by default because there's no good > >>+# place to install the PDF versions of the manpages. It's included mainly > >>+# because the magic ps2pdf options to embed fonts are needed to make the > >>output > >>+# look decent in the Chrome PDF viewer were really hard to find. > >>+man-pdf: $(man_MANS:=.pdf) $(dist_man_MANS:=.pdf) $(noinst_man_MANS:=.pdf) > >>+SUFFIXES += .1.pdf .5.pdf .8.pdf .1 .5 .8 > >>+PS2PDF_FLAGS = -dEmbedAllFonts=true -dPDFSETTINGS=/printer > >>+man2pdf = LANG=en_US.UTF-8 groff -p -T ps -man $< | ps2pdf > >>$(PS2PDF_FLAGS) - $@ > >>+.1.1.pdf: > >>+ $(man2pdf) > >>+.5.5.pdf: > >>+ $(man2pdf) > >>+.8.8.pdf: > >>+ $(man2pdf) > >>+endif > >>+endif > >>+ > >> include $(srcdir)/manpages.mk > >> $(srcdir)/manpages.mk: $(MAN_ROOTS) build-aux/sodepends.pl > >> @$(PERL) $(srcdir)/build-aux/sodepends.pl -I. -I$(srcdir) > >> $(MAN_ROOTS)>$(@F).tmp > >>diff --git a/NEWS b/NEWS > >>index 33732ff..37de06a 100644 > >>--- a/NEWS > >>+++ b/NEWS > >>@@ -23,7 +23,8 @@ post-v1.3.0 > >> and connectivity issues. This tool currently is not included in RH > >> or > >> Xen packages. > >> - RHEL packaging now supports integration with Red Hat network scripts. > >>- > >>+ - New "man-pdf" Makefile target to generate PDF versions of manpages, > >>+ for convenience. > >> > >> v1.3.0 - xx xxx xxxx > >> ------------------------ > >>diff --git a/configure.ac b/configure.ac > >>index 7bac871..a51b8db 100644 > >>--- a/configure.ac > >>+++ b/configure.ac > >>@@ -68,6 +68,7 @@ OVS_CHECK_SOCKET_LIBS > >> OVS_CHECK_LINKER_SECTIONS > >> OVS_CHECK_XENSERVER_VERSION > >> OVS_CHECK_GROFF > >>+OVS_CHECK_PS2PDF > >> > >> OVS_ENABLE_OPTION([-Wall]) > >> OVS_ENABLE_OPTION([-Wno-sign-compare]) > >>diff --git a/m4/openvswitch.m4 b/m4/openvswitch.m4 > >>index c59d5d4..0c90d7a 100644 > >>--- a/m4/openvswitch.m4 > >>+++ b/m4/openvswitch.m4 > >>@@ -354,3 +354,15 @@ AC_DEFUN([OVS_CHECK_GROFF], > >> ovs_cv_groff=no > >> fi]) > >> AM_CONDITIONAL([HAVE_GROFF], [test "$ovs_cv_groff" = yes])]) > >>+ > >>+dnl Checks for ps2pdf. > >>+AC_DEFUN([OVS_CHECK_PS2PDF], > >>+ [AC_CACHE_CHECK( > >>+ [for ps2pdf], > >>+ [ovs_cv_ps2pdf], > >>+ [if (echo quit | ps2pdf - -)>/dev/null 2>&1; then > >>+ ovs_cv_ps2pdf=yes > >>+ else > >>+ ovs_cv_ps2pdf=no > >>+ fi]) > >>+ AM_CONDITIONAL([HAVE_PS2PDF], [test "$ovs_cv_ps2pdf" = yes])]) > >>-- > >>1.7.4.4 > >> > >>_______________________________________________ > >>dev mailing list > >>dev@openvswitch.org > >>http://openvswitch.org/mailman/listinfo/dev > >_______________________________________________ > >dev mailing list > >dev@openvswitch.org > >http://openvswitch.org/mailman/listinfo/dev > > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
