Re: [yt-dev] Forthon 0.8.13

Yup -- my mistake for not doing so already. On Wed, Jul 9, 2014 at 2:34 PM, Douglas Harvey Rudd wrote:

Are the Forthon build commands for fKD going to be stripped out of yt-3.0/setup.py at some point (they're still there as of c6b0920).

Douglas Rudd Scientific Computing Consultant Research Computing Center drudd@uchicago.edu

On Jul 9, 2014, at 1:32 PM, Matthew Turk wrote:

...

Hi Doug,

I haven't seen this, but it doesn't surprise me that there has been some migration. For yt-3.0, we don't need Forthon anymore, so it might not be worth digging in too hard.

-Matt

On Wed, Jul 9, 2014 at 11:44 AM, Douglas Harvey Rudd wrote:

...

Hi all,

Has anyone tried to build YT using a recent version of Forthon? I've been building using 0.8.10 on an older Python installation but when I use any of 3 installations using Forthon 0.8.13 (pip installed, 0.8.15 is current) it fails due to misunderstanding how to construct the module fortran file from the module name:

Traceback (most recent call last): File "/software/python-2.7-2014q2-el6-x86_64/bin/Forthon", line 2, in <module> import Forthon.Forthon_builder File "/software/python-2.7-2014q2-el6-x86_64/lib/python2.7/site-packages/Forthon/Forthon_builder.py", line 87, in <module> raise Exception('Main fortran file can not be found, please specify using the fortranfile option') Exception: Main fortran file can not be found, please specify using the fortranfile option

....

subprocess.CalledProcessError: Command '['/software/python-2.7-2014q2-el6-x86_64/bin/Forthon', '-F', 'gfortran', '--compile_first', 'fKD_source', '--no2underscores', '--fopt', "'-O3'", 'fKD', 'fKD_source.f90']' returned non-zero exit status 1

Looking through the differences in Forthon_builder.py between those two versions looks like fortranfile is set to a default in the older version: 0.8.10:34 fortranfile = options.fortranfile or (pkg + '.' + options.fixed_suffix) 0.8.13:34 fortranfile = options.fortranfile

Adding a --fortranfile option for fKD.f90 seems to work, but I'm not certain this is the correct solution. I'll submit a PR for the following change unless anyone else has a better idea (or reason it's a bad idea):

setup.py:63 cmd = [Forthon_exe, "-F", "gfortran", "--compile_first", "fKD_source", "--no2underscores", "--fopt", "'-O3'", "fKD", "fKD_source.f90"]

->

cmd = [Forthon_exe, "-F", "gfortran", "--compile_first", "fKD_source", "--no2underscores", "--fopt", "'-O3'", "-f", "fKD.f90", "fKD", "fKD_source.f90"]

Douglas Rudd Scientific Computing Consultant Research Computing Center drudd@uchicago.edu

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

---

yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org

_______________________________________________ yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org