Re: [Plplot-devel] ndiff software commit

[Alan W. I. <Ala...@gm...>]

On 2020-03-02 10:02-0500 Hazen Babcock via Plplot-devel wrote:

>
> Hello,
>
> The ndiff software commit (a974e9802cc0b54ed33d9078b7767b29286c5684 I
> believe) added a lot of files and directories in the utils/ndiff
> directory that appear to be output files (checkXYZ.xyz). Perhaps this
> was a mistake?

Hi Hazen:

I implemented the CMake-based build system for ndiff years ago so to
answer your question I needed to take a quick look at the test part of
that build system again to confirm all was well.  If you look at the
test logic in ndiff/CMakeLists.txt the bash script ndiff_check.sh that
is configured refers to $1.xyz where xyz takes on the values "err",
"out", "opt", "in1", "in2".  Furthermore, if you look further at
ndiff/CMakeLists.txt that $1 argument to the script takes on the
values check001 through check009, and check010 through check026.

So from this quick check it appears the files you refer to are a
legitimate part of the test system so including them in the
commit was the right thing to do.

But thanks for asking because two heads are much better than
one, and you will likely catch one of my commits that needs fixing
some day.  But just not today.  :-)

Alan
__________________________
Alan W. Irwin

Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.org); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________

Linux-powered Science
__________________________