Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-103978

QtProtobuf: Optional fields need to be supported

    XMLWordPrintable

Details

    • Task
    • Resolution: Fixed
    • P2: Important
    • None
    • None
    • Protocol Buffers
    • None
    • 13
    • df8e7e1b8 (dev), 913aef61d (dev)
    • Foundation Sprint 70, Foundation Sprint 71, Foundation Sprint 72, Foundation Sprint 73, Foundation Sprint 74, Foundation Sprint 75, Foundation Sprint 76, Foundation Sprint 77, Foundation Sprint 78, Foundations Sprint 79, Foundations Sprint 80, Foundation Sprint 81, Foundation Sprint 82, Foundation Sprint 83, Foundation Sprint 84, Foundation Sprint 85, Foundation Sprint 86, Foundation Sprint 87, Foundation Sprint 88, Foundation Sprint 89, Foundation Sprint 90, Foundation Sprint 91, Foundation Sprint 92

    Description

      Currently trying to run qtprotobufgen on a .proto file with an optional field yields this

      Unable to execute
  D:/dev_tools/vcpkg/installed/x64-windows/tools/protobuf/protoc.exe:(1)
  basicmessages.proto: is a proto3 file that contains optional fields, but
  code generator protoc-gen-qtprotobufgen hasn't been updated to support
  optional fields in proto3.  Please ask the owner of this code generator to
  support proto3 optional

      To finish this task:

      • Implement in code generator
      • add tests for the generated interface
      • Test conformance

      The spec can be found here:
      https://github.com/protocolbuffers/protobuf/blob/v3.19.0/docs/field_presence.md

      Attachments

        Issue Links

          is required for

          User Story - Created by Jira Software - do not edit or delete. Issue type for a user story. QTBUG-109596 Qt GRPC Technology Preview scope in Qt 6.6

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              semlanik Alexey Edelev
              manordheim Mårten Nordheim
              Vladimir Minenko Vladimir Minenko
              Votes:
              1 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes