From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] add support for mandrake in alsaconf Date: Tue, 25 Nov 2003 13:16:18 +0100 Sender: alsa-devel-admin@lists.sourceforge.net Message-ID: References: <1069458051.11875.5.camel@cmn37.stanford.edu> <3FBF3808.1060104@gmx.de> <1069641542.7493.18.camel@cmn30.stanford.edu> <1069700300.2235.11.camel@cmn37.stanford.edu> <1069704189.2234.30.camel@cmn37.stanford.edu> Mime-Version: 1.0 (generated by SEMI 1.14.5 - "Awara-Onsen") Content-Type: text/plain; charset=US-ASCII Return-path: In-Reply-To: Errors-To: alsa-devel-admin@lists.sourceforge.net List-Help: List-Post: List-Subscribe: , List-Unsubscribe: , List-Archive: To: Thierry Vignaud Cc: Fernando Pablo Lopez-Lezcano , Lars Heineken , alsa-devel@lists.sourceforge.net List-Id: alsa-devel@alsa-project.org At Mon, 24 Nov 2003 20:33:40 +0000, Thierry Vignaud wrote: > > Fernando Pablo Lopez-Lezcano writes: > > > > > Options: > > > > a) define 'distribution="redhat"' for the mandrake detection > > > > code _if_ the oss options erasure for redhat works fine under > > > > mandrake (mandrake users should test this). > > > > b) add another handler for mandrake's modules.conf oss options > > > > (should be written taking into account whatever mandrake uses to > > > > configure sound so that the proper strings are removed from > > > > modules.conf). > > > > > > so why not doing such as threat all distros that provide > > > /etc/redhat-release the same way. i don't think there's any > > > distro out that provide /etc/redhat-release for compatibility that > > > would like to have users loose their module configuration: > > > > With the patch I supplied no distros that user /etc/redhat-release > > will lose their configuration (I hope, at least not due to the bug I > > introduced a long time ago which is fixed by the patch). Distros > > that are not RedHat or Fedora will execute the SuSE code path. > > > > Your patch is fine, I would say. But it has an assumption I don't > > quite like. I wrote "remove_sndconfig_block" (basing it on > > "remove_y2_block") by looking at all the strings that the sndconfig > > utility was adding to modules.conf (sndconfig was the way RedHat was > > doing sound configuration quite a while ago, I think this has > > changed). > > as for mandrake, we do not rely on sndconfig (though we still provide > it for isa pnp sound card owners) > > drakx (the installer) and draksound (the standalone soundconfig tool) > only write "sound-slot-X" aliases and "above XXX snd-pcm-oss" lines > for oss compatibility (if $module =~ /^snd-/) in /etc/modules.conf then remove_y2_block() would be enough. i'll take Thierry's first patch. it will be in pre3 release. Takashi ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. Does SourceForge.net help you be more productive? Does it help you create better code? SHARE THE LOVE, and help us help YOU! Click Here: http://sourceforge.net/donate/