GitLab

Summary:
When building Kaldi submodule, it requires to run `get_version.sh`, so that version header is available.
It was pointed that the script should run with `bash`, instead of `sh`.

Fixes https://github.com/pytorch/audio/issues/2268

Pull Request resolved: https://github.com/pytorch/audio/pull/2269

Reviewed By: carolineechen

Differential Revision: D34667726

Pulled By: mthrok

fbshipit-source-id: 761b82c54b58af2bfb2836cbe18c9708f853f1e1

Summary:
cc mthrok

Pull Request resolved: https://github.com/pytorch/audio/pull/2116

Reviewed By: mthrok

Differential Revision: D33368453

Pulled By: jdsgomes

fbshipit-source-id: 09cf3fe5ed6f771c2f16505633c0e59b0c27453c

It's wrong to depend on `${TORCH_LIBRARIES}` as it pulls in explicit
`libcuda.so.1` dependency, which violates the assumption that GPU
accelerated libraries should be loadable with no NVIDIA drivers installed

Instead, make it depend on `torch` target, which includes all necessary
Torch C++ API dependences

When building torchaudio from source, `get_version.sh` from kaldi is executed everytime,
which results in kaldi-bindings to be always rebuilt.

This commit add "if" guard to the part so that they are not always executed.