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

Build on macOS 15.1.1 broken - nanobind issue? #3284

Closed
cstricker opened this issue Dec 12, 2024 · 2 comments · Fixed by #3285
Closed

Build on macOS 15.1.1 broken - nanobind issue? #3284

cstricker opened this issue Dec 12, 2024 · 2 comments · Fixed by #3285
Labels

Comments

@cstricker
Copy link

Context

I have noticed for a couple of weeks that the build of neuron 9.0a breaks - quite early in the build process. At the moment, on none of my Ubuntu boxes, the build succeeds.

Overview of the issue

The issue is at the compile stage

Expected result/behavior

Should work...

NEURON setup

  • Version: neuron 9.0a - tested today after a fresh install
  • Installation method - cmake build
  • OS + Version: Ubuntu 24.10
  • Compiler + Version: clang++

Minimal working example - MWE

In file included from /home/stricker/.local/src/neuron/nrn/src/nrnpython/../neuron/container/mechanism_data.hpp:3:
/home/stricker/.local/src/neuron/nrn/src/nrnpython/../neuron/container/soa_container.hpp:885:77: error: a template argument list is expected after a name prefixed by the template keyword [-Wmissing-template-arg-list-after-template-kw]
885 | Callable tmp_callable = std::get<tag_index_v>(m_data).template for_each_vector(
| ^

Several errors come up.

Logs

CMakeBuild.txt

@cstricker cstricker added the bug label Dec 12, 2024
@alkino
Copy link
Member

alkino commented Dec 12, 2024

This is unrelated to nanobind. Seems more that you have a recent clang compiler with new warnings / errors.
Here is a fix #3285

Can you give us your version of clang?

@alkino alkino linked a pull request Dec 12, 2024 that will close this issue
@alkino
Copy link
Member

alkino commented Dec 12, 2024

In fact I found, this is a new valid error from clang-19.

JCGoran added a commit that referenced this issue Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants