search

rust-av / av-metrics

Quality metrics
MIT License
53 stars 10 forks source link

Failed to install when fetching from Cargo #266

Open vibhoothi opened 3 years ago

vibhoothi commented 3 years ago

When we try to install av-metrics with the help of the command cargo install av-metrics-tool it fails to install, error log

/home/mindfreeze/.cargo/registry/src/github.com-1ecc6299db9ec823/av-metrics
-tool-0.7.0/src/main.rs:359:34
    |
359 |     fn calculate_video_metric<D: Decoder>(
    |                      ^^^^^^^ not found in this scope
    |
help: consider importing one of these items
    |
1   | use av_metrics::video::decode::Decoder;
    |
1   | use crate::decode::Decoder;
    |

error[E0405]: cannot find trait `Decoder` in this scope

Full Error Log: https://paste.debian.net/plain/1185456

When we clone and build it works fine.

At the same time when we try to install like cargo install av-metrics alone, it does not work as mentioned in the readme, it is advised to have av-metrics-tool. So if we fix the confusion of having two av-metrics when we try to fix it will be better. error: specified packageav-metrics v0.6.2has no binaries

lu-zero commented 3 years ago

We should fix the av-metrics-tool version requirements and publish an update. Potentially we could rework the crates differently but that can be done later.