search

xiph / awcy

http://arewecompressedyet.com/
MIT License
72 stars 46 forks source link

Update Dockerfile to build dav1d #187

Closed rzumer closed 4 years ago

rzumer commented 4 years ago

This adds support for decoding rav1e output with dav1d from https://github.com/xiph/rd_tool/pull/51.

tdaede commented 4 years ago

I'm now hitting permission errors because Cargo is trying to use /usr/local/cargo, which isn't writable. I see you've already 777'd it but that doesn't seem to be enough.

It may also make sense to not run as root inside the container....

tdaede commented 4 years ago

The new Ubuntu seems to break scp copying files back, due to this:

https://bugzilla.redhat.com/show_bug.cgi?id=1722229

rzumer commented 4 years ago

If I understand the problem correctly, that's supposed to be solved by https://github.com/xiph/awcy/commit/7ed853ac16c0d3e064f3b2eddb9f1dabfe1a4e00, if not then it needs to be updated I guess.

I also ran into it before updating the to disco, though. Maybe that's a separate issue that I didn't come across.

Either way the rd_tool change can function without this.

tdaede commented 4 years ago

I think it's an unrelated issue. The chmod 777 doesn't affect directories and files that are created later.

I think what we should really do is set $HOME/$CARGO_HOME/$RUSTUP_HOME correctly for the runtime user. I'll see if I can do that.

rzumer commented 4 years ago

I'm not sure what "chmod 777" you are referring to, I set +w to the registry directory for users in a different PR, is that what you are referring to?