Todays CVS checkout fails on frv, ppceabi, mips, ...

Todays CVS checkout fails on frv, ppceabi, mips, ...

[Graham Stott]

Hi

Anyone know what's changed since the 16th to cause this :

---------------------------------------------
/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before "gdb_gregset_t"
/usr/local/src/uberbaum/gdb/gregset.h:32: warning: type defaults to `int' in declaration of `gdb_gregset_t'
/usr/local/src/uberbaum/gdb/gregset.h:32: warning: data definition has no type or storage class
/usr/local/src/uberbaum/gdb/gregset.h:33: error: parse error before "gdb_fpregset_t"
/usr/local/src/uberbaum/gdb/gregset.h:33: warning: type defaults to `int' in declaration of `gdb_fpregset_t'
/usr/local/src/uberbaum/gdb/gregset.h:33: warning: data definition has no type or storage class
/usr/local/src/uberbaum/gdb/gregset.h:45: error: parse error before '*' token
/usr/local/src/uberbaum/gdb/gregset.h:46: error: parse error before '*' token
/usr/local/src/uberbaum/gdb/gregset.h:52: error: parse error before '*' token
/usr/local/src/uberbaum/gdb/gregset.h:53: error: parse error before '*' token
/usr/local/src/uberbaum/gdb/linux-proc.c: In function `linux_do_thread_registers':
/usr/local/src/uberbaum/gdb/linux-proc.c:175: error: parse error before "gregs"
/usr/local/src/uberbaum/gdb/linux-proc.c:182: error: `gregs' undeclared (first use in this function)
/usr/local/src/uberbaum/gdb/linux-proc.c:182: error: (Each undeclared identifier is reported only once
/usr/local/src/uberbaum/gdb/linux-proc.c:182: error: for each function it appears in.)
/usr/local/src/uberbaum/gdb/linux-proc.c:190: error: `fpregs' undeclared (first use in this function)
---------------------------------------------

on all the cross toolchains (frv, ppceabi, mips, ...) I've tried to build
today following a cvs update. Everything built last night and I can't see
anything obvious in ChangeLogs.

Cheers
Graham

ps. i686-pc-linux-gnu native builds no problem,

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Daniel Jacobowitz]

On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
> Hi
> 
> Anyone know what's changed since the 16th to cause this :
> 
> ---------------------------------------------
> /usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
> "gdb_gregset_t"

Where's it getting included from?


-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Graham Stott]

Daniel Jacobowitz wrote:
> On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
> 
>>Hi
>>
>>Anyone know what's changed since the 16th to cause this :
>>
>>---------------------------------------------
>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
>>"gdb_gregset_t"
> 
> 
> Where's it getting included from?
> 
> 
gdb/linux-proc.c

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Daniel Jacobowitz]

On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
> Daniel Jacobowitz wrote:
> >On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
> >
> >>Hi
> >>
> >>Anyone know what's changed since the 16th to cause this :
> >>
> >>---------------------------------------------
> >>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
> >>"gdb_gregset_t"
> >
> >
> >Where's it getting included from?
> >
> >
> gdb/linux-proc.c

Is the target != host block at the end of configure.in triggering? 
Andrew changed that today but I can't see any problem with his change.

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Graham Stott]

Daniel Jacobowitz wrote:
> On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
> 
>>Daniel Jacobowitz wrote:
>>
>>>On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
>>>
>>>
>>>>Hi
>>>>
>>>>Anyone know what's changed since the 16th to cause this :
>>>>
>>>>---------------------------------------------
>>>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
>>>>"gdb_gregset_t"
>>>
>>>
>>>Where's it getting included from?
>>>
>>>
>>
>>gdb/linux-proc.c
> 
> 
> Is the target != host block at the end of configure.in triggering? 
> Andrew changed that today but I can't see any problem with his change.
> 

No it doesn't look like it's triggering :-(

I added "else echo "native" 1>&2" to that block and I'm seeing "native"
when I rerun configure.

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Graham Stott]

Daniel Jacobowitz wrote:
> On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
> 
>>Daniel Jacobowitz wrote:
>>
>>>On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
>>>
>>>
>>>>Hi
>>>>
>>>>Anyone know what's changed since the 16th to cause this :
>>>>
>>>>---------------------------------------------
>>>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
>>>>"gdb_gregset_t"
>>>
>>>
>>>Where's it getting included from?
>>>
>>>
>>
>>gdb/linux-proc.c
> 
> 
> Is the target != host block at the end of configure.in triggering? 
> Andrew changed that today but I can't see any problem with his change.
> 

I think I can see what's wrong ${host} and ${target} need to ${gdb_host_cpu}
and ${gdb_target_cpu} respect.

Graham

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Daniel Jacobowitz]

On Mon, Feb 17, 2003 at 08:33:28PM +0000, Graham Stott wrote:
> Daniel Jacobowitz wrote:
> >On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
> >
> >>Daniel Jacobowitz wrote:
> >>
> >>>On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
> >>>
> >>>
> >>>>Hi
> >>>>
> >>>>Anyone know what's changed since the 16th to cause this :
> >>>>
> >>>>---------------------------------------------
> >>>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
> >>>>"gdb_gregset_t"
> >>>
> >>>
> >>>Where's it getting included from?
> >>>
> >>>
> >>
> >>gdb/linux-proc.c
> >
> >
> >Is the target != host block at the end of configure.in triggering? 
> >Andrew changed that today but I can't see any problem with his change.
> >
> 
> I think I can see what's wrong ${host} and ${target} need to ${gdb_host_cpu}
> and ${gdb_target_cpu} respect.

Really?  What are $host and $target?  And how did you configure?

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Graham Stott]

Daniel Jacobowitz wrote:
> On Mon, Feb 17, 2003 at 08:33:28PM +0000, Graham Stott wrote:
> 
>>Daniel Jacobowitz wrote:
>>
>>>On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
>>>
>>>
>>>>Daniel Jacobowitz wrote:
>>>>
>>>>
>>>>>On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
>>>>>
>>>>>
>>>>>
>>>>>>Hi
>>>>>>
>>>>>>Anyone know what's changed since the 16th to cause this :
>>>>>>
>>>>>>---------------------------------------------
>>>>>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
>>>>>>"gdb_gregset_t"
>>>>>
>>>>>
>>>>>Where's it getting included from?
>>>>>
>>>>>
>>>>
>>>>gdb/linux-proc.c
>>>
>>>
>>>Is the target != host block at the end of configure.in triggering? 
>>>Andrew changed that today but I can't see any problem with his change.
>>>
>>
>>I think I can see what's wrong ${host} and ${target} need to ${gdb_host_cpu}
>>and ${gdb_target_cpu} respect.
> 
> 
> Really?  What are $host and $target?  And how did you configure?
> 

Both are empty.

A bog standard "/usr/local/src/uberbaum/configure --target=mips-elf"

If you look a few line up in gdb's configure script you'll see

gdb_host_cpu=$gdb_hogdb_host_cpu=$gdb_host_cpu
gdb_target_cpu=$gdb_target_cpu
nativefile=$nativefile

and after configuring gdb/config.status contains

gdb_host_cpu=i386
gdb_target_cpu=mips
nativefile=

Making my suggested change in my local tree configuring now triggers the host != target
block and my builds are now working.

Graham

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Daniel Jacobowitz]

On Mon, Feb 17, 2003 at 08:52:53PM +0000, Graham Stott wrote:
> Daniel Jacobowitz wrote:
> >On Mon, Feb 17, 2003 at 08:33:28PM +0000, Graham Stott wrote:
> >
> >>Daniel Jacobowitz wrote:
> >>
> >>>On Mon, Feb 17, 2003 at 07:54:33PM +0000, Graham Stott wrote:
> >>>
> >>>
> >>>>Daniel Jacobowitz wrote:
> >>>>
> >>>>
> >>>>>On Mon, Feb 17, 2003 at 07:45:03PM +0000, Graham Stott wrote:
> >>>>>
> >>>>>
> >>>>>
> >>>>>>Hi
> >>>>>>
> >>>>>>Anyone know what's changed since the 16th to cause this :
> >>>>>>
> >>>>>>---------------------------------------------
> >>>>>>/usr/local/src/uberbaum/gdb/gregset.h:32: error: parse error before 
> >>>>>>"gdb_gregset_t"
> >>>>>
> >>>>>
> >>>>>Where's it getting included from?
> >>>>>
> >>>>>
> >>>>
> >>>>gdb/linux-proc.c
> >>>
> >>>
> >>>Is the target != host block at the end of configure.in triggering? 
> >>>Andrew changed that today but I can't see any problem with his change.
> >>>
> >>
> >>I think I can see what's wrong ${host} and ${target} need to 
> >>${gdb_host_cpu}
> >>and ${gdb_target_cpu} respect.
> >
> >
> >Really?  What are $host and $target?  And how did you configure?
> >
> 
> Both are empty.

That's really confusing to me - shouldn't they be your host and
mips-unknown-elf?

-- 
Daniel Jacobowitz
MontaVista Software                         Debian GNU/Linux Developer

Re: Todays CVS checkout fails on frv, ppceabi, mips, ...

[Andrew Cagney]

> Hi
> 
> Anyone know what's changed since the 16th to cause this :

Just FYI,  I've yanked [my] patch.  Need to go back and test it some more.

Andrew