Re: CVS commit: wip/mupen64plus

Hi!

On Mon, Jan 14, 2013 at 05:17:56PM +0000, othyro wrote:
> Module name:	wip
> Committed by:	othyro
> Date:		Mon Jan 14 17:17:41 UTC 2013
> 
> Modified Files:
> 	wip/mupen64plus: Makefile
> 
> Log Message:
> Convert MesaLib and freetype2 to BUILD_DEPENDS and add glu to them. Bump
> PKGREVISION.

Diff contained:
@@ -32,12 +32,13 @@
                cd ${WRKSRC} && ${SETENV} ${INSTALL_ENV} ${BASH} m64p_install.sh;


-.include "../../graphics/MesaLib/buildlink3.mk"
+BUILD_DEPENDS+=        glu-[0-9]*:../../graphics/glu
+BUILD_DEPENDS+=        MesaLib-[0-9]*:../../graphics/MesaLib
 .include "../../mk/dlopen.buildlink3.mk"
 .include "../../mk/pthread.buildlink3.mk"
 #.include "../../x11/gtk2/buildlink3.mk"
 .include "../../devel/zlib/buildlink3.mk"
 .include "../../graphics/png/buildlink3.mk"
-.include "../../graphics/freetype2/buildlink3.mk"
+BUILD_DEPENDS+=        freetype2-[0-9]*:../../graphics/freetype2
 .include "../../devel/SDL/buildlink3.mk"
 .include "../../mk/bsd.pkg.mk"

This doesn't work that way -- when I try building it now, I get:
************************************ Building core library
make: Entering directory `/scratch/wip/mupen64plus/work/mupen64plus-bundle-src-1.99.5/source/mupen64plus-core/projects/unix'
Makefile:185: *** No FreeType 2 development libraries found!.  Stop.
make: Leaving directory `/scratch/wip/mupen64plus/work/mupen64plus-bundle-src-1.99.5/source/mupen64plus-core/projects/unix'
*** Error code 2

Not surprisingly, because the buildlink3 framework doesn't link files
that are BUILD_DEPENDS.

What you want is to include the buildlink3.mk files as usual, but set
"BUILDLINK_DEPMETHOD.foo=build" for appropriate values of "foo".

Please fix!

Thank you,
 Thomas

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122712