Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

falcON does not compile on MacOSX #105

Open
jcldc opened this issue Oct 14, 2022 · 4 comments
Open

falcON does not compile on MacOSX #105

jcldc opened this issue Oct 14, 2022 · 4 comments

Comments

@jcldc
Copy link
Collaborator

jcldc commented Oct 14, 2022

I fixed some compilation issues about falcON on macos platform (g++ and clang++), and removed again compilations warning.

The pb was coming from error and warning functions both declared in NEMO and clang/glibc and causing linking failures.

see falcON_compile branch

@teuben
Copy link
Owner

teuben commented Oct 24, 2022

I have a Catalina (10.15.x) running, but haven't seen any issues with gyrfalcON. Perhaps an older clang version.

@jcldc
Copy link
Collaborator Author

jcldc commented Oct 24, 2022

Actually gyrfalcON was compiling correctly before the modification, but not the ${NEMO}/usr/dehnen/falcON/src/public/acc manipulators. The reason is because warning and error functions are called from <defacc.h> and if you don't include <stdinc.h> before <defacc.h>, clang give an error message during the link, because warnings and error are also defined in clang library !!!!
When you install nemo using install_nemo script, gyrfalcON can be compiled but not these acc manipulators, at least from my macosx Sierra version.
Could you check that you have *.so files in ${NEMO}/usr/dehnen/falcON/acc, like this :

(base) mac-compile:acc grunner$ pwd
/Users/grunner/works/GIT/nemo/usr/dehnen/falcON/acc
(base) mac-compile:acc grunner$ ll
total 1440
drwxr-xr-x  18 grunner  staff   576B 17 oct 14:39 .
drwxr-xr-x  23 grunner  staff   736B 17 oct 14:39 ..
-rw-r--r--   1 grunner  staff     0B 17 oct 14:39 .dummy
-rwxr-xr-x   1 grunner  staff    65K 17 oct 14:39 Combined.so
-rwxr-xr-x   1 grunner  staff    23K 17 oct 14:39 Dehnen.so
-rwxr-xr-x   1 grunner  staff    24K 17 oct 14:39 DehnenMcLaughlin.so
-rwxr-xr-x   1 grunner  staff    95K 17 oct 14:39 DiscPot.so
-rwxr-xr-x   1 grunner  staff   105K 17 oct 14:39 GalPot.so
-rwxr-xr-x   1 grunner  staff    82K 17 oct 14:39 Halo.so
-rwxr-xr-x   1 grunner  staff    24K 17 oct 14:39 LogPot.so
-rwxr-xr-x   1 grunner  staff    20K 17 oct 14:39 MiyamotoNagai.so
-rwxr-xr-x   1 grunner  staff    75K 17 oct 14:39 Monopole.so
-rwxr-xr-x   1 grunner  staff    23K 17 oct 14:39 NFW.so
-rwxr-xr-x   1 grunner  staff    19K 17 oct 14:39 Plummer.so
-rwxr-xr-x   1 grunner  staff    23K 17 oct 14:39 Point.so
-rwxr-xr-x   1 grunner  staff    29K 17 oct 14:39 PotExp.so
-rwxr-xr-x   1 grunner  staff    71K 17 oct 14:39 Shrink.so
-rwxr-xr-x   1 grunner  staff    23K 17 oct 14:39 SoftKernel.so

@teuben
Copy link
Owner

teuben commented Oct 24, 2022

sounds like I need to add a regression test for this.

ugh, fish memory, the "make check" will do all the acc/manip tests of the dehnen code under "src/nbody/evolve/dehnen" entry.

@teuben
Copy link
Owner

teuben commented Oct 24, 2022

Indeed,my Catalina does not have them. They are formally copied in $NEMOOBJ/acc, which is also non-existent on my mac.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants