>>>>> "Roland" == Roland Mainz <roland.mainz at nrubsig.org> writes:

Roland> What about this: We disable the |XPG4CMDLIST()| macro in the
Roland> patch for now (e.g. ksh93r+) and only take those commands in
Roland> /usr/bin/ and /bin as builtins (I volunteer to crawl through the
Roland> sources and compare them against the Solaris versions).  

If I understood correctly, the /usr/bin (and /bin) builtins are
compatible with the Solaris version.  If that's true, then I think this
is fine from an interface point of view.

The fact that we need to crawl through code is a bad sign, though.
Duplicate code introduces the possibility of making a change in one
place and not the other, leading to bugs.  It would be better to
refactor, so that ksh93 and the Solaris binary can share common code.

cheers,
mike

Reply via email to