Opened 12 months ago

Closed 12 months ago

Last modified 4 months ago

#67417 closed defect (fixed)

qt5-qtlocation @5.15.8_1 build failure

Reported by: dbl001 (dbl) Owned by: MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: Cc: mascguy (Christopher Nielsen), chrstphrchvz (Christopher Chavez)
Port: qt5-qtlocation

Description

:info:build include/mbgl/util/unique_any.hpp:51:31: error: no member named 'move' in namespace 'std'
:info:build             vtable->move(std::move(rhs.storage), storage);
:info:build                          ~~~~~^
:info:build include/mbgl/util/unique_any.hpp:69:25: error: no member named 'move' in namespace 'std'
:info:build         unique_any(std::move(rhs)).swap(*this);
:info:build                    ~~~~~^
:info:build include/mbgl/util/unique_any.hpp:91:33: error: no type named 'move' in namespace 'std' 
:info:build             unique_any tmp(std::move(rhs));
:info:build                            ~~~~~^
:info:build include/mbgl/util/unique_any.hpp:94:39: error: no member named 'move' in namespace 'std'
:info:build                 rhs.vtable->move(std::move(storage), rhs.storage);
:info:build                                  ~~~~~^
:info:build include/mbgl/util/unique_any.hpp:98:35: error: no member named 'move' in namespace 'std'
:info:build                 vtable->move(std::move(tmp.storage), storage);
:info:build                              ~~~~~^
:info:build include/mbgl/util/unique_any.hpp:154:41: error: no member named 'move' in namespace 'std'; did you mean simply 'move'?
@                                                                               

Attachments (1)

main.log (2.3 MB) - added by dbl001 (dbl) 12 months ago.

Change History (12)

Changed 12 months ago by dbl001 (dbl)

Attachment: main.log added

comment:1 Changed 12 months ago by jmroot (Joshua Root)

Owner: set to MarcusCalhoun-Lopez
Status: newassigned
Summary: Error: rev-upgrade failed: Error rebuilding qt5-qtlocationqt5-qtlocation @5.15.8_1 build failure

comment:2 Changed 12 months ago by mascguy (Christopher Nielsen)

Cc: mascguy added

comment:3 Changed 12 months ago by mascguy (Christopher Nielsen)

It looks (?) like the appropriate Cxx standard is being specified during compilation, so it's possible that upstream is simply missing #include <utility> somewhere.

comment:4 Changed 12 months ago by mascguy (Christopher Nielsen)

Based on a quick review of upstream's code, it looks like #include <utility> is only missing from include/mbgl/util/unique_any.hpp.

comment:6 Changed 12 months ago by MarcusCalhoun-Lopez (Marcus Calhoun-Lopez)

#include <typeinfo>
#include <type_traits>
#include <stdexcept>

int main() {
    int x = 5;
    int y = std::move(x);
}

env SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX12.sdk clang++ main.cxx compiles fine.
env SDKROOT=/Library/Developer/CommandLineTools/SDKs/MacOSX13.sdk clang++ main.cxx give the error reported here.

Looking through some of the system header files, it seems that Xcode has been removing #include <utility> from header files.

comment:7 Changed 12 months ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In 9f6653ca55c4d87b7a5a26368dbddc6192a4f86a/macports-ports (master):

qt5-qtlocation: mapbox-gl-native: patch unique_any.hpp, for std::move

  • Include for <utility> missing from: mbgl/util/unique_any.hpp

Fixes: #67417

comment:8 in reply to:  4 Changed 5 months ago by barracuda156

Replying to mascguy:

Not directly related to this issue, but maybe you know a fix for #68508 – otherwise depends do not build on Sonoma.

comment:9 Changed 4 months ago by chrstphrchvz (Christopher Chavez)

The same fix is still needed for qt59-qtlocation, qt511-qtlocation, and qt513-qtlocation: https://github.com/macports/macports-ports/pull/21982

comment:10 Changed 4 months ago by chrstphrchvz (Christopher Chavez)

Cc: chrstphrchvz added

comment:11 Changed 4 months ago by chrstphrchvz (Christopher Chavez)

In 0027bd56ac4a68df4309f77a270fdc86c0028602/macports-ports (master):

qt(59|511|513)-qtlocation: add missing header include

See: #67417

Note: See TracTickets for help on using tickets.