mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: Philipp Zabel <p.zabel@pengutronix.de>
Cc: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [RFC 1/2] libva-headers: new package
Date: Fri, 25 Feb 2022 09:23:40 +0100	[thread overview]
Message-ID: <YhiSDGGFlNmFOXJG@pengutronix.de> (raw)
In-Reply-To: <20220223135352.604965-1-p.zabel@pengutronix.de>

On Wed, Feb 23, 2022 at 02:53:51PM +0100, Philipp Zabel wrote:
> Add a stripped down version of the libva package that installs only the
> headers and pkgconfig file required by Mesa.
> This avoids a circular dependency between Mesa, which requries libva
> headers, and libva, which may require Mesa to build GLX support.
> 
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
> ---
>  rules/libva-headers.in   |  9 +++++++
>  rules/libva-headers.make | 53 ++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 62 insertions(+)
>  create mode 100644 rules/libva-headers.in
>  create mode 100644 rules/libva-headers.make
> 
> diff --git a/rules/libva-headers.in b/rules/libva-headers.in
> new file mode 100644
> index 000000000000..a611b483fdb4
> --- /dev/null
> +++ b/rules/libva-headers.in
> @@ -0,0 +1,9 @@
> +## SECTION=multimedia_vaapi
> +
> +config LIBVA_HEADERS
> +	tristate
> +	select HOST_MESON
> +	select LIBDRM
> +	prompt "libva-headers"
> +	help
> +	  VA-API header files for mesalib
> diff --git a/rules/libva-headers.make b/rules/libva-headers.make
> new file mode 100644
> index 000000000000..5f80eead4c79
> --- /dev/null
> +++ b/rules/libva-headers.make
> @@ -0,0 +1,53 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2020 by Philipp Zabel <p.zabel@pengutronix.de>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_LIBVA_HEADERS) += libva-headers
> +
> +#
> +# Paths and names
> +#
> +LIBVA_HEADERS_VERSION	= $(LIBVA_VERSION)
> +LIBVA_HEADERS_MD5	= $(LIBVA_MD5)
> +LIBVA_HEADERS		= libva-headers-$(LIBVA_VERSION)
> +LIBVA_HEADERS_SUFFIX	= tar.bz2
> +LIBVA_HEADERS_URL	= $(LIBVA_URL)
> +LIBVA_HEADERS_SOURCE	= $(LIBVA_SOURCE)
> +LIBVA_HEADERS_DIR	= $(BUILDDIR)/$(LIBVA_HEADERS)
> +LIBVA_HEADERS_LICENSE	= $(LIBVA_LICENSE)
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +LIBVA_HEADERS_CONF_TOOL	:= meson
> +LIBVA_HEADERS_CONF_OPT	:= \
> +	$(CROSS_MESON_USR) \
> +	-Ddisable_drm=false \
> +	-Ddriverdir='' \
> +	-Denable_docs=false \
> +	-Denable_va_messaging=true \
> +	-Dwith_glx=no \
> +	-Dwith_wayland=no \
> +	-Dwith_x11=no \
> +
> +# ----------------------------------------------------------------------------
> +# Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/libva-headers.install.post:
> +	@$(call targetinfo)
> +	# only install headers and libva.pc
> +	@rm -f $(LIBVA_HEADERS_PKGDIR)/usr/lib/libva*
> +	@rm -f $(LIBVA_HEADERS_PKGDIR)/usr/lib/pkgconfig/libva-drm.pc

I think this should be done at the end of 'install' instead.

And you need to do the reverse in libva and add a dependency there.
Otherwise a 'ptxdist clean libva' will delete the header files and if mesa
is rebuilt then that will fail.

Michael

> +	@$(call world/install.post, LIBVA_HEADERS)
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.30.2
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de


      parent reply	other threads:[~2022-02-25  8:24 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-23 13:53 Philipp Zabel
2022-02-23 13:53 ` [ptxdist] [RFC 2/2] mesalib: enable VA support Philipp Zabel
2022-02-24 16:02 ` [ptxdist] [RFC 1/2] libva-headers: new package Roland Hieber
2022-02-25  8:23 ` Michael Olbrich [this message]

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=YhiSDGGFlNmFOXJG@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=p.zabel@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /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