Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Theo de Raadt
Ted Unangst wrote: > Scott Cheloha wrote: > > It doesn't mean anything. I guess I'm still gunshy about removing > > options and breaking things after the lock(1) thing. > > If the default behavior changes, and the option is now meaningless, but still > results in the *same* behavior, keep the

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Ted Unangst
Scott Cheloha wrote: > - while ((ch = getopt(argc, argv, "ad:f:jr:ut:z:")) != -1) > + while ((ch = getopt(argc, argv, "af:jr:uz:")) != -1) > switch(ch) { > case 'a': > slidetime = 1; > break; > - case 'd':

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Ted Unangst
Scott Cheloha wrote: > It doesn't mean anything. I guess I'm still gunshy about removing > options and breaking things after the lock(1) thing. If the default behavior changes, and the option is now meaningless, but still results in the *same* behavior, keep the option. The user still obtains

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Scott Cheloha
On Wed, Aug 07, 2019 at 06:08:45PM -0600, Theo de Raadt wrote: > Scott Cheloha wrote: > > > On Wed, Aug 07, 2019 at 05:52:54PM -0600, Theo de Raadt wrote: > > > Scott Cheloha wrote: > > > > > > > - while ((ch = getopt(argc, argv, "ad:f:jr:ut:z:")) != -1) > > > > + while ((ch =

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Theo de Raadt
Scott Cheloha wrote: > On Wed, Aug 07, 2019 at 05:52:54PM -0600, Theo de Raadt wrote: > > Scott Cheloha wrote: > > > > > - while ((ch = getopt(argc, argv, "ad:f:jr:ut:z:")) != -1) > > > + while ((ch = getopt(argc, argv, "af:jr:uz:")) != -1) > > > > You remove d and t, so: > > > > > +

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Scott Cheloha
On Wed, Aug 07, 2019 at 05:52:54PM -0600, Theo de Raadt wrote: > Scott Cheloha wrote: > > > - while ((ch = getopt(argc, argv, "ad:f:jr:ut:z:")) != -1) > > + while ((ch = getopt(argc, argv, "af:jr:uz:")) != -1) > > You remove d and t, so: > > > + case 'd': /* compat:

Re: Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Theo de Raadt
Scott Cheloha wrote: > - while ((ch = getopt(argc, argv, "ad:f:jr:ut:z:")) != -1) > + while ((ch = getopt(argc, argv, "af:jr:uz:")) != -1) You remove d and t, so: > + case 'd': /* compat: daylight saving time */ > break; Can't be

Removing the kernel timezone: date(1): drop -d dst and -t minutes_west

2019-08-07 Thread Scott Cheloha
This is the first in a series of patches that will remove timezone support from the kernel. Here, remove date(1) support for modifying the kernel's timezone. The flags are kept in the code so that scripts don't break immediately. ok? Index: date.c