Skip to content
This repository has been archived by the owner on Oct 28, 2022. It is now read-only.

Deprecate scripts/process_schemas #821

Open
david4096 opened this issue Feb 21, 2017 · 3 comments
Open

Deprecate scripts/process_schemas #821

david4096 opened this issue Feb 21, 2017 · 3 comments

Comments

@david4096
Copy link
Member

We had recently worked with having process schemas (which generates python bindings from the proto) as part of setup.py script. This allowed protoc to generate bindings for the proto as part of normal build processes without invoking another script.

In order to make our package more portable, we opted to include the bindings in the source tree. I would like to deprecate the call to the script directly and reinstitute process_schemas as part of the setup.py process and document this pattern.

@dcolligan
Copy link
Member

Didn't we move to the current arrangement of checking in the pb2 files because we couldn't guarantee that protoc was installed on the target machine? Has something changed to obviate that concern?

@david4096
Copy link
Member Author

Hi @dcolligan! Yep! And we can still check in the pb2, I just would like to not have to have process_schemas as a separate script to call but document the call to python setup.py develop. This also generates the version py, which I recently got stuck looking around for since it is in the .gitignore.

We can close this issue by clearly stating the requirements for building the proto using python, and demonstrating the development flow of running python setup.py develop after modifying the schemas.

@david4096
Copy link
Member Author

See solution in #828

We can close when it is clear just how and when this code path should be hit.

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

No branches or pull requests

2 participants