-
Notifications
You must be signed in to change notification settings - Fork 143
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
"mix test" doesn't work on Windows #281
Labels
Comments
@wingyplus I don't have a chance to test this since I don't have a Windows machine. Are you willing to give figuring this out a try? 🙃 |
whatyouhide
changed the title
mix test doesn't work on Windows
"mix test" doesn't work on Windows
Jun 25, 2022
Just a little investigation. Inspecting diff --git a/lib/protobuf/decoder.ex b/lib/protobuf/decoder.ex
index 2fa107e..2febe12 100644
--- a/lib/protobuf/decoder.ex
+++ b/lib/protobuf/decoder.ex
@@ -42,6 +42,9 @@ defmodule Protobuf.Decoder do
field_number = bsr(value, 3)
wire_type = band(value, 0b00000111)
+ IO.inspect(:stderr, byte_size(rest), [])
+ IO.inspect(:stderr, rest, binaries: :as_binaries)
+
if field_number != 0 do
handle_field(rest, field_number, wire_type, message, props)
else This is the result when I'm running on Windows: |
I'm encountering the same issue and having a hard time tracking down where the issue could be during decoding.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
If I run
mix test
on Windows, the project will raise error:After I investigate, I found that
mix escript.build
generate only unix shell script that run escript binary but not for Windows shell script. I try temporary by createprotoc-gen-elixir.bat
:NOTE: I copied this script from
mix escript.install
.And modify
mix.exs
with:After run
mix test
will change to:The text was updated successfully, but these errors were encountered: