Re: please test kawa-3.0 release candidate 2

[timo.myyra@wickedbsd.net (Timo Myyrä)]

Per Bothner <per@bothner.com> writes:

> On 09/02/2017 10:51 PM, Timo Myyrä wrote:
>
>> Running the tests won't work fully on OpenBSD yet, there are open issues on
>> Gitlab with patches for these:
>> https://gitlab.com/kashell/Kawa/issues/17
>> https://gitlab.com/kashell/Kawa/issues/18
>
> Oops.  Checked in now.
>
>> Man pages seem to copy nicely now but I looked at them with mandoc lint and they
>> contain few minor style errors:
>>
>> $ mandoc -Tlint qexo.man
>> mandoc: qexo.man:95:41: STYLE: whitespace at end of input line
>> mandoc: qexo.man:96:40: STYLE: whitespace at end of input line
>> mandoc: qexo.man:102:5: STYLE: unterminated quoted argument
>> mandoc: qexo.man:1:2: WARNING: missing date, using today's date: TH
>> mandoc: qexo.man:87:2: WARNING: skipping paragraph macro: P empty
>> mandoc: qexo.man:94:2: WARNING: skipping paragraph macro: P empty
>> mandoc: qexo.man:101:2: WARNING: skipping paragraph macro: P empty
>>
>> $ mandoc -Tlint kawa.man
>> mandoc: kawa.man:84:41: STYLE: whitespace at end of input line
>> mandoc: kawa.man:1:2: WARNING: missing date, using today's date: TH
>> mandoc: kawa.man:83:2: WARNING: skipping paragraph macro: P empty
>
> I checked in most of your patches.
>
> However, I'm not sure about how to add a date to the TH macro.
> Manually putting a date in there seems wrong.  Better might be
> to use the same date as the UPDATED macro in version.texi.
> Or maybe the date *should* be manually updated - these are files
> that are seldom updated anyway.

Hmm, probably best to add some code to update the date when building the
release. That way the date in man page would reflect the release date of
kawa. It would make the date shown in man page useful. 

Perhaps add some place holder in the man page and rewrite it in Makefile:

kawa.man:
        sed "s/#RELEASE_DATE#/$(date +%Y-%m-%d)/" kawa.man.tmp > kawa.man

qexo.man:
        sed "s/#RELEASE_DATE#/$(date +%Y-%m-%d)/" qexo.man.tmp > qexo.man

The above is just example, I haven't tested it yet.

Timo