* [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
@ 2011-03-12 10:30 Colin Guthrie
2011-03-12 10:47 ` Colin Guthrie
` (2 more replies)
0 siblings, 3 replies; 11+ messages in thread
From: Colin Guthrie @ 2011-03-12 10:30 UTC (permalink / raw)
To: alsa-devel
[-- Attachment #1: Type: text/plain, Size: 1637 bytes --]
Hi,
After doing a "simple":
git clone alsa-driver
git clone alsa-kernel
cd alsa-driver
./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
--with-moddir=updates/alsa:
The build bombed out pretty quickly with the following error.
The attached patch fixes it.
Hacking autoconf.h...
make[1]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include'
make -C sound prepare
make[2]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make prepare2
make[3]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
cp ../../alsa-kernel/include/ac97_codec.h .
patch -p0 -i ac97_codec.patch ac97_codec.h
patching file ac97_codec.h
Hunk #1 succeeded at 650 (offset 5 lines).
cp ../../alsa-kernel/include/core.h .
patch -p0 -i core.patch core.h
patching file core.h
cp ../../alsa-kernel/include/info.h .
cp: `../../alsa-kernel/include/info.h' and `./info.h' are the same file
make[3]: *** [info.h] Error 1
make[3]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make[2]: *** [prepare] Error 2
make[2]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make[1]: *** [prepare] Error 2
make[1]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include'
make: *** [dep] Error 1
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
[-- Attachment #2: 0001-Fix-broken-build-due-to-cp-symlink-behaviour.patch --]
[-- Type: text/x-patch, Size: 953 bytes --]
>From c8d828633824d1dc9eb1739c676fb1a8b961c523 Mon Sep 17 00:00:00 2001
From: Colin Guthrie <cguthrie@mandriva.org>
Date: Sat, 12 Mar 2011 10:28:38 +0000
Subject: [PATCH] Fix broken build due to cp+symlink behaviour
Sometimes the destination file is a symlink to the source file.
If this happens cp may complain:
cp: ./info.h' are the same file
This patch just adds a safety rm -f before hand which wont do any harm.
---
include/sound/Makefile | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/sound/Makefile b/include/sound/Makefile
index 4f3c782..f992c01 100644
--- a/include/sound/Makefile
+++ b/include/sound/Makefile
@@ -27,6 +27,7 @@ version.h: ../version.h
b=$$(basename $$s .h); \
if test -f $$b.patch; then \
echo "$$s: $$i $$b.patch" >> $@; \
+ echo " rm -f \$$@" >> $@; \
echo " cp \$$(INCSRC)/\$$@ ." >> $@; \
echo " patch -p0 -i $$b.patch \$$@" >> $@; \
echo >> $@; \
--
1.7.4.1
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-12 10:30 [PATCH] alsa-driver git repo: Build problems when copying symlink to itself Colin Guthrie
@ 2011-03-12 10:47 ` Colin Guthrie
2011-03-14 9:43 ` Takashi Iwai
2011-03-14 9:54 ` Jaroslav Kysela
2 siblings, 0 replies; 11+ messages in thread
From: Colin Guthrie @ 2011-03-12 10:47 UTC (permalink / raw)
To: alsa-devel
[-- Attachment #1: Type: text/plain, Size: 1557 bytes --]
'Twas brillig, and Colin Guthrie at 12/03/11 10:30 did gyre and gimble:
> Hi,
>
> After doing a "simple":
> git clone alsa-driver
> git clone alsa-kernel
> cd alsa-driver
> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
> --with-moddir=updates/alsa:
>
> The build bombed out pretty quickly with the following error.
>
> The attached patch fixes it.
And here is another error I stumbled on. The attached patch fixes this.
Hacking autoconf.h...
make[1]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include'
make -C sound prepare
make[2]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make prepare2
make[3]: Entering directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
ln -s ../../alsa-kernel/include/alc5623.h
ln: failed to create symbolic link `./alc5623.h': File exists
make[3]: *** [alc5623.h] Error 1
make[3]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make[2]: *** [prepare] Error 2
make[2]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
make[1]: *** [prepare] Error 2
make[1]: Leaving directory
`/home/colin/Development/Personal/ALSA/alsa-driver/include'
make: *** [dep] Error 1
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
[-- Attachment #2: 0001-Ensure-the-destination-of-the-symlink-doesn-t-yet-ex.patch --]
[-- Type: text/x-patch, Size: 752 bytes --]
>From d7b95b81a62faeab1e565946d91e4771b8128ce0 Mon Sep 17 00:00:00 2001
From: Colin Guthrie <cguthrie@mandriva.org>
Date: Sat, 12 Mar 2011 10:46:39 +0000
Subject: [PATCH] Ensure the destination of the symlink doesn't yet exists.
Similar to my last patch, the symlinking process can also fail due to the
destination file already being present. This ensures we wipe the file first.
---
include/sound/Makefile | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/sound/Makefile b/include/sound/Makefile
index f992c01..0e09df4 100644
--- a/include/sound/Makefile
+++ b/include/sound/Makefile
@@ -16,6 +16,7 @@ version.h: ../version.h
patch -p0 -i $< $@
%.h:
+ rm -f $@
ln -s $(INCSRC)/$@
.includes.tmp:
--
1.7.4.1
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-12 10:30 [PATCH] alsa-driver git repo: Build problems when copying symlink to itself Colin Guthrie
2011-03-12 10:47 ` Colin Guthrie
@ 2011-03-14 9:43 ` Takashi Iwai
2011-03-14 13:16 ` Colin Guthrie
2011-03-14 9:54 ` Jaroslav Kysela
2 siblings, 1 reply; 11+ messages in thread
From: Takashi Iwai @ 2011-03-14 9:43 UTC (permalink / raw)
To: Colin Guthrie; +Cc: alsa-devel
At Sat, 12 Mar 2011 10:30:48 +0000,
Colin Guthrie wrote:
>
> Hi,
>
> After doing a "simple":
> git clone alsa-driver
> git clone alsa-kernel
> cd alsa-driver
> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
> --with-moddir=updates/alsa:
>
> The build bombed out pretty quickly with the following error.
Can't reproduced with my trees. Could you try the GIT trees below
instead?
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/alsa-driver-build.git
git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
Takashi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-12 10:30 [PATCH] alsa-driver git repo: Build problems when copying symlink to itself Colin Guthrie
2011-03-12 10:47 ` Colin Guthrie
2011-03-14 9:43 ` Takashi Iwai
@ 2011-03-14 9:54 ` Jaroslav Kysela
2011-03-14 11:25 ` Colin Guthrie
2 siblings, 1 reply; 11+ messages in thread
From: Jaroslav Kysela @ 2011-03-14 9:54 UTC (permalink / raw)
To: Colin Guthrie; +Cc: alsa-devel
On Sat, 12 Mar 2011, Colin Guthrie wrote:
> Hi,
>
> After doing a "simple":
> git clone alsa-driver
> git clone alsa-kernel
> cd alsa-driver
> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
> --with-moddir=updates/alsa:
>
> The build bombed out pretty quickly with the following error.
>
> The attached patch fixes it.
>
> Hacking autoconf.h...
> make[1]: Entering directory
> `/home/colin/Development/Personal/ALSA/alsa-driver/include'
> make -C sound prepare
> make[2]: Entering directory
> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
> make prepare2
> make[3]: Entering directory
> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
> cp ../../alsa-kernel/include/ac97_codec.h .
> patch -p0 -i ac97_codec.patch ac97_codec.h
> patching file ac97_codec.h
> Hunk #1 succeeded at 650 (offset 5 lines).
> cp ../../alsa-kernel/include/core.h .
> patch -p0 -i core.patch core.h
> patching file core.h
> cp ../../alsa-kernel/include/info.h .
> cp: `../../alsa-kernel/include/info.h' and `./info.h' are the same file
I don't see this error using 'alsa-compile.sh --git' on Fedora 14... Make
sure that the alsa-driver tree is clean before build.
Jaroslav
-----
Jaroslav Kysela <perex@perex.cz>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 9:54 ` Jaroslav Kysela
@ 2011-03-14 11:25 ` Colin Guthrie
2011-03-14 13:11 ` Jaroslav Kysela
2011-03-14 13:16 ` Takashi Iwai
0 siblings, 2 replies; 11+ messages in thread
From: Colin Guthrie @ 2011-03-14 11:25 UTC (permalink / raw)
Cc: alsa-devel
'Twas brillig, and Jaroslav Kysela at 14/03/11 09:54 did gyre and gimble:
> On Sat, 12 Mar 2011, Colin Guthrie wrote:
>
>> Hi,
>>
>> After doing a "simple":
>> git clone alsa-driver
>> git clone alsa-kernel
>> cd alsa-driver
>> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
>> --with-moddir=updates/alsa:
>>
>> The build bombed out pretty quickly with the following error.
>>
>> The attached patch fixes it.
>>
>> Hacking autoconf.h...
>> make[1]: Entering directory
>> `/home/colin/Development/Personal/ALSA/alsa-driver/include'
>> make -C sound prepare
>> make[2]: Entering directory
>> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
>> make prepare2
>> make[3]: Entering directory
>> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
>> cp ../../alsa-kernel/include/ac97_codec.h .
>> patch -p0 -i ac97_codec.patch ac97_codec.h
>> patching file ac97_codec.h
>> Hunk #1 succeeded at 650 (offset 5 lines).
>> cp ../../alsa-kernel/include/core.h .
>> patch -p0 -i core.patch core.h
>> patching file core.h
>> cp ../../alsa-kernel/include/info.h .
>> cp: `../../alsa-kernel/include/info.h' and `./info.h' are the same file
>
> I don't see this error using 'alsa-compile.sh --git' on Fedora 14... Make
> sure that the alsa-driver tree is clean before build.
Both trees are clean.
They are clones from
git://git.alsa-project.org/alsa-driver.git
and
git://git.alsa-project.org/alsa-kernel.git
I'm trying Takashi's other repos, now but the sound-2.6 clone is taking
about 2h and counting....
Also Jaroslav, are you the list admin? I've always send mails to this
list via gmane, but as reply-all is the etiquette, I want to make sure
I'm registered and such for posting. But the mailman admin page wont let
me subscribe or send me password reminders (and thus wont let me
unsubscribe to resubscribe either :s) Not sure how to deal with the
chicken and egg problem (other than changing my email address which I'll
probably do at some point soon for other reasons anyway).
Cheers
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 11:25 ` Colin Guthrie
@ 2011-03-14 13:11 ` Jaroslav Kysela
2011-03-14 13:16 ` Takashi Iwai
1 sibling, 0 replies; 11+ messages in thread
From: Jaroslav Kysela @ 2011-03-14 13:11 UTC (permalink / raw)
To: Colin Guthrie; +Cc: ALSA development
On Mon, 14 Mar 2011, Colin Guthrie wrote:
> 'Twas brillig, and Jaroslav Kysela at 14/03/11 09:54 did gyre and gimble:
>> On Sat, 12 Mar 2011, Colin Guthrie wrote:
>>
>>> Hi,
>>>
>>> After doing a "simple":
>>> git clone alsa-driver
>>> git clone alsa-kernel
>>> cd alsa-driver
>>> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
>>> --with-moddir=updates/alsa:
>>>
>>> The build bombed out pretty quickly with the following error.
>>>
>>> The attached patch fixes it.
>>>
>>> Hacking autoconf.h...
>>> make[1]: Entering directory
>>> `/home/colin/Development/Personal/ALSA/alsa-driver/include'
>>> make -C sound prepare
>>> make[2]: Entering directory
>>> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
>>> make prepare2
>>> make[3]: Entering directory
>>> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
>>> cp ../../alsa-kernel/include/ac97_codec.h .
>>> patch -p0 -i ac97_codec.patch ac97_codec.h
>>> patching file ac97_codec.h
>>> Hunk #1 succeeded at 650 (offset 5 lines).
>>> cp ../../alsa-kernel/include/core.h .
>>> patch -p0 -i core.patch core.h
>>> patching file core.h
>>> cp ../../alsa-kernel/include/info.h .
>>> cp: `../../alsa-kernel/include/info.h' and `./info.h' are the same file
>>
>> I don't see this error using 'alsa-compile.sh --git' on Fedora 14... Make
>> sure that the alsa-driver tree is clean before build.
>
> Both trees are clean.
>
> They are clones from
>
> git://git.alsa-project.org/alsa-driver.git
> and
> git://git.alsa-project.org/alsa-kernel.git
What's 'alsa-driver/include/sound/info.h' in your tree after the error?
It's a symlink?
I tried compilation from both of my kmirror/kernel git repos and it looks
OK.
> Also Jaroslav, are you the list admin? I've always send mails to this
> list via gmane, but as reply-all is the etiquette, I want to make sure
> I'm registered and such for posting. But the mailman admin page wont let
I subscribed your address 'gmane@colin.guthr.ie' manually.
Jaroslav
-----
Jaroslav Kysela <perex@perex.cz>
Linux Kernel Sound Maintainer
ALSA Project, Red Hat, Inc.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 11:25 ` Colin Guthrie
2011-03-14 13:11 ` Jaroslav Kysela
@ 2011-03-14 13:16 ` Takashi Iwai
1 sibling, 0 replies; 11+ messages in thread
From: Takashi Iwai @ 2011-03-14 13:16 UTC (permalink / raw)
To: Colin Guthrie; +Cc: alsa-devel
At Mon, 14 Mar 2011 11:25:02 +0000,
Colin Guthrie wrote:
>
> 'Twas brillig, and Jaroslav Kysela at 14/03/11 09:54 did gyre and gimble:
> > On Sat, 12 Mar 2011, Colin Guthrie wrote:
> >
> >> Hi,
> >>
> >> After doing a "simple":
> >> git clone alsa-driver
> >> git clone alsa-kernel
> >> cd alsa-driver
> >> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
> >> --with-moddir=updates/alsa:
> >>
> >> The build bombed out pretty quickly with the following error.
> >>
> >> The attached patch fixes it.
> >>
> >> Hacking autoconf.h...
> >> make[1]: Entering directory
> >> `/home/colin/Development/Personal/ALSA/alsa-driver/include'
> >> make -C sound prepare
> >> make[2]: Entering directory
> >> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
> >> make prepare2
> >> make[3]: Entering directory
> >> `/home/colin/Development/Personal/ALSA/alsa-driver/include/sound'
> >> cp ../../alsa-kernel/include/ac97_codec.h .
> >> patch -p0 -i ac97_codec.patch ac97_codec.h
> >> patching file ac97_codec.h
> >> Hunk #1 succeeded at 650 (offset 5 lines).
> >> cp ../../alsa-kernel/include/core.h .
> >> patch -p0 -i core.patch core.h
> >> patching file core.h
> >> cp ../../alsa-kernel/include/info.h .
> >> cp: `../../alsa-kernel/include/info.h' and `./info.h' are the same file
> >
> > I don't see this error using 'alsa-compile.sh --git' on Fedora 14... Make
> > sure that the alsa-driver tree is clean before build.
>
> Both trees are clean.
>
> They are clones from
>
> git://git.alsa-project.org/alsa-driver.git
> and
> git://git.alsa-project.org/alsa-kernel.git
>
> I'm trying Takashi's other repos, now but the sound-2.6 clone is taking
> about 2h and counting....
If you have another kernel git tree, you can clone more quickly via
--reference option.
% git clone --reference /somewhere/linus-2.6.git ..../sound-2.6.git
Takashi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 9:43 ` Takashi Iwai
@ 2011-03-14 13:16 ` Colin Guthrie
2011-03-14 13:21 ` Takashi Iwai
0 siblings, 1 reply; 11+ messages in thread
From: Colin Guthrie @ 2011-03-14 13:16 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
'Twas brillig, and Takashi Iwai at 14/03/11 09:43 did gyre and gimble:
> At Sat, 12 Mar 2011 10:30:48 +0000,
> Colin Guthrie wrote:
>>
>> Hi,
>>
>> After doing a "simple":
>> git clone alsa-driver
>> git clone alsa-kernel
>> cd alsa-driver
>> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
>> --with-moddir=updates/alsa:
>>
>> The build bombed out pretty quickly with the following error.
>
> Can't reproduced with my trees. Could you try the GIT trees below
> instead?
>
> git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/alsa-driver-build.git
> git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
OK, so I should have explained it a bit better - tho' in actual fact my
fixes wont really help much....
I was using the trees @
git://git.alsa-project.org/alsa-driver.git
and
git://git.alsa-project.org/alsa-kernel.git
I tried cloning the repos you suggested and ran the script and all was well.
The problem I generally encountered was that as time passes (as it has
since the last time I used my clones) and you do a git pull, the patches
that are needed change.
When these change, the file alsa-driver/includes/sound/.includes (which
is generated by alsa-driver/includes/sound/Makefile) is *not* updated.
Therefore, the patch information is bogus and incorrect.
Really the build script needs to rm that file or the Makefile has to be
able to update it accordingly.
I also run into the same problem when trying to chase down a regression
(a pre-bisect checks). I was not able to rebuild the older drivers
easily without rm'ing the above .includes file. Obviously even with the
two patches I attached previously, it only papers over the problem as
the .include file is simply out of date. Correcting the .include file
should avoid the need for my changes.
Am I missing a trick here or is it part of the bi-secting process?
Also please see my other mail about *how* to do a bisect here - while
keeping alsa-driver and alsa-kernel in sync....
In an ideal world I need to downgrade to 1.0.23 or 1.0.22 of ALSA to
check to see if the problem still existed, but can't find a super easy
way to do that (other than perhaps just building myself a really old
kernel from my distro's VCS).
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 13:16 ` Colin Guthrie
@ 2011-03-14 13:21 ` Takashi Iwai
2011-03-14 15:06 ` Colin Guthrie
0 siblings, 1 reply; 11+ messages in thread
From: Takashi Iwai @ 2011-03-14 13:21 UTC (permalink / raw)
To: Colin Guthrie; +Cc: alsa-devel
At Mon, 14 Mar 2011 13:16:43 +0000,
Colin Guthrie wrote:
>
> 'Twas brillig, and Takashi Iwai at 14/03/11 09:43 did gyre and gimble:
> > At Sat, 12 Mar 2011 10:30:48 +0000,
> > Colin Guthrie wrote:
> >>
> >> Hi,
> >>
> >> After doing a "simple":
> >> git clone alsa-driver
> >> git clone alsa-kernel
> >> cd alsa-driver
> >> ./gitcompile --with-debug=full --with-isapnp=yes --with-sequencer=yes
> >> --with-moddir=updates/alsa:
> >>
> >> The build bombed out pretty quickly with the following error.
> >
> > Can't reproduced with my trees. Could you try the GIT trees below
> > instead?
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/alsa-driver-build.git
> > git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound-2.6.git
>
> OK, so I should have explained it a bit better - tho' in actual fact my
> fixes wont really help much....
>
>
> I was using the trees @
>
> git://git.alsa-project.org/alsa-driver.git
> and
> git://git.alsa-project.org/alsa-kernel.git
>
>
> I tried cloning the repos you suggested and ran the script and all was well.
>
> The problem I generally encountered was that as time passes (as it has
> since the last time I used my clones) and you do a git pull, the patches
> that are needed change.
>
> When these change, the file alsa-driver/includes/sound/.includes (which
> is generated by alsa-driver/includes/sound/Makefile) is *not* updated.
> Therefore, the patch information is bogus and incorrect.
But you did clean up before update, right?
"make clean" should remove .includes file.
> Really the build script needs to rm that file or the Makefile has to be
> able to update it accordingly.
>
> I also run into the same problem when trying to chase down a regression
> (a pre-bisect checks). I was not able to rebuild the older drivers
> easily without rm'ing the above .includes file. Obviously even with the
> two patches I attached previously, it only papers over the problem as
> the .include file is simply out of date. Correcting the .include file
> should avoid the need for my changes.
>
> Am I missing a trick here or is it part of the bi-secting process?
>
> Also please see my other mail about *how* to do a bisect here - while
> keeping alsa-driver and alsa-kernel in sync....
Use alsa-kmirror.
Takashi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 13:21 ` Takashi Iwai
@ 2011-03-14 15:06 ` Colin Guthrie
2011-03-14 15:14 ` Takashi Iwai
0 siblings, 1 reply; 11+ messages in thread
From: Colin Guthrie @ 2011-03-14 15:06 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
'Twas brillig, and Takashi Iwai at 14/03/11 13:21 did gyre and gimble:
> At Mon, 14 Mar 2011 13:16:43 +0000,
>> When these change, the file alsa-driver/includes/sound/.includes (which
>> is generated by alsa-driver/includes/sound/Makefile) is *not* updated.
>> Therefore, the patch information is bogus and incorrect.
>
> But you did clean up before update, right?
> "make clean" should remove .includes file.
Hmm, I probably didn't actually. :s
I guess the bisecting process etc. for this repos must be a bit
different to other areas? Would I need a full make clean and recompile
at each stage of a bisect (not that it takes all that long).
>> Really the build script needs to rm that file or the Makefile has to be
>> able to update it accordingly.
>>
>> I also run into the same problem when trying to chase down a regression
>> (a pre-bisect checks). I was not able to rebuild the older drivers
>> easily without rm'ing the above .includes file. Obviously even with the
>> two patches I attached previously, it only papers over the problem as
>> the .include file is simply out of date. Correcting the .include file
>> should avoid the need for my changes.
>>
>> Am I missing a trick here or is it part of the bi-secting process?
>>
>> Also please see my other mail about *how* to do a bisect here - while
>> keeping alsa-driver and alsa-kernel in sync....
>
> Use alsa-kmirror.
Hmmm, OK. Any page describing how to do this anywhere? e.g. how to
clone, bisect etc. on older versions of the drivers?
Trying to find out the details of how to debug and bisect alsa drivers
seems to be pretty hard (and I don't consider myself a total noob here,
tho' it feels like it!)
Thanks for any insights :)
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] alsa-driver git repo: Build problems when copying symlink to itself
2011-03-14 15:06 ` Colin Guthrie
@ 2011-03-14 15:14 ` Takashi Iwai
0 siblings, 0 replies; 11+ messages in thread
From: Takashi Iwai @ 2011-03-14 15:14 UTC (permalink / raw)
To: Colin Guthrie; +Cc: alsa-devel
At Mon, 14 Mar 2011 15:06:30 +0000,
Colin Guthrie wrote:
>
> 'Twas brillig, and Takashi Iwai at 14/03/11 13:21 did gyre and gimble:
> > At Mon, 14 Mar 2011 13:16:43 +0000,
> >> When these change, the file alsa-driver/includes/sound/.includes (which
> >> is generated by alsa-driver/includes/sound/Makefile) is *not* updated.
> >> Therefore, the patch information is bogus and incorrect.
> >
> > But you did clean up before update, right?
> > "make clean" should remove .includes file.
>
> Hmm, I probably didn't actually. :s
>
> I guess the bisecting process etc. for this repos must be a bit
> different to other areas? Would I need a full make clean and recompile
> at each stage of a bisect (not that it takes all that long).
Yes. But bisecting is difficult with the combined tree.
> >> Really the build script needs to rm that file or the Makefile has to be
> >> able to update it accordingly.
> >>
> >> I also run into the same problem when trying to chase down a regression
> >> (a pre-bisect checks). I was not able to rebuild the older drivers
> >> easily without rm'ing the above .includes file. Obviously even with the
> >> two patches I attached previously, it only papers over the problem as
> >> the .include file is simply out of date. Correcting the .include file
> >> should avoid the need for my changes.
> >>
> >> Am I missing a trick here or is it part of the bi-secting process?
> >>
> >> Also please see my other mail about *how* to do a bisect here - while
> >> keeping alsa-driver and alsa-kernel in sync....
> >
> > Use alsa-kmirror.
>
> Hmmm, OK. Any page describing how to do this anywhere? e.g. how to
> clone, bisect etc. on older versions of the drivers?
It's not trivial because these are basically two individual trees.
They aren't always in sync. alsa-driver tree is fixed once when some
build errors are detected, for example. So, the only sane way would
be to use alsa-kmirror. This is often still not trivial because of
build breakage...
Nowadays usually the bisect is done inside the kernel tree.
Takashi
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2011-03-14 15:14 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-12 10:30 [PATCH] alsa-driver git repo: Build problems when copying symlink to itself Colin Guthrie
2011-03-12 10:47 ` Colin Guthrie
2011-03-14 9:43 ` Takashi Iwai
2011-03-14 13:16 ` Colin Guthrie
2011-03-14 13:21 ` Takashi Iwai
2011-03-14 15:06 ` Colin Guthrie
2011-03-14 15:14 ` Takashi Iwai
2011-03-14 9:54 ` Jaroslav Kysela
2011-03-14 11:25 ` Colin Guthrie
2011-03-14 13:11 ` Jaroslav Kysela
2011-03-14 13:16 ` Takashi Iwai
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.