changes to bankdef.xml format

["Mark M. Ito" <marki@jlab.org>]

Folks,

I've just ehecked in some changes to src/libraries/include/bankdef.xml 
that may effect you when you try to build the code. The warning is that 
if you update bankdef.xml, you must use an updated version of 
$PRIMEX_SCRIPTS, i. e., one that contains an updated version of 
make_bank_code.pl.

The change is to have a separate tag, "bankflag" as a daughter tag 
within the bank specification. That way it is explicit that more than 
one bankflag can be thrown. Further, the hard-wired default of the 
BANKFLAG_WRITE_OUT flag getting turned on for all banks in 
InitializeBank has been removed. Now the default write/no-write behavior 
is set on a bank-by-bank basis in bankdef.xml, using the bankflag tag. 
The default has been set so that all banks with the rawbank flag set and 
the EVENTID bank are written out. Others are not (by default). This 
correlation of the raw flag and the write-out flag was done by hand and 
is, as such, explicit in bankdef.xml; it is not enforced in any code.

This change was motivated by the config bank being overwritten by config 
banks being read-in from skim files. The run-time setting of config 
parameters was thus defeated.

  -- Mark