Re: [PATCH 1/1] sysfsutils: install libs to base_libdir

[Saul Wold <sgw@linux.intel.com>]

On 08/09/2012 06:19 AM, wenzong fan wrote:
> On 08/08/2012 04:44 PM, Koen Kooi wrote:
>>
>> Op 8 aug. 2012, om 08:09 heeft wenzong fan<wenzong.fan@windriver.com>
>> het volgende geschreven:
>>
>>> On 08/07/2012 08:23 PM, Koen Kooi wrote:
>>>>
>>>> Op 7 aug. 2012, om 14:14 heeft wenzong.fan@windriver.com het
>>>> volgende geschreven:
>>>>
>>>>> From: Wenzong Fan<wenzong.fan@windriver.com>
>>>>>
>>>>> Fix the warning form pcmciautils building:
>>>>>
>>>>> "WARNING: QA Issue: pcmciautils: /sbin/pccardctl, installed in the
>>>>> base_prefix, requires a shared library under exec_prefix (/usr):
>>>>> libsysfs.so.2 =>   /usr/lib/libsysfs.so.2"
>>>>>
>>>>> Just refer to Debian/Ubuntu distributions, install sysfsutils libs
>>>>> to "/lib*" instead of "/usr/lib*".
>>>>>
>>>>> [YOCTO #2907]
>>>>>
>>>>> Signed-off-by: Wenzong Fan<wenzong.fan@windriver.com>
>>>>> ---
>>>>> meta/recipes-core/sysfsutils/sysfsutils_2.1.0.bb |    4 +++-
>>>>> 1 file changed, 3 insertions(+), 1 deletion(-)
>>>>>
>>>>> diff --git a/meta/recipes-core/sysfsutils/sysfsutils_2.1.0.bb
>>>>> b/meta/recipes-core/sysfsutils/sysfsutils_2.1.0.bb
>>>>> index 3e4fdda..f01d19b 100644
>>>>> --- a/meta/recipes-core/sysfsutils/sysfsutils_2.1.0.bb
>>>>> +++ b/meta/recipes-core/sysfsutils/sysfsutils_2.1.0.bb
>>>>> @@ -8,7 +8,7 @@ LICENSE_libsysfs = "LGPLv2.1"
>>>>> LIC_FILES_CHKSUM =
>>>>> "file://COPYING;md5=3d06403ea54c7574a9e581c6478cc393 \
>>>>>
>>>>> file://cmd/GPL;md5=d41d4e2e1e108554e0388ea4aecd8d27 \
>>>>>
>>>>> file://lib/LGPL;md5=b75d069791103ffe1c0d6435deeff72e"
>>>>> -PR = "r3"
>>>>> +PR = "r4"
>>>>>
>>>>> SRC_URI = "${SOURCEFORGE_MIRROR}/linux-diag/sysfsutils-${PV}.tar.gz \
>>>>>       file://sysfsutils-2.0.0-class-dup.patch \
>>>>> @@ -24,3 +24,5 @@ PACKAGES =+ "libsysfs libsysfs-dev
>>>>> libsysfs-staticdev"
>>>>> FILES_libsysfs = "${libdir}/lib*${SOLIBS}"
>>>>> FILES_libsysfs-dev = "${libdir}/lib*${SOLIBSDEV} ${includedir}"
>>>>> FILES_libsysfs-staticdev = "${libdir}/lib*.a"
>>>>
>>>> Ahem! ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
> Yep, this works but looks confused, do you mean the 'export libdir'
> should be put ahead of FILES_lib*?
>
>  From the build logs, only change is $libdir: '/usr/lib -> /lib',
> details please refer to attached file.
>
Wenzong,
We are looking for buildhistory info, not a diff of your build log.

buildhistory can be activated as follows in your local.conf:

INHERIT += "buildhistory"
BUILDHISTORY_COMMIT = "1"

Then you can build before and after your change, and use the script 
buildhistory_diff to produce the output we are looking for.

Thanks
	Sau!

> Thanks
> Wenzong
>
>>>>
>>>>> +
>>>>> +export libdir = "${base_libdir}"
>>>>
>>>> And can you post the buildhistory diff to see if anything else moved
>>>> when it shouldn't have e.g. package config files?
>>>
>>> I make a diff to the build dir, only the location of 'lib/' was moved:
>>
>> Please post the buildhistory diff, that is much more informative. And
>> you haven't responded to the FILES_lib* oversight.
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>