svn commit: r455665 - head/audio/zynaddsubfx

[Kurt Jaeger]

Hi!

> > Log:
> >   audio/zynaddsubfx: fix build on cur, 10.x by force llvm40
> 
> Commit message does not explain what was the original problem, so it's
> hard to judge whether attempted solution is correct or not.

Yes, as usual the saying goes: look at the PR, and you'll find the
links to the build logs that failed. hps will probably dig into it
if he finds time.

> > @@ -41,8 +41,8 @@ GUI_LIB_DEPENDS=		libfltk.so:x11-toolkits/fltk \
> >  				libfontconfig.so:x11-fonts/fontconfig
> >  GUI_ON=				-DGuiModule:STRING=fltk
> >  GUI_CMAKE_OFF=			-DGuiModule:STRING=OFF
> > -GUI_USE+=			XORG=x11,xext,xfixes,xcursor,ice,sm,xpm,xinerama,xft
> > -GUI_USE+=			GL=glu
> > +GUI_USE+=			XORG=x11,xext,xfixes,xcursor,xrender,ice,sm,xpm,xinerama,xft
> > +GUI_USE+=			GL=gl,glu
> 
> This change was not explained in the commit log.

It was missing, we found that during testbuilds, and added it as well.

Would we add it if it would not be missing ?

> > +.include <bsd.port.options.mk>
> 
> What is this .include <bsd.port.options.mk> needed for?

hps added it, and so it went in. I doubt it's necessary, so hps
can remove it when he updates the port the next time.

> > +# Build fix for ZynAddSubFx
> > +BUILD_DEPENDS+=	clang40:devel/llvm40
> > +CC=		clang40
> > +CXX=		clang++40
> 
> ./danfe

-- 
pi at FreeBSD.org         +49 171 3101372                3 years to go !