Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 03 Feb 2003 13:51:59 -0700 (MST)
From:      "M. Warner Losh" <imp@bsdimp.com>
To:        myevmenk@exodus.net
Cc:        hackers@FreeBSD.ORG, questions@FreeBSD.ORG, brian@FreeBSD.ORG, doconnor@gsoft.com.au
Subject:   Re: [PATCH] PPP in -direct mode does not execute any chat scripts
Message-ID:  <20030203.135159.94296629.imp@bsdimp.com>
In-Reply-To: <3E3ED4DF.7090904@exodus.net>
References:  <45258A4365C6B24A9832BFE224837D552B1283@sjdcex01.int.exodus.net> <3E3ED4DF.7090904@exodus.net>

next in thread | previous in thread | raw e-mail | index | archive | help
In message: <3E3ED4DF.7090904@exodus.net>
            Maksim Yevmenkin <myevmenk@exodus.net> writes:
: Dear Hackers,
: 
: Please find attached patch that adds new option to the PPP.
: 
: 
: run-scripts-in-direct-mode
: 	Default: Disabled. This allows to run chat scripts in
: 	direct mode.
: 
: did i miss anything? objections? comments? reviews?

Maybe it would be better to call this "force-scripts" or something
more general purpose so that one could force the use of scripts
anywhere.  Making direct mode have a special case override seems a
little too restrictive.  The heart of the patch would become:

-  if (dl->physical->type & (PHYS_DIRECT|PHYS_DEDICATED))
+  if (!Enabled(dl->bundle, OPT_FORCE_SCRIPTS) &&
+      (dl->physical->type & (PHYS_DIRECT|PHYS_DEDICATED))
     /* Ignore scripts */
     runscripts = 0;

with corresponding changes to the docs, defines, etc.

Warner


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030203.135159.94296629.imp>