-
Notifications
You must be signed in to change notification settings - Fork 38
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
Optional fields not supported #244
Comments
Just ran into this, any progress? |
Sorry, all activities are temporary frozen, because of the migration to Qt6. I will try to cover this ASAP. :( |
Hello, have also encountered the problem described here and would like to add the following information. From version 3.12 and up to version 3.15 protoc still needs the addition of --experimental_allow_proto3_optional, which is unfortunately not supported with Qt6 using qt_add_protobuf.
A workaround here is to edit the file lib/cmake/Qt6ProtobufTools/Qt6ProtobufToolsMacros.cmake
Following this, one comes very close to the problem described here, since the error message that now arises is identical.
More information about supporting optional can be found in the protobuf repository and I would start by implementing the method King regards |
Hi, @glaubnichtalles the 'optional' fields support is planned for Qt6: QTBUG-103978 Unfortunately I don't plan any backporting from Qt6 to this repo because of complexity. |
Hi, I can't open https://bugreports.qt.io/browse/QTBUG-103978 even after login. The problem does exist in Qt 6.5.1 and Qt 6.6.0 (beta). |
Hi, thanks for the development regarding "Optional fields" support. My latest tests with Qt 6.6.0 beta 3 show that there are problems with the use of reserved c++ keywords. It would be great if for example the following entries could be added to the tests for the proto files:
Kind reagrds |
Optional fields aren't supported, resulting in this error when you try to use a file with optional fields:
The text was updated successfully, but these errors were encountered: