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

Compilation errors #263

Open
julemai opened this issue May 1, 2024 · 6 comments
Open

Compilation errors #263

julemai opened this issue May 1, 2024 · 6 comments

Comments

@julemai
Copy link

julemai commented May 1, 2024

Hi there,

I cloned the Develop branch and try to compile it (using make) on MacOS/Linux. I am using mpicxx version 15.0.0. I am however getting error messages like:

In file included from streamnet.cpp:56:
./linklib.h:281:2: error: static assertion failed: MPI_Type_extent was removed in MPI-3.0.  Use MPI_Type_get_extent instead.
        MPI_Type_extent(MPI_LONG, &extent);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/opt/homebrew/Cellar/open-mpi/5.0.3/include/mpi.h:3183:31: note: expanded from macro 'MPI_Type_extent'
#define MPI_Type_extent(...)  THIS_FUNCTION_WAS_REMOVED_IN_MPI30(MPI_Type_extent, MPI_Type_get_extent)
                              ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/opt/homebrew/Cellar/open-mpi/5.0.3/include/mpi.h:361:59: note: expanded from macro 'THIS_FUNCTION_WAS_REMOVED_IN_MPI30'
#define THIS_FUNCTION_WAS_REMOVED_IN_MPI30(func, newfunc) static_assert(0, #func " was removed in MPI-3.0.  Use " #newfunc " instead.")
                                                          ^             ~

Would you want me to resolve these in a separate branch and create a pull request in case I am able to resolve them? Should I work off the Develop branch or the Master? Or would you like to look into this? Which compiler version are you using. I could also just try to downgrade to an older compiler.

Thanks,
Julie

@julemai
Copy link
Author

julemai commented May 1, 2024

Please see here the full output of the compilation (including all warnings and error messages).
TauDEM_compile_output.txt

@dtarb
Copy link
Owner

dtarb commented May 1, 2024

If you can resolve thse in a branch from Develop and make a pull request, that would be great. I use a PC with Windows, however have not worked on TauDEM for a while.

@julemai
Copy link
Author

julemai commented May 2, 2024

Sounds good. I will try if I get that running.

Could you give me write permissions? I currently can't create a branch.

@julemai
Copy link
Author

julemai commented May 2, 2024

Resolved all errors and some additional warnings. Just waiting to be able to publish the branch and get the pull request done. :)

@dtarb
Copy link
Owner

dtarb commented May 2, 2024

@julemai I'd prefer it if you cloned the repository and made a pull request against the code in your clone. See https://github.com/orgs/community/discussions/49745 (or elsewhere) for guidance on this.

@julemai
Copy link
Author

julemai commented May 3, 2024

I don't get that running. It just doesn't let me publish any branch that I create even following above.

I will just work with the local version I have now. I also saw that there is a pull request from 2022 that might solve the outdated MPI commands already.

I'm happy if you close this issue.

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