[Kde-games-devel] Re: KGoldrunner now uses TagaroAudio, OpenAL and SndFile

[Ian Wadham]

On Wednesday 11 May 2011 11:54:09 pm Parker Coates wrote:
> On Wed, May 11, 2011 at 05:02, Ian Wadham wrote:
> > On Tuesday 10 May 2011 7:54:15 am Mathias Kraus wrote:
> >> If you want to make OpenAL and SndFile a hard dependency for
> >> KGoldrunner, I'd also make it a hard dependency for granatier because
> >> the sound is so much better and I think if two games use it, it might be
> >> a good reason for a hard dependency.
> >
> > Yes I do want to make OpenAL and SndFile a hard dependency for
> > KGoldrunner. Tomorrow is the last day to decide for Granatier.  Did you
> > see my last reply?
> >
> > I am poised, ready to commit (first thing Thursday) a version of
> > kgoldrunner/CMakeLists.txt that has the line:
> >    find_package(OpenAL REQUIRED)
> > instead of:
> >    macro_optional_find_package(OpenAL QUIET)
> >
> > I am not sure if this is the correct way to do it.  OpenAL *still* lists
> > as an optional external package during the kdegames build.  But at least,
> > KGoldrunner fails to build if OpenAL is not found.  Will the Release Team
> > pick up on that?  Or should I warn them?
> 
> You should also update kdegames/CMakeLists.txt to check for OpenAL and
> SndFile and then attempt to build KGoldrunner only if they're found,
> much the same as is already done for OpenGL, QCA and Python.
> 
I wrote to Stefan privately about that. He has replied on this list that we
should keep the build details and TagaroAudio source code at the KGr and
Granatier level for now, but do a proper job with libtagaro (or whatever it
is eventually called) for KDE 4.8, including move of kdegames to git.

Kubrick is not built if there is no OpenGL support, because it is useless
without that.  KSudoku builds OK, but conditionally builds the 3-D variant
and always builds the 2-D variants.  The KGoldrunner case is rather like
KSudoku.  KGoldrunner can be built without sound and works fine, but
maybe it is less fun.  So I am looking either for OpenAL sound or no sound.

Cheers, Ian W.