On Thu, Feb 12, 2009 at 2:26 PM, Roland Mainz <roland.ma...@nrubsig.org> wrote: > timeless wrote: >> 2009/2/11 Roland Mainz <roland.ma...@nrubsig.org>: >> > Index: src/cmd/ksh/builtins/alias.c >> > +/* >> > + * If the requested command is not an alias load it explicitly >> ... >> > + * if an executable with the same name sits in a PATH element >> > + * before /usr/bin (e.g. /usr/xpg4/bin/ls would be executed >> > + * before /usr/bin/ls if would look like >> > + * PATH=/usr/xpg4/bin:/usr/bin). >> >> I'm aware you're merely moving the comment, however, "if would look >> like" doesn't sound correct. >> >> perhaps "if the path was something like" > > Fixed... I've created a new webrev (see > http://cr.opensolaris.org/~gisburn/ksh93_integration/ksh93_fix_cr6800929_cr6793120_20090212_002_webrev/) > which fixes the comment and re-adds the $(ROOTALIASPROG) targets to the > "install" rule in usr/src/cmd/ksh/builtins/Makefile which somehow went > missing.
the change looks right. _______________________________________________ opensolaris-code mailing list opensolaris-code@opensolaris.org http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
