search

didactic-drunk / sodium.cr

Crystal wrapper for the libsodium crypto API
MIT License
49 stars 13 forks source link

Docker bugfix #3. #6

Closed didactic-drunk closed 4 years ago

m-o-e commented 4 years ago

docker run -ti crystallang/crystal:latest /bin/bash -c -- "echo >shard.yml -e 'name: test\nversion: 0.0.1\ndependencies:\n sodium:\n github: didactic-drunk/sodium.cr\n commit: 633f9c1621c21bcc429625cbf074f98a84a932ca'; shards && echo >test.cr 'require \"sodium\"'; crystal test.cr"

Yup, that's an improvement. Doesn't fail shards or the build, but at least prints the wget error on top, which leads in the right direction.

didactic-drunk commented 4 years ago

Think I got it. Verify?

m-o-e commented 4 years ago

docker run -ti crystallang/crystal:latest /bin/bash -c -- "echo >shard.yml -e 'name: test\nversion: 0.0.1\ndependencies:\n sodium:\n github: didactic-drunk/sodium.cr\n commit: 57f6467'; shards && (echo >test.cr 'require \"sodium\"'; crystal test.cr)"

Noice! :+1:

Now it fails on shards exactly like it should. :)

m-o-e commented 4 years ago

docker run -ti crystallang/crystal:latest /bin/bash -c -- "apt update; apt install wget; echo >shard.yml -e 'name: test\nversion: 0.0.1\ndependencies:\n sodium:\n github: didactic-drunk/sodium.cr\n commit: 57f6467'; shards && (echo >test.cr 'require \"sodium\"'; crystal test.cr)"

PS: And works fine when wget is present. Also as it should. 👌