Re: [PATCH i-g-t 1/3] lib/uwildmat: Move to a dedicated vendored library directory

[Peter Senna Tschudin <peter.senna@intel.com>]

On 13.09.2024 12:44, Kamil Konieczny wrote:
> From: Ryszard Knop <ryszard.knop@intel.com>
> 
> This directory is in the igt tree for convenience. It's not immediately
> clear that this is a vendored library, so move it into vendor/ and make
> Meson add that directory for easy includes etc.
> 

Reviewed-by: Peter Senna Tschudin <peter.senna@linux.intel.com>
> Signed-off-by: Ryszard Knop <ryszard.knop@intel.com>
> ---
>  lib/meson.build                      | 6 +++++-
>  lib/{ => vendor}/uwildmat/uwildmat.c | 0
>  lib/{ => vendor}/uwildmat/uwildmat.h | 0
>  3 files changed, 5 insertions(+), 1 deletion(-)
>  rename lib/{ => vendor}/uwildmat/uwildmat.c (100%)
>  rename lib/{ => vendor}/uwildmat/uwildmat.h (100%)
> 
> diff --git a/lib/meson.build b/lib/meson.build
> index 1c83bc5c3..b10410c8a 100644
> --- a/lib/meson.build
> +++ b/lib/meson.build
> @@ -95,7 +95,6 @@ lib_sources = [
>  	'igt_pm.c',
>  	'igt_dummyload.c',
>  	'igt_store.c',
> -	'uwildmat/uwildmat.c',
>  	'igt_kmod.c',
>  	'igt_ktap.c',
>  	'igt_panfrost.c',
> @@ -117,6 +116,9 @@ lib_sources = [
>  	'xe/xe_query.c',
>  	'xe/xe_spin.c',
>  	'xe/xe_util.c',
> +
> +	# Vendored libraries:
> +	'vendor/uwildmat/uwildmat.c',
>  ]
>  
>  lib_deps = [
> @@ -135,6 +137,8 @@ lib_deps = [
>  	zlib
>  ]
>  
> +inc = [ inc, include_directories('vendor') ]
> +
>  if libdrm_nouveau.found()
>  	lib_deps += libdrm_nouveau
>  	lib_sources += [
> diff --git a/lib/uwildmat/uwildmat.c b/lib/vendor/uwildmat/uwildmat.c
> similarity index 100%
> rename from lib/uwildmat/uwildmat.c
> rename to lib/vendor/uwildmat/uwildmat.c
> diff --git a/lib/uwildmat/uwildmat.h b/lib/vendor/uwildmat/uwildmat.h
> similarity index 100%
> rename from lib/uwildmat/uwildmat.h
> rename to lib/vendor/uwildmat/uwildmat.h

Intel Semiconductor AG
Registered No. 020.30.913.786-7
Registered Office: Dufourstrasse 101 , 8008 Zurich, Switzerland