From nobody Wed Oct 05 15:49:18 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 4MjK065cCBz4V5bs; Wed, 5 Oct 2022 15:55:02 +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 4MjK062bsSz3kHK; Wed, 5 Oct 2022 15:55:02 +0000 (UTC) (envelope-from amdmi3@amdmi3.ru) Received: from amdmi3.ru (localhost [127.0.0.1]) by sprint-2.amdmi3.ru (Postfix) with SMTP id 62A3A188E79; Wed, 5 Oct 2022 18:55:54 +0300 (MSK) Received: by amdmi3.ru (nbSMTP-1.00) for uid 1000 amdmi3@amdmi3.ru; Wed, 5 Oct 2022 18:55:54 +0300 (MSK) Date: Wed, 5 Oct 2022 18:49:18 +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: c698371b0910 - main - devel/sdl20: fix armv7-related check in SDL_atomic.h Message-ID: References: <202209291355.28TDt7Ln090671@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-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Rspamd-Queue-Id: 4MjK062bsSz3kHK X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-ThisMailContainsUnwantedMimeParts: N * Piotr Kubaj (pkubaj@anongoth.pl) wrote: > Do you want me to put that patch in files now or, since it's > already committed to ports and will be removed anyway with the next > SDL update, it's ok as it is now? It's ok as it is now. It doesn't apply to SDL 2.24.1 update I'm working on now though, can it be safely removed with the update? > On 22-10-05 16:04:21, Dmitry Marakasov wrote: > > * Piotr Kubaj (pkubaj@FreeBSD.org) wrote: > > > > > URL: https://cgit.FreeBSD.org/ports/commit/?id=c698371b0910e792db5690126bde955456fdccd2 > > > > > > commit c698371b0910e792db5690126bde955456fdccd2 > > > Author: Piotr Kubaj > > > AuthorDate: 2022-09-29 13:51:25 +0000 > > > Commit: Piotr Kubaj > > > CommitDate: 2022-09-29 13:54:40 +0000 > > > > > > devel/sdl20: fix armv7-related check in SDL_atomic.h > > > > > > Fixes build errors like: > > > In file included from /usr/ports/games/openrct2/work/OpenRCT2-0.4.1/src/openrct2-ui/CursorRepository.h:12: In file included from /usr/local/include/SDL2/SDL.h:35: /usr/local/include/SDL2/SDL_atomic.h:243:28: error: '__ARM_ARCH__' is not defined, evaluates to 0 [-Werror,-Wundef] #elif (defined(__arm__) && __ARM_ARCH__ >= 7) || defined(__aarch64__) > > > --- > > > devel/sdl20/Makefile | 4 ++++ > > > devel/sdl20/distinfo | 4 +++- > > > 2 files changed, 7 insertions(+), 1 deletion(-) > > > > > > diff --git a/devel/sdl20/Makefile b/devel/sdl20/Makefile > > > index d84a9679030b..6e8c2c390ba6 100644 > > > --- a/devel/sdl20/Makefile > > > +++ b/devel/sdl20/Makefile > > > @@ -1,9 +1,13 @@ > > > PORTNAME= sdl2 > > > DISTVERSION= 2.24.0 > > > +PORTREVISION= 1 > > > CATEGORIES= devel > > > MASTER_SITES= https://www.libsdl.org/release/ > > > DISTNAME= SDL2-${PORTVERSION} > > > > > > +PATCH_SITES= https://github.com/libsdl-org/SDL/commit/ > > > +PATCHFILES= a204ef50bbf20b77083c0a14124e673efc3a53c6.patch:-p1 > > > > I think I've already asked, but I'm not sure it was you. Please do not > > ever introduce this kind of unmaintainable ugliness to my ports. > > All patches should be put under files/. > > > > > MAINTAINER= amdmi3@FreeBSD.org > > > COMMENT= Cross-platform multimedia development API > > > WWW= https://www.libsdl.org/ > > > diff --git a/devel/sdl20/distinfo b/devel/sdl20/distinfo > > > index 3f836d60852a..c78d82f18d9c 100644 > > > --- a/devel/sdl20/distinfo > > > +++ b/devel/sdl20/distinfo > > > @@ -1,3 +1,5 @@ > > > -TIMESTAMP = 1661880729 > > > +TIMESTAMP = 1664457277 > > > SHA256 (SDL2-2.24.0.tar.gz) = 91e4c34b1768f92d399b078e171448c6af18cafda743987ed2064a28954d6d97 > > > SIZE (SDL2-2.24.0.tar.gz) = 7520541 > > > +SHA256 (a204ef50bbf20b77083c0a14124e673efc3a53c6.patch) = 2cc9607f42acc8b00ac058cbc5fcba3d7037db21db679533afd483824dd5b658 > > > +SIZE (a204ef50bbf20b77083c0a14124e673efc3a53c6.patch) = 1168 -- Dmitry Marakasov . amdmi3@amdmi3.ru ..: https://github.com/AMDmi3 https://amdmi3.ru/