Index | Thread | Search

From:
Walter Alejandro Iglesias <wai@roquesor.com>
Subject:
Re: cal: add option to highlight the current day
To:
Theo de Raadt <deraadt@openbsd.org>
Cc:
Johannes Thyssen Tishman <jtt@openbsd.org>, Job Snijders <job@bsd.nl>, tech@openbsd.org, landry@openbsd.org, patrick keshishian <pkeshish@gmail.com>
Date:
Sun, 5 Jul 2026 20:04:43 +0200

Download raw body.

Thread
On Sun, Jul 05, 2026 at 10:49:13AM -0600, Theo de Raadt wrote:
> Walter Alejandro Iglesias <wai@roquesor.com> 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