All of 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.