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