Skip to content

Commit

Permalink
Fix handling of BUILD_SUFFIX in remote-gdb script
Browse files Browse the repository at this point in the history
When CONFIG_BUILD_SUFFIX is enabled, the target-* folders in build_dir
and staging_dir have this suffix in the name, but not the
toolchain directories. When detecting the names for "arch" and "libc",
also accept the suffix and do not use it for the toolchain path.

Signed-off-by: Thomas Langer <[email protected]>
  • Loading branch information
langerth authored and hauke committed Sep 1, 2019
1 parent 413c68d commit 035906f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions scripts/remote-gdb
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,9 @@ if( opendir SD, "$Bin/../staging_dir" )
{
my ( $tid, $arch, $libc, @arches );

if( $ARGV[1] =~ m!\btarget-(.+?)_(([^/_]+libc|musl)[^/_]*)\b!i )
if( $ARGV[1] =~ m!\btarget-(.+?)_([^/_]+libc|musl)_?([^/]*).*\b!i )
{
print("Using target $1 ($2)\n");
print("Using target $1 ($2, $3)\n");
($arch, $libc) = ($1, $2);
}
else
Expand All @@ -26,10 +26,10 @@ if( opendir SD, "$Bin/../staging_dir" )

while( defined( my $e = readdir SD ) )
{
if( -d "$Bin/../staging_dir/$e" && $e =~ /^target-(.+?)_(([^_]+libc|musl).*)/i )
if( -d "$Bin/../staging_dir/$e" && $e =~ /^target-(.+?)_([^\/_]+libc|musl)_?([^\/]*).*/i )
{
push @arches, [ $1, $2 ];
printf(" %2d) %s (%s)\n", @arches + 0, $1, $2);
printf(" %2d) %s (%s %s)\n", @arches + 0, $1, $2, $3);
}
}

Expand Down

0 comments on commit 035906f

Please sign in to comment.