> Am 11.06.2021 um 22:13 schrieb Stuart Henderson <[email protected]>:
> 
> On 2021/06/11 21:59, Omar Polo wrote:
>> Klemens Nanni <[email protected]> writes:
>>>> On Fri, Jun 11, 2021 at 08:32:18PM +0200, Omar Polo wrote:
>>>>> textproc/cloc' pod2man.mk fails to generate a manpage because the date
>>>>> command fails.  Don't know why it doesn't stop the build, but the
>>>>> following seems to fix it.
>>>>> Index: Makefile
>>>>> ===================================================================
>>>>> RCS file: /home/cvs/ports/textproc/cloc/Makefile,v
>>>>> retrieving revision 1.15
>>>>> diff -u -p -r1.15 Makefile
>>>>> --- Makefile    1 Jun 2020 21:24:48 -0000    1.15
>>>>> +++ Makefile    11 Jun 2021 18:22:49 -0000
>>>>> @@ -4,6 +4,7 @@ COMMENT=        count lines of code
>>>>> V=            1.86
>>>>> DISTNAME=        cloc-${V}
>>>>> +REVISION=        0
>>>>> CATEGORIES=        textproc devel
>>>>> Index: patches/patch-Unix_pod2man_mk
>>>>> ===================================================================
>>>>> RCS file: patches/patch-Unix_pod2man_mk
>>>>> diff -N patches/patch-Unix_pod2man_mk
>>>>> --- /dev/null    1 Jan 1970 00:00:00 -0000
>>>>> +++ patches/patch-Unix_pod2man_mk    11 Jun 2021 18:23:27 -0000
>>>>> @@ -0,0 +1,14 @@
>>>>> +$OpenBSD$
>>>>> +
>>>>> +Index: Unix/pod2man.mk
>>>>> +--- Unix/pod2man.mk.orig
>>>>> ++++ Unix/pod2man.mk
>>>>> +@@ -40,7 +40,7 @@ PACKAGE        ?= package
>>>>> + # Optional variables to set
>>>>> + MANSECT        ?= 1
>>>>> + PODCENTER    ?= User Commands
>>>>> +-PODDATE        ?= $$(date --utc --date="@$${SOURCE_DATE_EPOCH:-$$(date 
>>>>> +%s)}" "+%Y-%m-%d")
>>>>> ++PODDATE        ?= $$(date -ujf "%s" "$${SOURCE_DATE_EPOCH:-$$(date 
>>>>> +%s)}" "+%Y-%m-%d")
>>> SOURCE_DATE_EPOCH is not set, neither by cloc WRKSRC or our Makefile,
>>> so the "today" default, e.g. "2021-06-11" kicks in.
>>> That means that manual page and therefore the package content changes
>>> every day;  wouldn't it be wiser to amend FAKE_FLAGS with this?
>>>   PODDATE=<date of release>
>>> No need for a patch fixing what's already flawed (in our build
>>> environment), imho.
>> Yep, it seems way better :)
>> Index: Makefile
>> ===================================================================
>> RCS file: /home/cvs/ports/textproc/cloc/Makefile,v
>> retrieving revision 1.15
>> diff -u -p -r1.15 Makefile
>> --- Makefile    1 Jun 2020 21:24:48 -0000    1.15
>> +++ Makefile    11 Jun 2021 19:56:40 -0000
>> @@ -4,6 +4,7 @@ COMMENT=        count lines of code
>> V=            1.86
>> DISTNAME=        cloc-${V}
>> +REVISION=        0
>> CATEGORIES=        textproc devel
>> @@ -25,7 +26,8 @@ RUN_DEPENDS=        devel/p5-Algorithm-Diff \
>> BUILD_DEPENDS+=        ${RUN_DEPENDS}
>> FAKE_FLAGS=        prefix="${PREFIX}" \
>> -            man_prefix="${PREFIX}"
>> +            man_prefix="${PREFIX}" \
>> +            PODDATE=2020-05-19
>> USE_GMAKE=        Yes
> 
> How about automating the datestamp?

I like that. ok jung@

Note, port needs an update anyways, as
there is a newer cloc release available.

But maybe we should try adding a fix upstream,
like done for OS X here:
https://github.com/AlDanial/cloc/commit/32d0a28f44fc8687fd5dbfc94e9b197cc68b7f60


> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/textproc/cloc/Makefile,v
> retrieving revision 1.15
> diff -u -p -r1.15 Makefile
> --- Makefile    1 Jun 2020 21:24:48 -0000    1.15
> +++ Makefile    11 Jun 2021 20:11:30 -0000
> @@ -4,6 +4,7 @@ COMMENT=        count lines of code
> 
> V=            1.86
> DISTNAME=        cloc-${V}
> +REVISION=        0
> 
> CATEGORIES=        textproc devel
> 
> @@ -25,7 +26,8 @@ RUN_DEPENDS=        devel/p5-Algorithm-Diff \
> BUILD_DEPENDS+=        ${RUN_DEPENDS}
> 
> FAKE_FLAGS=        prefix="${PREFIX}" \
> -            man_prefix="${PREFIX}"
> +            man_prefix="${PREFIX}" \
> +            PODDATE=`stat -t %Y-%m-%d -f %Sm ${WRKSRC}/cloc`
> 
> USE_GMAKE=        Yes

Reply via email to