[PATCH] Fix building gdb-7.0 on x86_64-*-freebsd

Joel Brobecker brobecker@adacore.com
Mon Oct 19 19:53:00 GMT 2009


Hi Sergio,

> Thank you for this patch.  I believe you could add a comment explaining what 
> this number means (just like you did above).  What do you think?
> Just my two cents, as usual :-).

I just applied the following patch (head only)

2009-10-19  Joel Brobecker  <brobecker@adacore.com>

        * amd64fbsd-nat.c (amd64fbsd_supply_pcb): Add comment.

Writing the comment made me think that we may have chosen the wrong
version number, depending on the angle we look at things from: If we
want to always be able to build, then we should have chosen version
74, since there are a few days where version 74 is missing the field
and yet the GDB code still tries to access it for that versio number.
However, if we're trying to have our code always use these fields
whenver possible, then 75 is the right choice. Although we'll fail
to build for any sources that's in between the moment the fields were
removed and the moment the version was bumped to 75 (a time period
of about a few days), we'll be able use these fields for the rest
of version 74 of the sources.

Based on the number of days this window is open, I would say that this
situation is highly unlikely, and so we don't really need to worry about
it.

-- 
Joel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: comment.diff
Type: text/x-diff
Size: 1041 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20091019/7d977c95/attachment.bin>


More information about the Gdb-patches mailing list