Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error compiling for ARM GCC-5 #1157

Closed
jairoareyes opened this issue Mar 17, 2023 · 4 comments
Closed

Error compiling for ARM GCC-5 #1157

jairoareyes opened this issue Mar 17, 2023 · 4 comments

Comments

@jairoareyes
Copy link

Hi, firstly, thanks for your work.

I'm having an issue when trying to compile the library for a device which uses the compiler arm-linux-gnueabihf-gcc-5.

The device is an embedded system manufactured by FriendlyCore, here you can find more information about the device. In sum, this is a device that has an OS based on Ubuntu 16.04.

I understand that this library uses C++14, and as far as I know the GCC-5 compiler supports C++14, so in theory should not be any problem compiling this. However, when compiling, it shows an error related to strings:

sqlite_orm.h:19752:96: error: binding ‘const string {aka const std::__cxx11::basic_string<char>}’ to reference of type ‘std::__cxx11::basic_string<char>&’ discards qualifiers
                               table.for_each_column([this, colInfo, &tableName = table.name, db](auto& c
                                                                                              ^

I'm working with the version of this commit:

10d843ed (HEAD) appveyor: Updated description for clang C++20 environment

If you need the sysroot or toolchain just let me know, everything is publish on the manufacturer website.
Thanks in advance.

@fnc12
Copy link
Owner

fnc12 commented Mar 17, 2023

Hi. Thank you for your feedback!

The error looks ridiculous cause &tableName = table.name binding must work as is. Looks like a GCC issue

@trueqbit
Copy link
Collaborator

trueqbit commented Mar 17, 2023

It's a bug fixed in gcc 7, see #1145 (comment)

@fnc12
Copy link
Owner

fnc12 commented Oct 8, 2023

@jairoareyes looks like it is not an issue anymore. We'd like to close it. Can we do something else related to it?

@fnc12
Copy link
Owner

fnc12 commented Oct 13, 2023

closing due to inactivity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants