From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ed@altlinux.ru>
To: ALT Linux kernel packages development <devel-kernel@altlinux.ru>
From: ed@altlinux.ru (Ed V. Bartosh)
Organization: ALT Linux
Date: Tue, 29 Jul 2003 21:13:32 +0400
Message-ID: <m3k7a1xnlv.fsf@altlinux.ru>
User-Agent: Gnus/5.1002 (Gnus v5.10.2) XEmacs/21.4 (Portable Code, linux)
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Cc: ALT Devel discussion list <devel@altlinux.ru>
Subject: [devel] modules.conf
X-BeenThere: devel@altlinux.ru
X-Mailman-Version: 2.1.2
Precedence: list
Reply-To: ALT Devel discussion list <devel@altlinux.ru>
List-Id: ALT Devel discussion list  <devel.altlinux.ru>
List-Unsubscribe: <http://altlinux.ru/mailman/listinfo/devel>,
	<mailto:devel-request@altlinux.ru?subject=unsubscribe>
List-Archive: </pipermail/devel>
List-Post: <mailto:devel@altlinux.ru>
List-Help: <mailto:devel-request@altlinux.ru?subject=help>
List-Subscribe: <http://altlinux.ru/mailman/listinfo/devel>,
	<mailto:devel-request@altlinux.ru?subject=subscribe>
X-List-Received-Date: Tue, 29 Jul 2003 18:14:54 -0000
Archived-At: <http://lore.altlinux.org/devel/m3k7a1xnlv.fsf@altlinux.ru/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

--=-=-=
Content-Type: text/plain; charset=koi8-r
Content-Transfer-Encoding: 8bit

Hello, All

Предлагаю несколько изменить схему формирования 
modules.conf, включив в него только include-ы, а эти самые инклюды
генерить каждой подсистеме (kudzu, lm_sensors, etc) отдельно. 
Это по-моему логичнее, красивее и надежнее, каждая подсистема пишет
только свой файл, а не переписывает общий.

В связи с этим у меня есть несколько простеньких патчей (патчи
приатачены), которые могут быть небесполезны в данной схеме:

1. Для автоматической генерации modules.conf.sensors, он позволяет
   сделать что-нить типа этого:
start()
{
    SYSCONF_LMSENSORS='/etc/sysconfig/lm_sensors'

    [ -f "$SYSCONF_LMSENSORS" ] || /usr/sbin/sensors-detect > /dev/null 2>&1 </dev/null

    return 0
}
...
ну, или что-нибудь более умное :) и автоматом получать дефолтную
конфигурацию lm_sensors. 
При условии, что в modules.conf будет соответствующий include :)

2. Для обучения mkinitrd понимать эти самые include в modules.conf

Что скажет общественность ?

-- 
Best regards,
Ed V. Bartosh

--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment;
	filename=lm_sensors-2.8.0-modules_conf_sensor-generation.patch

--- lm_sensors-2.8.0/prog/detect/sensors-detect	2003-07-08 17:07:59 +0400
+++ lm_sensors-2.8.0/prog/detect/sensors-detect.new	2003-07-25 17:48:45 +0400
@@ -3723,6 +3723,16 @@
   print "#----cut here----\n";
   print $configfile;
   print "#----cut here----\n";
+  print "\nDo you want to generate /etc/modules.conf.sensor ? (YES/no): ";
+  unless (<STDIN> =~ /^\s*[Nn]/) {
+	open(MODULES_CONF, ">/etc/modules.conf.sensor")
+        or die "Sorry, can't open /etc/modules.conf.sensor ($!)?!?";
+	print MODULES_CONF $configfile;
+	close(MODULES_CONF);
+	print "You should add 'include /etc/modules.conf.sensor' to\n",
+	"your /etc/modules.conf to activate these changes.";
+  }
+
   print "\nTo load everything that is needed, add this to some /etc/rc* ",
         "file:\n\n";
   print "#----cut here----\n";

--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment;
	filename=mkinitrd-modules_conf_includes-alt.patch

--- mkinitrd-2.8.7/mkinitrd~	2003-03-24 19:07:31 +0300
+++ mkinitrd-2.8.7/mkinitrd	2003-07-29 17:48:12 +0400
@@ -83,7 +83,7 @@
 pause=
 MODULES=
 
-MODULES_CONF=/etc/modules.conf
+MODULES_CONF='/etc/modules.conf '`egrep -e '^include[ ]+' /etc/modules.conf | sed -e 's/^include[ ]\+\(.\+\)/\1/' | tr '\\n' ' '`
 FSTAB_FILE=/etc/fstab
 MODULES_DIR=
 
@@ -192,7 +192,7 @@
 {
 	[ -z "$noscsi" ] || return
 
-	local scsimodules=`/bin/egrep -s '(alias|probeall)[ 	]+scsi_hostadapter' "$MODULES_CONF" |
+	local scsimodules=`/bin/egrep -s '(alias|probeall)[ 	]+scsi_hostadapter' $MODULES_CONF |
 		/bin/grep -v '^[ 	]*#' |LC_COLLATE=C /bin/sort -u |/bin/awk '{$1=$2="";print}'`
 	
 	local n
@@ -241,7 +241,7 @@
 	fi
 
 	if /bin/grep -s '^/dev/ataraid' "$FSTAB_FILE" |/bin/fgrep -v noauto >/dev/null 2>&1; then
-		local ataraidmodules=`/bin/egrep -s '(alias|probeall)[ 	]+ataraid_hostadapter' "$MODULES_CONF" |
+		local ataraidmodules=`/bin/egrep -s '(alias|probeall)[ 	]+ataraid_hostadapter' $MODULES_CONF |
 			/bin/grep -v '^[ 	]*#' |LC_COLLATE=C /bin/sort -u |/bin/awk '{$1=$2="";print}'`
 		local n
 		for n in $ataraidmodules; do
@@ -386,7 +386,7 @@
 		local n="${m##*/}"
 		n="${n%.o}"
 
-		options=`/bin/sed -ne "s/^options[ 	]\\+$n[ 	]\\+//p" "$MODULES_CONF"`
+		options=`/bin/sed -ne "s/^options[ 	]\\+$n[ 	]\\+//p" $MODULES_CONF`
 
 		Debug "Loading module $m with options '$options'"
 		/bin/cat >>"$RCFILE" <<EOF

--=-=-=--