This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA/commit] pre-initialize python-value.c:values_in_python (bug in MacOS linker)


Hello,

We noticed that GDB does not build on x86-macos and x86_64-macos when
configured --without-python (we found this out because one of our
machine no longer has python installed on it). The problem is that
the linker complains that it cannot file values_in_python, referenced
from value.c. This is a bug, since we verified that the symbol was
defined inside python-value.o which was also part of the archive.
We think that this might be related to the fact that the symbol is
defined as "C" (common). We changed the code to add an initial value
of NULL (which in C is strictly equivalent to the current code),
which changed our variable to the Data section and, ta da, allowed
GDB to link.

2008-12-05  Joel Brobecker  <brobecker@adacore.com>

        * python/python-value.c (values_in_python): Add specific initialization
        to NULL to work-around a MacOS linker bug.

Tested on x86-linux, both using --with-python and --without-python.
No regression.

I think this is pretty safe and safe contained, and doesn't reduce
the quality of the code, so I'm planning on committing this on Monday
when back from weekend holiday (I'm in Europe). Any objection? Should
I add a comment?

Thanks,
-- 
Joel

Attachment: python-value.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]