openal-soft .pc file compiling is broken i need help please

[LinuxDonald]

I found today that the .pc file compiling is broken in openal-soft.

When i compile openal-soft i become this .pc file:

prefix=/usr
exec_prefix=${prefix}
libdir=${exec_prefix}//usr/lib64
includedir=${prefix}/include

Name: OpenAL
Description: OpenAL is a cross-platform 3D audio API
Requires:
Version: 1.12.854
Libs: -L${libdir} -lopenal
Cflags: -I${includedir}

Can anyone please help me to fix it? Upstream need this patch too.
I have the CMakeLists.txt added to this mail.

[Rex Dieter]

LinuxDonald wrote:
> I found today that the .pc file compiling is broken in openal-soft.
>
> When i compile openal-soft i become this .pc file:
>
> prefix=/usr
> exec_prefix=${prefix}
> libdir=${exec_prefix}//usr/lib64
> includedir=${prefix}/include
>
> Name: OpenAL
> Description: OpenAL is a cross-platform 3D audio API
> Requires:
> Version: 1.12.854
> Libs: -L${libdir} -lopenal
> Cflags: -I${includedir}
>
> Can anyone please help me to fix it? Upstream need this patch too.
> I have the CMakeLists.txt added to this mail.

I can take a look (probably tomorrow).

-- Rex

[Braden McDaniel]

On 4/1/10 2:17 PM, LinuxDonald wrote:
> I found today that the .pc file compiling is broken in openal-soft.
>
> When i compile openal-soft i become this .pc file:
>
> prefix=/usr
> exec_prefix=${prefix}
> libdir=${exec_prefix}//usr/lib64
> includedir=${prefix}/include
>
> Name: OpenAL
> Description: OpenAL is a cross-platform 3D audio API
> Requires:
> Version: 1.12.854
> Libs: -L${libdir} -lopenal
> Cflags: -I${includedir}
>
> Can anyone please help me to fix it? Upstream need this patch too.
> I have the CMakeLists.txt added to this mail.

It looks like this line in CMakeLists.txt:
> SET(libdir "${exec_prefix}/${LIB_INSTALL_DIR}")

should be
> SET(libdir "${LIB_INSTALL_DIR}")

[LinuxDonald]

Am 01.04.2010 23:01, schrieb Braden McDaniel:
> On 4/1/10 2:17 PM, LinuxDonald wrote:
> > I found today that the .pc file compiling is broken in openal-soft.
> >
> > When i compile openal-soft i become this .pc file:
> >
> > prefix=/usr
> > exec_prefix=${prefix}
> > libdir=${exec_prefix}//usr/lib64
> > includedir=${prefix}/include
> >
> > Name: OpenAL
> > Description: OpenAL is a cross-platform 3D audio API
> > Requires:
> > Version: 1.12.854
> > Libs: -L${libdir} -lopenal
> > Cflags: -I${includedir}
> >
> > Can anyone please help me to fix it? Upstream need this patch too.
> > I have the CMakeLists.txt added to this mail.
>
> It looks like this line in CMakeLists.txt:
> > SET(libdir "${exec_prefix}/${LIB_INSTALL_DIR}")
>
> should be
> > SET(libdir "${LIB_INSTALL_DIR}")

Yeah that fixed:
libdir=/usr/lib64

But what about that:
exec_prefix=${prefix}
includedir=${prefix}/include
?
That are broken to in the .pc file :(

[Braden McDaniel]

On Sun, 2010-04-04 at 17:18 +0200, LinuxDonald wrote:
> Am 01.04.2010 23:01, schrieb Braden McDaniel:

[snip]
> > It looks like this line in CMakeLists.txt:
> > > SET(libdir "${exec_prefix}/${LIB_INSTALL_DIR}")
> >
> > should be
> > > SET(libdir "${LIB_INSTALL_DIR}")
>
> Yeah that fixed:
> libdir=/usr/lib64
>
> But what about that:
> exec_prefix=${prefix}
> includedir=${prefix}/include
> ?
> That are broken to in the .pc file :(

Those don't look broken to me.  Why do you think they are?

See "man pkg-config" under METADATA FILE SYNTAX.

[LinuxDonald]

Am 05.04.2010 03:11, schrieb Braden McDaniel:
> On Sun, 2010-04-04 at 17:18 +0200, LinuxDonald wrote:
> > Am 01.04.2010 23:01, schrieb Braden McDaniel:
>
> [snip]
> > > It looks like this line in CMakeLists.txt:
> > > > SET(libdir "${exec_prefix}/${LIB_INSTALL_DIR}")
> > >
> > > should be
> > > > SET(libdir "${LIB_INSTALL_DIR}")
> >
> > Yeah that fixed:
> > libdir=/usr/lib64
> >
> > But what about that:
> > exec_prefix=${prefix}
> > includedir=${prefix}/include
> > ?
> > That are broken to in the .pc file :(
>
> Those don't look broken to me.  Why do you think they are?
>
> See "man pkg-config" under METADATA FILE SYNTAX.

Don´t must be there real path ala:

includedir=${prefix}/include ala includedir=/usr/include/AL ?

[Braden McDaniel]

On Mon, 2010-04-05 at 13:09 +0200, LinuxDonald wrote:
> Am 05.04.2010 03:11, schrieb Braden McDaniel:
> > On Sun, 2010-04-04 at 17:18 +0200, LinuxDonald wrote:
> > > Am 01.04.2010 23:01, schrieb Braden McDaniel:
> >
> > [snip]
> > > > It looks like this line in CMakeLists.txt:
> > > > > SET(libdir "${exec_prefix}/${LIB_INSTALL_DIR}")
> > > >
> > > > should be
> > > > > SET(libdir "${LIB_INSTALL_DIR}")
> > >
> > > Yeah that fixed:
> > > libdir=/usr/lib64
> > >
> > > But what about that:
> > > exec_prefix=${prefix}
> > > includedir=${prefix}/include
> > > ?
> > > That are broken to in the .pc file :(
> >
> > Those don't look broken to me.  Why do you think they are?
> >
> > See "man pkg-config" under METADATA FILE SYNTAX.
>
> Don´t must be there real path ala:
>
> includedir=${prefix}/include ala includedir=/usr/include/AL ?

Not according to the section of the man page to which I just directed
you.