Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 25 Nov 2017 15:28:36 -0500
From:      Pedro Giffuni <pfg@FreeBSD.org>
To:        Mark Millard <markmi@dsl-only.net>
Cc:        FreeBSD Toolchain <freebsd-toolchain@freebsd.org>, FreeBSD Current <freebsd-current@freebsd.org>
Subject:   Re: head -r326193 (e.g.) buildworld broken: rejecting some . . ./usr/src/amd64.amd64/tmp/usr/include/stdio.h content
Message-ID:  <79bca8ee-7961-8e51-10bd-a65154ecd430@FreeBSD.org>
In-Reply-To: <5D3C8011-1B5F-45CB-8390-9A6E9862FB47@dsl-only.net>
References:  <5E1C6F4E-C573-4B99-AF96-FCB756B0A75B@dsl-only.net> <5D3C8011-1B5F-45CB-8390-9A6E9862FB47@dsl-only.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Thank you for the report ...


On 11/25/17 15:15, Mark Millard wrote:
> [Quick top post:]
>
> Reverting to -r326192 and rebuilding avoided the issue.
>   
> Prior notes:
>
> On 2017-Nov-25, at 12:02 PM, Mark Millard <markmi at dsl-only.net> wrote:
>
>> For example,
>>
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:449:47: error: function definition is not allowed here
>> static __inline int __sputc(int _c, FILE *_p) {
>>
>> for:
>>
>> /*
>> * The __sfoo macros are here so that we can
>> * define function versions in the C library.
>> */
>> #define __sgetc(p) (--(p)->_r < 0 ? __srget(p) : (int)(*(p)->_p++))
>> #if defined(__GNUC__) && defined(__STDC__)
>> static __inline int __sputc(int _c, FILE *_p) {
>>         if (--_p->_w >= 0 || (_p->_w >= _p->_lbfsize && (char)_c != '\n'))
>>                 return (*_p->_p++ = _c);
>>         else
>>                 return (__swbuf(_c, _p));
>> }
>> #else
>> /*
>> * This has been tuned to generate reasonable code on the vax using pcc.
>> */
>> #define __sputc(c, p) \
>>         (--(p)->_w < 0 ? \
>>                 (p)->_w >= (p)->_lbfsize ? \
>>                         (*(p)->_p = (c)), *(p)->_p != '\n' ? \
>>                                 (int)*(p)->_p++ : \
>>                                 __swbuf('\n', p) : \
>>                         __swbuf((int)(c), p) : \
>>                 (*(p)->_p = (c), (int)*(p)->_p++))
>> #endif
>>
>>
>>
>> More of the prefix of the messages:
>>
>>
>>
>> --- var.o ---
>> In file included from /usr/src/bin/sh/var.c:70:
>> In file included from /usr/src/bin/sh/myhistedit.h:35:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/histedit.h:48:
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:255:6: warning: declaration of built-in function 'fprintf' requires inclusion of the header <stdio.h> [-Wbuiltin-requires-header]
>> int      fprintf(FILE * __restrict, const char * __restrict, ...);
>>          ^
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:260:6: warning: declaration of built-in function 'fscanf' requires inclusion of the header <stdio.h> [-Wbuiltin-requires-header]
>> int      fscanf(FILE * __restrict, const char * __restrict, ...);
>>          ^
>> --- csh_make ---
>> Building /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/rescue/rescue/usr/src/bin/csh/sh.init.o
>> --- sh_make ---
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:284:6: warning: declaration of built-in function 'vfprintf' requires inclusion of the header <stdio.h> [-Wbuiltin-requires-header]
>> int      vfprintf(FILE * __restrict, const char * __restrict,
>>          ^
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:293:6: warning: declaration of built-in function 'vfscanf' requires inclusion of the header <stdio.h> [-Wbuiltin-requires-header]
>> int      vfscanf(FILE * __restrict, const char * __restrict, __va_list)
>>          ^
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/stdio.h:449:47: error: function definition is not allowed here
>> static __inline int __sputc(int _c, FILE *_p) {
>>                                               ^
>> In file included from /usr/src/bin/sh/var.c:70:
>> In file included from /usr/src/bin/sh/myhistedit.h:35:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/histedit.h:254:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/wchar.h:69:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/_ctype.h:97:
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/runetype.h:96:1: error: function definition is not allowed here
>> {
>> ^
>> In file included from /usr/src/bin/sh/var.c:70:
>> In file included from /usr/src/bin/sh/myhistedit.h:35:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/histedit.h:254:
>> In file included from /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/wchar.h:69:
>> /usr/obj/amd64_clang/amd64.amd64/usr/src/amd64.amd64/tmp/usr/include/_ctype.h:101:1: error: function definition is not allowed here
>> {
>> ^
>> . . . (much omitted) . . .

I can't reproduce it ...
I have been running tinderbox builds with the changes for a while.

Jenkins wasn't affected either:
https://ci.freebsd.org/job/FreeBSD-head-amd64-build/6040/

I have seen problems on arm with zstd though.

Pedro.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?79bca8ee-7961-8e51-10bd-a65154ecd430>