public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Cc: linux-media@vger.kernel.org, gjasny@googlemail.com,
	hverkuil@xs4all.nl, kieran.bingham@ideasonboard.com,
	nicolas@ndufresne.ca, p.zabel@pengutronix.de, rosenp@gmail.com,
	sakari.ailus@iki.fi, sean@mess.org, user.vdr@gmail.com,
	xavier.claessens@collabora.com, deborah.brouwer@collabora.com,
	ariel@vanguardiasur.com.ar, ezequiel@vanguardiasur.com.ar,
	tomi.valkeinen@ideasonboard.com
Subject: Re: [v4l-utils] [PATCH v10 3/5] meson: Add support for doxygen documentation
Date: Mon, 20 Mar 2023 12:38:11 +0100	[thread overview]
Message-ID: <20230320123811.4b120519@coco.lan> (raw)
In-Reply-To: <20230223164441.22719-4-laurent.pinchart@ideasonboard.com>

Em Thu, 23 Feb 2023 18:44:39 +0200
Laurent Pinchart <laurent.pinchart@ideasonboard.com> escreveu:

> From: Ariel D'Alessandro <ariel.dalessandro@collabora.com>
> 

> +doxygen_install_dir = join_paths(get_option('datadir'), 'doc',
> +                                 '@0@'.format(meson.project_name()))
> +
> +doxygen_output = []
> +if get_option('doxygen-html')
> +    doxygen_output += 'html'
> +endif
> +if get_option('doxygen-man')
> +    doxygen_output += 'man'
> +endif
> +
> +custom_target('doxygen',
> +              input : doxyfile,
> +              output : doxygen_output,
> +              command : [prog_doxygen, doxyfile],
> +              install : true,
> +              install_dir : doxygen_install_dir)

This is not right. It makes man pages to be installed inside:
	/usr/share/doc/v4l-utils/man/man3

It should be, instead:

	usr/share/man/man3

I suspect you'll need a separate custom_target for man.

Please fix it ASAP.

Thanks,
Mauro

  reply	other threads:[~2023-03-20 11:38 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-02-23 16:44 [v4l-utils] [PATCH v10 0/5] Switch build system to meson Laurent Pinchart
2023-02-23 16:44 ` [v4l-utils] [PATCH v10 1/5] Add support for meson building Laurent Pinchart
2023-03-12 14:31   ` Gregor Jasny
2023-03-12 14:49     ` Laurent Pinchart
2023-03-13  7:57       ` Gregor Jasny
2023-03-13  8:11         ` Laurent Pinchart
2023-03-14  7:03           ` Gregor Jasny
2023-03-18 18:17             ` Mauro Carvalho Chehab
2023-03-18 23:06               ` Laurent Pinchart
2023-03-19  0:20                 ` Mauro Carvalho Chehab
2023-03-13  9:09         ` Sean Young
2023-03-13 10:15           ` Laurent Pinchart
2023-02-23 16:44 ` [v4l-utils] [PATCH v10 2/5] Copy Doxygen configuration file to doc/ Laurent Pinchart
2023-02-23 16:44 ` [v4l-utils] [PATCH v10 3/5] meson: Add support for doxygen documentation Laurent Pinchart
2023-03-20 11:38   ` Mauro Carvalho Chehab [this message]
2023-02-23 16:44 ` [v4l-utils] [PATCH v10 4/5] Drop autoconf/automake support Laurent Pinchart
2023-02-23 16:44 ` [v4l-utils] [PATCH v10 5/5] meson: Use -include to include config.h instead of -I Laurent Pinchart

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230320123811.4b120519@coco.lan \
    --to=mchehab@kernel.org \
    --cc=ariel.dalessandro@collabora.com \
    --cc=ariel@vanguardiasur.com.ar \
    --cc=deborah.brouwer@collabora.com \
    --cc=ezequiel@vanguardiasur.com.ar \
    --cc=gjasny@googlemail.com \
    --cc=hverkuil@xs4all.nl \
    --cc=kieran.bingham@ideasonboard.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=nicolas@ndufresne.ca \
    --cc=p.zabel@pengutronix.de \
    --cc=rosenp@gmail.com \
    --cc=sakari.ailus@iki.fi \
    --cc=sean@mess.org \
    --cc=tomi.valkeinen@ideasonboard.com \
    --cc=user.vdr@gmail.com \
    --cc=xavier.claessens@collabora.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox