Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

Old descriptor.proto #17

Open
rfevang opened this issue Jan 25, 2017 · 3 comments
Open

Old descriptor.proto #17

rfevang opened this issue Jan 25, 2017 · 3 comments

Comments

@rfevang
Copy link

rfevang commented Jan 25, 2017

The descriptor.proto bundled is pretty old, and lacks options used in the built-in proto3 types (cc_enable_arenas, csharp_namespace, go_package and objc_class_prefix). The result is an annoying list of errors found if one uses any of these protos. Any chance of updating this file?

As a side note, I can't figure out how to specify my own descriptor.proto file. It's not clear what the options field is relative to (the default just says google/protobuf/descriptor.proto, where is that google folder located?). Nothing I've done seems to make a difference, whether I put in an absolute path, one relative to my home directory or absolute gibberish. It still figures out that java_package is a valid option, but not that go_package is one.

@elchtzeasar
Copy link

I am also having issues with this. I have made sure to add the path to "my own" google/protobuf/descriptor.proto in project settings -> Protocol Buffer -> Import Paths, which does not work. Specifying an absolute path does not help either.

@stephenh
Copy link

I also added an import path, and see it being picked up in the xtext output:

[Worker-12] indexing platform:/resource/mirror/build/extracted-include-protos/main/google/protobuf/descriptor.proto

But it's also picking up the packaged descriptor, and seems to be preferring that:

[Worker-12] indexing platform:/resource/com.google.protobuf/google/protobuf/descriptor.proto

E.g. I still have errors due to go_package, cc_enable_arenas, etc.

At this point I think I'll have to uninstall the protobuf-dt plugin, as I don't see anyway to resolve/ignore these errors.

@DoneSpeak
Copy link

image

I have this error as well. It upset me very much.

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

No branches or pull requests

4 participants