On Tue, 13 Dec 2022, Jason McIntyre wrote:

> i was not rejecting your diff. it is there for the relevant developers to 
> take or leave, as they see fit.
>
> there is no point in having two entries either way. it is not a case of 
> anyone having to shut up, nor of users having to work anything out.

Having one Advent entry like 12/SunFirst could give a user the wrong
impression that Advent is always on the first Sunday of December.

If there is to be only one entry it might be safer to just hard
code a date and put an asterisk next to it like other moving dates.

Next year, 2023, Advent is on Dec 3:

12/03*  First Sunday of Advent (4th Sunday before Christmas)

One could also use 12/01* (good for 2024) as many commercial Advent
calendars start on that date:

12/01*  First Sunday of Advent (4th Sunday before Christmas)

>
> On 13 December 2022 15:05:07 GMT, Richard Narron <[email protected]> wrote:
> >On Tue, 13 Dec 2022, Jason McIntyre wrote:
> >
> >> On Mon, Dec 05, 2022 at 01:29:26PM -0800, Richard Narron wrote:
> >> > Here is an new version of the 5 patches with changes to two files:
> >> >
> >> > calendars/calendar.christian
> >> > advent.c
> >> >
> >> > The file, calendar.christian, Advent entries are changed
> >> > from:
> >> >
> >> >   11/SunLast      First Sunday of Advent (4th Sunday before Christmas)
> >> >   12/SunFirst     First Sunday of Advent (4th Sunday before Christmas)
> >> >
> >> > to:
> >> >
> >> >   Advent  First Sunday of Advent (4th Sunday before Christmas)
> >> >
> >>
> >> for now i have removed "11/SunLast". that should give us at least a
> >> couple of years to deliberate this diff.
> >>
> >> unless someone picks it up, i will gently remind fans of
> >> calendar.christain of its description in calendar(1):
> >>
> >>     calendar.christian    Christian holidays (should be updated
> >>                           yearly by the local system
> >>                           administrator so that roving holidays
> >>                           are set correctly for the current
> >>                           year).
> >
> >The fact that there are two Advent entries in the calendar.christian table
> >and the man page mentioning "fixing" the calendar.christian table reminds
> >me of an old lightbulb joke:
> >
> >--------------------------------------------------------------------
> >How many hardware engineers does it take to change a lightbulb?
> >None: "We'll fix it in software."
> >
> >How many software engineers does it take to change a lightbulb?
> >None: "We'll document it in the manual."
> >
> >How many tech writers does it take to change a lightbulb?
> >None: "The user can work it out."
> >-------------------------------------------------------------------
> >
> >If you are not going to use the code you might as well leave both
> >Advent entries 11/SunLast and 12/SunFirst alone because one of them will
> >be correct and one will be wrong.
> >
> >I took a different approach from "We'll document it in the manual" and
> >"The user can work it out".
> >
> >I shut up and coded.
> >
>

Richard Narron

Reply via email to