From patchwork Wed Dec 29 01:57:56 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: GitHub pull_request - opened X-Patchwork-Id: 528819 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7F362C433EF for ; Wed, 29 Dec 2021 01:58:49 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id B8D721739; Wed, 29 Dec 2021 02:57:57 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz B8D721739 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1640743127; bh=qs+gCC6MrXwU0LwSwJF6LaJV9C0nkdp7peVAJzGBz5A=; h=From:To:In-Reply-To:References:Subject:Date:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=Yui8qEu0G6xft1odK4fqh6e4Tlc3p7fC+eAqPsGr/sxw18dsG2BS1lEqMQqpu9jPG r/vtDt2x8gT1i8bXVSlf0Fq5Ca56QXug65TY6w8/pB/nKGYS6T+iqeUEdK/R461w2L hXhehxYDycUnDOxPyRBBZCeBRaHg2mCIsgp6QgEw= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 66CB1F801EC; Wed, 29 Dec 2021 02:57:57 +0100 (CET) Received: by alsa1.perex.cz (Postfix, from userid 50401) id D73BEF80224; Wed, 29 Dec 2021 02:57:56 +0100 (CET) Received: from webhooks-bot.alsa-project.org (gate.perex.cz [77.48.224.242]) by alsa1.perex.cz (Postfix) with ESMTP id 121C0F800E9 for ; Wed, 29 Dec 2021 02:57:53 +0100 (CET) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 121C0F800E9 MIME-Version: 1.0 From: GitHub issues - edited To: alsa-devel@alsa-project.org In-Reply-To: <1640743072750150919-webhooks-bot@alsa-project.org> References: <1640743072750150919-webhooks-bot@alsa-project.org> Subject: [patch] /usr/share/alsa/speaker-test/sample_map.csv is not accurate, which does not affect operation of speaker-test Message-Id: <20211229015756.D73BEF80224@alsa1.perex.cz> Date: Wed, 29 Dec 2021 02:57:56 +0100 (CET) X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" alsa-project/alsa-utils issue #133 was edited from ZjYwMj: With alsa-utils 1.2.6: ``` ``` The directory ``/usr/share/alsa/samples/`` does not exist. It sounds as if ``speaker-test -t wav`` plays the files mentioned in the diff. This inaccuracy does not affect the operation of ``speaker-test``. Do the paths to the wav files hard coded within ``speaker-test``? Does only their ``basename`` counts? Still, I find it a minor bug. Why is ``/usr/share/alsa/speaker-test/sample_map.csv`` distributed anyway? Am I misunderstanding [Install sound samples to correct places](https://github.com/alsa-project/alsa-utils/commit/fb2f99f1f99c430d1ca263be84d919aa808601c3)? Is it for informational purposes? Was it used in the past, and no longer serves its purpose? Is it planed to be used in the future? Is it something else? [speaker-test.c:745](https://github.com/alsa-project/alsa-utils/blob/00be486131129a6d209de62202d2a4974638127f/speaker-test/speaker-test.c#L745) and [746](https://github.com/alsa-project/alsa-utils/blob/00be486131129a6d209de62202d2a4974638127f/speaker-test/speaker-test.c#L746) has the ``basename``s hardcoded. And there is also the default value for ``speaker-test --wavdir`` of ``/usr/share/sounds/alsa``. Issue URL : https://github.com/alsa-project/alsa-utils/issues/133 Repository URL: https://github.com/alsa-project/alsa-utils --- a/usr/share/alsa/speaker-test/sample_map.csv 2021-12-06 18:44:25.000000000 +0000 +++ b/usr/share/alsa/speaker-test/sample_map.csv 2021-12-28 19:48:02.390305957 +0000 @@ -1,2 +1,2 @@ -0, "Front Left", "/usr/share/alsa/samples/Front_Left.wav" -1, "Front Right", "/usr/share/alsa/samples/Front_Right.wav" +0, "Front Left", "/usr/share/sounds/alsa/Front_Left.wav" +1, "Front Right", "/usr/share/sounds/alsa/Front_Right.wav"