[OE-core] [PATCH] perl: Remove the changes of bin, lib etc. as the defaults will give sane values.
Martin Ertsås
mertsas at cisco.com
Mon Sep 10 05:51:59 UTC 2012
On 09/06/12 21:43, Mark Hatle wrote:
> On 9/6/12 2:11 PM, Saul Wold wrote:
>> On 09/06/2012 01:29 AM, Martin Ertsaas wrote:
>>> The old config put stuff in
>>> ${perldir}/image/usr/local/oecore-x86_64/sysroot/x86_64-unknownsdk-linux/usr/local/oecore-x86_64/sysroots/x86_64-unknownsdk-linux/usr
>>> instead of in
>>> ${perldir}/image/usr/local/oecore-x86_64/sysroot/x86_64-nimbussdk-linux/usr
>>> as it should.
>>
>> I am not sure we want to remove that sed code, since allows for
>> different ${libdir} than /usr/lib, such as a flattened /lib, same with
>> the ${exec_prefix}.
>
> Ya, the sed lines are specifically there to enable the end user to
> modify the exec_prefix and libdir to their local distribution
> policies. If this is no longer necessary, we can remove it.. but
> someone needs to verify they are being used properly. (Flattened
> filesystem and multilibs are the two places I know it's used today.)
>
> --Mark
Ok. That is ok. However, I think we would need to fix it to give a sane
default value, like
${perldir}/image/usr/local/oecore-x86_64/sysroot/x86_64-unknownsdk-linux/usr.
Not the part after image is duplicated in the install directory, which
makes do_install fail if you try to use the standard directory.
- Martin
>
>> Sau!
>>
>>
>>> ---
>>> meta/recipes-devtools/perl/perl_5.14.2.bb | 19 ++++++++-----------
>>> 1 files changed, 8 insertions(+), 11 deletions(-)
>>>
>>> diff --git a/meta/recipes-devtools/perl/perl_5.14.2.bb
>>> b/meta/recipes-devtools/perl/perl_5.14.2.bb
>>> index 3c1360f..04e2597 100644
>>> --- a/meta/recipes-devtools/perl/perl_5.14.2.bb
>>> +++ b/meta/recipes-devtools/perl/perl_5.14.2.bb
>>> @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM =
>>> "file://Copying;md5=2b4c6ffbcfcbdee469f02565f253d81a \
>>> # We need gnugrep (for -I)
>>> DEPENDS = "virtual/db grep-native"
>>> DEPENDS += "gdbm zlib"
>>> -PR = "r9"
>>> +PR = "r10"
>>>
>>> # 5.10.1 has Module::Build built-in
>>> PROVIDES += "libmodule-build-perl"
>>> @@ -145,16 +145,13 @@ do_configure() {
>>>
>>> ${@base_contains('DISTRO_FEATURES', 'largefile', '',
>>> 'do_nolargefile', d)}
>>>
>>> - # Update some paths in the configuration
>>> - sed -i -e 's, at DESTDIR@,${prefix},g' \
>>> - -e 's, at ARCH@-thread-multi,,g' \
>>> - -e 's, at ARCH@,${TARGET_ARCH}-${TARGET_OS},g' \
>>> - -e 's, at STAGINGDIR@,${STAGING_DIR_HOST},g' \
>>> - -e "s%\([
>>> \"^\',=]\+\)/usr/include%\1${STAGING_INCDIR}%g" \
>>> - -e "s%\([ \"^\',=]\+\)/usr/lib/%\1${libdir}/%g" \
>>> - -e "s%\([ \"^\',=]\+\)/usr/%\1${exec_prefix}/%g" \
>>> - -e "s%/perl5%/perl%g" \
>>> - config.sh-${TARGET_ARCH}-${TARGET_OS}
>>> + # Update some paths in the configuration
>>> + sed -i -e 's, at DESTDIR@,${prefix},g' \
>>> + -e 's, at ARCH@-thread-multi,,g' \
>>> + -e 's, at ARCH@,${TARGET_ARCH}-${TARGET_OS},g' \
>>> + -e 's, at STAGINGDIR@,${STAGING_DIR_HOST},g' \
>>> + -e "s%/perl5%/perl%g" \
>>> + config.sh-${TARGET_ARCH}-${TARGET_OS}
>>>
>>> case "${TARGET_ARCH}" in
>>> x86_64 | powerpc | s390)
>>>
>>
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core at lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>>
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core at lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
More information about the Openembedded-core
mailing list