From padb at googlecode.com Mon Dec 5 18:31:43 2011 From: padb at googlecode.com (padb at googlecode.com) Date: Mon, 05 Dec 2011 18:31:43 +0000 Subject: [padb] r448 committed - Tweak autogen rules after testing on Ubuntu 11.10 Message-ID: <20cf300fae13c974e204b35c87b4@google.com> Revision: 448 Author: apittman at gmail.com Date: Mon Dec 5 10:30:43 2011 Log: Tweak autogen rules after testing on Ubuntu 11.10 http://code.google.com/p/padb/source/detail?r=448 Modified: /trunk/src/Makefile.am ======================================= --- /trunk/src/Makefile.am Tue Dec 22 02:45:13 2009 +++ /trunk/src/Makefile.am Mon Dec 5 10:30:43 2011 @@ -1,7 +1,7 @@ bin_SCRIPTS = padb libexec_PROGRAMS = minfo minfo_CFLAGS = -Wall -minfo_LDFLAGS = -ldl +minfo_LDADD = -ldl minfo_SOURCES = minfo.c mpi_interface.h EXTRA_DIST = padb From padb at googlecode.com Thu Dec 15 20:39:55 2011 From: padb at googlecode.com (padb at googlecode.com) Date: Thu, 15 Dec 2011 20:39:55 +0000 Subject: [padb] r449 committed - Make backend_debugger a entry in gdb rather than inner_conf so that... Message-ID: <20cf3074d632b29dfe04b4277c5b@google.com> Revision: 449 Author: apittman at gmail.com Date: Thu Dec 15 12:39:20 2011 Log: Make backend_debugger a entry in gdb rather than inner_conf so that when gdb is called from the outer there aren't any warnings http://code.google.com/p/padb/source/detail?r=449 Modified: /trunk/src/padb ======================================= --- /trunk/src/padb Tue Jul 12 13:50:37 2011 +++ /trunk/src/padb Thu Dec 15 12:39:20 2011 @@ -624,7 +624,7 @@ # The number of spaces to indent each new level by when displaying trees. This # value used to be 2 until I added the indentation count in the left column, now # we have that reduce this number to use less screen space. -$conf{tree_indent} = '1'; +$conf{tree_indent} = '1'; $conf{launch_mode} = 'local,rmgr,ssh,pdsh,clush'; @@ -6177,8 +6177,14 @@ seq => 1, }; + if ( defined $inner_conf{backend_debugger} ) { + $gdb->{backend_debugger} = $inner_conf{backend_debugger}; + } else { + $gdb->{backend_debugger} = $conf{backend_debugger}; + } + my $cmd = 'gdb --interpreter=mi -q'; - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { $cmd = 'idb --interpreter=mi2 -q'; } if ( defined $core ) { @@ -6290,7 +6296,7 @@ $gdb->{runtime}{mpich2} = 1; } - if ( $inner_conf{backend_debugger} ne 'idb' ) { + if ( $gdb->{backend_debugger} ne 'idb' ) { gdb_n_send( $gdb, '-gdb-set print address off' ); gdb_n_send( $gdb, '-gdb-set language auto' ); } @@ -6321,7 +6327,7 @@ my %p = _gdb_send_real_async_wait( $gdb, "-target-attach $pid" ); - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { gdb_attach_post( $gdb, $pid ); return $pid; } @@ -6396,7 +6402,7 @@ print { $gdb->{debugfd} } "$seq$cmd\n"; } my %r = gdb_n_next_result( $gdb, $seq ); - if ( $inner_conf{backend_debugger} ne 'idb' + if ( $gdb->{backend_debugger} ne 'idb' and $gdb->{attached} and $r{seq} ne $seq ) { @@ -6426,7 +6432,7 @@ sub _gdb_send_real_async_wait { my ( $gdb, $cmd ) = @_; my $seq = $gdb->{seq}; - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { return; } my %r = gdb_n_next_result( $gdb, $seq ); @@ -6449,7 +6455,7 @@ return; } - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { return; } @@ -6470,7 +6476,7 @@ return; } - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { return; } @@ -6813,7 +6819,7 @@ # As idb doesn't confirm that it's attached properly it's impossible # to do this test correctly here. - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { return %res; } @@ -7945,7 +7951,7 @@ gdb_send( $gdb, "-stack-select-frame $frame->{level}" ); my $print_cmd = '2'; - if ( $inner_conf{backend_debugger} eq 'idb' ) { + if ( $gdb->{backend_debugger} eq 'idb' ) { $print_cmd = '1'; } my %r = gdb_send_addr( $gdb,