From 5b4a5f2bf4fb1440241091cba32eec0d4adcd615 Mon Sep 17 00:00:00 2001 From: jlam Date: Thu, 18 Mar 2004 09:12:08 +0000 Subject: Fix serious bug where BUILDLINK_PACKAGES wasn't being ordered properly by moving the inclusion of buildlink3.mk files outside of the protected region. This bug would be seen by users that have set PREFER_PKGSRC or PREFER_NATIVE to non-default values. BUILDLINK_PACKAGES should be ordered so that for any package in the list, that package doesn't depend on any packages to the left of it in the list. This ordering property is used to check for builtin packages in the correct order. The problem was that including a buildlink3.mk file for correctly ensured that was removed from BUILDLINK_PACKAGES and appended to the end. However, since the inclusion of any other buildlink3.mk files within that buildlink3.mk was in a region that was protected against multiple inclusion, those dependencies weren't also moved to the end of BUILDLINK_PACKAGES. --- audio/SDL_mixer/buildlink3.mk | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'audio/SDL_mixer') diff --git a/audio/SDL_mixer/buildlink3.mk b/audio/SDL_mixer/buildlink3.mk index b18af6d5313..6b7920d0c0b 100644 --- a/audio/SDL_mixer/buildlink3.mk +++ b/audio/SDL_mixer/buildlink3.mk @@ -1,4 +1,4 @@ -# $NetBSD: buildlink3.mk,v 1.6 2004/03/05 19:25:06 jlam Exp $ +# $NetBSD: buildlink3.mk,v 1.7 2004/03/18 09:12:08 jlam Exp $ BUILDLINK_DEPTH:= ${BUILDLINK_DEPTH}+ SDL_MIXER_BUILDLINK3_MK:= ${SDL_MIXER_BUILDLINK3_MK}+ @@ -14,9 +14,8 @@ BUILDLINK_PACKAGES+= SDL_mixer BUILDLINK_DEPENDS.SDL_mixer+= SDL_mixer>=1.2.5nb2 BUILDLINK_PKGSRCDIR.SDL_mixer?= ../../audio/SDL_mixer BUILDLINK_INCDIRS.SDL_mixer?= include/SDL +.endif # SDL_MIXER_BUILDLINK3_MK .include "../../devel/SDL/buildlink3.mk" -.endif # SDL_MIXER_BUILDLINK3_MK - BUILDLINK_DEPTH:= ${BUILDLINK_DEPTH:S/+$//} -- cgit v1.2.3