The changes look good to me. Irek
On 2/12/09, 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. > > > ---- > > Bye, > Roland > > -- > __ . . __ > (o.\ \/ /.o) roland.ma...@nrubsig.org > \__\/\/__/ MPEG specialist, C&&JAVA&&Sun&&Unix programmer > /O /==\ O\ TEL +49 641 3992797 > (;O/ \/ \O;) > > _______________________________________________ > opensolaris-code mailing list > opensolaris-code@opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/opensolaris-code > _______________________________________________ opensolaris-code mailing list opensolaris-code@opensolaris.org http://mail.opensolaris.org/mailman/listinfo/opensolaris-code
