Fix Makefile.in in data-directory
Jie Zhang
jie@codesourcery.com
Thu Oct 21 00:10:00 GMT 2010
On 10/21/2010 04:13 AM, Doug Evans wrote:
> On Wed, Oct 20, 2010 at 5:09 AM, Jie Zhang<jie@codesourcery.com> wrote:
>>
>> The current install rule in the Makefile.in of data-directory uses recursive make but does not pass FLAGS_TO_PASS. This causes an issue when people try to override prefix with command line option "make install", i.e. data-directory will still be installed under the configure time prefix. This patch fixes it. OK?
>
> [Apologies for the resend.]
>
> Thanks for the bug report. Forgot about that.
>
> This patch is not ok though. A parallel make (e.g. -jN) might try to
> build all and install-only simultaneously.
>
> maybe:
> $(MAKE) install-only $(FLAGS_TO_PASS)
>
>
> For reference sake, there are other instances of "install: all install-only".
> They need to be fixed too.
I don't think this is an issue. I think in the last more than 10 years,
no one did "make -j all install" since the same instance in
gdb/Makefile.in has been there since it was created. Actually if you do
that in current gdb, it will not install gdb at all. I usually issue
make -j4 all
make install
to make gdb. I believe people have been used to do that in this way
after so many years. So I think it's not an issue to do the same for
data-directory. No?
--
Jie Zhang
CodeSourcery
More information about the Gdb-patches
mailing list