Re: [oe] [meta-oe][PATCH v2] xmlsec1: Fix configure QA error caused by host lookup path

["Anatol Belski" <anbelski@linux.microsoft.com>]

Hi,

On 1/15/2021 3:56 AM, akuster808 wrote:
>
> On 1/14/21 9:36 AM, Anatol Belski wrote:
>> The configure script contains hardcoded lookup paths to /usr and other
>> paths that might interfere with the host. These are overwritten with the
>> staging dir locations for Poky compatibility.
> The QA message has "xmlsec1-1.2.30" so this appears it needs to be
> backported to Gatesgarth and Dunfell.
>
> Please confirm.

Correct, the initial issue is in Dunfell. It should be either backported 
or I was intended to file a patch against Dunfell, whatever is 
preferred. Please let me know. The v2 is the version to be taken.

Thanks

Anatol

>
> -armin
>> Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
>> ---
>>   .../xmlsec1/ensure-search-path-non-host.patch | 22 +++++++++++++++++++
>>   .../recipes-support/xmlsec1/xmlsec1_1.2.31.bb |  1 +
>>   2 files changed, 23 insertions(+)
>>   create mode 100644 meta-oe/recipes-support/xmlsec1/xmlsec1/ensure-search-path-non-host.patch
>>
>> diff --git a/meta-oe/recipes-support/xmlsec1/xmlsec1/ensure-search-path-non-host.patch b/meta-oe/recipes-support/xmlsec1/xmlsec1/ensure-search-path-non-host.patch
>> new file mode 100644
>> index 0000000000..a5a298af0d
>> --- /dev/null
>> +++ b/meta-oe/recipes-support/xmlsec1/xmlsec1/ensure-search-path-non-host.patch
>> @@ -0,0 +1,22 @@
>> +xmlsec1: Fix configure QA error caused by host lookup path
>> +
>> +ERROR: mc:my-sdk:xmlsec1-1.2.30-r0 do_configure: QA Issue: This autoconf log indicates errors, it looked at host include and/or library paths while determining system capabilities.
>> +
>> +It will eventually arise after the configure QA as the configure script should only look at the staging sysroot dir, not at the host.
>> +
>> +Upstream-Status: Inappropriate [embedded specific]
>> +Signed-off-by: Anatol Belski <anbelski@linux.microsoft.com>
>> +
>> +--- a/configure.ac.orig   2021-01-13 14:37:42.254991177 +0000
>> ++++ b/configure.ac        2021-01-13 14:40:56.546269330 +0000
>> +@@ -250,8 +250,8 @@
>> + dnl ==========================================================================
>> + dnl Common installation locations
>> + dnl ==========================================================================
>> +-COMMON_INCLUDE_DIR="/usr/include /usr/local/include"
>> +-COMMON_LIB_DIR="/usr/lib /usr/lib64 /usr/local/lib"
>> ++COMMON_INCLUDE_DIR="${STAGING_INCDIR}"
>> ++COMMON_LIB_DIR="${STAGING_LIBDIR}"
>> + case $host in
>> +     i*86-*-linux-gnu)   COMMON_LIB_DIR="$COMMON_LIB_DIR /usr/lib/i386-linux-gnu" ;;
>> +     x86_64-*-linux-gnu) COMMON_LIB_DIR="$COMMON_LIB_DIR /usr/lib/x86_64-linux-gnu" ;;
>> diff --git a/meta-oe/recipes-support/xmlsec1/xmlsec1_1.2.31.bb b/meta-oe/recipes-support/xmlsec1/xmlsec1_1.2.31.bb
>> index 803016c96e..3464f2f43c 100644
>> --- a/meta-oe/recipes-support/xmlsec1/xmlsec1_1.2.31.bb
>> +++ b/meta-oe/recipes-support/xmlsec1/xmlsec1_1.2.31.bb
>> @@ -19,6 +19,7 @@ SRC_URI = "http://www.aleksey.com/xmlsec/download/${BP}.tar.gz \
>>       file://xmlsec1-examples-allow-build-in-separate-dir.patch \
>>       file://0001-nss-nspr-fix-for-multilib.patch \
>>       file://run-ptest \
>> +    file://ensure-search-path-non-host.patch \
>>       "
>>   
>>   SRC_URI[md5sum] = "182f5cbdfb8ddbac535c04470c170dc3"
>>
>> 
>>