Re: cal: add option to highlight the current day

On Sun, Jul 05, 2026 at 10:49:13AM -0600, Theo de Raadt wrote:
> Walter Alejandro Iglesias <[email protected]> wrote:
> 
> > On Sun, Jul 05, 2026 at 08:10:22AM -0600, Theo de Raadt wrote:
> > > +                       if (is_today(today, month, year))
> > > +                               printf("\033[7m");
> > > 
> > > Not going to happen.
> > > 
> > > 
> > 
> > That's from my diff, not Johannes'.  I guess you don't like it because
> > it would break old terminals that doesn't support ANSI escape sequences.
> > Given Johannes have used tgetstr he is already aware of this.  The above
> > is just a rookie mistake on my part. :-)
> 
> "it would break old terminals that doesn't support ANSI escape sequences"
> 
> Wow.
> 

If instead of mocking you explained why you don't like the above the
next time I contribute a patch to the system I'll take it into account
and we all benefit.  I didn't dedicate my life to this like you did, yet
bugs were fixed thanks to me that you didn't fix.

-- 
Walter