Opened 13 years ago
Closed 3 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 13 years ago by haianjxw@…
comment:2 Changed 11 years ago by ryandesign (Ryan Carsten Schmidt)
Cc: | pengyu.ut@… added |
---|
Has duplicate #40563.
comment:3 follow-up: 7 Changed 11 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 5 years ago by ryandesign (Ryan Carsten Schmidt)
comment:5 Changed 5 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 5 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 3 years ago by ryandesign (Ryan Carsten Schmidt)
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
Replying to ryandesign@…:
It works with: