Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
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

  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