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]

Re: [PATCH] Move gnulib to top level


On 6/13/19 1:48 PM, Tom Tromey wrote:
> This patch moves the gdb/gnulib subdirectory to the top level.
> 
> It adjusts the top-level build system to build gnulib when necessary,
> and changes gdb to use this.  However, gdbserver still builds its own
> copy of gnulib, just from the new source location.
> 
> A small hack was needed to ensure that gnulib is only built when gdb
> is enabled.  The Makefile only provides an ordering -- the directory
> must be mentioned in configdirs to actually be compiled at all.
> 
> Most of the patch is just a "git mv" of gnulib, though a few minor
> path adjustments were needed in some files there.
> 

Also, gnulib/configure.ac is using ACX_LARGEFILE now.

> Tested by the buildbot.
> 

I think we'll need the following, though not necessarily
in this patch:

 - a gnulib/README file
 - to list gnulib in top/MAINTAINERS

LGTM.

I'm sure you know, but JIC, the top-level changes will
need to go to gcc too.

Thanks,
Pedro Alves


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