Update compile.md instructions - `make install`

sandialabs / CACTUS

CACTUS (Code for Axial and Cross-flow TUrbine Simulation) is a turbine performance simulation code, based on a free wake vortex method, to study wind turbines and marine hydrokinetic (MHK) devices.

BSD 3-Clause "New" or "Revised" License

18 stars 12 forks source link

Update compile.md instructions - `make install` #23

Closed whophil closed 3 years ago

whophil commented 4 years ago

make install is required in order to get the cactus file to be moved to the bin/ folder.

kmruehl commented 4 years ago

@whophil this partially worked. cactus was moved to the bin/ directory, but it was stripped of its .exe extension and thus running PATH=$PATH:../../bin pytest runreg.py fails (because an executable is not found). However, running PATH=$PATH:../../build pytest runreg.py runs and all 3 tests passed.

whophil commented 4 years ago

Pushed a few changes which ensure that the .exe extension is kept. make install is no longer required, just running make should do the trick. Does this resolve your issue?

kmruehl commented 3 years ago

@whophil sorry it's taken my literal months to get back to you on this, but the updated instructions allow me to run the regression tests, and all three pass. Thanks! I'll merge this PR now.

kmruehl commented 3 years ago

@whophil I'm going to delete this branch now that it's merged