From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: gjasny@googlemail.com,
Ariel D'Alessandro <ariel.dalessandro@collabora.com>
Cc: linux-media@vger.kernel.org, hverkuil@xs4all.nl, sean@mess.org,
p.zabel@pengutronix.de, ezequiel@vanguardiasur.com.ar,
nicolas@ndufresne.ca, kieran.bingham@ideasonboard.com,
xavier.claessens@collabora.com, nicolas.dufresne@collabora.com,
user.vdr@gmail.com, sakari.ailus@iki.fi,
mchehab+huawei@kernel.org, rosenp@gmail.com
Subject: Re: [v4l-utils v5 0/5] Add support for meson building
Date: Wed, 15 Dec 2021 23:05:54 +0200 [thread overview]
Message-ID: <YbpYsgFEXvVKZguI@pendragon.ideasonboard.com> (raw)
In-Reply-To: <YVsOleFU6cmIOnmZ@pendragon.ideasonboard.com>
Hi Gregor,
On Mon, Oct 04, 2021 at 05:24:21PM +0300, Laurent Pinchart wrote:
> On Wed, May 12, 2021 at 03:49:41PM -0300, Ariel D'Alessandro wrote:
> > Hi all,
> >
> > Another attempt to support meson build, addressing comments/fixes from
> > reviewers.
> >
> > This patchset was developed on top of v4l-utils current master branch:
> >
> > commit f0c7e3d71eaf4182bae7eb3ee0e43b4eeb047ea9
> > Author: Hans Verkuil <hverkuil-cisco@xs4all.nl>
> >
> > v4l2-ctl: add '--set-edid type=list' support
> >
> > You can find the patchset pushed to the following branch:
> >
> > https://gitlab.collabora.com/adalessandro/v4l-utils/-/tree/v4l-utils-meson-v5
> >
> > Feedback appreciated as always :-)
>
> Reviving a bit of an old thread, I'd like to know what is blocking the
> merge of this series. In particular, Gregor, as you maintain v4l-utils,
> are you happy with this series, or do you have a list of features you
> want to see implemented (or of bugs that needs to be fixed) before you
> will consider merging it ?
Ping if I may.
Ariel, in case it can be useful, I've pushed
git://linuxtv.org/pinchartl/v4l-utils.git meson
with two fixups.
> > Changes from v4:
> >
> > * Dropped install_subdir() work-around to install empty dir. Supported
> > now in meson v0.57.
> > * Bump to meson v0.57. Replaced deprecated methods.
> > * Added override_dependency() calls on libraries.
> > * Properly check for bpf and SDL2 dependencies.
> > * Removed libraries_private field from libv4lconvert pkgconfig.
> > * Documented meson static builds support.
> > * Set default compiler standard options.
> > * Added bpf check in keytable.
> >
> > Changes from v3:
> >
> > * Split Doxygen support patch.
> > * Properly generate config.h configuration file.
> >
> > Changes from v2:
> >
> > * Updated to current master branch.
> > * Moved README to markdown syntax.
> > * Documented meson build.
> > * Set PACKAGE_VERSION config macro as quoted.
> >
> > Changes from v1:
> >
> > * Updated project version to 1.21.0.
> > * Fixed clang includes in keytable bpf compilation.
> > * Improved variable reutilization in meson scripts.
> > * Set libraries version and soversion.
> > * Control symbol visibility.
> > * Install empty 'protocols' directory in keytable user dir.
> > * Fixed svg filenames in qv4l2 and qvidcap.
> > * Added support for Doxygen documentation (html and man).
> > * Updated required meson version to v0.53 (for fs module).
> > * Added new files to EXTRA_DIST in each Makefile.am.
> >
> > Regards,
> >
> > Ariel D'Alessandro (5):
> > Move README to markdown syntax
> > Add support for meson building
> > Copy Doxygen configuration file to doc/
> > meson: Add support for doxygen documentation
> > Makefile.am: Distribute meson related files
> >
> > .gitignore | 1 +
> > INSTALL.meson.md | 142 +
> > Makefile.am | 5 +-
> > README | 274 --
> > README.md | 332 +++
> > contrib/Makefile.am | 3 +-
> > contrib/cobalt-ctl/Makefile.am | 2 +
> > contrib/cobalt-ctl/meson.build | 8 +
> > contrib/decode_tm6000/Makefile.am | 2 +
> > contrib/decode_tm6000/meson.build | 14 +
> > contrib/gconv/Makefile.am | 2 +-
> > contrib/gconv/meson.build | 44 +
> > contrib/meson.build | 13 +
> > contrib/rds-saa6588/Makefile.am | 2 +
> > contrib/rds-saa6588/meson.build | 7 +
> > contrib/test/Makefile.am | 3 +-
> > contrib/test/meson.build | 143 +
> > contrib/xc3028-firmware/Makefile.am | 2 +-
> > contrib/xc3028-firmware/meson.build | 11 +
> > doc/Doxyfile.in | 2352 +++++++++++++++++
> > doc/meson.build | 34 +
> > lib/Makefile.am | 2 +-
> > lib/libdvbv5/Makefile.am | 2 +-
> > lib/libdvbv5/meson.build | 159 ++
> > lib/libv4l-mplane/Makefile.am | 2 +
> > lib/libv4l-mplane/meson.build | 23 +
> > lib/libv4l1/Makefile.am | 2 +-
> > lib/libv4l1/meson.build | 62 +
> > lib/libv4l2/Makefile.am | 2 +-
> > lib/libv4l2/meson.build | 71 +
> > lib/libv4l2rds/Makefile.am | 2 +
> > lib/libv4l2rds/meson.build | 37 +
> > lib/libv4lconvert/Makefile.am | 2 +-
> > lib/libv4lconvert/meson.build | 117 +
> > lib/meson.build | 11 +
> > libdvbv5-po/meson.build | 3 +
> > meson.build | 334 +++
> > meson_options.txt | 50 +
> > utils/Makefile.am | 2 +-
> > utils/cec-compliance/Makefile.am | 2 +-
> > utils/cec-compliance/meson.build | 22 +
> > utils/cec-ctl/Makefile.am | 2 +-
> > utils/cec-ctl/meson.build | 18 +
> > utils/cec-follower/Makefile.am | 2 +-
> > utils/cec-follower/meson.build | 19 +
> > utils/cx18-ctl/Makefile.am | 2 +
> > utils/cx18-ctl/meson.build | 8 +
> > utils/dvb/Makefile.am | 2 +-
> > utils/dvb/meson.build | 70 +
> > utils/gen_media_bus_format_codes.sh | 7 +
> > utils/gen_media_bus_format_names.sh | 7 +
> > utils/ir-ctl/Makefile.am | 2 +
> > utils/ir-ctl/meson.build | 23 +
> > utils/ivtv-ctl/Makefile.am | 2 +
> > utils/ivtv-ctl/meson.build | 13 +
> > utils/keytable/Makefile.am | 3 +-
> > utils/keytable/bpf_protocols/Makefile.am | 3 +-
> > .../bpf_protocols/clang_sys_includes.sh | 9 +
> > utils/keytable/bpf_protocols/meson.build | 31 +
> > utils/keytable/meson.build | 81 +
> > utils/keytable/rc_keymaps/meson.build | 151 ++
> > utils/libcecutil/Makefile.am | 2 +-
> > utils/libcecutil/meson.build | 45 +
> > utils/libmedia_dev/Makefile.am | 2 +-
> > utils/libmedia_dev/meson.build | 14 +
> > utils/libv4l2util/Makefile.am | 2 +-
> > utils/libv4l2util/meson.build | 16 +
> > utils/media-ctl/Makefile.am | 2 +
> > utils/media-ctl/meson.build | 43 +
> > utils/meson.build | 46 +
> > utils/qv4l2/Makefile.am | 3 +-
> > utils/qv4l2/meson.build | 80 +
> > utils/qvidcap/Makefile.am | 3 +-
> > utils/qvidcap/meson.build | 82 +
> > utils/rds-ctl/Makefile.am | 1 +
> > utils/rds-ctl/meson.build | 13 +
> > utils/v4l2-compliance/Makefile.am | 2 +-
> > utils/v4l2-compliance/meson.build | 58 +
> > utils/v4l2-ctl/Makefile.am | 2 +-
> > utils/v4l2-ctl/meson.build | 75 +
> > utils/v4l2-dbg/Makefile.am | 2 +-
> > utils/v4l2-dbg/meson.build | 16 +
> > utils/v4l2-sysfs-path/Makefile.am | 2 +
> > utils/v4l2-sysfs-path/meson.build | 14 +
> > v4l-utils-po/meson.build | 3 +
> > 85 files changed, 4987 insertions(+), 299 deletions(-)
> > create mode 100644 INSTALL.meson.md
> > delete mode 100644 README
> > create mode 100644 README.md
> > create mode 100644 contrib/cobalt-ctl/meson.build
> > create mode 100644 contrib/decode_tm6000/meson.build
> > create mode 100644 contrib/gconv/meson.build
> > create mode 100644 contrib/meson.build
> > create mode 100644 contrib/rds-saa6588/meson.build
> > create mode 100644 contrib/test/meson.build
> > create mode 100644 contrib/xc3028-firmware/meson.build
> > create mode 100644 doc/Doxyfile.in
> > create mode 100644 doc/meson.build
> > create mode 100644 lib/libdvbv5/meson.build
> > create mode 100644 lib/libv4l-mplane/meson.build
> > create mode 100644 lib/libv4l1/meson.build
> > create mode 100644 lib/libv4l2/meson.build
> > create mode 100644 lib/libv4l2rds/meson.build
> > create mode 100644 lib/libv4lconvert/meson.build
> > create mode 100644 lib/meson.build
> > create mode 100644 libdvbv5-po/meson.build
> > create mode 100644 meson.build
> > create mode 100644 meson_options.txt
> > create mode 100644 utils/cec-compliance/meson.build
> > create mode 100644 utils/cec-ctl/meson.build
> > create mode 100644 utils/cec-follower/meson.build
> > create mode 100644 utils/cx18-ctl/meson.build
> > create mode 100644 utils/dvb/meson.build
> > create mode 100755 utils/gen_media_bus_format_codes.sh
> > create mode 100755 utils/gen_media_bus_format_names.sh
> > create mode 100644 utils/ir-ctl/meson.build
> > create mode 100644 utils/ivtv-ctl/meson.build
> > create mode 100755 utils/keytable/bpf_protocols/clang_sys_includes.sh
> > create mode 100644 utils/keytable/bpf_protocols/meson.build
> > create mode 100644 utils/keytable/meson.build
> > create mode 100644 utils/keytable/rc_keymaps/meson.build
> > create mode 100644 utils/libcecutil/meson.build
> > create mode 100644 utils/libmedia_dev/meson.build
> > create mode 100644 utils/libv4l2util/meson.build
> > create mode 100644 utils/media-ctl/meson.build
> > create mode 100644 utils/meson.build
> > create mode 100644 utils/qv4l2/meson.build
> > create mode 100644 utils/qvidcap/meson.build
> > create mode 100644 utils/rds-ctl/meson.build
> > create mode 100644 utils/v4l2-compliance/meson.build
> > create mode 100644 utils/v4l2-ctl/meson.build
> > create mode 100644 utils/v4l2-dbg/meson.build
> > create mode 100644 utils/v4l2-sysfs-path/meson.build
> > create mode 100644 v4l-utils-po/meson.build
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2021-12-15 21:05 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-12 18:49 [v4l-utils v5 0/5] Add support for meson building Ariel D'Alessandro
2021-05-12 18:49 ` [v4l-utils v5 1/5] Move README to markdown syntax Ariel D'Alessandro
2021-05-12 18:49 ` [v4l-utils v5 2/5] Add support for meson building Ariel D'Alessandro
2021-05-12 18:49 ` [v4l-utils v5 3/5] Copy Doxygen configuration file to doc/ Ariel D'Alessandro
2021-05-12 18:49 ` [v4l-utils v5 4/5] meson: Add support for doxygen documentation Ariel D'Alessandro
2021-05-12 18:49 ` [v4l-utils v5 5/5] Makefile.am: Distribute meson related files Ariel D'Alessandro
2021-05-13 8:56 ` [v4l-utils v5 0/5] Add support for meson building Kieran Bingham
2021-05-17 20:55 ` Ariel D'Alessandro
2021-05-17 22:13 ` Kieran Bingham
2021-05-18 7:23 ` Mauro Carvalho Chehab
2021-05-18 10:18 ` Kieran Bingham
2021-05-19 11:07 ` Mauro Carvalho Chehab
2021-06-16 13:36 ` Ariel D'Alessandro
2021-06-16 14:26 ` Ariel D'Alessandro
2021-06-16 14:59 ` Mauro Carvalho Chehab
2021-06-16 15:06 ` Xavier Claessens
2021-06-16 15:10 ` Nicolas Dufresne
2021-06-16 15:11 ` Laurent Pinchart
2021-10-04 14:24 ` Laurent Pinchart
2021-12-15 21:05 ` Laurent Pinchart [this message]
2021-11-18 9:03 ` Tomi Valkeinen
2021-11-18 10:09 ` Tomi Valkeinen
2021-12-15 21:07 ` Laurent Pinchart
2021-11-18 10:39 ` Laurent Pinchart
2021-12-15 21:10 ` 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=YbpYsgFEXvVKZguI@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=ariel.dalessandro@collabora.com \
--cc=ezequiel@vanguardiasur.com.ar \
--cc=gjasny@googlemail.com \
--cc=hverkuil@xs4all.nl \
--cc=kieran.bingham@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=mchehab+huawei@kernel.org \
--cc=nicolas.dufresne@collabora.com \
--cc=nicolas@ndufresne.ca \
--cc=p.zabel@pengutronix.de \
--cc=rosenp@gmail.com \
--cc=sakari.ailus@iki.fi \
--cc=sean@mess.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.