On April 11th, 2013, 11:06 p.m. CEST, John Dennis wrote:
It's nice you added FindOpenLMI.cmake, that's one of solving the problem and it much more consistent with CMake.
I don't see where openlmi.pc.in gets converted from a .in to the final file with substitutions nor where it gets installed.
I'm not sure I see the value in the variable substitutions in openlmi.pc.in.
openlmi.pc.in does not follow the conventions for pc files, please review the existing pc files in /usr/lib/pkgconfig, This document gives an overview: http://people.freedesktop.org/~dbn/pkg-config-guide.html
Off the top of my head I think the OpenLMI.pc file might look like this:
------------------------------------------------------
prefix=/usr
exec_prefix=${prefix}
includedir=${prefix}/include/openlmi
libdir=${exec_prefix}/lib
Name: OpenLMI
Description: OpenLMI provider support
Version: 1.0.0
Libs: -lopenlmicommon
Cflags: -I${includedir}
------------------------------------------------------
I think I might keep the changes necessary for building (e.g. the .cmake and .pc) file independent of the header file name change, the two are unrelated.
> I don't see where openlmi.pc.in gets converted from a .in to the final file with substitutions nor where it gets installed.
see src/CMakeLists.txt:
17 configure_file(openlmi.pc.in openlmi.pc)
18 install(FILES openlmi.pc DESTINATION lib${LIB_SUFFIX}/pkgconfig)
> I'm not sure I see the value in the variable substitutions in openlmi.pc.in.
The variable substitution is there mainly because of substitution of version with actual version of OpenLMI-providers.
> openlmi.pc.in does not follow the conventions for pc files, please review the existing pc files in /usr/lib/pkgconfig
I based the .pc file on some random out of /usr/lib/pkgconfig, but it seems I picked a wrong one, will fix.
- Radek
On April 11th, 2013, 4:02 p.m. CEST, Radek Novacek wrote:
Review request for OpenLMI Developers.
By Radek Novacek.
Updated April 11, 2013, 4:02 p.m.
Repository:
openlmi-providers
Description
Support for using libopenlmicommon by external providers
* add FindOpenLMI.cmake module
* add pkgconfig for OpenLMI
* rename globals.c/h to openlmi.c/h
* add symlink with major version to openlmicommon library
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=948948
|
Diffs
- CMakeLists.txt (230ed263388a62fe4a293479ca7bb9744e3fcfd9)
- cmake/modules/FindOpenLMI.cmake (PRE-CREATION)
- src/CMakeLists.txt (73189110171562bdcbaf56faff7d2d07b325a076)
- src/account/LMI_AccountCapabilitiesProvider.c (1636ab7ad7b894bd9800f4fd8a20d22e9533fab2)
- src/account/LMI_AccountManagementCapabilitiesProvider.c (a4499be4d318495be6519d42b040e7da966835b9)
- src/account/LMI_AccountManagementServiceCapabilitiesProvider.c (febc4f31de52049b1c12049fbd3d687c0bf508b3)
- src/account/LMI_AccountManagementServiceProvider.c (1af6cf4f55f574d3bf665202ff7b55224a8d9009)
- src/account/LMI_AccountOnSystemProvider.c (ab84097648ec2cbd883276f8e981d7c596a9ca5e)
- src/account/LMI_AccountProvider.c (c4a68f7c71d0f77821bdf95aec7ae60ffae0b455)
- src/account/LMI_AccountSettingDataProvider.c (92d3b28a23bb04e625f65deb7573ab7baacea372)
- src/account/LMI_AssignedAccountIdentityProvider.c (323a582217cd1c101863cc489b3e4ee94aa3a4ff)
- src/account/LMI_AssignedGroupIdentityProvider.c (a9037f1bd86a39ef21869796a692a0ec85affb08)
- src/account/LMI_EnabledAccountCapabilitiesProvider.c (7cda823209f2d94ddc4f1fd8f7de3c210495d033)
- src/account/LMI_GroupProvider.c (a9c547724ecf8e66f92efb1677ee3b6f348f6051)
- src/account/LMI_HostedAccountManagementServiceProvider.c (dd03a5fc144bc8f4927fe9ebe7d598c05d771265)
- src/account/LMI_IdentityProvider.c (92c2b0119a3dd81ca62be99c6e09f894f46841cd)
- src/account/LMI_MemberOfGroupProvider.c (dddd2e80952d7a1bedc0a0291ada56b5d7a1aea9)
- src/account/LMI_OwningGroupProvider.c (317b7b4a4ec002acda6aa2b6a933a76ca41369a9)
- src/account/LMI_ServiceAffectsIdentityProvider.c (64e218b4068c696dbb5479d7e73c2f38b4fc0cb5)
- src/fan/LMI_FanAssociatedSensorProvider.c (d75bf5cc39b8a28d17980ab763d48831d762387c)
- src/fan/LMI_FanProvider.c (26af16ea3c119c4ea9fd8812e5b8cf9ed5007347)
- src/fan/LMI_FanSensorProvider.c (47bd9d529931adc4ac6a1b15d759067bb63f324a)
- src/fan/fan.c (30385034233c7abe4eb76939e8abc38f9819cf4c)
- src/globals.h ()
- src/globals.c (7e58817fbaf87d25d4ec343d8589ec74f1224a8b)
- src/hardware/LMI_AssociatedProcessorCacheMemoryProvider.c (3d06a65f7d24423400384f6bbfb70ecd8280319b)
- src/hardware/LMI_PCIDeviceProvider.c (e879484a53bc29aa6109ad31a25a9616f03ca3e0)
- src/hardware/LMI_ProcessorCacheMemoryProvider.c (afc5d7f0488c506dddabd438f7af56578f464856)
- src/hardware/LMI_ProcessorCapabilitiesProvider.c (c7711242aa14d6ea585eed2f4337f772a2d6a176)
- src/hardware/LMI_ProcessorChipProvider.c (47ba488995976313c1def5a4269e55b1bbf051c3)
- src/hardware/LMI_ProcessorChipRealizesProvider.c (6dfd4bfb91bd25b1a4c13d816766ae7b56e5286d)
- src/hardware/LMI_ProcessorElementCapabilitiesProvider.c (322e966103f6e4acc6b69ec7269505bd3c6cff5f)
- src/hardware/LMI_ProcessorProvider.c (99b0fad7d1e36efbb1e823d20cda74bf9b577235)
- src/hardware/cpuinfo.h (5ec5dd5083fe73548a434a3312c0a371c0c0d961)
- src/hardware/dmidecode.h (9190ea4ac26e82889b24184fc40072f007f9c836)
- src/hardware/lscpu.h (5dc5bd5cbf01471ce8a6e98ba8f93386336a3898)
- src/hardware/sysfs.h (0e0b523ec6918b028ae0b17b9c756b76e2e686e8)
- src/hardware/utils.h (933c0427b127c8658b98b4b07ca06a2c9125c48a)
- src/logicalfile/file.h (b1b3358f0f48811a1b255d8e842ebcfaf38c2f2e)
- src/openlmi.pc.in (PRE-CREATION)
- src/power/LMI_AssociatedPowerManagementServiceProvider.c (fd6ffdbe5826d6e14503c4377301e37e67a3fe81)
- src/power/LMI_ConcreteJobProvider.c (4841852f63cd9398f65c83866562c214f70b625c)
- src/power/LMI_ElementCapabilitiesProvider.c (3a49518a26e2c3b523bac8289dcdf6f720ae7267)
- src/power/LMI_HostedPowerManagementServiceProvider.c (889dd2cf211935da0d75d9af0064e38ea81cea3e)
- src/power/LMI_PowerManagementServiceProvider.c (655d5948faa672a82b2c659f4a716e423ba5b3a7)
- src/service/LMI_ServiceProvider.c (7c8ffdb242913dbe997f0ad802a9128f5f864c67)
View Diff
|