oskaritimperi/nimtwirp

Twirp RPC framework for Nim

nimtwirp's Issues

grpc not mentioned in this repo; shouldn't it be?

• seems like this allows implementing grpc servers in nim; why not mention grpc in readme and package keyword/description?
  Or maybe I'm misunderstanding it

• grpc supports Bi-directional streaming (both for input and output); is that the case for nimtwirp? wasn't clear from readme

links

• see also: Could this be used for gRPC? · Issue #9 · oswjk/nimpb

nimble develop: Cannot develop packages that are binaries only

nimble develop
       Tip: 3 messages have been suppressed, use --verbose to show them.
     Error: Cannot develop packages that are binaries only.

is that really a binary only package?

nimtwirp_build.nim(40, 11) Error: 'nil' is now invalid for 'string'; compile with --nilseqs:on for a migration period; usage of '==' is a user-defined error

nimble install
Verifying dependencies for [email protected]
Warning: No nimblemeta.json file found in /Users/timothee/.nimble/pkgs/choosenim-0.3.2
Info: Dependency on nimpb@any version already satisfied
Verifying dependencies for [email protected]
Info: Dependency on nimpb_protoc@any version already satisfied
Verifying dependencies for [email protected]
Installing [email protected]
Building nimtwirp/nimtwirp/nimtwirp_build using c backend
Tip: 138 messages have been suppressed, use --verbose to show them.
Error: Build failed for package: nimtwirp
... Details:
... Execution failed with exit code 1
... Command: "/Users/timothee/.nimble//bin/nim" c --noBabelPath -d:release --path:"/Users/timothee/.nimble/pkgs/nimpb-0.2.0" --path:"/Users/timothee/.nimble/pkgs/nimpb_protoc-0.1.0" -o:"/Users/timothee/git_clone/nim/nimtwirp/nimtwirp/nimtwirp_build" "/Users/timothee/git_clone/nim/nimtwirp/nimtwirp/nimtwirp_build.nim"
... Output: Hint: used config file '/Users/timothee/git_clone/nim/Nim/config/nim.cfg' [Conf]
... Hint: used config file '/Users/timothee/.config/nim/nim.cfg' [Conf]
... Hint: used config file '/Users/timothee/.config/nim/config.nims' [Conf]
... /Users/timothee/.config/nim/timn/config.nims D20180822T214914
... /Users/timothee/git_clone/nim/nimtwirp/nimtwirp/nimtwirp_build.nim(5, 8) Warning: Use the 'parseopt' module instead; parseopt2 is deprecated [Deprecated]
... import parseopt2
... ^
... /Users/timothee/git_clone/nim/nimtwirp/nimtwirp/nimtwirp_build.nim(20, 1) template/generic instantiation from here
... /Users/timothee/git_clone/nim/nimtwirp/nimtwirp/nimtwirp_build.nim(40, 11) Error: 'nil' is now invalid for 'string'; compile with --nilseqs:on for a migration period; usage of '==' is a user-defined error
... if outdir == nil:

service_twirp.nim should have 1st line say: `# Generated by ... Do not edit!` to make clear it's auto-generated

• service_twirp.nim should have 1st line say: # Generated by ... Do not edit! to make clear it's auto-generated
• ideally all generated files (including binaries) should go under a build dir that is gitignored

`nimble test` should run examples (instead of using a makefile)