Opened 3 years ago
Last modified 3 weeks ago
#62459 assigned defect
wxMaxima @21.02.0: error: unknown type name 'nullptr_t'; did you mean 'std::nullptr_t'?
| Reported by: | dyne2meter | Owned by: | MSoegtropIMC |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.6.4 |
| Keywords: | elcapitan | Cc: | cooljeanius (Eric Gallager) |
| Port: | wxMaxima |
Description
Build on legacy system (MacOS 10.11.6); main log attached
Attachments (1)
Change History (6)
Changed 3 years ago by dyne2meter
| Attachment: | main.log.gz added |
|---|
comment:1 Changed 3 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | GitHub: MSoegtropIMC removed |
|---|---|
| Keywords: | elcapitan added; nullptr_t Autocomplete_Popup.cpp removed |
| Owner: | set to MSoegtropIMC |
| Status: | new → assigned |
| Summary: | unknown type name 'nullptr_t' error building wxMaxima → wxMaxima @21.02.0: error: unknown type name 'nullptr_t'; did you mean 'std::nullptr_t'? |
comment:2 Changed 3 years ago by jmroot (Joshua Root)
This code is likely trying to get nullptr_t in the global namespace by including <stddef.h>. Firstly, nullptr_t is a C++11 feature, and if a C++11 compiler is needed to build this port, it should indicate it by setting compiler.cxx_standard 2011. Secondly, <stddef.h> is a C compatibility header, and using those in C++ is deprecated. So while the headers on El Capitan may be buggy in that <stddef.h> doesn't provide all the functionality it should, it's more future-proof to include <cstddef> instead and use std::nullptr_t, which should neatly fix the El Capitan issue as well.
comment:3 Changed 3 years ago by MSoegtropIMC
I reported this upstream (https://github.com/wxMaxima-developers/wxmaxima/issues/1523) and asked if they want to change their code to use std::bullptr_t and <cstddef> or force C++11. As far as I understood @jmroot, both options would work.
comment:4 Changed 14 months ago by MSoegtropIMC
According to upstream developers and since there have no further reports for 2 years, I guess this can be closed.
To whom it may concern: please close this ticket
comment:5 Changed 3 weeks ago by cooljeanius (Eric Gallager)
| Cc: | cooljeanius added |
|---|
main.log