FS#44516 - [unittestpp] wrong directory for include files and missing include-file UnitTest++.h
Attached to Project:
Community Packages
Opened by Uwe Koloska (kolewu) - Thursday, 09 April 2015, 21:05 GMT
Last edited by Eli Schwartz (eschwartz) - Thursday, 03 August 2017, 22:51 GMT
Opened by Uwe Koloska (kolewu) - Thursday, 09 April 2015, 21:05 GMT
Last edited by Eli Schwartz (eschwartz) - Thursday, 03 August 2017, 22:51 GMT
|
Details
Description:
Upstream (now on github -- see my other report https://bugs.archlinux.org/task/44514) installs include files in include/UnitTest++ (see: https://github.com/unittest-cpp/unittest-cpp/blob/master/CMakeLists.txt#L51 -- that's a new addition, but if I remember correctly intended before) and this has been the case with the AUR package 1.4-2 as well. (the AUR version 1.4-3 changed that though) Don't know whether there is an Arch rule that permits mixed case for include directories -- but then there are some more packages that do break this rule: find /usr/include/ -type d -a -name "[A-Z]*" I don't think it's a good idea to change an include directories name. Searching for all relevant information, I found that there is a file missing in the package: UnitTest++.h that is mentioned in the documentation: /usr/share/doc/unittestpp/UnitTest++.html (but it's not in the new install-target -- what a mess ...) Some searching later: since all include files are now in one directory and this include directory is reported correctly by pkg-config (and my package lenmus get's it right in the cmake search), it's not as important what the name is -- but I would prefer the name that upstream has chosen. Additional info: * 1.4-6 * commit of the install target: https://github.com/unittest-cpp/unittest-cpp/commit/13a6c9e6ada95a291cdcf132ba2a0122f4a924a7 |
This task depends upon
Closed by Eli Schwartz (eschwartz)
Thursday, 03 August 2017, 22:51 GMT
Reason for closing: Fixed
Additional comments about closing: fixed pc file: unittestpp 1.6.1-1
fixed installation directory: unittestpp 2.0.0-1
Thursday, 03 August 2017, 22:51 GMT
Reason for closing: Fixed
Additional comments about closing: fixed pc file: unittestpp 1.6.1-1
fixed installation directory: unittestpp 2.0.0-1
-Cflags: -I${includedir}/unittest++
+Cflags: -I${includedir}/UnitTest++