Re: [PATCH] gprofng/testsuite: restrict testing to native configurations

[Vladimir Mezentsev <vladimir.mezentsev@oracle.com>]

LGTM.

Thank you for the gprofng improvement.
-Vladimir

On 12/19/22 07:06, Jan Beulich wrote:
> The binaries involved in testing gprofng are native ones, and hence a
> cross build of binutils won't really test intended functionality. Since
> this testing takes quite a bit of time (typically more than running all
> of binutils, gas, and ld testsuites together), restrict the testing to
> native configurations only.
>
> --- a/gprofng/testsuite/gprofng.display/display.exp
> +++ b/gprofng/testsuite/gprofng.display/display.exp
> @@ -24,6 +24,13 @@ if {[info exists env(LC_ALL)]} {
>   set env(LC_ALL) "C"
>   
>   set pltf [exec uname -i]
> +
> +# Native testing only for now.
> +if { ![istarget $pltf-*-*]
> +     && (![string match i?86 $pltf] || ![istarget i?86-*-*]) } then {
> +    return
> +}
> +
>   switch $pltf {
>     x86_64 {
>       # Columns in the table represent: