On Sat, Feb 11, 2012 at 2:50 PM, Josef 'Jeff' Sipek <span dir="ltr"><<a href="mailto:jeffpc@josefsipek.net">jeffpc@josefsipek.net</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Fri, Feb 10, 2012 at 07:53:09PM +0000, Bayard Bell wrote:<br>
> I know that jeffpc mentioned having a patch for issues with an earlier<br>
> version, but it wasn't attached to the issue. I don't know, either, if it's<br>
> still applicable to the recent version to which I've just upgraded. I trust<br>
> that will come out in the course of review.<br>
><br>
> <a href="https://bitbucket.org/buffyg/oi-build/changeset/3b8c1e923e14" target="_blank">https://bitbucket.org/buffyg/oi-build/changeset/3b8c1e923e14</a><br>
<br>
</div></div>utmp_update is broken (it doesn't do what POSIX says it should) and so it's<br>
really an illumos bug.  I talked with urxvt folks quite a bit about it, and<br>
IIRC, they were going to include a workaround.  Looking at the Changelog it<br>
looks like that didn't happen.<br>
<br>
The copyright is wrong unless you took this from Oracle.<br>
<br>
And three small nits...<br>
<br>
1) Why the OSOL CDDL header instead of the Illumos CDDL header?<br></blockquote><div><br>I cleaned up the copyright just after submitting. See previous response for rest.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

2) Why keep all those commented out lines in there?<br></blockquote><div><br>Ditto.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
3) What's the reasoning behind setting $PATH?<br></blockquote><div><br>Copying and generally assuming that most stuff that stuff that builds with gcc is happier preferring GNU tools to SVR4 ones.<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
Jeff.<br>
<br>
P.S. FWIW, the correct way to fix utmp_update is to rip it out and write it<br>
from scratch.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
If I have trouble installing Linux, something is wrong. Very wrong.<br>
                - Linus Torvalds<br>
<br>
_______________________________________________<br>
oi-dev mailing list<br>
<a href="mailto:oi-dev@openindiana.org">oi-dev@openindiana.org</a><br>
<a href="http://openindiana.org/mailman/listinfo/oi-dev" target="_blank">http://openindiana.org/mailman/listinfo/oi-dev</a><br>
</font></span></blockquote></div><br>