Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 93286 - alsa-driver causes collision-protect errors
Summary: alsa-driver causes collision-protect errors
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Other
: High normal
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-05-19 20:52 UTC by Georgi Georgiev
Modified: 2005-10-31 10:25 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
alsa-driver-1.0.9b.ebuild (alsa-driver-1.0.9b.ebuild,4.07 KB, text/plain)
2005-08-15 13:06 UTC, Dick Marinus
Details
alsa-driver-1.0.9b.ebuild (alsa-driver-1.0.9b.ebuild,4.12 KB, text/plain)
2005-10-02 13:05 UTC, Dick Marinus
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Georgi Georgiev 2005-05-19 20:52:48 UTC
emerge alsa-driver-1.0.9_rc3 with collision-protect on, causes collision-protect errors on the following files:

/lib/modules/${KV_FULL}/modules.{alias,ccwmap,dep,ieee1394map,inputmap,isapnpmap,pcimap,symbols,submap}

I patched and tested that the following solves the problem:

@@ -128,6 +128,8 @@
                mv ${D}/lib/modules/${KV_FULL}/kernel/sound ${D}/lib/modules/${KV_FULL}/${PN}
                rmdir ${D}/lib/modules/${KV_FULL}/kernel &> /dev/null
        fi
+
+       rm -f -v ${D}/lib/modules/${KV_FULL}/modules.*
 }
 
 pkg_postinst() {
Comment 1 Dick Marinus 2005-07-11 11:25:17 UTC
quite annoying indeed :(
Comment 2 Jonathan Smith (RETIRED) gentoo-dev 2005-07-18 05:57:02 UTC
any word?
Comment 3 Dick Marinus 2005-08-15 13:06:52 UTC
Created attachment 66018 [details]
alsa-driver-1.0.9b.ebuild

--- /usr/portage/media-sound/alsa-driver/alsa-driver-1.0.9b.ebuild     
2005-07-21 06:05:50.000000000 +0200
+++ alsa-driver-1.0.9b.ebuild	2005-08-15 21:58:16.000000000 +0200
@@ -64,6 +64,7 @@
	fi

	convert_to_m ${S}/Makefile
+	sed -ie "s/\(.*depmod\)/#\1/" Makefile || die
 }

 src_compile() {
@@ -98,16 +99,7 @@


 src_install() {
-	dodir /usr/include/sound
-
-	make DESTDIR=${D} install || die "make install failed"
-
-	# Provided by alsa-headers now
-	rm -rf ${D}/usr/include/sound
-
-	# We have our own scripts in alsa-utils
-	test -e ${D}/etc/init.d/alsasound && rm ${D}/etc/init.d/alsasound
-	test -e ${D}/etc/rc.d/init.d/alsasound && rm
${D}/etc/rc.d/init.d/alsasound
+	make DESTDIR=${D} install-modules || die "make install failed"

	dodoc CARDS-STATUS INSTALL FAQ README WARNING TODO

==
this really fix things ;-) maybe DESTDIR should be changed to ${D}/${PN} so
that the last 5 lines from src_install can be removed...
Comment 4 Dick Marinus 2005-10-02 13:05:18 UTC
Created attachment 69747 [details]
alsa-driver-1.0.9b.ebuild

--- /usr/portage/media-sound/alsa-driver/alsa-driver-1.0.9b.ebuild     
2005-08-26 00:35:47.000000000 +0200
+++ media-sound/alsa-driver/alsa-driver-1.0.9b.ebuild	2005-10-02
21:59:32.000000000 +0200
@@ -1,6 +1,6 @@
 # Copyright 1999-2005 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header:
/var/cvsroot/gentoo-x86/media-sound/alsa-driver/alsa-driver-1.0.9b.ebuild,v 1.9
2005/08/25 22:17:04 chriswhite Exp $
+# $Header: $

 inherit linux-mod flag-o-matic eutils

@@ -64,6 +64,7 @@
	fi

	convert_to_m ${S}/Makefile
+	sed -ie "s/\(.*depmod\)/#\1/" Makefile || die
 }

 src_compile() {
@@ -98,16 +99,7 @@


 src_install() {
-	dodir /usr/include/sound
-
-	make DESTDIR=${D} install || die "make install failed"
-
-	# Provided by alsa-headers now
-	rm -rf ${D}/usr/include/sound
-
-	# We have our own scripts in alsa-utils
-	test -e ${D}/etc/init.d/alsasound && rm ${D}/etc/init.d/alsasound
-	test -e ${D}/etc/rc.d/init.d/alsasound && rm
${D}/etc/rc.d/init.d/alsasound
+	make DESTDIR=${D} install-modules || die "make install failed"

	dodoc CARDS-STATUS INSTALL FAQ README WARNING TODO



updated to current version in portage
Comment 5 Diego Elio Pettenò (RETIRED) gentoo-dev 2005-10-31 10:25:36 UTC
Thanks, fixed in CVS