bogus xml in si, appbat

Amused myself by tossing the si xml at
an xml parser, and it wasn't happy.
Seems that if an entity is defined to
read in another file, then entities
defined in that file become relative
to it's location.  The nALFS sample
prototypes for LFS follow this rule,
but our stuff does not.

To give this in more concrete terms,

<!ENTITY % system_entities SYSTEM "entities/system">

appears in LSB.xml.

entities/system in turn contains a ton
of definitions like this:

<!ENTITY prep SYSTEM "bootstrap/prep.xml">
<!ENTITY prep-directories SYSTEM "bootstrap/creating_dirs.xml">
...

but an xml parser gets unhappy about that because
it takes this to mean prep will be found as

entities/bootstrap/prep.xml

that is, relative to the directory the system_entities
file was found in.

To make the parser happy, the above two would
have to re rewritten as:

<!ENTITY prep SYSTEM "../bootstrap/prep.xml">
<!ENTITY prep-directories SYSTEM "../bootstrap/creating_dirs.xml">


Of course, the former is a bit easier to write.

Any idea why ours has been changed?  Should it
change back someday?

Mats