Renaming the default loaded "init" scripts and added command line arg for "bare-mode"

- bro.init was renamed to base/init-bare.bro and base/all.bro
  was renamed to init-default.bro.

- To run in "bare mode" with only the init-bare.bro and no other
  scripts from base/, use either -b or --bare-mode.

- The environment variable to run in "bare mode" has been removed.

scripts/base/init-default.bro

@@ -1,10 +1,8 @@
 ##! This script loads everything in the base/ script directory.  If you want
-##! to run Bro without all of these scripts loaded by default, you can define
+##! to run Bro without all of these scripts loaded by default, you can use
-##! the BRO_NO_BASE_SCRIPTS environment variable to any value.  You can also
+##! the -b (--bare-mode) command line argument.  You can also copy the "@load"
-##! copy the "@load" lines from this script to your own script to load only
+##! lines from this script to your own script to load only the scripts that 
-##! the scripts that you actually want.
+##! you actually want.
-
-@if ( getenv("BRO_NO_BASE_SCRIPTS") == "" )
 
 @load base/utils/site
 @load base/utils/addrs
@@ -17,7 +15,8 @@
 @load base/utils/strings
 @load base/utils/thresholds
 
-# This has some weird interplay between types and BiFs so it's loaded in bro.init
+# This has some deep interplay between types and BiFs so it's 
+# loaded in base/init-bare.bro
 #@load base/frameworks/logging
 @load base/frameworks/notice
 @load base/frameworks/dpd
@@ -40,5 +39,3 @@
 @load base/protocols/ssh
 @load base/protocols/ssl
 @load base/protocols/syslog
-
-@endif

src/main.cc

@@ -138,6 +138,7 @@ void usage()
 	fprintf(stderr, "bro version %s\n", bro_version());
 	fprintf(stderr, "usage: %s [options] [file ...]\n", prog);
 	fprintf(stderr, "    <file>                         | policy file, or read stdin\n");
+	fprintf(stderr, "    -b|--bare-mode                 | don't load scripts from the base/ directory\n");
 	fprintf(stderr, "    -d|--debug-policy              | activate policy file debugging\n");
 	fprintf(stderr, "    -e|--exec <bro code>           | augment loaded policies by given code\n");
 	fprintf(stderr, "    -f|--filter <filter>           | tcpdump filter\n");
@@ -349,6 +350,7 @@ int main(int argc, char** argv)
 	char* seed_load_file = getenv("BRO_SEED_FILE");
 	char* seed_save_file = 0;
 	char* user_pcap_filter = 0;
+	int bare_mode = false;
 	int seed = 0;
 	int dump_cfg = false;
 	int to_xml = 0;
@@ -358,6 +360,7 @@ int main(int argc, char** argv)
 	int RE_level = 4;
 
 	static struct option long_opts[] = {
+		{"bare-mode",	no_argument,		0,	'b'},
 		{"debug-policy",	no_argument,		0,	'd'},
 		{"dump-config",		no_argument,		0,	'g'},
 		{"exec",		required_argument,	0,	'e'},
@@ -438,7 +441,7 @@ int main(int argc, char** argv)
 	opterr = 0;
 
 	char opts[256];
-	safe_strncpy(opts, "B:D:e:f:I:i:K:n:p:R:r:s:T:t:U:w:x:X:y:Y:z:CFGLOPSWdghvZ",
+	safe_strncpy(opts, "B:D:e:f:I:i:K:n:p:R:r:s:T:t:U:w:x:X:y:Y:z:CFGLOPSWbdghvZ",
 		     sizeof(opts));
 
 #ifdef USE_PERFTOOLS
@@ -448,6 +451,10 @@ int main(int argc, char** argv)
 	int op;
 	while ( (op = getopt_long(argc, argv, opts, long_opts, &long_optsind)) != EOF )
 		switch ( op ) {
+		case 'b':
+			bare_mode = true;
+			break;
+			
 		case 'd':
 			fprintf(stderr, "Policy file debugging ON.\n");
 			g_policy_debug = true;
@@ -675,8 +682,9 @@ int main(int argc, char** argv)
 	timer_mgr = new PQ_TimerMgr("<GLOBAL>");
 	// timer_mgr = new CQ_TimerMgr();
 
-	add_input_file("base/bro.init");
+	add_input_file("base/init-bare.bro");
-	add_input_file("base/all.bro");
+	if ( ! bare_mode )
+		add_input_file("base/init-default.bro");
 
 	if ( optind == argc &&
 	     read_files.length() == 0 && flow_files.length() == 0 &&

testing/btest/Baseline/policy.misc.default-loaded-scripts/canonified_loaded_scripts.log

@@ -1,5 +1,5 @@
 # depth	name
-0	scripts/base/bro.init
+0	scripts/base/init-bare.bro
 1	build/src/const.bif.bro
 1	build/src/types.bif.bro
 1	build/src/strings.bif.bro
@@ -10,7 +10,7 @@
 2	scripts/base/frameworks/logging/./main.bro
 3	build/src/logging.bif.bro
 2	scripts/base/frameworks/logging/./writers/ascii.bro
-0	scripts/base/all.bro
+0	scripts/base/init-default.bro
 1	scripts/base/utils/site.bro
 2	scripts/base/utils/./patterns.bro
 1	scripts/base/utils/addrs.bro

testing/btest/policy/frameworks/logging/adapt-filter.bro

@@ -1,5 +1,5 @@
 
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT 
+# @TEST-EXEC: bro -b %INPUT 
 # @TEST-EXEC: btest-diff ssh-new-default.log
 # @TEST-EXEC: test '!' -e ssh.log
 

testing/btest/policy/frameworks/logging/ascii-binary.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/ascii-empty.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 redef LogAscii::output_to_stdout = F;

testing/btest/policy/frameworks/logging/ascii-escape.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT 
+# @TEST-EXEC: bro -b %INPUT 
 # @TEST-EXEC: btest-diff ssh.log
 
 redef LogAscii::separator = "||";

testing/btest/policy/frameworks/logging/ascii-options.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 redef LogAscii::output_to_stdout = F;

testing/btest/policy/frameworks/logging/ascii-timestamps.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff test.log
 
 module Test;

testing/btest/policy/frameworks/logging/attr-extend.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/attr.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/disable-stream.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: test '!' -e ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/empty-event.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/events.bro

@@ -1,5 +1,5 @@
 
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT >output
+# @TEST-EXEC: bro -b %INPUT >output
 # @TEST-EXEC: btest-diff output
 
 module SSH;

testing/btest/policy/frameworks/logging/exclude.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/file.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/include.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/no-local.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: test '!' -e ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/path-func.bro

@@ -1,5 +1,5 @@
 
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: ( ls static-*; cat static-* ) >output
 # @TEST-EXEC: btest-diff output
 

testing/btest/policy/frameworks/logging/pred.bro

@@ -1,5 +1,5 @@
 
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT 
+# @TEST-EXEC: bro -b %INPUT 
 # @TEST-EXEC: btest-diff ssh.success.log
 # @TEST-EXEC: btest-diff ssh.failure.log
 

testing/btest/policy/frameworks/logging/remove.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro -B logging %INPUT
+# @TEST-EXEC: bro -b -B logging %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 # @TEST-EXEC: btest-diff ssh.failure.log
 

testing/btest/policy/frameworks/logging/rotate-custom.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro -r %DIR/rotation.trace %INPUT | egrep "test|test2" | sort >out
+# @TEST-EXEC: bro -b -r %DIR/rotation.trace %INPUT | egrep "test|test2" | sort >out
 # @TEST-EXEC: for i in `ls test*.log | sort`; do printf '> %s\n' $i; cat $i; done | sort | uniq >>out
 # @TEST-EXEC: btest-diff out
 

testing/btest/policy/frameworks/logging/stdout.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT >output
+# @TEST-EXEC: bro -b %INPUT >output
 # @TEST-EXEC: btest-diff output
 # @TEST-EXEC: test '!' -e ssh.log
 

testing/btest/policy/frameworks/logging/test-logging.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;

testing/btest/policy/frameworks/logging/types.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 #
 # Testing all possible types.

testing/btest/policy/frameworks/logging/unset-record.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff testing.log
 
 redef enum Log::ID += { TESTING };

testing/btest/policy/frameworks/logging/vec.bro

@@ -1,5 +1,5 @@
 #
-# @TEST-EXEC: BRO_NO_BASE_SCRIPTS=1 bro %INPUT
+# @TEST-EXEC: bro -b %INPUT
 # @TEST-EXEC: btest-diff ssh.log
 
 module SSH;