atwhaley / cfast

Automatically exported from code.google.com/p/cfast
0 stars 2 forks source link

Three "simple" suggestions for Cfast on Linux #22

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Dear CFast Team,
I compiled the actual source code for cfast (not the GUI, just CFast) on 
Ubuntu 8.04 and it works. Here are three suggestions, which I think make 
it easier for other interested persons to do the same...

Maybe you can change this few "issues" at the source directory, I think 
they should have no influence for a windows system

1. Rename SIZES.fi to sizes.fi, because Linux is case sensitive. The file 
is included as sizes.fi in the SOURCE Code, so Linux doesn't find 
SIZES.fi. I renamed it by hand, then it works.

2. In file "output.f" there is a call "USE DFLIB" (subroutine 
deleteoutputfiles (outputfile), line 3014). I think, this call is not 
required, because there are only "standard calls" in this function. If you 
do not have a Visual Fortran compiler (or the dflib library), there is an 
error (I got one). I compiled succesfully without the "USE DFLIB" call, so 
I think it is not necessary for a succesful run and can be deleted in the 
code.

3. Compiler Directives in "input.f": Are these directives for actual PC's 
required? The Intel Compiler gave me some warnings, maybe they can 
removed???

Attached to this post you will find a makefile which based on the FDS5 
makefile. With this makefile I compiled the code with an Intel Fortran 
compiler on Ubuntu 8.04. Maybe you can put it into the source directory.

Best Regards from Germany
Christian Rogsch

Original issue reported on code.google.com by crog...@gmail.com on 26 May 2008 at 11:05

Attachments:

GoogleCodeExporter commented 9 years ago
1. sizes.fi (or SIZES.fi) is an empty file that contained various parameters in 
earlier versions of the model.  All references to sizes.fi have been removed 
from 
the model as of google code revision 36.

2. The use DFLIB should have been a use IFPORT for the INTEL Fortran complier.  
There are a number of references to the IFPORT library in CFAST.

3. The compiler directives are also a holdover from earlier versions.  We'll 
flag 
these for removal in a future version.  New issue #27 added to cover this.

Original comment by cfastdev@gmail.com on 5 Jun 2008 at 5:43