This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] revised FreeBSD PowerPC support
- From: Andreas Tobler <andreast-list at fgznet dot ch>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Sun, 03 Feb 2013 16:58:58 +0100
- Subject: Re: [patch] revised FreeBSD PowerPC support
- References: <510C38DB.1020407@fgznet.ch> <20130203151153.GB5265@adacore.com>
Hi Joel,
On 03.02.13 16:11, Joel Brobecker wrote:
> Hi Andreas,
>
>> 2013-02-01 Andreas Tobler <andreast@fgznet.ch>
>>
>> * Makefile.in (ALL_TARGET_OBS): Add powerpc FreeBSD files.
>> * configure.host: Add powerpc*-*-freebsd* target.
>> * configure.tgt: Add target info for powerpc*-*-freebsd*.
>> * ppcfbsd-nat.c, ppcfbsd-tdep.h, ppcfbsd-tdep.c: New files.
>> * config/powerpc/fbsd.mh: New file.
>
> I only quickly scanned your patch, and just happened to notice
> that the .mh file is missing a copyright header. I know that most
> .mh files do not have one, and yours is probably small enough that
> we could get away with it, but we're trying to be a little more
> consistent about this now. Thank you!
Np, I was looking at the others and left it away. I'll add one.
Thanks, for the feedback!
Andreas
>> Index: config/powerpc/fbsd.mh
>> --- /dev/null
>> +++ config/powerpc/fbsd.mh
>> @@ -0,0 +1,5 @@
>> +# Host: FreeBSD/powerpc
>> +NATDEPFILES= fbsd-nat.o fork-child.o inf-ptrace.o ppcfbsd-nat.o bsd-kvm.o
>> +
>> +LOADLIBES= -lkvm
>> +
>
>