-
Notifications
You must be signed in to change notification settings - Fork 122
Use of protocol buffers #440
Comments
Currently there isn't any way to integrate the proto file generation into the build_runner build system. I chose to distribute the generated files to allow for users to more easily depend on this package without needing to install the protobuf compiler and generate them themselves out of band from the standard dart build. I still don't think there is a very good alternative. Thanks for pointing out the new version. I'll start the process to get the generated files updated. |
But why is it angular_component's responsibility to support protobuf at all? Wouldn't it make more sense to remove everything related to protobuf, and let whoever wants protobuf support add it on the outside? |
These components are an open source version of the material widgets used in applications at Google. Our intention for this project was to keep them in sync and avoid diverging the source. |
I'm having the same problem:
Would there be any workaround to use |
@supermuka feel free to use my fork referenced in PR (#443): https://github.com/s2nventures/angular_components |
This is pretty annoying. Could you split |
Update protobuf to 1.0.1 and regenerate and dartfmt .pb.dart files. Fixes angulardart#440 ("Once and for all!" "But..." "ONCE AND FOR ALL!")
There are issues with the build at the moment on the latest dev releases of the SDK. I left the PR open to signal that this is still an issue that will need to be resolved when the next sync does happen.
The project is still active development internally but the goal right now is to produce a more automated sync to github strategy that doesn't require as much manual intervention from engineers. |
I ran this into exact same problem for the last couple of days so a sync that would fix the version problem in the mean time would be appreciated. |
MaterialDatePicker uses protobuf to define a Date and a DateRange class. The protobuf dependency is currently locked to an older version, and prevents use of recent fixes in protobuf. Dependency overrides won't work, as the generated .pb.dart files fail to compile while depending on protobuf 0.14
Is there a rationale behind embedding conversion to protobuf formatted classes in angular_components? Looks to me like this shouldn't really be in angular_components at all?
The text was updated successfully, but these errors were encountered: