Opened 14 years ago
Closed 4 years ago
#34517 closed defect (fixed)
stardict: variable length array of non-POD element type
| Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | alexandernx@… |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | 2.1.1 |
| Keywords: | clang | Cc: | pengyu.ut@… |
| Port: | stardict |
Description
stardict 3.0.2 fails to build on the Lion buildbot with the error:
t_lookupdata.cpp:20:30: error: variable length array of non-POD element type 'std::vector<gchar *>'
std::vector<gchar *> reslist[dictmask.size()];
^
1 error generated.
make[2]: *** [t_lookupdata.o] Error 1
Maybe updating to 3.0.3 will help, maybe not.
Change History (8)
comment:1 Changed 14 years ago by haianjxw@…
comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | pengyu.ut@… added |
|---|
Has duplicate #40563.
comment:3 follow-up: 7 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
| Keywords: | clang added; lion removed |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
3.0.4 still has the problem so I filed an upstream bug report and blacklisted clang in r111503.
comment:4 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
comment:5 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
The original problem is still happening on older OS versions:
But I didn't see the problem when building on High Sierra. Maybe this is a feature introduced in newer versions of clang, and we need to blacklist older ones.
comment:7 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)
Replying to ryandesign:
3.0.4 still has the problem so I filed an upstream bug report
https://code.google.com/archive/p/stardict-3/issues/58
which was auto imported to:
https://github.com/huzheng001/stardict-3/issues/58
Upstream says they fixed it in 2013, but their fix appears to have been ineffective. I've added a note to the GitHub issue.
comment:8 Changed 4 years ago by ryandesign (Ryan Carsten Schmidt)
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |

Replying to ryandesign@…:
It works with: