jan damborsky wrote: > Hi, > > could I please ask for reviewing the easy fix for following bug ? > > 4336 AI image should have link to /usr/has/bin/vi > http://defect.opensolaris.org/bz/show_bug.cgi?id=4336 > > webrev is available at: > http://cr.opensolaris.org/~dambi/bug-4336 >
Something seems wrong to me here. The AI image manifest includes /usr/bin/vi and /usr/has/bin/vi in the bootroot_contents list, and yet we have this code in ai_bootroot_configure doing seemingly similar things. I'm not at all following why this doesn't work after /usr is mounted from the image, either; is it just a matter that /usr/bin/vi isn't in the package area before we start archiving into the root archive? If so, then that seems like the better fix. Dave > Thank you, > Jan > > Modules affected: > ----------------- > * Distribution Constructor (ai_bootroot_configure script) > > Testing done: > ------------- > * AI image created with DC containing the fix > * AI image booted and verified that vi can be > invoked without need to provide absolute path > > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
