parmetis 4.0.2 #13733

Closed
wants to merge 1 commit into
from

3 participants

@wdeconinck

Changes:

  • Updates parmetis to 4.0.2
  • Add mpi dependency

Issue with workaround:

When mpi is compiled with llvm-gcc, and this brew install is NOT explicitly set with "--use-llvm",
the cflag "-march=native" from "clang" fails the CMake configuration,
saying that CMAKE_C_COMPILER was not able to compile a C program.

When this cflag is removed, everything works fine.

@jacknagel

Squash to a single commit, also there is still an extra new blank line at the end of the file.

@Sharpie Sharpie commented on the diff Aug 28, 2012
Library/Formula/parmetis.rb
def install
- system "make", "config", "prefix=#{prefix}"
- system 'make install'
+ # Install Metis
+ Dir.chdir 'metis' do
+ # Wrapper around a CMake configuration
+ system "make" , "config" , "shared=1" , "prefix=#{prefix}"
+ system 'make install'
@Sharpie
Sharpie added a line comment Aug 28, 2012

Hmmm... does this actually install Metis? If so, this creates a nasty clash with the metis formula that we should take care of.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@wdeconinck

Yes, it does install metis. At the time I created this, a metis formula did not exist yet.

I see 2 options:

  • two separate formula's
  • metis formula with "--parallel" option (which then depends on mpi and installs parmetis)

Same should then be applied to the scotch and pt-scotch formulae.

I also noticed that in the mean time the metis formula is already at version 5.x, while parmetis is still at 4.0.2

@adamv

Pulled an alternate update for this

@adamv adamv closed this Oct 10, 2012
@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.