Update diesel to 0.11. - Githubissues

jimmycuadra commented 7 years ago

This also updates the ruma-dev Docker image, because this new version of Diesel requires clang. (See https://github.com/ruma/docker-ruma-dev/commit/90e4968e41a4dd59c270481a40e4df6b0bb634bb)

For some reason, these changes caused script/cargo test to fail with:

     Running target/debug/deps/ruma-4252fc14c8afc8b3
/source/target/debug/deps/ruma-4252fc14c8afc8b3: error while loading shared libraries: libsodium.so.18: cannot open shared object file: No such file or directory
error: test failed

unless I explicitly set LD_LIBRARY_PATH to /usr/local/lib. I'm not too happy about that, so I'm hoping someone can figure out why that is suddenly necessary when it wasn't before. I checked the shell environment in the older version of the Docker image, and LD_LIBRARY_PATH is not set there either, so the problem is not that it's missing in the new image.

mujx commented 7 years ago

Maybe you need to run ldconfig after the libsodium compilation in the Dockerfile?

jimmycuadra commented 7 years ago

That did the trick! Thanks!

ruma / homeserver

Update diesel to 0.11. #170