Closed GoogleCodeExporter closed 9 years ago
GoogleCodeExporter
commented
9 years ago Please disregard/close this one. It was me missing a few lines I needed to
adjust
at the top of the makefile. Sorry!Original comment by jamie.l...@gmail.com on 4 Aug 2009 at 2:29
GoogleCodeExporter
commented
9 years ago No problem -- are you saying that the makefile that is maintained in the FDS
Repository is missing something, or is it OK?Original comment by mcgra...@gmail.com on 4 Aug 2009 at 2:39
GoogleCodeExporter
commented
9 years ago The makefile in the repository is fine. I use a slighty different one for
my OS and had neglected to update a few key lines that are correct in the
repository.Original comment by jamie.l...@gmail.com on 4 Aug 2009 at 3:09
GoogleCodeExporter
commented
9 years ago Should we add the lines to the makefile that is on-line? Our goal is to have a
makefile that works on as many different platforms as possible, so that we can
better control all the various compiler options that make one version of FDS
different from another.Original comment by mcgra...@gmail.com on 4 Aug 2009 at 3:12
GoogleCodeExporter
commented
9 years ago That would be great. The only addition I have is for serial compile of Gnu
Linux 64:
gnu_linux_64 : FFLAGS = -O3 -unroll -static -frecord-marker=4
gnu_linux_64 : CFLAGS = -O3 -unroll -static
gnu_linux_64 : FCOMPL = mpif90
gnu_linux_64 : CCOMPL = mpicc
gnu_linux_64 : obj = fds5_serial_linux_64
gnu_linux_64 : setup $(obj_serial)
$(FCOMPL) $(FFLAGS) -o $(obj) $(obj_serial)Original comment by jamie.l...@gmail.com on 4 Aug 2009 at 3:26
GoogleCodeExporter
commented
9 years ago Why do you use mpif90 and mpicc for a serial compile?Original comment by mcgra...@gmail.com on 4 Aug 2009 at 4:04
GoogleCodeExporter
commented
9 years ago I hadn't noticed...that is probably the result of a copy and paste that somehow
worked. I suspect I would be better off with gfortran and gcc instead. I will
try
to change to that and see what happens, then post the result here. It may be a
day
or two before I can give it a shot.Original comment by jamie.l...@gmail.com on 4 Aug 2009 at 4:16
GoogleCodeExporter
commented
9 years ago Take a look at what we currently have listed for the gnu compilers. I'd like to
make
these as consistent as possible. Sometimes we get compiler options that are so
specific to a particular machine/OS that the options work for no one else.Original comment by mcgra...@gmail.com on 4 Aug 2009 at 4:48
GoogleCodeExporter
commented
9 years ago The following code works. It is also consistent with the current listing for
the
mpi version of gnu_linux_64 as well as for the serial version of gnu_linux_32.
gnu_linux_64 : FFLAGS = -O3 -unroll -static -frecord-marker=4
gnu_linux_64 : CFLAGS = -O3 -unroll -static
gnu_linux_64 : FCOMPL = gfortran
gnu_linux_64 : CCOMPL = gcc
gnu_linux_64 : obj = fds5_serial_linux_64
gnu_linux_64 : setup $(obj_serial)
$(FCOMPL) $(FFLAGS) -o $(obj) $(obj_serial)
Original comment by jamie.l...@gmail.com on 4 Aug 2009 at 4:58
GoogleCodeExporter
commented
9 years ago I just committed the change. Note that I call the executable fds5_gnu_linux_64
to
conform to our standard practice
fds5_compiler_OS_bit
ThanksOriginal comment by mcgra...@gmail.com on 4 Aug 2009 at 5:06
GoogleCodeExporter
commented
9 years ago Original comment by mcgra...@gmail.com on 23 Nov 2009 at 9:44
Original issue reported on code.google.com by
jamie.l...@gmail.comon 4 Aug 2009 at 2:21