Re: patch #8947: Don't try to execute a data file

Rob Savoye <[email protected]>
Newsgroups gmane.comp.sysutils.dejagnu.general
Organization Linaro
Message-ID <[email protected]>
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 -
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.