Amended to fix s/xzing/zxing/ typo.

Thanks!

Just noticed, looking at a test-run of make install, that my zxing.pc does not look like I expected:

prefix=/home/axel/contrib/zxing-cpp.install
exec_prefix=${prefix}
libdir=
includedir=

Ideas?

Oh, sorry, I should have tested before submitting. I think we need to update the cmake code that generates the final .pc file too. Will submit fix later today.

@axxel: Fix here: #469

(This time I did a build to verify the change.)

Thanks. Your fix works but I don't understand why it is required because a) the formerly used non-FULL versions should be defined iff the FULL versions are and b) the fix that has been used and working on the NixOS side was exactly what your original PR did.

Anyway, I reorganized the .pc file and CMakeConfig generation and this takes also care of the missing GNUInstallDirs include. That said, I'd appreciate if you could test the code especially with respect to those changes. I don't have a setup that properly tests the validity of the generated .pc file.

Thanks. Your fix works but I don't understand why it is required because a) the formerly used non-FULL versions should be defined iff the FULL versions are and b) the fix that has been used and working on the NixOS side was exactly what your original PR did.

I think cmake itself defines CMAKE_INSTALL_<dir>, but require GNUInstallDirs for CMAKE_INSTALL_FULL_<dir>. (I don't know about anything this working on NixOS with -DCMAKE_INSTALL_<dir>=/absolute without using GNUInstallDirs.)

Anyway, I reorganized the .pc file and CMakeConfig generation and this takes also care of the missing GNUInstallDirs include. That said, I'd appreciate if you could test the code especially with respect to those changes. I don't have a setup that properly tests the validity of the generated .pc file.

libdir= and includedir= look fine, and are the only variables used by the .pc file itself. I don't know if anyone is using prefix= (which is now removed).