Hi, On Fri, Sep 20, 2013 at 4:47 AM, Pedro Alves <pal...@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