Re: [ALPHA RELEASE] module-init-tools 0.9-alpha

[Rusty Russell]

In message <[e-mail]> you write:
> On Fri, 29 Nov 2002, Rusty Russell wrote:
> 
> > 0.9-alpha Version
> 
> Your make moveold script didn't take into account that the modutils from 
> RedHat and Keith Owens have lsmod, modprobe, and rmmod as symlinks to 
> insmod.  I worked around that by doing a "cp insmod lsmod", etc.

It should (Debian does the same thing, and it works here):

# Don't just move symlinks, reset them to point to xxx.old.
moveold:
	if [ "`echo $(DESTDIR)$(sbindir) | tr -s / /`" = /sbin ]; then :;    \
	else								     \
		echo $@ usually only makes sense when installing into /sbin; \
		exit 1;							     \
	fi
	for f in insmod lsmod modprobe rmmod depmod; do			\
		if [ -L /sbin/$$f ]; then				\
			ln -sf `readlink /sbin/$$f`.old /sbin/$$f;	\
		fi;							\
		mv /sbin/$$f /sbin/$$f.old;				\
	done

What part of this doesn't work for you?

> I took a stock 2.5.50-bk and added the namei.c patch and the patch to 
> module.h out of the NEWS file.  The RedHat 8.0 bootup scripts did not 
> autoload the pcmcia_core, yenta_socket, ds, orinoco_cs, orinoco, and 
> hermes modules, but I could load them by hand.  Unfortuantely, it still 
> does not allow the ethernet interface to be brought up an configured.

Argh, 0.9-alpha has a bug: I changed /etc/modprobe.conf to
/tmp/modprobe.conf for testing and didn't revert it before release
(you can see it trying to open "/tmp/modprobe.conf" in the strace in
your next mail).

> [root@lap root]# modprobe orinoco_cs
> WARNING: Error inserting hermes (/lib/modules/2.4.18-18.8.0/kernel/drivers/net/wireless/hermes.o): Unknown symbol in module

Once again, a testing bug: I removed backwards compatibility so I
could test on my stable machine.

I've fixed both these in 0.9-beta.  If you would test one more time
for me, I'd appreciate it.  (Or, simply apply the patch below, which
turns 0.9-alpha into 0.9-beta).

Thankyou *very* much for your persistent testing!
Rusty.
--
  Anyone who quotes me in their sig is an idiot. -- Rusty Russell.

diff -urN module-init-tools-0.9-alpha/ChangeLog module-init-tools-0.9-beta/ChangeLog
--- module-init-tools-0.9-alpha/ChangeLog	2002-11-29 21:45:25.000000000 +1100
+++ module-init-tools-0.9-beta/ChangeLog	2002-11-29 21:49:23.000000000 +1100
@@ -3,6 +3,8 @@
   CONFIG_KALLSYMS.
 o Fixed extra newline in "in use by" message.
 o Fixed parsing for new-style /proc/modules.
+o Fixed version parsing code (thanks to Adam Richter's report)
+o Fixed "running out of filedescriptors" (Adam Richter)	
 o Implemented options in modprobe
 o Implemented install in modprobe
 o Implemented options in modules.conf2modprobe.conf
diff -urN module-init-tools-0.9-alpha/configure module-init-tools-0.9-beta/configure
--- module-init-tools-0.9-alpha/configure	2002-11-29 21:45:49.000000000 +1100
+++ module-init-tools-0.9-beta/configure	2002-12-02 13:10:43.000000000 +1100
@@ -783,7 +783,7 @@
 
 PACKAGE=module-init-tools
 
-VERSION="0.9-alpha"
+VERSION="0.9-beta"
 
 if test "`cd $srcdir && pwd`" != "`pwd`" && test -f $srcdir/config.status; then
   { echo "configure: error: source directory already configured; run "make distclean" there first" 1>&2; exit 1; }
diff -urN module-init-tools-0.9-alpha/configure.in module-init-tools-0.9-beta/configure.in
--- module-init-tools-0.9-alpha/configure.in	2002-11-29 21:44:19.000000000 +1100
+++ module-init-tools-0.9-beta/configure.in	2002-12-02 13:10:29.000000000 +1100
@@ -2,7 +2,7 @@
 
 AC_CANONICAL_SYSTEM
 
-AM_INIT_AUTOMAKE(module-init-tools,"0.9-alpha")
+AM_INIT_AUTOMAKE(module-init-tools,"0.9-beta")
 
 AC_PROG_CC
 
diff -urN module-init-tools-0.9-alpha/modprobe.c module-init-tools-0.9-beta/modprobe.c
--- module-init-tools-0.9-alpha/modprobe.c	2002-11-29 21:46:47.000000000 +1100
+++ module-init-tools-0.9-beta/modprobe.c	2002-12-02 13:10:19.000000000 +1100
@@ -638,11 +638,7 @@
 				   { "syslog", 0, NULL, 's' },
 				   { NULL, 0, NULL, 0 } };
 
-#if 0
 #define DEFAULT_CONFIG "/etc/modprobe.conf"
-#else
-#define DEFAULT_CONFIG "/tmp/modprobe.conf"
-#endif
 
 int main(int argc, char *argv[])
 {
@@ -658,9 +654,7 @@
 	struct module_options *modoptions = NULL;
 	struct module *start;
 
-#if 0
 	try_old_version("modprobe", argv);
-#endif
 
 	while ((opt = getopt_long(argc, argv, "vVC:o:knqsc", options, NULL)) != -1){
 		switch (opt) {