svn commit: r355301 - head/usr.sbin/bhyve

Rodney W. Grimes freebsd at gndrsh.dnsmgr.net
Thu Dec 5 19:21:36 UTC 2019


> On 12/4/19 9:35 PM, Rodney W. Grimes wrote:
> >> I see, thanks for the pointers.
> >> It looks like cfmakeraw() and tcsetattr() were what I was looking for.
> >> A bhyve-specific printf wrapper looks the right solution to me.
> >> I can try to sketch a patch for you guys to review, if that's useful.
> >>
> >> Cheers,
> >>   Vincenzo
> > 
> > Meanwhile could you please revert the commit, and add a note to
> > D22552 to the effects that this was not the right solution?
> 
> I don't think we have to revert as it isn't that big of a deal.  Just
> fixing it going forward is probably fine.

I disagree.  Not reverting this leads to a bunch of un-needed changes
in the forward moving review, clouding the view of what is really
being changed in that new review which is now a mix of undoing this
and adding a new set of macros to deal with end of line.

-- 
Rod Grimes                                                 rgrimes at freebsd.org


More information about the svn-src-all mailing list