ocaml / dune / 23339

7%

Build Type

push

github

Committed by web-flow

Commit Message

fix(pkg): fix downloads on windows (#9252)

opam invokes the cygwin version of curl which means passing % in a unix
shell is fine. We are however calling the Windows version of curl on
Windows which means that % is not a valid token in a command line
enviornment.

To get around this and make it more cross platform, we quote the
outputted error code on our invocation of curl in with double quotes.
This allows the command to work on Windows and all the platforms it
worked on before.

Signed-off-by: Ali Caglayan <alizter@gmail.com>

Run Details

4 of 29 new or added lines in 1 file covered. (13.79%)

3 existing lines in 1 file now uncovered.

2801 of 40083 relevant lines covered (6.99%)

28418.69 hits per line