From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.3 required=5.0 tests=MAILING_LIST_MULTI, RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL autolearn=ham autolearn_force=no version=3.4.4 Received: (qmail 25076 invoked from network); 24 Oct 2020 15:48:14 -0000 Received: from mother.openwall.net (195.42.179.200) by inbox.vuxu.org with ESMTPUTF8; 24 Oct 2020 15:48:14 -0000 Received: (qmail 22045 invoked by uid 550); 24 Oct 2020 15:48:12 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 22005 invoked from network); 24 Oct 2020 15:48:11 -0000 Date: Sat, 24 Oct 2020 11:47:59 -0400 From: Rich Felker To: Chen Wang Cc: musl Message-ID: <20201024154759.GF534@brightrain.aerifal.cx> References: <20201023084853.GV2947641@port70.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Subject: Re: Re: =?utf-8?B?W211c2xd5Zue5aSN?= =?utf-8?Q?=3A?= issue when running libc-test On Fri, Oct 23, 2020 at 04:53:14PM +0800, Chen Wang wrote: > Yes, I think it should be conditional silenced for musl, otherwise > it has blocked following builds, such as for main.exe and > runtest.exe. > So I think it is an issue, right? If this is happening, you probably did something with CFLAGS or such that broke the build. As written, the libc-test build system is supposed to treat errors as successful runs of the make target that just produce the error report as their output; they should not stop the build. Rich > 发件人： Szabolcs Nagy > 发送时间： 2020-10-23 16:48 > 收件人： Chen Wang > 抄送： musl > 主题： Re: [musl]回复: issue when running libc-test > * Chen Wang [2020-10-23 15:19:05 +0800]: > > Correct some comments in last email. The issue is not due to lacking of header file, it should be due to some constants are lost, I paste the whole error report here FYR: > > > > ../install/bin/musl-gcc -Isrc/common -Isrc/common -pipe -std=c99 -D_POSIX_C_SOURCE=200809L -Wall -Wno-unused-function -Wno-missing-braces -Wno-unused -Wno-overflow -Wno-unknown-pragmas -fno-builtin -frounding-math -Werror=implicit-function-declaration -Werror=implicit-int -Werror=pointer-sign -Werror=pointer-arith -g -D_FILE_OFFSET_BITS=64 -pedantic-errors -Werror -Wno-unused -D_XOPEN_SOURCE=700 -c -o src/api/unistd.o src/api/unistd.c 2>src/api/unistd.o.err || echo BUILDERROR src/api/unistd.o; cat src/api/unistd.o.err > > BUILDERROR src/api/unistd.o > > src/api/unistd.c: In function ‘f’: > > src/api/unistd.c:87:3: error: ‘_CS_POSIX_V7_THREADS_CFLAGS’ undeclared (first use in this function) > > 87 | C(_CS_POSIX_V7_THREADS_CFLAGS) > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ > > src/api/unistd.c:5:21: note: in definition of macro ‘C’ > > 5 | #define C(n) switch(n){case n:;} > > | ^ > > src/api/unistd.c:87:3: note: each undeclared identifier is reported only once for each function it appears in > > 87 | C(_CS_POSIX_V7_THREADS_CFLAGS) > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~ > > src/api/unistd.c:5:21: note: in definition of macro ‘C’ > > 5 | #define C(n) switch(n){case n:;} > > | ^ > > src/api/unistd.c:88:3: error: ‘_CS_POSIX_V7_THREADS_LDFLAGS’ undeclared (first use in this function) > > 88 | C(_CS_POSIX_V7_THREADS_LDFLAGS) > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > src/api/unistd.c:5:21: note: in definition of macro ‘C’ > > 5 | #define C(n) switch(n){case n:;} > > | ^ > > src/api/unistd.c:117:3: error: ‘_PC_TIMESTAMP_RESOLUTION’ undeclared (first use in this function) > > 117 | C(_PC_TIMESTAMP_RESOLUTION) > > | ^~~~~~~~~~~~~~~~~~~~~~~~ > > src/api/unistd.c:5:21: note: in definition of macro ‘C’ > > 5 | #define C(n) switch(n){case n:;} > > | ^ > > src/api/unistd.c:238:3: error: ‘_SC_XOPEN_UUCP’ undeclared (first use in this function) > > 238 | C(_SC_XOPEN_UUCP) > > | ^~~~~~~~~~~~~~ > > src/api/unistd.c:5:21: note: in definition of macro ‘C’ > > 5 | #define C(n) switch(n){case n:;} > > | ^ > > > > and this would cause the following main.exe and runtest.exe failed to be built. > > this is expected: > > posix requires these macros but musl does not define them. > in practice they are unlikely to be needed so i guess > these should be silenced in some way.