* [patch] testsuite: support board_info timeouts
@ 2010-08-10 4:08 DJ Delorie
2010-08-10 5:09 ` Mike Stump
2011-06-02 16:49 ` DJ Delorie
0 siblings, 2 replies; 5+ messages in thread
From: DJ Delorie @ 2010-08-10 4:08 UTC (permalink / raw)
To: gcc-patches
Is there any reason why we don't support board-level timeouts? It's
really hard to specify timeouts for sid-based embedded targets with
lots of multilibs (or just one, sometimes).
It's certainly better than "really REALLY ugly" which is the only
other option at that point.
* lib/timeout.exp (timeout): Add board_info support.
2010-08-09 Thomas Koenig <tkoenig@gcc.gnu.org>
Index: lib/timeout.exp
===================================================================
--- lib/timeout.exp (revision 163048)
+++ lib/timeout.exp (working copy)
@@ -43,12 +43,14 @@ proc timeout_value { args } {
if [info exists individual_timeout] {
set val $individual_timeout
} elseif [info exists tool_timeout] {
set val $tool_timeout
} elseif [target_info exists gcc,timeout] {
set val [target_info gcc,timeout]
+ } elseif [board_info target exists gcc,timeout] {
+ set val [board_info target gcc,timeout]
} else {
# This is really, REALLY ugly, but this is the default from
# remote.exp deep within DejaGnu.
set val 300
}
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch] testsuite: support board_info timeouts
2010-08-10 4:08 [patch] testsuite: support board_info timeouts DJ Delorie
@ 2010-08-10 5:09 ` Mike Stump
2011-06-02 16:49 ` DJ Delorie
1 sibling, 0 replies; 5+ messages in thread
From: Mike Stump @ 2010-08-10 5:09 UTC (permalink / raw)
To: DJ Delorie; +Cc: gcc-patches
n Aug 9, 2010, at 8:48 PM, DJ Delorie wrote:
> Is there any reason why we don't support board-level timeouts?
I don't know of any... If it helps you, I'd say, lets put it in...
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch] testsuite: support board_info timeouts
2010-08-10 4:08 [patch] testsuite: support board_info timeouts DJ Delorie
2010-08-10 5:09 ` Mike Stump
@ 2011-06-02 16:49 ` DJ Delorie
2011-06-02 17:28 ` Mike Stump
1 sibling, 1 reply; 5+ messages in thread
From: DJ Delorie @ 2011-06-02 16:49 UTC (permalink / raw)
To: gcc-patches
I never got feedback from the testsuite maintainers on this one...
> Date: Mon, 9 Aug 2010 23:48:31 -0400
> From: DJ Delorie <dj@redhat.com>
> Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
>
>
> Is there any reason why we don't support board-level timeouts? It's
> really hard to specify timeouts for sid-based embedded targets with
> lots of multilibs (or just one, sometimes).
>
> It's certainly better than "really REALLY ugly" which is the only
> other option at that point.
>
> * lib/timeout.exp (timeout): Add board_info support.
>
> 2010-08-09 Thomas Koenig <tkoenig@gcc.gnu.org>
> Index: lib/timeout.exp
> ===================================================================
> --- lib/timeout.exp (revision 163048)
> +++ lib/timeout.exp (working copy)
> @@ -43,12 +43,14 @@ proc timeout_value { args } {
> if [info exists individual_timeout] {
> set val $individual_timeout
> } elseif [info exists tool_timeout] {
> set val $tool_timeout
> } elseif [target_info exists gcc,timeout] {
> set val [target_info gcc,timeout]
> + } elseif [board_info target exists gcc,timeout] {
> + set val [board_info target gcc,timeout]
> } else {
> # This is really, REALLY ugly, but this is the default from
> # remote.exp deep within DejaGnu.
> set val 300
> }
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch] testsuite: support board_info timeouts
2011-06-02 16:49 ` DJ Delorie
@ 2011-06-02 17:28 ` Mike Stump
2011-06-02 18:16 ` DJ Delorie
0 siblings, 1 reply; 5+ messages in thread
From: Mike Stump @ 2011-06-02 17:28 UTC (permalink / raw)
To: DJ Delorie; +Cc: gcc-patches
On Jun 2, 2011, at 9:48 AM, DJ Delorie wrote:
> I never got feedback from the testsuite maintainers on this one...
Ok.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch] testsuite: support board_info timeouts
2011-06-02 17:28 ` Mike Stump
@ 2011-06-02 18:16 ` DJ Delorie
0 siblings, 0 replies; 5+ messages in thread
From: DJ Delorie @ 2011-06-02 18:16 UTC (permalink / raw)
To: Mike Stump; +Cc: gcc-patches
Thanks! Committed.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-06-02 18:16 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-08-10 4:08 [patch] testsuite: support board_info timeouts DJ Delorie
2010-08-10 5:09 ` Mike Stump
2011-06-02 16:49 ` DJ Delorie
2011-06-02 17:28 ` Mike Stump
2011-06-02 18:16 ` DJ Delorie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).