Make machine setup scripts work on MacOS

mabarnes / moment_kinetics

Other

2 stars 4 forks source link

Make machine setup scripts work on MacOS #184

Closed johnomotani closed 3 months ago

johnomotani commented 4 months ago

Replace realpath with Python in machine setup script. realpath is not necessarily portable, while we already require Python to be available for the Julia-downloader script. Using Python's os module should be pretty portable.

Fixes #183.

johnomotani commented 4 months ago

@mabarnes could you see if this PR fixes the issue in #183 please?

mabarnes commented 4 months ago

i've just pulled the latest source and still get the same error referencing realpath... :(

johnomotani commented 4 months ago

@mabarnes are you using the changes from this PR (which hasn't been merged into anything else yet)? The realpath command line shouldn't be being called at all!