This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC/TileGX 1/2] fix gdbserver build failure
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Jiong Wang <jiwang at tilera dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 17 Sep 2013 06:43:09 -0700
- Subject: Re: [RFC/TileGX 1/2] fix gdbserver build failure
- Authentication-results: sourceware.org; auth=none
- References: <50F9148F dot 3010602 at tilera dot com> <5237C847 dot 3040908 at tilera dot com> <20130917123042 dot GD3132 at adacore dot com> <52384C0E dot 1070808 at tilera dot com> <5238574F dot 1000509 at tilera dot com> <20130917133023 dot GG3132 at adacore dot com> <52385AEF dot 5070404 at tilera dot com>
> linux-tile-low.o is the target file of TileGX. (it's name is confusing ?)
>
> previous commit by the following wrongly removed the tilegx target file.
>
> commit f7fc28008a4d76dd6e55eb77070eb18e956406a9
> Author: Luis Machado <luisgpm@br.ibm.com>
> Date: Thu Aug 22 23:46:27 2013 +0000
>
> - srv_tgtobj="linux-low.o linux-tile-low.o linux-osdata.o linux-procfs.o"
> - srv_tgtobj="${srv_tgtobj} linux-ptrace.o"
> + srv_tgtobj="$srv_linux_obj linux-osdata.o"
Ok. Thank you for that, I had missed that commit in my research.
That goes to show how important it is to send detailed descriptions
of the problem when sending a patch. Otherwise, we end up wasting
time trying to figure it out again.
I knew what the file was about, and I knew it made sense. But I'd like
every patch to be explained on this list before it gets checked in.
This is for later on, when someone tries to figure out why a patch
was applied, at least it's documented in one public place.
Please go ahead and commit.
--
Joel