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.

Sign up for GitHub

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

Jump to bottom

gRPCAlchemy can't import generated proto files on Windows #34

Closed
tommasoturchi opened this issue Apr 30, 2020 · 1 comment
Closed

gRPCAlchemy can't import generated proto files on Windows #34

tommasoturchi opened this issue Apr 30, 2020 · 1 comment
Assignees
@GuangTianLi
Labels
enhancement New feature or request

Comments

@tommasoturchi
Copy link

tommasoturchi commented Apr 30, 2020 •
edited
Loading

  • gRPCAlchemy version: 0.5.1
  • Python version: 3.7.6
  • Operating System: Windows

Description

gRPCAlchemy can't import generated proto files on Windows, see this error:

ModuleNotFoundError: No module named 'protos\hellomessage_pb2'

I think this is due to https://github.com/GuangTianLi/grpcalchemy/blob/master/grpcalchemy/utils.py#L104 and https://github.com/GuangTianLi/grpcalchemy/blob/master/grpcalchemy/server.py#L119 replacing '/' with '.', since on windows the directory separator is '\', as you can see from the error message.

Hope this helps :-)
@tommasoturchi tommasoturchi changed the title gRPCAlchemy can't import the generated proto files on Windows gRPCAlchemy can't import generated proto files on Windows Apr 30, 2020
@GuangTianLi GuangTianLi self-assigned this Jun 12, 2020
@GuangTianLi GuangTianLi added the enhancement New feature or request label Jun 12, 2020
GuangTianLi added a commit that referenced this issue Jun 15, 2020
@GuangTianLi
🐛 fix [#34] compatibility in windows
47346c4
@GuangTianLi
Copy link
Owner

Fixed in 0.6.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@GuangTianLi GuangTianLi

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tommasoturchi @GuangTianLi