[rfa/rfc] Build libcommon.a for gdb and gdbserver

Yao Qi yao@codesourcery.com
Wed Jan 19 23:26:00 GMT 2011


On 01/18/2011 08:19 AM, Tom Tromey wrote:
>
> One idea would be to make new top-level directories in src, then either
> use the generic --disable-DIR configure option, or add a new one to
> specify that you just want gdbserver.
>
> Another idea would be to just have gdb/configure, but have a special
> Makefile target that only builds gdbserver.  Then all the configury
> would be unified; maybe this is the best option, for that reason.

I thought of this, but it makes gdb/configure in mess when we move 
everything to gdb/configure.  Finally, IMO, it is better to create 
separate Makefile.in/configure.ac in common/, whic scales well when we 
move more files to common/ later.

> One final option would be to put gdbserver and common into a single
> directory, so we keep just a single "extra" configure script.
>
> Tom>  I see that common/Makefile.in doesn't have automatic dependency
> Tom>  tracking.  I think it should, especially if we are planning to move more
> Tom>  and more code there.  This would be simpler with a single configure:-)
>
>>> I'll fix that.
>
> FWIW, it may be worth just switching to automake.  It is quite a bit of
> work to do for gdb, but for gdbserver it might not be so bad.

I tried to fix it, but hard to do, unless we migrate common/ to 
automake, which I don't think worthwhile doing so.

In this new patch,
1. Makefile.in and configure.ac are still created as last one.  The 
reason for this is explained above.
2. Automatic dependency tracking of common/Makefile.in is not resolved.
3. Including different headers when building libcommon.a for gdb and 
gdbserver.
4  Building libcommon.a for gdb and gdbserver respectively.

Comments?

-- 
Yao Qi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gdb-gdbserver-common-0119.patch
Type: text/x-patch
Size: 140561 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20110119/0d67c812/attachment.bin>


More information about the Gdb-patches mailing list