Content-Transfer-Encoding: 7bit
Content-Type: multipart/mixed; boundary="_----------=_1708199115283790"
MIME-Version: 1.0
Date: Sat, 17 Feb 2024 19:45:15 +0000
From: "Thomas Klausner" <wiz@netbsd.org>
Subject: CVS commit: pkgsrc/multimedia/gmplayer
To: pkgsrc-changes@NetBSD.org
Reply-To: wiz@netbsd.org
Message-Id: <20240217194515.4F60EF9F2@cvs.NetBSD.org>
Sender: pkgsrc-changes-owner@NetBSD.org
Precedence: bulk

This is a multi-part message in MIME format.

--_----------=_1708199115283790
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"

Module Name:	pkgsrc
Committed By:	wiz
Date:		Sat Feb 17 19:45:15 UTC 2024

Modified Files:
	pkgsrc/multimedia/gmplayer: Makefile

Log Message:
gmplayer: fix build

Apply mplayer's

revision 1.113
date: 2024-01-27 03:11:04 +0100;  author: riastradh;  state: Exp;  lines: +16 -1;  commitid: YiogfS4wLLyaV1WE;
multimedia/mplayer: Fix cross-build.

This removes our local patch to pass CFLAGS to compile the the
build-time tool codec-cfg.  These CFLAGS don't work for cross-builds,
but all we really need is to make libavutil/avutil.h.  So instead of
patching the makefile, just pass BUILDLINK_CPPFLAGS through into the
HOST_CC invocation that compiles it, in native builds.

And in cross builds, sleazily pass the include path under the
localbase in the destdir, so that we don't have to pull in ffmpeg5 as
a tool dependency -- libavutil/*.h seems to work here, and may
actually be correct because it pulls in libavutil/avconfig.h and
codec-cfg might want that to describe the target system rather than
the build system.


To generate a diff of this commit:
cvs rdiff -u -r1.163 -r1.164 pkgsrc/multimedia/gmplayer/Makefile

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.


--_----------=_1708199115283790
Content-Disposition: inline
Content-Length: 1336
Content-Transfer-Encoding: binary
Content-Type: text/x-diff; charset=us-ascii

Modified files:

Index: pkgsrc/multimedia/gmplayer/Makefile
diff -u pkgsrc/multimedia/gmplayer/Makefile:1.163 pkgsrc/multimedia/gmplayer/Makefile:1.164
--- pkgsrc/multimedia/gmplayer/Makefile:1.163	Mon Jan 22 13:17:02 2024
+++ pkgsrc/multimedia/gmplayer/Makefile	Sat Feb 17 19:45:15 2024
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.163 2024/01/22 13:17:02 ryoon Exp $
+# $NetBSD: Makefile,v 1.164 2024/02/17 19:45:15 wiz Exp $
 #
 # NOTE: if you are updating both mplayer and gmplayer, you must ensure
 # that *both* distinfo files contain the correct, up-to-date files and
@@ -108,6 +108,18 @@ CONFIGURE_ARGS+=	--enable-gui --disable-
 			--confdir=${PREFIX}/share/mplayer
 CONFIGURE_ARGS+=	--enable-xvmc
 
+.if ${USE_CROSS_COMPILE:tl} == "yes"
+CONFIGURE_ARGS+=	--target=${MACHINE_ARCH:Q}-${OPSYS:Q}
+# Reaching into the destdir with a -I flag to NATIVE_CC is kinda
+# sleazy, but this will probably work and it saves the trouble of
+# pulling in a gigantic tool dependency.
+CONFIGURE_ARGS+=	--host-cc=${NATIVE_CC:Q}\ -I${_CROSS_DESTDIR:Q}${PREFIX:Q}/include/ffmpeg5
+#TOOL_DEPENDS+=		ffmpeg5>=0:../../multimedia/ffmpeg5
+#CONFIGURE_ARGS+=	--host-cc=${NATIVE_CC:Q}\ -I${TOOLBASE:Q}/include/ffmpeg5
+.else
+CONFIGURE_ARGS+=	--host-cc=${CC:Q}\ ${BUILDLINK_CPPFLAGS:Q}
+.endif
+
 INSTALLATION_DIRS=	bin share/mplayer/skins
 
 do-install:


--_----------=_1708199115283790--