Reliance on non-standard location parameters

[dvzrv]

Hi! 👋
I am currently trying to package this library for Arch Linux as it is now an implied requirement for ardour.

I noticed that in

LibAAF/CMakeLists.txt

Lines 62 to 64 in 3e4c2cd

	set( BIN_INSTALL_PATH "/usr/local/bin/" )
	set( LIB_INSTALL_PATH "/usr/local/lib/" )
	set( HEADERS_INSTALL_PATH "/usr/local/include/" )

you are hardcoding non-standard location parameters.

This is very bad for downstream packaging as one can not override these locations without patching the CMakeLists.txt.

Please instead rely on GNUInstallDirs, which is a builtin facility of cmake. Reinventing the wheel here will unfortunately be very painful for anyone else otherwise.

All that downstreams should have to pass in to cmake when it comes to defining install location is -D CMAKE_INSTALL_PREFIX=/usr to modify the prefix used for the target location.

I can try and have a look whether I can provide a patch for this.

[agfline]

Hi,
Never used cmake before, so any help is appreciated ;)
I will fix that soon, thanks for pointing that out and for the explanations.