Message ID | 1456149467-19460-1-git-send-email-tuliom@linux.vnet.ibm.com |
---|---|
State | Dropped |
Headers |
Received: (qmail 49700 invoked by alias); 22 Feb 2016 13:58:09 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <libc-alpha.sourceware.org> List-Unsubscribe: <mailto:libc-alpha-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 49668 invoked by uid 89); 22 Feb 2016 13:58:07 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.8 required=5.0 tests=AWL, BAYES_00, KAM_LAZY_DOMAIN_SECURITY, RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy=H*r:sendmail, H*r:sSMTP, magno, H*r:emulation X-HELO: e24smtp01.br.ibm.com X-IBM-Helo: d24dlp01.br.ibm.com X-IBM-MailFrom: tuliom@linux.vnet.ibm.com X-IBM-RcptTo: libc-alpha@sourceware.org From: "Tulio Magno Quites Machado Filho" <tuliom@linux.vnet.ibm.com> To: fweimer@redhat.com Cc: libc-alpha@sourceware.org Subject: [PATCH] Add missing include to posix/tst-dir.c Date: Mon, 22 Feb 2016 10:57:47 -0300 Message-Id: <1456149467-19460-1-git-send-email-tuliom@linux.vnet.ibm.com> In-Reply-To: <56B8C43A.4040705@redhat.com> References: <56B8C43A.4040705@redhat.com> X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16022213-1524-0000-0000-00000502D4E1 |
Commit Message
Tulio Magno Quites Machado Filho
Feb. 22, 2016, 1:57 p.m. UTC
A recent change to posix/tst-dir.c started to use macros from libc-internal.h but didn't include it, causing build issues. 2016-02-22 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> * posix/tst-dir.c: Include libc-internal.h. --- posix/tst-dir.c | 1 + 1 file changed, 1 insertion(+)
Comments
* Tulio Magno Quites Machado Filho: > A recent change to posix/tst-dir.c started to use macros from > libc-internal.h but didn't include it, causing build issues. > > 2016-02-22 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> > > * posix/tst-dir.c: Include libc-internal.h. I have never seen a need to include <libc-internal.h> like this before. How are you building the tests that you need this? Thanks, Florian
Florian Weimer <fw@deneb.enyo.de> writes: > * Tulio Magno Quites Machado Filho: > >> A recent change to posix/tst-dir.c started to use macros from >> libc-internal.h but didn't include it, causing build issues. >> >> 2016-02-22 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> >> >> * posix/tst-dir.c: Include libc-internal.h. > > I have never seen a need to include <libc-internal.h> like this > before. How are you building the tests that you need this? There is nothing special: configure --prefix=/usr --with-cpu=power8 --enable-add-ons make make -k check I'm using GCC 4.8.4. It seems we already have some testcases doing that: $ find . -type f \( -name 'test-*' -o -name 'tst-*' -o -name 'bug*' \) \ | xargs grep -n libc-internal.h ./stdio-common/tst-printfsz.c:5:#include <libc-internal.h> ./stdio-common/tst-unlockedio.c:23:#include <libc-internal.h> ./stdio-common/test-vfprintf.c:28:#include <libc-internal.h> ./stdio-common/tst-sprintf.c:5:#include <libc-internal.h> ./stdio-common/bug21.c:2:#include <libc-internal.h> ./stdio-common/tst-printf.c:29:#include <libc-internal.h> ./string/tst-endian.c:6:#include <libc-internal.h> ./misc/tst-error1.c:6:#include <libc-internal.h>
* Tulio Magno Quites Machado Filho: > Florian Weimer <fw@deneb.enyo.de> writes: > >> * Tulio Magno Quites Machado Filho: >> >>> A recent change to posix/tst-dir.c started to use macros from >>> libc-internal.h but didn't include it, causing build issues. >>> >>> 2016-02-22 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com> >>> >>> * posix/tst-dir.c: Include libc-internal.h. >> >> I have never seen a need to include <libc-internal.h> like this >> before. How are you building the tests that you need this? > > There is nothing special: > configure --prefix=/usr --with-cpu=power8 --enable-add-ons > make > make -k check > > I'm using GCC 4.8.4. Thanks, found the difference, it's due to the #includes in sysdeps/i386/nptl/tls.h and sysdeps/x86_64/nptl/tls.h. Perhaps we can get rid of that to avoid future such breakage. Patch is fine.
Florian Weimer <fw@deneb.enyo.de> writes: > Thanks, found the difference, it's due to the #includes in > sysdeps/i386/nptl/tls.h and sysdeps/x86_64/nptl/tls.h. Perhaps we can > get rid of that to avoid future such breakage. > > Patch is fine. I just noticed Stefan sent another patch before me: http://patchwork.sourceware.org/patch/10980/ Stefan, feel free to push it. ;-)
diff --git a/posix/tst-dir.c b/posix/tst-dir.c index d0e4878..4b15f34 100644 --- a/posix/tst-dir.c +++ b/posix/tst-dir.c @@ -26,6 +26,7 @@ #include <string.h> #include <unistd.h> #include <sys/stat.h> +#include <libc-internal.h> /* We expect four arguments: