#59321 closed defect (fixed)
qt56-qtscript @5.6.3_2: : error: unknown type name 'This'
Reported by: | kencu (Ken) | Owned by: | MarcusCalhoun-Lopez (Marcus Calhoun-Lopez) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | tehcog (tehcog), chrstphrchvz (Christopher Chavez) | |
Port: | qt56-qtscript |
Description
This port defaults to clang-9.0 for building on 10.7, but this is a failure, in the end (log attached). The first error is as above, but a litany of errors follow.
The build succeeds with clang-3.7, but not clang-5.0.
qt56-qtscript @5.6.3_2 (active) platform='darwin 11' archs='x86_64' date='2019-10-13T13:06:49-0700'
Attachments (1)
Change History (9)
Changed 5 years ago by kencu (Ken)
Attachment: | qt56-qtscript-10.7-fail-clang-9.0.log added |
---|
comment:1 Changed 5 years ago by kencu (Ken)
comment:2 Changed 5 years ago by tehcog (tehcog)
Cc: | tehcog added |
---|
comment:3 Changed 5 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
C++20 will likely have a header file called version. Qt Script has a file called VERSION. On a case-insensitive file systems, the wrong file will be found.
comment:4 Changed 5 years ago by kencu (Ken)
Indeed. I build most of these older ports with clang-5.0 (which I hope we never get rid of), and don't see how we could realistically spend the time to upgrade all these older ports to build with clang-9.0. I don't plan to do that, myself.
This one would only build with clang-3.7 however, on the day I tried it.
Perhaps someone on the wild wide web has found the time to make it build with new, different, stricter compilers.
comment:5 Changed 5 years ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:6 Changed 11 months ago by chrstphrchvz (Christopher Chavez)
This issue was addressed upstream only since 5.12: https://codereview.qt-project.org/c/qt/qtscript/+/254871
So the fix is still needed for qt59-qtscript and qt511-qtscript: https://github.com/macports/macports-ports/pull/21972
comment:7 Changed 11 months ago by chrstphrchvz (Christopher Chavez)
Cc: | chrstphrchvz added |
---|
Has "duplicate", although with qt58 59325