Clay Baenziger wrote: > Hi all, > This is a simple code-review for the addition of three libraries to > the SPARC microroot to support cut(1) used by the live-fs-root script. > This is due to a change in cut(1) in build 128 from the onnv push for > '6793719 RFE: Update /usr/bin/cut to AT&T AST "cut"' I believe. > > Code review at: > http://cr.opensolaris.org/~clayb/12833/ >
The bug report is sub-minimal: should include the exact failure mode, at a minimum (not to mention your notes above about what caused it and some of the notes in the email you'd sent internally - as I've said long ago, if you feel the need to write more explanation in the review message, that means you didn't put enough in the bug report. Don't make future code archaeologists have to piece the story together :-) I'm slightly confused why this fix is correct, as the actual path for these libraries would appear to be /usr/lib/sparcv9, not /usr/lib/64. Are we relying on some side-effect of the file resolution method working here? Why isn't there a similar problem (and fix required) on x86? Dave
