Petsc conifg opts changed by hajgato · Pull Request #446 · easybuilders/easybuild-easyblocks

hajgato · 2014-07-30T18:33:49Z

No description provided.

…sybuild-easyblocks into petsc_conifg_opts_changed

hpcugentbot · 2014-07-30T18:33:52Z

Automatic reply from Jenkins: Can I test this?

wpoely86 · 2014-07-31T07:52:30Z

Extend this to cmd = "%s make %s" % (self.cfg['prebuildopts'], self.cfg['buildopts'])

Can you not simpily set self.cfg['parallel'] to None and use the configuremake build_step?

wpoely86 · 2014-07-31T08:09:01Z

ok to test

if suitesparse if loops corrected

def build_step removed, and -j issue for make solved using self.cfg['paeallel'] to None.

wpoely86 · 2014-07-31T13:45:25Z

this needs to be a indenting level deeper (in the function)

But then it should be earlier in the file, right?

the location is fine (in the configure_step() ) but the code is on the same level as the methods. It needs to happen in a method 😉

JensTimmerman · 2014-09-10T13:23:19Z

I think it's best to also set LIBS and LD_FLAGS in configopts?
(btw, is the -- needed?)

hajgato · 2014-09-10T14:47:44Z

I am not sure whether we need -- or not. According to configure --help we need, but based on some of the messages written by configure is not needed. I would stick to the version suggested by configure --help.

I am also not completely sure about setting the LIBS and LD_FLAGS options as well, because this is not a standard configure script, actually it is a python script and it is quite picky how different options are set.

If we set LIBS and LD_FLAGS then they needed to set probably for the previous versions as well.

Anyway, from the config log I can see the following:

https://gist.github.com/hajgato/3b13d060d869f225868f

So, it looks that the libs and LD_FLAGS are correctly set.

JensTimmerman · 2014-09-11T14:43:20Z

alright, correct, seems to work here,
however, FIAT and scientificpython are not recognized in the configure step, so these should also be set somehow?

hajgato · 2014-09-12T08:05:40Z

@JensTimmerman Well, the options are there in the configure part. I think if it does not recognize FIAT and scientificpython , their lines would not be there. Anyway, here is the full buildlog:

https://gist.github.com/3fb8bb1125d44a7c0e1b

hajgato · 2014-09-12T08:30:48Z

From software/PETSc/3.5.1-intel-2014b-Python-2.7.8/conf/configure.log

TEST configureLibrary from config.packages.scientificpython(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/packages/scientificpython.py:39)
TESTING: configureLibrary from config.packages.scientificpython(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/packages/scientificpython.py:39)
  Find an installation ando check if it can work with PETSc
==================================================================================
Checking for a functional scientificpython
        Checking location User specified root directory SCIENTIFICPYTHON
        Added directory /apps/brussel/ivybridge/software/ScientificPython/2.8.1-intel-2014b-Python-2.7.8/lib/python/site-packages to Python path
================================================================================
TEST checkSharedLibrary from config.packages.scientificpython(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/package.py:566)
TESTING: checkSharedLibrary from config.packages.scientificpython(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/package.py:566)
  By default we don't care about checking if the library is shared
      Popping language C
      Pushing language C
================================================================================
TEST configureLibrary from config.packages.fiat(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/packages/fiat.py:35)
TESTING: configureLibrary from config.packages.fiat(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/packages/fiat.py:35)
  Find an installation ando check if it can work with PETSc
==================================================================================
Checking for a functional fiat
        Checking location User specified root directory FIAT
        Added directory /apps/brussel/ivybridge/software/FIAT/1.1-intel-2014b-Python-2.7.8/lib/python/site-packages to Python path
================================================================================
TEST checkSharedLibrary from config.packages.fiat(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/package.py:566)
TESTING: checkSharedLibrary from config.packages.fiat(/tmp/hajgato/build/PETSc/3.5.1/intel-2014b-Python-2.7.8/petsc-3.5.1/config/BuildSystem/config/package.py:566)
  By default we don't care about checking if the library is shared
      Popping language C
================================================================================

hajgato · 2014-09-12T08:36:25Z

Apparently FIAT and scientificpython in the PYTHONPATH

JensTimmerman · 2014-09-12T12:31:40Z

Yeah, I might have read the config output wrong, it is listed, but has nothing behind it, which was looking weird to me,
anyway, this worked for me :-)

boegel · 2014-10-09T11:38:44Z

@hajgato: correct me if I'm wrong, but this is just an unrolled version of the loop in the else part, but with BLACS excluded?

If so, we should rework this to:

deps = ["FFTW", "ScaLAPACK"] if LooseVersion(self.version) < LooseVersion("3.5"): deps.append("BLACS") # loop in else part over 'deps'

Balazs Hajgato added 3 commits July 30, 2014 10:07

PETSc 3.5.1 with intel 2014b

387627d

PETSc 3.5.1 with intel 2014b

c35bd87

Merge branch 'imkl_11.1.x_fftw_mpi_wrappers' of github.com:hajgato/ea…

0b76493

…sybuild-easyblocks into petsc_conifg_opts_changed

hajgato mentioned this pull request Jul 30, 2014

PETSc 3.5.1 with intel 2014b easybuilders/easybuild-easyconfigs#985

Merged

wpoely86 reviewed Jul 31, 2014
View reviewed changes

Balázs Hajgató added 2 commits July 31, 2014 15:05

Update petsc.py

c50e4cb

if suitesparse if loops corrected

Update petsc.py

227d2a2

def build_step removed, and -j issue for make solved using self.cfg['paeallel'] to None.

wpoely86 reviewed Jul 31, 2014
View reviewed changes

Update petsc.py

576778e

boegel mentioned this pull request Aug 4, 2014

petsc has changed some config parameters in version 3.5 #448

Closed

SCALAPACK_INC_DIR now is set for Intel toolchain

f79d3fb

JensTimmerman reviewed Sep 10, 2014
View reviewed changes

boegel reviewed Oct 9, 2014
View reviewed changes

Conversation

hajgato commented Jul 30, 2014

Uh oh!

hpcugentbot commented Jul 30, 2014

Uh oh!

wpoely86 Jul 31, 2014

Choose a reason for hiding this comment

Uh oh!

wpoely86 commented Jul 31, 2014

Uh oh!

wpoely86 Jul 31, 2014

Choose a reason for hiding this comment

Uh oh!

hajgato Jul 31, 2014

Choose a reason for hiding this comment

Uh oh!

wpoely86 Jul 31, 2014

Choose a reason for hiding this comment

Uh oh!

JensTimmerman Sep 10, 2014

Choose a reason for hiding this comment

Uh oh!

hajgato commented Sep 10, 2014

Uh oh!

JensTimmerman commented Sep 11, 2014

Uh oh!

hajgato commented Sep 12, 2014

Uh oh!

hajgato commented Sep 12, 2014

Uh oh!

hajgato commented Sep 12, 2014

Uh oh!

JensTimmerman commented Sep 12, 2014

Uh oh!

boegel Oct 9, 2014

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants