* [PATCH 1/2] openssl_1.1: avoid using += with an over-ride
@ 2018-07-06 19:05 Andre McCurdy
2018-07-06 19:05 ` [PATCH 2/2] openssl_1.1: minor recipe formatting tweaks etc Andre McCurdy
2018-07-07 8:52 ` [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Alexander Kanavin
0 siblings, 2 replies; 4+ messages in thread
From: Andre McCurdy @ 2018-07-06 19:05 UTC (permalink / raw)
To: openembedded-core
Using += with an over-ride can be a source of confusion so try to
avoid the construct in core recipes.
The current usage is incorrect and prevents the aarch64 and musl
specific config options from being active together.
Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
---
meta/recipes-connectivity/openssl/openssl_1.1.0h.bb | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
index 94b75eb..ac2ad81 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
@@ -122,12 +122,12 @@ do_configure () {
#| engines/afalg/e_afalg.c:110:20: error: '__NR_eventfd' undeclared (first use in this function)
#| return syscall(__NR_eventfd, n);
#| ^~~~~~~~~~~~
-EXTRA_OECONF_aarch64 += "no-afalgeng"
+EXTRA_OECONF_append_aarch64 = " no-afalgeng"
#| ./libcrypto.so: undefined reference to `getcontext'
#| ./libcrypto.so: undefined reference to `setcontext'
#| ./libcrypto.so: undefined reference to `makecontext'
-EXTRA_OECONF_libc-musl += "-DOPENSSL_NO_ASYNC"
+EXTRA_OECONF_append_libc-musl = " -DOPENSSL_NO_ASYNC"
do_install () {
oe_runmake DESTDIR="${D}" MANDIR="${mandir}" MANSUFFIX=ssl install
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] openssl_1.1: minor recipe formatting tweaks etc
2018-07-06 19:05 [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Andre McCurdy
@ 2018-07-06 19:05 ` Andre McCurdy
2018-07-07 8:52 ` [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Alexander Kanavin
1 sibling, 0 replies; 4+ messages in thread
From: Andre McCurdy @ 2018-07-06 19:05 UTC (permalink / raw)
To: openembedded-core
Drop redundant setting of S to its default value and re-order
variables to align more closely to the OE style-guide.
Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
---
.../recipes-connectivity/openssl/openssl_1.1.0h.bb | 39 ++++++++++------------
1 file changed, 18 insertions(+), 21 deletions(-)
diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
index ac2ad81..80f11a2 100644
--- a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
+++ b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
@@ -8,21 +8,28 @@ SECTION = "libs/network"
LICENSE = "openssl"
LIC_FILES_CHKSUM = "file://LICENSE;md5=d57d511030c9d66ef5f5966bee5a7eff"
-BBCLASSEXTEND = "native nativesdk"
-
-SRC_URI[md5sum] = "5271477e4d93f4ea032b665ef095ff24"
-SRC_URI[sha256sum] = "5835626cde9e99656585fc7aaa2302a73a7e1340bf8c14fd635a62c66802a517"
-
SRC_URI = "http://www.openssl.org/source/openssl-${PV}.tar.gz \
file://run-ptest \
file://openssl-c_rehash.sh \
file://0001-Take-linking-flags-from-LDFLAGS-env-var.patch \
"
-S = "${WORKDIR}/openssl-${PV}"
+SRC_URI[md5sum] = "5271477e4d93f4ea032b665ef095ff24"
+SRC_URI[sha256sum] = "5835626cde9e99656585fc7aaa2302a73a7e1340bf8c14fd635a62c66802a517"
inherit lib_package multilib_header ptest
+#| engines/afalg/e_afalg.c: In function 'eventfd':
+#| engines/afalg/e_afalg.c:110:20: error: '__NR_eventfd' undeclared (first use in this function)
+#| return syscall(__NR_eventfd, n);
+#| ^~~~~~~~~~~~
+EXTRA_OECONF_append_aarch64 = " no-afalgeng"
+
+#| ./libcrypto.so: undefined reference to `getcontext'
+#| ./libcrypto.so: undefined reference to `setcontext'
+#| ./libcrypto.so: undefined reference to `makecontext'
+EXTRA_OECONF_append_libc-musl = " -DOPENSSL_NO_ASYNC"
+
do_configure () {
os=${HOST_OS}
case $os in
@@ -118,17 +125,6 @@ do_configure () {
perl ./Configure ${EXTRA_OECONF} --prefix=$useprefix --openssldir=${libdir}/ssl-1.1 --libdir=${libdirleaf} $target
}
-#| engines/afalg/e_afalg.c: In function 'eventfd':
-#| engines/afalg/e_afalg.c:110:20: error: '__NR_eventfd' undeclared (first use in this function)
-#| return syscall(__NR_eventfd, n);
-#| ^~~~~~~~~~~~
-EXTRA_OECONF_append_aarch64 = " no-afalgeng"
-
-#| ./libcrypto.so: undefined reference to `getcontext'
-#| ./libcrypto.so: undefined reference to `setcontext'
-#| ./libcrypto.so: undefined reference to `makecontext'
-EXTRA_OECONF_append_libc-musl = " -DOPENSSL_NO_ASYNC"
-
do_install () {
oe_runmake DESTDIR="${D}" MANDIR="${mandir}" MANSUFFIX=ssl install
oe_multilib_header openssl/opensslconf.h
@@ -152,10 +148,11 @@ do_install_ptest() {
sed -i 's/$target{shared_extension_simple}/".so.ptest"/' ${D}${PTEST_PATH}/test/recipes/90-test_shlibload.t
}
-RDEPENDS_${PN}-ptest += "perl-module-file-spec-functions bash python"
-
-FILES_${PN} =+ " ${libdir}/ssl-1.1/*"
-
PACKAGES =+ "${PN}-engines"
+
+FILES_${PN} =+ "${libdir}/ssl-1.1/*"
FILES_${PN}-engines = "${libdir}/engines-1.1"
+RDEPENDS_${PN}-ptest += "perl-module-file-spec-functions bash python"
+
+BBCLASSEXTEND = "native nativesdk"
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH 1/2] openssl_1.1: avoid using += with an over-ride
2018-07-06 19:05 [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Andre McCurdy
2018-07-06 19:05 ` [PATCH 2/2] openssl_1.1: minor recipe formatting tweaks etc Andre McCurdy
@ 2018-07-07 8:52 ` Alexander Kanavin
2018-07-09 3:07 ` Andre McCurdy
1 sibling, 1 reply; 4+ messages in thread
From: Alexander Kanavin @ 2018-07-07 8:52 UTC (permalink / raw)
To: Andre McCurdy; +Cc: OE-core
We really need a QA warning for this issue. Nobody's going to remember
what the correct way is. Can you look into it please?
Alex
2018-07-06 21:05 GMT+02:00 Andre McCurdy <armccurdy@gmail.com>:
> Using += with an over-ride can be a source of confusion so try to
> avoid the construct in core recipes.
>
> The current usage is incorrect and prevents the aarch64 and musl
> specific config options from being active together.
>
> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
> ---
> meta/recipes-connectivity/openssl/openssl_1.1.0h.bb | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
> index 94b75eb..ac2ad81 100644
> --- a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
> +++ b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
> @@ -122,12 +122,12 @@ do_configure () {
> #| engines/afalg/e_afalg.c:110:20: error: '__NR_eventfd' undeclared (first use in this function)
> #| return syscall(__NR_eventfd, n);
> #| ^~~~~~~~~~~~
> -EXTRA_OECONF_aarch64 += "no-afalgeng"
> +EXTRA_OECONF_append_aarch64 = " no-afalgeng"
>
> #| ./libcrypto.so: undefined reference to `getcontext'
> #| ./libcrypto.so: undefined reference to `setcontext'
> #| ./libcrypto.so: undefined reference to `makecontext'
> -EXTRA_OECONF_libc-musl += "-DOPENSSL_NO_ASYNC"
> +EXTRA_OECONF_append_libc-musl = " -DOPENSSL_NO_ASYNC"
>
> do_install () {
> oe_runmake DESTDIR="${D}" MANDIR="${mandir}" MANSUFFIX=ssl install
> --
> 1.9.1
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH 1/2] openssl_1.1: avoid using += with an over-ride
2018-07-07 8:52 ` [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Alexander Kanavin
@ 2018-07-09 3:07 ` Andre McCurdy
0 siblings, 0 replies; 4+ messages in thread
From: Andre McCurdy @ 2018-07-09 3:07 UTC (permalink / raw)
To: OE-core
On Sat, Jul 7, 2018 at 1:52 AM, Alexander Kanavin
<alex.kanavin@gmail.com> wrote:
> We really need a QA warning for this issue. Nobody's going to remember
> what the correct way is. Can you look into it please?
It should be a parsing error, not a QA warning. If a feature of the
language is mistakenly used so often (100% of the time, as far as I
can tell) then we should just have the parser reject it.
> 2018-07-06 21:05 GMT+02:00 Andre McCurdy <armccurdy@gmail.com>:
>> Using += with an over-ride can be a source of confusion so try to
>> avoid the construct in core recipes.
>>
>> The current usage is incorrect and prevents the aarch64 and musl
>> specific config options from being active together.
>>
>> Signed-off-by: Andre McCurdy <armccurdy@gmail.com>
>> ---
>> meta/recipes-connectivity/openssl/openssl_1.1.0h.bb | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
>> index 94b75eb..ac2ad81 100644
>> --- a/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
>> +++ b/meta/recipes-connectivity/openssl/openssl_1.1.0h.bb
>> @@ -122,12 +122,12 @@ do_configure () {
>> #| engines/afalg/e_afalg.c:110:20: error: '__NR_eventfd' undeclared (first use in this function)
>> #| return syscall(__NR_eventfd, n);
>> #| ^~~~~~~~~~~~
>> -EXTRA_OECONF_aarch64 += "no-afalgeng"
>> +EXTRA_OECONF_append_aarch64 = " no-afalgeng"
>>
>> #| ./libcrypto.so: undefined reference to `getcontext'
>> #| ./libcrypto.so: undefined reference to `setcontext'
>> #| ./libcrypto.so: undefined reference to `makecontext'
>> -EXTRA_OECONF_libc-musl += "-DOPENSSL_NO_ASYNC"
>> +EXTRA_OECONF_append_libc-musl = " -DOPENSSL_NO_ASYNC"
>>
>> do_install () {
>> oe_runmake DESTDIR="${D}" MANDIR="${mandir}" MANSUFFIX=ssl install
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-07-09 3:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-07-06 19:05 [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Andre McCurdy
2018-07-06 19:05 ` [PATCH 2/2] openssl_1.1: minor recipe formatting tweaks etc Andre McCurdy
2018-07-07 8:52 ` [PATCH 1/2] openssl_1.1: avoid using += with an over-ride Alexander Kanavin
2018-07-09 3:07 ` Andre McCurdy
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox