public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
* testsuite/gdb.gdbtk
@ 2008-04-22  6:51 Denis PILAT
  2008-04-22 18:53 ` testsuite/gdb.gdbtk Maciej W. Rozycki
  0 siblings, 1 reply; 7+ messages in thread
From: Denis PILAT @ 2008-04-22  6:51 UTC (permalink / raw)
  To: insight

I'm wondering why this directory is not delivered within insight 
distribution.
Is there anybody who knows if this testsuite is maintained and working 
well ?

Thanks
Denis

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
  2008-04-22  6:51 testsuite/gdb.gdbtk Denis PILAT
@ 2008-04-22 18:53 ` Maciej W. Rozycki
  2008-04-23 11:23   ` testsuite/gdb.gdbtk Denis PILAT
  0 siblings, 1 reply; 7+ messages in thread
From: Maciej W. Rozycki @ 2008-04-22 18:53 UTC (permalink / raw)
  To: Denis PILAT; +Cc: insight

On Tue, 22 Apr 2008, Denis PILAT wrote:

> I'm wondering why this directory is not delivered within insight 
> distribution.
> Is there anybody who knows if this testsuite is maintained and working 
> well ?

 It mostly worked for me for the mipsisa32-sde-elf target and the
mips-sim-sde32 target board the last time I tried -- that's a
configuration using the GNU simulator.  There may have been two or three
failures which I had no time to investigate unfortunately, but the
majority of tests passed.  I have no access to that configuration anymore,
but I hope that sounds encouraging to you.  As to maintenance, I would not
call that very active, but I am sure if you find a bug and prepare a fix
which you will post here, it will be gladly accepted. :-)

  Maciej

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
  2008-04-22 18:53 ` testsuite/gdb.gdbtk Maciej W. Rozycki
@ 2008-04-23 11:23   ` Denis PILAT
  2008-04-23 14:54     ` testsuite/gdb.gdbtk Maciej W. Rozycki
  2008-09-12 14:20     ` testsuite/gdb.gdbtk Denis PILAT
  0 siblings, 2 replies; 7+ messages in thread
From: Denis PILAT @ 2008-04-23 11:23 UTC (permalink / raw)
  To: Maciej W. Rozycki, insight

Thanks for this answer, but I don't understand why gdb.gdbtk does not 
belong to insight distribution.
I've tried on insight6.8 for native configuration  i686-pc-linux-gnu and 
get 29 unexpected failures (and 523 pass).

About patches, would somebody accept the following one that avoid having 
an error when TCL_LIBRARY variable is not set ?
Thanks,

--
Denis

2008-04-23  Denis Pilat  <denis.pilat@st.com>

	* insight-support.exp (gdbtk_start): test TCL_LIBRARY env. variable
	  before being unset.


Index: insight-support.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.gdbtk/insight-support.exp,v
retrieving revision 1.2
diff -u -p -r1.2 insight-support.exp
--- insight-support.exp 8 Feb 2007 17:12:50 -0000       1.2
+++ insight-support.exp 23 Apr 2008 11:05:58 -0000
@@ -98,8 +98,9 @@ proc gdbtk_start {test} {
   set env(SRCDIR) $abs_srcdir
   set env(GDBTK_VERBOSE) 1
   set env(GDBTK_LOGFILE) [to_tcl_path [file join $objdir gdb.log]]
-  unset -nocomplain env(TCL_LIBRARY)
-
+  if {[info exists env(TCL_LIBRARY)]} {
+    unset -nocomplain env(TCL_LIBRARY)
+  }
   set err [catch {exec $INSIGHT -nx -q --tclcommand=$test} res]
   if { $err } {
     perror "Execing $INSIGHT failed: $res"






Maciej W. Rozycki wrote:
> On Tue, 22 Apr 2008, Denis PILAT wrote:
>
>   
>> I'm wondering why this directory is not delivered within insight 
>> distribution.
>> Is there anybody who knows if this testsuite is maintained and working 
>> well ?
>>     
>
>  It mostly worked for me for the mipsisa32-sde-elf target and the
> mips-sim-sde32 target board the last time I tried -- that's a
> configuration using the GNU simulator.  There may have been two or three
> failures which I had no time to investigate unfortunately, but the
> majority of tests passed.  I have no access to that configuration anymore,
> but I hope that sounds encouraging to you.  As to maintenance, I would not
> call that very active, but I am sure if you find a bug and prepare a fix
> which you will post here, it will be gladly accepted. :-)
>
>   Maciej
>
>   

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
  2008-04-23 11:23   ` testsuite/gdb.gdbtk Denis PILAT
@ 2008-04-23 14:54     ` Maciej W. Rozycki
  2008-04-23 18:12       ` testsuite/gdb.gdbtk Keith Seitz
  2008-09-12 14:20     ` testsuite/gdb.gdbtk Denis PILAT
  1 sibling, 1 reply; 7+ messages in thread
From: Maciej W. Rozycki @ 2008-04-23 14:54 UTC (permalink / raw)
  To: Denis PILAT; +Cc: insight

On Wed, 23 Apr 2008, Denis PILAT wrote:

> Thanks for this answer, but I don't understand why gdb.gdbtk does not 
> belong to insight distribution.

 Well, I suppose there is nothing to understand here -- I am not entirely
sure how exactly distributions from the src/ tree are prepared, but that's
quite likely a matter of the directory having been accidentally omitted
from the list in a script somewhere.

  Maciej

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
  2008-04-23 14:54     ` testsuite/gdb.gdbtk Maciej W. Rozycki
@ 2008-04-23 18:12       ` Keith Seitz
  0 siblings, 0 replies; 7+ messages in thread
From: Keith Seitz @ 2008-04-23 18:12 UTC (permalink / raw)
  To: Denis PILAT; +Cc: insight

On Wed, 23 Apr 2008, Denis PILAT wrote:

>> Thanks for this answer, but I don't understand why gdb.gdbtk does not 
>> belong to insight distribution.

I think what is happening is that I am forgetting to tag this directory 
for the release. [Gdb group doesn't tag insight any more...]

No great conspiracy: just my ignorance. I will jot it down in my release 
instructions so that I don't forget to do it again!

Thank you for pointing this out!

Keith

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
  2008-04-23 11:23   ` testsuite/gdb.gdbtk Denis PILAT
  2008-04-23 14:54     ` testsuite/gdb.gdbtk Maciej W. Rozycki
@ 2008-09-12 14:20     ` Denis PILAT
       [not found]       ` <48D30B30.4020407@redhat.com>
  1 sibling, 1 reply; 7+ messages in thread
From: Denis PILAT @ 2008-09-12 14:20 UTC (permalink / raw)
  To: Maciej W. Rozycki, insight; +Cc: Denis PILAT

I forgot to ping you about this patch.
-- 
Denis


>
> About patches, would somebody accept the following one that avoid 
> having an error when TCL_LIBRARY variable is not set ?
> Thanks,
>
> -- 
> Denis
>
> 2008-04-23  Denis Pilat  <denis.pilat@st.com>
>
>     * insight-support.exp (gdbtk_start): test TCL_LIBRARY env. variable
>       before being unset.
>
>
> Index: insight-support.exp
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/gdb.gdbtk/insight-support.exp,v
> retrieving revision 1.2
> diff -u -p -r1.2 insight-support.exp
> --- insight-support.exp 8 Feb 2007 17:12:50 -0000       1.2
> +++ insight-support.exp 23 Apr 2008 11:05:58 -0000
> @@ -98,8 +98,9 @@ proc gdbtk_start {test} {
>   set env(SRCDIR) $abs_srcdir
>   set env(GDBTK_VERBOSE) 1
>   set env(GDBTK_LOGFILE) [to_tcl_path [file join $objdir gdb.log]]
> -  unset -nocomplain env(TCL_LIBRARY)
> -
> +  if {[info exists env(TCL_LIBRARY)]} {
> +    unset -nocomplain env(TCL_LIBRARY)
> +  }
>   set err [catch {exec $INSIGHT -nx -q --tclcommand=$test} res]
>   if { $err } {
>     perror "Execing $INSIGHT failed: $res"
>
>
>
>
>
>
> Maciej W. Rozycki wrote:
>> On Tue, 22 Apr 2008, Denis PILAT wrote:
>>
>>  
>>> I'm wondering why this directory is not delivered within insight 
>>> distribution.
>>> Is there anybody who knows if this testsuite is maintained and 
>>> working well ?
>>>     
>>
>>  It mostly worked for me for the mipsisa32-sde-elf target and the
>> mips-sim-sde32 target board the last time I tried -- that's a
>> configuration using the GNU simulator.  There may have been two or three
>> failures which I had no time to investigate unfortunately, but the
>> majority of tests passed.  I have no access to that configuration 
>> anymore,
>> but I hope that sounds encouraging to you.  As to maintenance, I 
>> would not
>> call that very active, but I am sure if you find a bug and prepare a fix
>> which you will post here, it will be gladly accepted. :-)
>>
>>   Maciej
>>
>>   
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: testsuite/gdb.gdbtk
       [not found]       ` <48D30B30.4020407@redhat.com>
@ 2008-09-20 20:41         ` Denis PILAT
  0 siblings, 0 replies; 7+ messages in thread
From: Denis PILAT @ 2008-09-20 20:41 UTC (permalink / raw)
  To: Keith Seitz; +Cc: insight

Just committed !
-- 
Denis

Keith Seitz wrote:
> Denis PILAT wrote:
>> I forgot to ping you about this patch.
>
> That's okay, I forgot to look at it. Again, apparently.
>
> If you still have this lying around, by all means, check it in.
>
> Keith
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2008-09-19  8:20 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-04-22  6:51 testsuite/gdb.gdbtk Denis PILAT
2008-04-22 18:53 ` testsuite/gdb.gdbtk Maciej W. Rozycki
2008-04-23 11:23   ` testsuite/gdb.gdbtk Denis PILAT
2008-04-23 14:54     ` testsuite/gdb.gdbtk Maciej W. Rozycki
2008-04-23 18:12       ` testsuite/gdb.gdbtk Keith Seitz
2008-09-12 14:20     ` testsuite/gdb.gdbtk Denis PILAT
     [not found]       ` <48D30B30.4020407@redhat.com>
2008-09-20 20:41         ` testsuite/gdb.gdbtk Denis PILAT

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).