1) If we do decide to adopt this usage in packages then it should go into the package.py class for GNUPackage not directly into the hypre one.
2) If we do decide to adopt this usage then it should print to the screen in a "box" for each package it is doing this stuff, because it is likely fragile and needs to be told explicitly to the user and it being done and what its alternative is (should it break) 3) If we do decide to adopt this usage Satish will need to figure out how to run ALL the nightly builds using the --with-batch option so he can run all the nightly builds using the --with-batch option automatically so the -host option can get properly tested on a variety of systems. I think it is worth trying this but fear it is too fragile and will break often, hence we need proper testing before sticking into master. Barry > On Oct 10, 2017, at 3:48 PM, Satish Balay <[email protected]> wrote: > > I never understood --host option of autoconf - and it would rarely > work for me. > > For packages where '--with-batch --download-package' did not work - > our suggstion so far was to install them manually. > > I'm hesitant to add this patch [it might work on one setup - but not > the other?] - but Matt/Jed might have better suggestions. > > cc:ing petsc-dev > > Satish > > On Tue, 10 Oct 2017, Stefano Zampini wrote: > >> Satish, >> >> today I had some problems installing PETSc with --download-hypre in batched >> mode. Here is my fix >> >> diff --git a/config/BuildSystem/config/packages/hypre.py >> b/config/BuildSystem/config/packages/hypre.py >> index e374532..3c607c9 100644 >> --- a/config/BuildSystem/config/packages/hypre.py >> +++ b/config/BuildSystem/config/packages/hypre.py >> @@ -49,6 +49,9 @@ class Configure(config.package.GNUPackage): >> libs = ' '.join(libs) >> args.append('--with-MPI-libs="'+libs+'"') >> >> + if self.argDB['with-batch']: >> + args.append('--host') >> + >> # tell hypre configure not to look for blas/lapack [and not use >> hypre-internal blas] >> >> args.append('--with-blas-lib="'+self.libraries.toString(self.blasLapack.dlib)+'"') >> args.append('--with-lapack-lib=" "') >> >> >> >
