On 03/22/2016 01:15 AM, Ben Elliston wrote:

> +    # Don't try to execute a data file, which is used by some tests
> +    # to compare output.
> +    if {[regexp "^\.txt$" $prog]} {
> +      return ""
> +    }
> 
> Wouldn't it be better to not call unix_load with such an argument?

  Unix_load is used to download a data file to the target, and the
output of the executable is compared. This is used by some of the GCC tests.

> Also, the patch contains a proc definition called make_absolute.  It's
> redundant -- Tcl offers `file normalize'.

  I can drop that from the patch, I didn't write it.

        - rob -


_______________________________________________
DejaGnu mailing list
DejaGnu@gnu.org
https://lists.gnu.org/mailman/listinfo/dejagnu

Reply via email to