From: Takashi Iwai <tiwai@suse.de>
To: Pavel Roskin <proski@gnu.org>
Cc: perex@suse.cz, alsa-devel@lists.sourceforge.net
Subject: Re: Driver headers used by alsa-lib
Date: Tue, 08 Oct 2002 13:14:58 +0200 [thread overview]
Message-ID: <s5hlm592nd9.wl@alsa2.suse.de> (raw)
In-Reply-To: <Pine.LNX.4.44.0210041255490.23670-100000@marabou.research.att.com>
At Fri, 4 Oct 2002 13:20:32 -0400 (EDT),
Pavel Roskin wrote:
>
> Hello!
>
> There is another configuration problem that needs to be fixed. I had ALSA
> already installed (CVS version as of a week ago or so). Today I compiled
> alsa-driver, alsa-lib and alsa-util and then started installing them.
>
> I found that "make install" caused recompilation in alsa-lib. It used to
> be like that, but this time I wanted to do it right.
>
> I found that the reason for recompilation is new files in
> /usr/include/sound/, in particular asoundef.h. It turmed out that there
> are two different files with this name - one is
> alsa-lib/include/asoundef.h, the other is alsa-kernel/include/asoundef.h.
>
> Object files under alsa-lib were compiled against headers in
> /usr/include/sound/, and "make install" in alsa-kernel installed new
> headers there. The dependency was recorded by the compiler, so the
> alsa-lib object files became out of date.
>
> It seems to me that there are at least two things wrong here. First is
> that the kernel should not be installing its headers in places where they
> can be accidentally used. Most software should not need those headers at
> all.
>
> For example, Linux kernel includes are available under linux and asm in
> /usr/include. When I'm using them, I'm well aware of the fact that it's
> essentially a private kernel interface and that I should avoid it if
> possible.
well, not all people are aware of that inclusion of kernel headers
will break portability ;)
> If I include <sound/asoundef.h>, I would assume that it's something for
> user-space software, not for kernel drivers. Maybe kernel headers should
> be installed under /usr/include/sound/kernel if we install them at all?
sound header files are not necessary except for alsa-lib and hardware
specific applications like alsa-tools' ones.
i think /usr/include/sound is confusing. it looks like normal header
files...
> The other problem is that alsa-lib should be using its own headers from
> its own distribution rather than the preinstalled ones, otherwise we have
> a whole new problem with compiling against an older library.
right.
actually i use the extracted alsa-kernel header files to build
alsa-lib rpm package, too, so why not in the alsa-lib tree itself?
we need only three files from alsa-kernel, namely, asound.h,
asoundef.h and asequencer.h. they can be copied into alsa-lib tree,
for example, under alsa-lib/include/alsa-kernel directory, which will
be not copied to the public place.
or, we can keep the alsa-kernel header files on
/usr/include/alsa/kernel directory, so that we don't need two distinct
directories which may confuse people. (even in this case, the three
files above should be duplicated in alsa-lib, though.)
Jaroslav, how about this?
Takashi
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
next prev parent reply other threads:[~2002-10-08 11:14 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-10-04 17:20 Driver headers used by alsa-lib Pavel Roskin
2002-10-04 17:48 ` Peter L Jones
2002-10-08 11:14 ` Takashi Iwai [this message]
2002-10-09 6:58 ` Jaroslav Kysela
2002-10-09 9:55 ` Takashi Iwai
2002-10-09 10:06 ` Takashi Iwai
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=s5hlm592nd9.wl@alsa2.suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@lists.sourceforge.net \
--cc=perex@suse.cz \
--cc=proski@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox