On Fri, 20 Aug 2010, Stephen Warren wrote:
> Jaroslav Kysela wrote:
>> Stephen Warren wrote:
>>> Jaroslav Kysela wrote:
>>>> Stephen Warren wrote:
>>>>> Unfortunately, it doesn't seem to work:
>>>>>
>>>>> ALSAKERNELDIR=../sound-2.6/sound ./gitcompile
>>>>
>>>> It should be:
>>>>
>>>> ALSAKERNELDIR=../sound-2.6 ./gitcompile
>>>
>>> sound-2.6 is a complete Linux kernel tree, not just the ALSA drivers.
>>> Hence, the second part of the following test fails, because last.c is in
>>> ../sound-2.6/sound/last.c not ../sound-2.6/last.c
>>>
>>> if [ ! -d $ALSAKERNELDIR -o ! -f $ALSAKERNELDIR/last.c ]; then
>>> echo "No alsa-kmirror is found."
>>
>> You're right. But the Makefile accepts both trees. This patch should solve
>> this issue:
>
> OK, that patch works OK, but I actually ended up getting the exact same issue.
>
> make[3]: Warning: File `.includes' has modification time 0.04 s in the future
> cp ../../alsa-kernel/include/*.h .
> cp: cannot stat `../../alsa-kernel/include/*.h': No such file or directory
>
> I looked into that further this time, and I found it's because
> alsa-driver-build/alsa-kernel/include points at ../sound-2.6/include/sound,
> but should have an extra "../" on the front. Doing the following fixes this:
>
> ALSAKERNELDIR=`cd ../sound-2.6 && pwd` ./gitcompile
>
> Doesn't this affect the ../alsa-kmirror case too? Perhaps everyone who checks
> out alsa-kmirror places it inside the alsa-driver-build directory, or symlinks
> it themselves first?
>
> Perhaps gitcompile can add the following right before the make:
>
> ALSAKERNELDIR=`cd ${ALSAKERNELDIR} && pwd`
Good catch. The fix is in my tree now:
http://git.alsa-project.org/?p=alsa-driver.git;a=commitdiff;h=59ca79b3253de1c160ee446a67d3d5fd2b4dba92
Jaroslav
-----
Jaroslav Kysela <perex@xxxxxxxx>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
