whisper.cpp icon indicating copy to clipboard operation
whisper.cpp copied to clipboard

Generalize install locations

Open trixirt opened this issue 1 year ago • 3 comments

On Fedora the install location is dependent on the arch ex/ for x86_64 libraries install to /usr/lib64

So copy how llama-cpp does its install, replacing 'llama' with 'whisper' in the cmake variables.

trixirt avatar Jan 20 '24 13:01 trixirt

The change seem to break Windows builds. Any hope to fix it?

petterreinholdtsen avatar Jun 16 '24 04:06 petterreinholdtsen

This patch is now in conflict with the latest master branch.

@ggerganov Is there any hope to get install rules included in the cmake setup? I guess Windows compatibility is required?

petterreinholdtsen avatar Feb 07 '25 18:02 petterreinholdtsen

Is there any hope to get install rules included in the cmake setup? I guess Windows compatibility is required?

@petterreinholdtsen I thought they are already included. If it does not work on Windows, please propose a patch to fix it.

ggerganov avatar Feb 08 '25 09:02 ggerganov