[petsc-dev] Introducing new "test harness" to PETSc

[Barry Smith]

> On Jan 18, 2017, at 2:47 PM, Satish Balay <balay at mcs.anl.gov> wrote:
> 
> wPETSC_DIR is one of the petscconf variables.
> 
> Perhaps we should start a branch origin/scott/testharness-update2 or
> something to push all new changes..

    Simple stand alone feature additions and bug fixes (such as support for comments) can got into their own branches and moved relatively quickly to master.  I don't want another two month scott/testharness-update2 with all kinds of random crap in it if we can avoid that.
> 
> 
> [and change wPETSC_DIR to W_PETSC_DIR everywhere? There is still some
> inconsistancy wrt lowercase/uppercase in these 2 scripts]

W_PETSC_DIR   is really ugly.


> 
> Satish
> 
> -----
> 
> diff --git a/config/example_template.py b/config/example_template.py
> index 6296f46..2f9050e 100755
> --- a/config/example_template.py
> +++ b/config/example_template.py
> @@ -9,6 +9,7 @@ exec=@EXEC@
> testname='@TESTNAME@'
> label=@LABEL@
> runfiles=@LOCALRUNFILES@
> +wPETSC_DIR=@WPETSC_DIR@
> petsc_dir=@PETSC_DIR@
> 
> . ${petsc_dir}/config/petsc_harness.sh
> diff --git a/config/gmakegentest.py b/config/gmakegentest.py
> index 79894e7..fe449a8 100755
> --- a/config/gmakegentest.py
> +++ b/config/gmakegentest.py
> @@ -244,6 +244,7 @@ class generateExamples(Petsc):
>     subst['rm']=self.conf['RM']
>     subst['grep']=self.conf['GREP']
>     subst['petsc_lib_dir']=self.conf['PETSC_LIB_DIR']
> +    subst['wpetsc_dir']=self.conf['wPETSC_DIR']
> 
>     return subst
> 
> On Wed, 18 Jan 2017, Scott Kruger wrote:
> 
>> 
>> 
>> 
>> In the current version of the new test system, there is support for 'requires:
>> DATAFILESPATH' which then keys off what is configured,
>> similar to any other requirement on how PETSc is configured.
>> 
>> How would I get wPETSC_DIR?
>> 
>> Scott
>> 
>> 
>> On 1/18/17 8:09 AM, Satish Balay wrote:
>>> On Tue, 17 Jan 2017, Barry Smith wrote:
>>> 
>>>> In this case the data files is in the PETSc share directory so you should
>>>> use the ${PETSC_DIR} to point to it. Only files in the DATAFILESPATH
>>>> directory should use $DATAFILESPATH
>>>> 
>>> 
>>> Hm - we use wPETSC_DIR so that the path is  valid for windows binaries
>>> aswell. Perhaps
>>> the test harness will need a fix for this..
>>> 
>>> $ git grep wPETSC_DIR |grep makefile |head -3
>>> src/ksp/ksp/examples/tutorials/makefile:	-@${MPIEXEC} -n 2 ./ex10 -f0
>>> ${wPETSC_DIR}/share/petsc/datafiles/matrices/spd-real-int${PETSC_INDEX_SIZE}-float${PETSC_SCALAR_SIZE}
>>>> ex10_1.tmp 2>&1; \
>>> src/ksp/ksp/examples/tutorials/makefile:	-@${MPIEXEC} -n 2 ./ex10
>>> -pc_type asm -pc_asm_sub_mat_type aijviennacl -f0
>>> ${wPETSC_DIR}/share/petsc/datafiles/matrices/spd-real-int${PETSC_INDEX_SIZE}-float${PETSC_SCALAR_SIZE}
>>>> ex10_asm_viennacl.tmp 2>&1; \
>>> src/ksp/ksp/examples/tutorials/makefile:	-@${MPIEXEC} -n 1 ./ex63
>>> --filedir=${wPETSC_DIR}/share/petsc/datafiles/matrices/
>>> --filename=amesos2_test_mat0.mtx --solver=SuperLU --print-residual=true
>>> -ksp_monitor -pc_type lu -pc_factor_mat_solver_package superlu -ksp_view
>>> -ksp_converged_reason  > ex63_1.tmp 2>&1;	  \
>>> 
>>> Satish
>>> 
>> 
>> 
>