Re: /usr/bin/calendar broken on current

From: Dimitry Andric <dim_at_FreeBSD.org>
Date: Fri, 09 Nov 2012 13:52:24 +0100
On 2012-11-09 08:26, Greg 'groggy' Lehey wrote:> On Thursday,  8 November 2012 at 22:58:37 -0800, Manfred Antar wrote:
>> Sometime in the last week calendar stopped working.
>> not sure the cause
>> here is some of the output:
>> /usr/share/calendar/calendar.music:231:17: warning: missing terminating ' character [-Winvalid-pp-token]
>> 12/16   Don McLean's "American Pie" is released, 1971
>>                    ^
>
> This is unexpected fallout from the transition from gcc to clang.
> calendar invokes cpp, and it seems that clang's cpp doesn't like what
> it sees.  This patch works around the issue:
>
> --- pathnames.h	(revision 242777)
> +++ pathnames.h	(working copy)
> _at__at_ -32,5 +32,5 _at__at_
>
>   #include <paths.h>
>
> -#define	_PATH_CPP	"/usr/bin/cpp"
> +#define	_PATH_CPP	"/usr/bin/gcpp"
>   #define	_PATH_INCLUDE	"/usr/share/calendar"
>
> Clearly that's not the solution.  I'll investigate.

Looks like yet another cpp -traditional abuse.  Clang will most likely
never support traditional preprocessing.

It is probably better to just use sed or awk for this kind of trickery.
Received on Fri Nov 09 2012 - 11:52:21 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:32 UTC