The dev/cmp bench command panics

[nico-abram]

After patching the dead bitbucket link for thingy-floor0.ogg to http://achurch.org/hg/libnogg/raw-file/tip/tests/data/thingy-floor0.ogg in /dev/cmp/src/lib.rs, it panics within vorbis-rs when running cargo run --release -- bench:

thread 'main' panicked at 'attempted to zero-initialize type `vorbisfile_sys::ov_callbacks`, which is invalid', C:\Users\Nick\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib/rustlib/src/rust\library\core\src\mem\mod.rs:623:9

I opened tomaka/vorbis-rs#19, and opened a PR into vorbisfile-sys. If that gets merged, then a simple change in vorbis-rs should fix this specific panic, however the master branch for vorbis-rs also has other issues (I was also unable to compile it on windows, ran into a missing pthread.h C compiler error, but that could be because of a mistake on my part) , and I'm not entirely sure if the library is still maintained (Even if vorbis-rs is still maintained the link error also involves vorbis-encoder which looks unmaintained to me)

I'm not sure what a good path forward would be

[est31]

vorbisfile PR: tomaka/vorbisfile-sys#2

[est31]

@nico-abram regarding the build error, it's a bit weird. The CI for lewton also runs on windows.

[nico-abram]

I opened an issue for it here tomaka/vorbisfile-sys#3

lewton and /dev/cmp both compile fine for me on windows, the problem was the vorbisfile-sys master branch (I think there were probably changes in it's build script since the version from crates io that /dev/cmp is using)

[nico-abram]

It seems the windows issue was recently fixed but for vorbisfile-sys to work it needs a vorbis-rs release to point to

[est31]

There is PR with a fix: tomaka/vorbis-rs#20

As it's not merged yet, I've made f655fa2 to make the CI apply the PR as a workaround.