Can you take a look at $out/bin. There are scripts with broken shebangs like ' /bin/csh' and windows binaries inside and unrelated files like ./bin/uncrustify.cfg

@Mic92, thanks for the help and feedback. For now, I opted to just remove the offending files you mentioned. I hope this is a satisfying solution for now. I will probably need to return to this later in order to add support for MPI.

For the next iteration on this package it also might be interesting to make use of openblas as a replacement for both blas and lapack, which should probably give a good speedup compared to the reference implementation.

@jamtrott : thanks for this package; I'm currently looking at adding MPI support and have a few questions:

• is it ok for you if we add openmpi support by default or do you want it to be optional?
• any reason why using configureFlagsArray inside the preconfigure script and not using the configureFlags attribute?
• I can also try to make use of openblas as suggested by @knedlsepp if you agree

@bzizou, my pleasure.

• It is probably simpler to enable MPI by default, so I'm fine with that.
• I don't have a good answer to your second question. I seem to recall having some problems with using the configureFlags attribute, although I can't remember the specifics.
• If you can switch to openblas, then that's great.

I look forward to seeing your pull request!

@bzizou: I've learned that the conda-forge is always a good place to look for some ideas on how to best build various stuff: https://github.com/conda-forge/petsc-feedstock/blob/master/recipe/build.sh#L7
It seems they build it with:

• openblas
• suitesparse
• mpi

Feel free to have a look at: https://github.com/knedlsepp/nixpkgs/tree/extend-petsc
It compiles, but the python stuff installed in bin still needs correct shebangs.

My WIP is here: https://github.com/Gricad/nix-ciment-channel/tree/master/ciment/petsc
It compiles but tests are failing... :-(
I fixed the petscmpiexec script (shebang + incorrect sed separator)

Update: now working (with openblasCompat), updated link above

@bzizou: Got petsc compiling and passing the checkPhase. This was probably due to using openblas instead of openblasCompat.
This now should support openblas, openmpi, suitesparse, hwloc, scotch, fftw, netcdf, hdf5, superlu and metis.
Compared to homebrew, there is still room for improvement:

• parmetis
• scalapack
• hypre
• sundials
• homebrew builds both with-scalar-type=real and with-scalar-type=complex
• It seems most of the programs installed to bin are not really useful. Should they even be installed?

Yes, tests are passing with openblasCompat!! Ok, continuing :-)

For fftw, I think that we first need to add openmpi support into it...

@bzizou: I actually already did that: https://github.com/knedlsepp/nixpkgs/commits/extend-petsc

@knedlsepp : should'nt you push it upstream by making a PR?

Definitely! I was thinking we should do that in one chunk with a "petsc: Add missing features"-PR once we are satisfied with it.

Well, yes. Do you want to make this PR? My WIP is now very similar to your's and my time is over for this two days on this topic.
Here's what I currently have working: Gricad@ee78e6d

I think that you can simply add the fftw stuff and add my petscmpiexec patch (and no more remove it in postinstall)