Skip to content
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

Protobuf library should upgrade to protobuf 3.3 #2609

Closed
reyoung opened this issue Jun 26, 2017 · 5 comments
Closed

Protobuf library should upgrade to protobuf 3.3 #2609

reyoung opened this issue Jun 26, 2017 · 5 comments

Comments

@reyoung
Copy link
Collaborator

reyoung commented Jun 26, 2017

The current release of protobuf is 3.3.2, but Paddle is using protobuf 3.1.0. In protobuf 3.1.0, there are several bugs which affect our code:

@reyoung
Copy link
Collaborator Author

reyoung commented Jun 26, 2017

@wangkuiyi @gangliao Should we upgrade our protobuf library? If we should, I could give a PR tomorrow.

@gangliao
Copy link
Contributor

@Xreki I don't have the environment to build the protobuf 3.3.2 on ARM, @Xreki Can you please build this version? Is it feasible to upgrade protobuf?

@Xreki
Copy link
Contributor

Xreki commented Jun 27, 2017

I have tried to build the newest protobuf before and met some problem. In the building process of the newest version, there is a need to generate a binary file and run the binary file.
In cmake/libprotoc.cmake

add_executable(js_embed ${protobuf_source_dir}/src/google/protobuf/compiler/js/embed.cc)
add_custom_command(
  OUTPUT ${protobuf_source_dir}/src/google/protobuf/compiler/js/well_known_types_embed.cc
  DEPENDS js_embed ${js_well_known_types_sources}
  COMMAND js_embed ${js_well_known_types_sources} > ${protobuf_source_dir}/src/google/protobuf/compiler/js/well_known_types_embed.cc
)

Simple command make was used, thus all the targets of Protobuf will be made. I will try another detailed command like make libprotobuf to not build protoc.

@reyoung
Copy link
Collaborator Author

reyoung commented Jun 27, 2017

This issue is blocked by Protobuf's cmake. We cannot upgrade our protobuf from 3.1 to 3.3 because, in 3.2, cmake cannot build a cross-compile protobuf.

The PRs are added and not merged.

@qingqing01
Copy link
Contributor

The protocolbuffers/protobuf#3878 fixed cross-compiling for android.

@reyoung reyoung closed this as completed Feb 7, 2018
heavengate pushed a commit to heavengate/Paddle that referenced this issue Aug 16, 2021
* add ttfnet enhance

* add doc

* fix pafnet training
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants