[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-12 Thread terry.guo at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #38 from Terry Guo --- (In reply to Kai Tietz from comment #37) > I confirm that in libgcc we still have an issue ... > Could you please make a new report for libgcc's libgcov-util.c for it. > > Thanks in advance Reported it at http

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-12 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #37 from Kai Tietz --- I confirm that in libgcc we still have an issue ... Could you please make a new report for libgcc's libgcov-util.c for it. Thanks in advance

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-12 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #36 from Kai Tietz --- Well, I guess that you missed to reconfigure gcc. By checking current source is the include of ftw.h guarded by HAVE_FTW_H check, which get defined by configure if header is found.

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-12 Thread terry.guo at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Terry Guo changed: What|Removed |Added CC||terry.guo at arm dot com --- Comment #35 fro

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Kai Tietz changed: What|Removed |Added Status|REOPENED|RESOLVED Resolution|---

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #33 from Kai Tietz --- Author: ktietz Date: Tue Feb 10 14:14:58 2015 New Revision: 220584 URL: https://gcc.gnu.org/viewcvs?rev=220584&root=gcc&view=rev Log: 2015-02-10 Rainer Emrich PR gcov-profile/61889 * gcov-tool.c: Re

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #32 from Kai Tietz --- Author: ktietz Date: Tue Feb 10 14:13:13 2015 New Revision: 220582 URL: https://gcc.gnu.org/viewcvs?rev=220582&root=gcc&view=rev Log: 2015-02-10 Rainer Emrich PR gcov-profile/61889 * libgcc/libgcov-

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #31 from Rainer Emrich --- (In reply to Kai Tietz from comment #30) > Yes, this patch slipped under my radar. It would be good if you - Rainer - > would have pinged on it. As far as I recalled I awaited at that time a full > patch b

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #30 from Kai Tietz --- Yes, this patch slipped under my radar. It would be good if you - Rainer - would have pinged on it. As far as I recalled I awaited at that time a full patch by Rong on this subject. (See comment #16 and after

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #29 from Rainer Emrich --- Am 10.02.2015 12:12, schrieb jakub at gcc dot gnu.org: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 > > --- Comment #28 from Jakub Jelinek --- > http://gcc.gnu.org/ml/gcc-patches/2014-09/msg02137.

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #28 from Jakub Jelinek --- http://gcc.gnu.org/ml/gcc-patches/2014-09/msg02137.html ?

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #27 from Kai Tietz --- (In reply to Jakub Jelinek from comment #26) > Instead of the #undef mkdir you'd IMHO better just use (mkdir) (filename) > in the second case. > Anyway, if you've posted your patch to gcc-patches, you should be

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #26 from Jakub Jelinek --- Instead of the #undef mkdir you'd IMHO better just use (mkdir) (filename) in the second case. Anyway, if you've posted your patch to gcc-patches, you should be pinging it until it is reviewed. Kai, can you

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-10 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Rainer Emrich changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-09 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Jeffrey A. Law changed: What|Removed |Added Status|NEW |RESOLVED CC|

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-02-09 Thread tbsaunde at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #23 from tbsaunde at gcc dot gnu.org --- Author: tbsaunde Date: Tue Feb 10 03:40:20 2015 New Revision: 220566 URL: https://gcc.gnu.org/viewcvs?rev=220566&root=gcc&view=rev Log: Support gcov-tool without ftw.h gcc/ PR gcov-pr

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2015-01-21 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Jakub Jelinek changed: What|Removed |Added CC||jakub at gcc dot gnu.org --- Comment #22

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-14 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #21 from Kai Tietz --- (In reply to xur from comment #20) > Thanks for the comments. I'll work on this to get it fixed this time. > > Let me understand your idea correctly: > We will have two patches: The first one will check FTW-API

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread xur at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #20 from xur at google dot com --- Thanks for the comments. I'll work on this to get it fixed this time. Let me understand your idea correctly: We will have two patches: The first one will check FTW-API and make the gcov-tool build co

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #19 from Kai Tietz --- Hi Xur, I asked you in my intial support to check for existance of FTW-API, and not to implement it for Win32. So first, send patch checking in a valid way if API can be used. The ftw/nftw emulation you wrote

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread xur at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #18 from xur at google dot com --- This patch is after Kai Tietz's comment. and it does check the nfw headers. On Mon, Oct 13, 2014 at 2:40 PM, fxcoudert at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #17 from Francois-Xavier Coudert --- (In reply to xur from comment #16) > I sent a patch to fix this, a few weeks ago, but I have got the review > or approval. > https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00186.html Kai Tietz, min

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread xur at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #16 from xur at google dot com --- I sent a patch to fix this, a few weeks ago, but I have got the review or approval. https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00186.html Honza, could you take a quick look? -Rong On Mon, Oct

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #15 from StaffLeavers at arm dot com --- tony.wang no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.com T

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #14 from StaffLeavers at arm dot com --- tony.wang no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.com T

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Francois-Xavier Coudert changed: What|Removed |Added Status|WAITING |NEW CC|tony.wa

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #13 from StaffLeavers at arm dot com --- tony.wang no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.com T

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #12 from StaffLeavers at arm dot com --- tony.wang no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.com T

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-13 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #11 from Rainer Emrich --- Dear friends this issue seems to become a never ending story. In my understanding the person causing the issue is responsible for a fix. There are several hints in this thread how to solve the issue. So plea

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-10-09 Thread fxcoudert at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Francois-Xavier Coudert changed: What|Removed |Added CC||fxcoudert at gcc dot gnu.org -

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-09-24 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #9 from Rainer Emrich --- Created attachment 33548 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33548&action=edit Proposed patch to fix the mingw case.

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-09-24 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #8 from Rainer Emrich --- (In reply to xur from comment #7) > OK. I'll fix this and submit another patch. What is the status for that? > > On Wed, Aug 20, 2014 at 11:26 AM, ktietz at gcc dot gnu.org > wrote: > > https://gcc.gnu.org

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-08-20 Thread xur at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #7 from xur at google dot com --- OK. I'll fix this and submit another patch. On Wed, Aug 20, 2014 at 11:26 AM, ktietz at gcc dot gnu.org wrote: > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 > > --- Comment #6 from Kai Tietz

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-08-20 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #6 from Kai Tietz --- Yes, I remember. I didn't comment on it. The following checks aren't ok. '#if !defined(_WIN32)' you should disable those parts *only* if ftw API isn't present. This you should check by a HAVE_FTW_H configure-ch

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-08-20 Thread xur at google dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 --- Comment #5 from xur at google dot com --- I sent a patch for this a few days ago: https://www.mail-archive.com/gcc-patches@gcc.gnu.org/msg87155.html It's pending review. Thanks, -Rong On Wed, Aug 20, 2014 at 10:36 AM, ktietz at gcc dot g

[Bug gcov-profile/61889] [5 Regression] gcov-tool.c uses nftw, ftw.h

2014-08-20 Thread ktietz at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61889 Kai Tietz changed: What|Removed |Added Status|NEW |WAITING Known to fail|4.10.0