Non-Lite Protos

[dhulihan]

Hi, I'm trying to include a phonenumbers.PhoneNumber proto in another proto file (a v3 file that is not using LITE_RUNTIME) like so:

// phone_number.proto
syntax = "proto3";

package phoneproto;

option go_package="github.com/foo/bar/phoneproto";

import "github.com/nyaruka/phonenumbers/phonenumber.proto";

message PhoneNumberRequest {
    phonenumbers.PhoneNumber phone_number = 1;
}

but i'm getting an error when compiling to go:

phone_number.proto: Files that do not use optimize_for = LITE_RUNTIME cannot import files which do use this option.  This file is not lite, but it imports "github.com/nyaruka/phonenumbers/phonenumber.proto" which is.

I really like that protos are available, but it would be nice to be able to reuse your proto files that don't use LITE_RUNTIME.

[dhulihan]

Ideas for addressing this:

• provide a non-lite file in a different protobuf package, eg: in protobuf/heavy/phonenumber.proto.
• Figure out some way to ignore option optimize_for = LITE_RUNTIME;. I haven't found a way to do this yet.

[dhulihan]

It looks like this directive is no longer supported.

In Proto3 syntax files, "optimized_for=LITE_RUNTIME" is no longer supported.

https://github.com/protocolbuffers/protobuf/blob/master/CHANGES.txt#L1826
https://github.com/protocolbuffers/protobuf/blob/v3.0.0-beta-3/CHANGES.txt#L130