dejagnu
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: How to set "isremote" ?


From: Christophe Lyon
Subject: Re: How to set "isremote" ?
Date: Wed, 5 Jun 2019 15:52:21 +0200

On Wed, 5 Jun 2019 at 12:42, Pedro Alves <address@hidden> wrote:
>
> On 6/4/19 5:22 PM, Christophe Lyon wrote:
> > Hi,
> >
> > I've been debugging a problem where we clear "isremote" with:
> > unset_board_info isremote
> > set_board_info isremote 0
> > but this isn't taken into account correctly by is_remote (in
> > framework.exp), when we use target variants, because is_remote removes
> > the target variant specifications.
> >
> > For instance, if we use --target-board=qemu, everything works well,
> > but if we use --target-board=qemu/-mcpu=XXX, then:
> > ==========
> > unset_board_info isremote
> > set_board_info isremote 0
> > ==========
> > clear the "isremote" flag for board=="qemu/-mcpu=XXX", but subsequent
> > calls to is_remote actually check the isremote flagfor board=="qemu"
> > (after removing the part after the "/")
> >
> > Is that a bug in is_remote, or is there a clean way of setting "isremote"?
>
> Here's how GDB works around this:
>
> https://sourceware.org/git/?p=binutils-gdb.git;a=blob;f=gdb/testsuite/boards/local-board.exp;h=6e0448c58a94afc87e23c33359593a89fecd76fe;hb=HEAD#l20
>
>   20 global board
>   21 global board_info
>   22 # Remove any target variant specifications from the name.
>   23 set baseboard [lindex [split $board "/"] 0]
>   24 set board_info($baseboard,isremote) 0
>

Thanks for confirming that this kind of workaround is needed (Applying
these lines to our .exp file does the trick)

Christophe

> Thanks,
> Pedro Alves



reply via email to

[Prev in Thread] Current Thread [Next in Thread]