[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <7caf7242-1ca4-462a-b3d0-627258df3f1b@linuxfoundation.org>
Date: Thu, 19 Sep 2024 09:40:34 -0600
From: Shuah Khan <skhan@...uxfoundation.org>
To: Abdul Rahim <abdul.rahim@...ahoo.com>, perex@...ex.cz, tiwai@...e.com,
broonie@...nel.org, shuah@...nel.org
Cc: linux-sound@...r.kernel.org, linux-kernel@...r.kernel.org,
Shuah Khan <skhan@...uxfoundation.org>
Subject: Re: [PATCH] kselftest/alsa: add silent flag to reduce noise
On 9/15/24 17:41, Abdul Rahim wrote:
> When ALSA is not installed on the users system, the error:
>
> "Package alsa was not found in the pkg-config search path.
> Perhaps you should add the directory containing `alsa.pc'
> to the PKG_CONFIG_PATH environment variable
> Package 'alsa', required by 'virtual:world', not found"
>
extra line?
> is printed 3 times, which generates unnecessary noise.
> Hence, Remove unnecessary noise using `--silence-errors` on LDLIBS
> assignment, so the message is printed only once.
I would say this message is alerting the user that the package is missing.
Why would you want to delete it?
>
> Signed-off-by: Abdul Rahim <abdul.rahim@...ahoo.com>
> ---
> tools/testing/selftests/alsa/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/alsa/Makefile b/tools/testing/selftests/alsa/Makefile
> index 25be68025290..cd022fc869fb 100644
> --- a/tools/testing/selftests/alsa/Makefile
> +++ b/tools/testing/selftests/alsa/Makefile
> @@ -2,7 +2,7 @@
> #
>
> CFLAGS += $(shell pkg-config --cflags alsa) $(KHDR_INCLUDES)
> -LDLIBS += $(shell pkg-config --libs alsa)
> +LDLIBS += $(shell pkg-config --silence-errors --libs alsa)
> ifeq ($(LDLIBS),)
> LDLIBS += -lasound
> endif
thanks,
-- Shuah
Powered by blists - more mailing lists