This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] GDB Hurd Fixes
- From: David Michael <fedora dot dm0 at gmail dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: bug-hurd at gnu dot org, gdb-patches at sourceware dot org
- Date: Fri, 20 Sep 2013 11:17:08 -0400
- Subject: Re: [RFC] GDB Hurd Fixes
- Authentication-results: sourceware.org; auth=none
- References: <CAEvUa7n-dgyoBajgCAkp5V7mkb6i-64Hq2Hsjj9qHPKRLR7r2Q at mail dot gmail dot com> <523C0B99 dot 7080109 at redhat dot com>
Hi,
On Fri, Sep 20, 2013 at 4:47 AM, Pedro Alves <palves@redhat.com> wrote:
> On 09/20/2013 01:43 AM, David Michael wrote:
>> (Copying gdb-patches this time.)
> But, we're missing all the context on the gdb-patches@ side.
Sorry about that--here's an explanation of the problems in GDB's build
process with current Hurd:
First, mig has stopped using the "auto" keyword in its output.[1]
Without that keyword, gdb/reply_mig_hack.awk fails to match a
necessary pattern and outputs a bad gdb/process_reply_S.c file. The
first change I made adds a new pattern to the script in addition to
the old one, so it should work with both old and new mig binaries.
Next, new function definitions were added (then renamed) in
<hurd/process_reply.defs>.[2] In the generated
gdb/process_reply_S.raw, some of the new functions match patterns in
gdb/reply_mig_hack.awk in a different order than expected, producing
bad output again. The second change I made to the script ensures a
necessary definition is found before writing output. (It may be
preferable to add a "parse_phase = 6" instead.)
Also because of [2], linking fails due to missing some new functions
in gdb/process_reply_S.c. I just extended the way other unused
functions from process_reply.defs were handled previously in
gdb/gnu-nat.c.
Thanks.
David
[1] http://git.savannah.gnu.org/cgit/hurd/mig.git/commit/?id=b53836447df7230cd5665a7ccabd2a6e1a6607e5
[2] http://git.savannah.gnu.org/cgit/hurd/hurd.git/commit/?id=e19cc6184fb99394845d56e6e915fea9805e5c28