Message ID | 20200109081731.42472-1-ldurfina@tachyum.com |
---|---|
State | New, archived |
Headers |
Received: (qmail 117285 invoked by alias); 9 Jan 2020 08:17:54 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <gdb-patches.sourceware.org> List-Unsubscribe: <mailto:gdb-patches-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:gdb-patches-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/gdb-patches/> List-Post: <mailto:gdb-patches@sourceware.org> List-Help: <mailto:gdb-patches-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 117274 invoked by uid 89); 9 Jan 2020 08:17:53 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-19.9 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, SPF_PASS autolearn=ham version=3.3.1 spammy=Updated, H*r:0800 X-HELO: mail.tachyum.com Received: from mail.tachyum.com (HELO mail.tachyum.com) (66.160.133.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 09 Jan 2020 08:17:52 +0000 Received: from THQ-IP-EX1.tachyum.com (relayhost.tachyum.com [10.7.1.6]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by mail.tachyum.com (Postfix) with ESMTPS id 64E0264E for <gdb-patches@sourceware.org>; Thu, 9 Jan 2020 00:17:54 -0800 (PST) Received: from tachyum-sk-server.tachyum.com (10.7.241.13) by THQ-IP-EX1.tachyum.com (10.7.1.6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1779.2; Thu, 9 Jan 2020 00:17:44 -0800 From: Lukas Durfina <ldurfina@tachyum.com> To: <gdb-patches@sourceware.org> Subject: [PATCH] gdb/testsuite: Updated includes Date: Thu, 9 Jan 2020 09:17:31 +0100 Message-ID: <20200109081731.42472-1-ldurfina@tachyum.com> In-Reply-To: <20200107175458.GA31232@adacore.com> References: <20200107175458.GA31232@adacore.com> MIME-Version: 1.0 Content-Type: text/plain |
Commit Message
ldurfina@tachyum.com
Jan. 9, 2020, 8:17 a.m. UTC
Fixed compilation with musl. --- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.base/fileio.c | 3 +-- 2 files changed, 6 insertions(+), 2 deletions(-)
Comments
Hello, I had assumed you pushed the patch, but in double-checking, it does not seem that way. Apologies for the confusion. The patch is OK to wait, but can you include the ChangeLog entry in the revision log? It's been pretty traditional in the GDB project to have the revision log of the commit be the same as what's sent by email. I'll take this opportunity to make some suggestions on how to make the revision log a little more useful to those who use "git log" (I do it a lot myself). Typically, it looks like this: | Fix compilation error with musl in gdb/testsuite/gdb.base/fileio.c | | <in most cases, a short description of the problem to better | understand the circumstances of the change should be provided here> | | gdb/ChangeLog: | | * gdb.base/fileio.c: Remove #include of <sys/errno.h>. | Replace #include of <sys/fcntl.h> by <fcntl.h>. Thank you! On Thu, Jan 09, 2020 at 09:17:31AM +0100, Lukas Durfina wrote: > Fixed compilation with musl. > --- > gdb/testsuite/ChangeLog | 5 +++++ > gdb/testsuite/gdb.base/fileio.c | 3 +-- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog > index 3f243641b8..243721f6cb 100644 > --- a/gdb/testsuite/ChangeLog > +++ b/gdb/testsuite/ChangeLog > @@ -1,3 +1,8 @@ > +2020-01-09 Lukas Durfina <ldurfina@tachyum.com> > + > + * gdb.base/fileio.c: Remove #include of <sys/errno.h>. > + Replace #include of <sys/fcntl.h> by <fcntl.h>. > + > 2020-01-06 Andrew Burgess <andrew.burgess@embecosm.com> > > * gdb.base/backtrace.c: New file. > diff --git a/gdb/testsuite/gdb.base/fileio.c b/gdb/testsuite/gdb.base/fileio.c > index 7f482a34d3..0f201518b7 100644 > --- a/gdb/testsuite/gdb.base/fileio.c > +++ b/gdb/testsuite/gdb.base/fileio.c > @@ -1,13 +1,12 @@ > #include <stdio.h> > #include <stdlib.h> > #include <string.h> > -#include <sys/errno.h> > #include <sys/types.h> > -#include <sys/fcntl.h> > #include <sys/stat.h> > #include <sys/time.h> > #include <errno.h> > #include <sys/wait.h> > +#include <fcntl.h> > #include <unistd.h> > #include <time.h> > /* TESTS : > -- > 2.17.1
> > Hello, > > I had assumed you pushed the patch, but in double-checking, it does not > seem that way. Apologies for the confusion. Hi Joel, Sorry, I was not sure what you mean by pushing. I assume I cannot push directly to repo, so I sent a patch. Thanks for a hint. I am going to send an updated patch in the following email. Best regards, Lukas > > The patch is OK to wait, but can you include the ChangeLog entry in the > revision log? It's been pretty traditional in the GDB project to have the > revision log of the commit be the same as what's sent by email. I'll take this > opportunity to make some suggestions on how to make the revision log a > little more useful to those who use "git log" (I do it a lot myself). Typically, it > looks like this: > > | Fix compilation error with musl in gdb/testsuite/gdb.base/fileio.c > | > | <in most cases, a short description of the problem to better > | understand the circumstances of the change should be provided here> > | > | gdb/ChangeLog: > | > | * gdb.base/fileio.c: Remove #include of <sys/errno.h>. > | Replace #include of <sys/fcntl.h> by <fcntl.h>. > > Thank you! > > On Thu, Jan 09, 2020 at 09:17:31AM +0100, Lukas Durfina wrote: > > Fixed compilation with musl. > > --- > > gdb/testsuite/ChangeLog | 5 +++++ > > gdb/testsuite/gdb.base/fileio.c | 3 +-- > > 2 files changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index > > 3f243641b8..243721f6cb 100644 > > --- a/gdb/testsuite/ChangeLog > > +++ b/gdb/testsuite/ChangeLog > > @@ -1,3 +1,8 @@ > > +2020-01-09 Lukas Durfina <ldurfina@tachyum.com> > > + > > + * gdb.base/fileio.c: Remove #include of <sys/errno.h>. > > + Replace #include of <sys/fcntl.h> by <fcntl.h>. > > + > > 2020-01-06 Andrew Burgess <andrew.burgess@embecosm.com> > > > > * gdb.base/backtrace.c: New file. > > diff --git a/gdb/testsuite/gdb.base/fileio.c > > b/gdb/testsuite/gdb.base/fileio.c index 7f482a34d3..0f201518b7 100644 > > --- a/gdb/testsuite/gdb.base/fileio.c > > +++ b/gdb/testsuite/gdb.base/fileio.c > > @@ -1,13 +1,12 @@ > > #include <stdio.h> > > #include <stdlib.h> > > #include <string.h> > > -#include <sys/errno.h> > > #include <sys/types.h> > > -#include <sys/fcntl.h> > > #include <sys/stat.h> > > #include <sys/time.h> > > #include <errno.h> > > #include <sys/wait.h> > > +#include <fcntl.h> > > #include <unistd.h> > > #include <time.h> > > /* TESTS : > > -- > > 2.17.1 > > -- > Joel
> Sorry, I was not sure what you mean by pushing. > I assume I cannot push directly to repo, so I sent a patch. > Thanks for a hint. I am going to send an updated patch in the following email. I see. I had assumed that you have "Write After Approval" privileges, which gives you privileges to push after your patch has been reviewed by a maintainer and approved. The requirements for you to be given write privileges are that you have a GDB Copyright Assignment on file with the FSF and that you submit at least one good patch (this patch qualifies). For this patch, I will simply push it for you, once you send the updated version. Normally, we require that all patches are covered by a copyright assignment (mentioned above), but since your patch is small, we can include it under the "tiny change" rule. For the future, if you think you'll be sending further contributions to GDB, you should start the copyright assignment process. I can send you the initial paperwork if interested -- just let me know in private. Thanks
diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 3f243641b8..243721f6cb 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2020-01-09 Lukas Durfina <ldurfina@tachyum.com> + + * gdb.base/fileio.c: Remove #include of <sys/errno.h>. + Replace #include of <sys/fcntl.h> by <fcntl.h>. + 2020-01-06 Andrew Burgess <andrew.burgess@embecosm.com> * gdb.base/backtrace.c: New file. diff --git a/gdb/testsuite/gdb.base/fileio.c b/gdb/testsuite/gdb.base/fileio.c index 7f482a34d3..0f201518b7 100644 --- a/gdb/testsuite/gdb.base/fileio.c +++ b/gdb/testsuite/gdb.base/fileio.c @@ -1,13 +1,12 @@ #include <stdio.h> #include <stdlib.h> #include <string.h> -#include <sys/errno.h> #include <sys/types.h> -#include <sys/fcntl.h> #include <sys/stat.h> #include <sys/time.h> #include <errno.h> #include <sys/wait.h> +#include <fcntl.h> #include <unistd.h> #include <time.h> /* TESTS :