From nobody Wed Jan 12 15:17:15 2022 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id E3DE519531C5; Wed, 12 Jan 2022 15:19:53 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from sprint-2.amdmi3.ru (sprint-2.amdmi3.ru [IPv6:2a0a:2b41:94:cb5e::]) by mx1.freebsd.org (Postfix) with ESMTP id 4JYrpK5S4Xz3JCx; Wed, 12 Jan 2022 15:19:53 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from amdmi3.ru (localhost [IPv6:::1]) by sprint-2.amdmi3.ru (Postfix) with SMTP id C293F188CE5; Wed, 12 Jan 2022 18:19:44 +0300 (MSK) Received: by amdmi3.ru (nbSMTP-1.00) for uid 1000 amdmi3@amdmi3.ru; Wed, 12 Jan 2022 18:19:44 +0300 (MSK) Date: Wed, 12 Jan 2022 18:17:15 +0300 From: Dmitry Marakasov To: Piotr Kubaj Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 74615207da3d - main - audio/wavplay: fix ONLY_FOR_ARCHS Message-ID: References: <202201121250.20CCojGJ020504@gitrepo.freebsd.org> List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Rspamd-Queue-Id: 4JYrpK5S4Xz3JCx X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N * Piotr Kubaj (pkubaj@anongoth.pl) wrote: > this is wrong. If there is endianness issue, why is powerpc64le removed? > > Either the issue is something else (and ONLY_FOR_ARCHS_REASON > needs to be fixed) or powerpc64le should be readded. It may only be readded if it's confirmed that wavplay correctly plays sound there. Endianess may not be the only issue. > On 22-01-12 12:50:45, Dmitry Marakasov wrote: > > The branch main has been updated by amdmi3: > > > > URL: https://cgit.FreeBSD.org/ports/commit/?id=74615207da3d4e9b707ef5713256456bcd19d6c3 > > > > commit 74615207da3d4e9b707ef5713256456bcd19d6c3 > > Author: Dmitry Marakasov > > AuthorDate: 2022-01-12 11:02:38 +0000 > > Commit: Dmitry Marakasov > > CommitDate: 2022-01-12 12:44:23 +0000 > > > > audio/wavplay: fix ONLY_FOR_ARCHS > > > > - Remove powerpc* from ONLY_FOR_ARCHS, which were incorrectly added > > on a sole basis that the port builds. It needs to be runtested. > > - Add ONLY_FOR_ARCHS_REASON to prevent such mistake in future. > > > > PR: 261061 > > --- > > audio/wavplay/Makefile | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/audio/wavplay/Makefile b/audio/wavplay/Makefile > > index bfc5c7a82577..d16999356e4d 100644 > > --- a/audio/wavplay/Makefile > > +++ b/audio/wavplay/Makefile > > @@ -12,7 +12,8 @@ COMMENT= Wav player and recorder > > LICENSE= GPLv2 # only > > LICENSE_FILE= ${WRKSRC}/COPYING > > > > -ONLY_FOR_ARCHS= amd64 armv6 armv7 i386 powerpc64 powerpc64le > > +ONLY_FOR_ARCHS= amd64 armv6 armv7 i386 > > +ONLY_FOR_ARCHS_REASON= does not play sound correctly due to endian issues > > > > CFLAGS+= -D_WANT_SEMUN -- Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D amdmi3@amdmi3.ru ..: https://github.com/AMDmi3 https://amdmi3.ru