This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: target-delegates.c needs some TLC [was Re: [OB PATCH] target.h (to_traceframe_info): Fix TARGET_DEFAULT_RETURN]
- From: Doug Evans <dje at google dot com>
- To: Yao Qi <yao at codesourcery dot com>
- Cc: Doug Evans <xdje42 at gmail dot com>, Tom Tromey <tromey at redhat dot com>, Hui Zhu <hui_zhu at mentor dot com>, gdb-patches ml <gdb-patches at sourceware dot org>
- Date: Sun, 9 Mar 2014 22:34:15 -0700
- Subject: Re: target-delegates.c needs some TLC [was Re: [OB PATCH] target.h (to_traceframe_info): Fix TARGET_DEFAULT_RETURN]
- Authentication-results: sourceware.org; auth=none
- References: <CAP9bCMTwuW_2_3Dn+JynOxtgyv862ghs8GZgOyUm8MbEcfb=Tw at mail dot gmail dot com> <531D2739 dot 3000508 at codesourcery dot com>
On Sun, Mar 9, 2014 at 7:45 PM, Yao Qi <yao@codesourcery.com> wrote:
> On 03/06/2014 01:20 PM, Doug Evans wrote:
>> While I realize we don't want to require perl for building gdb (and I
>> for one would never advocate it), I wonder if we can do at least a bit
>> better.
>
> perl is required by git, and GDB source is version controlled by git.
> I assume that most of the host machine used for gdb development have
> git and perl installed. People may get source from release, and
> generate patch on top of it (without git/perl installed). It isn't
> common, is it?
Apologies for not being clear enough.
This is not, and never has been, an issue of needing perl installed per se.
The issue is whether we should impose on people the requirement of
some perl knowledge, however minimal, when one turns on
--enable-maintainer-mode.
If people decide to, fine by me. But I was not prepared to do that
without a discussion.
> Anyway, we can check whether perl is installed. If perl is installed,
> run make-target-delegates during make and error out if new-generated
> target-delegates.c is different from the one in gdb source. Is it good?
> If yes, I'll submit a patch for this.
There is no need for this AFAICS.
Does AM_MAINTAINER_MODE do this for autoconf?
Maybe it does and I can't find it.
And if it doesn't do so for autoconf, why do it for perl?