* [PATCH] [meta-swupdate] Fix build error of dependence. @ 2020-06-04 12:26 zhengruoqin 2020-06-04 7:22 ` [yocto] " Stefano Babic 0 siblings, 1 reply; 9+ messages in thread From: zhengruoqin @ 2020-06-04 12:26 UTC (permalink / raw) To: yocto; +Cc: Zheng Ruoqing -c -o mongoose/mongoose.o mongoose/mongoose.c | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such file or directory Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> --- recipes-support/swupdate/swupdate.inc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/recipes-support/swupdate/swupdate.inc b/recipes-support/swupdate/swupdate.inc index 9ea0a8a..3d9a3fa 100644 --- a/recipes-support/swupdate/swupdate.inc +++ b/recipes-support/swupdate/swupdate.inc @@ -123,6 +123,9 @@ python () { elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: depends += ' mbedtls' + if 'CONFIG_MONGOOSESSL=y\n' in features: + depends += ' openssl' + if 'CONFIG_JSON=y\n' in features: depends += ' json-c' -- 2.17.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-04 12:26 [PATCH] [meta-swupdate] Fix build error of dependence zhengruoqin @ 2020-06-04 7:22 ` Stefano Babic 2020-06-08 8:49 ` zhengruoqin 0 siblings, 1 reply; 9+ messages in thread From: Stefano Babic @ 2020-06-04 7:22 UTC (permalink / raw) To: zhengruoqin, yocto; +Cc: swupdate Hi Zheng, there is a ML for SWUpdate and meta-swupdate (see CC), we are cross-posting here: On 04.06.20 14:26, zhengruoqin wrote: > -c -o mongoose/mongoose.o mongoose/mongoose.c > | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such file > or directory > > Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> > --- > recipes-support/swupdate/swupdate.inc | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/recipes-support/swupdate/swupdate.inc b/recipes-support/swupdate/swupdate.inc > index 9ea0a8a..3d9a3fa 100644 > --- a/recipes-support/swupdate/swupdate.inc > +++ b/recipes-support/swupdate/swupdate.inc > @@ -123,6 +123,9 @@ python () { > elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: > depends += ' mbedtls' > > + if 'CONFIG_MONGOOSESSL=y\n' in features: > + depends += ' openssl' > + openSSL is not the only supported SSL library, the Webserver runs also with mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is used as signing and crypto library. > if 'CONFIG_JSON=y\n' in features: > depends += ' json-c' > > > > > Best regards, Stefano Babic ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-04 7:22 ` [yocto] " Stefano Babic @ 2020-06-08 8:49 ` zhengruoqin 2020-06-08 8:55 ` [swupdate] " Stefano Babic 0 siblings, 1 reply; 9+ messages in thread From: zhengruoqin @ 2020-06-08 8:49 UTC (permalink / raw) To: Stefano Babic, yocto; +Cc: swupdate Hi Stefano > there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- > posting here: Thank you, I got it. > openSSL is not the only supported SSL library, the Webserver runs also with > mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is > used as signing and crypto library. > > > if 'CONFIG_JSON=y\n' in features: > > depends += ' json-c' OK, I'll update my patch and send V2 version. -------------------------------------------------- Zheng Ruoqin Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) ADDR.: No.6 Wenzhu Road, Software Avenue, Nanjing, 210012, China MAIL : zhengrq.fnst@cn.fujistu.com > -----Original Message----- > From: Stefano Babic <sbabic@denx.de> > Sent: Thursday, June 4, 2020 3:22 PM > To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com>; > yocto@yoctoproject.org > Cc: swupdate <swupdate@googlegroups.com> > Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. > > Hi Zheng, > > there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- > posting here: > > On 04.06.20 14:26, zhengruoqin wrote: > > -c -o mongoose/mongoose.o mongoose/mongoose.c > > | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such > > | file > > or directory > > > > Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> > > --- > > recipes-support/swupdate/swupdate.inc | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/recipes-support/swupdate/swupdate.inc > > b/recipes-support/swupdate/swupdate.inc > > index 9ea0a8a..3d9a3fa 100644 > > --- a/recipes-support/swupdate/swupdate.inc > > +++ b/recipes-support/swupdate/swupdate.inc > > @@ -123,6 +123,9 @@ python () { > > elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: > > depends += ' mbedtls' > > > > + if 'CONFIG_MONGOOSESSL=y\n' in features: > > + depends += ' openssl' > > + > > openSSL is not the only supported SSL library, the Webserver runs also with > mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is > used as signing and crypto library. > > > if 'CONFIG_JSON=y\n' in features: > > depends += ' json-c' > > > > > > > > > > > > Best regards, > Stefano Babic > > > > > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-08 8:49 ` zhengruoqin @ 2020-06-08 8:55 ` Stefano Babic 2020-06-18 7:08 ` zhengruoqin 0 siblings, 1 reply; 9+ messages in thread From: Stefano Babic @ 2020-06-08 8:55 UTC (permalink / raw) To: Zheng, Ruoqin, Stefano Babic, yocto; +Cc: swupdate Hi Zheng, On 08.06.20 10:49, Zheng, Ruoqin wrote: > Hi Stefano > >> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >> posting here: > Thank you, I got it. > >> openSSL is not the only supported SSL library, the Webserver runs also with >> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >> used as signing and crypto library. >> >>> if 'CONFIG_JSON=y\n' in features: >>> depends += ' json-c' > OK, I'll update my patch and send V2 version. > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set CONFIG_MONGOOSESSL, SSL is already set. The only way to go into the issue is to modify directly your defconfig file instead of running "bitbake -c menuconfig" and then apply the diffconfig or the new defconfig. But this is of course a wrong way, exactly as in kernel. Try to run "mnuconfig" and then you can apply the fragment to your diffconfig (fragments are supported, too). Best regards, Stefano > -------------------------------------------------- > Zheng Ruoqin > Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) > ADDR.: No.6 Wenzhu Road, Software Avenue, > Nanjing, 210012, China > MAIL : zhengrq.fnst@cn.fujistu.com > > > >> -----Original Message----- >> From: Stefano Babic <sbabic@denx.de> >> Sent: Thursday, June 4, 2020 3:22 PM >> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com>; >> yocto@yoctoproject.org >> Cc: swupdate <swupdate@googlegroups.com> >> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. >> >> Hi Zheng, >> >> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >> posting here: >> >> On 04.06.20 14:26, zhengruoqin wrote: >>> -c -o mongoose/mongoose.o mongoose/mongoose.c >>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such >>> | file >>> or directory >>> >>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> >>> --- >>> recipes-support/swupdate/swupdate.inc | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/recipes-support/swupdate/swupdate.inc >>> b/recipes-support/swupdate/swupdate.inc >>> index 9ea0a8a..3d9a3fa 100644 >>> --- a/recipes-support/swupdate/swupdate.inc >>> +++ b/recipes-support/swupdate/swupdate.inc >>> @@ -123,6 +123,9 @@ python () { >>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: >>> depends += ' mbedtls' >>> >>> + if 'CONFIG_MONGOOSESSL=y\n' in features: >>> + depends += ' openssl' >>> + >> >> openSSL is not the only supported SSL library, the Webserver runs also with >> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >> used as signing and crypto library. >> >>> if 'CONFIG_JSON=y\n' in features: >>> depends += ' json-c' >>> >>> >>> >>> >>> >> >> Best regards, >> Stefano Babic >> >> >> >> >> > > > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
* [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-08 8:55 ` [swupdate] " Stefano Babic @ 2020-06-18 7:08 ` zhengruoqin 2020-06-18 7:42 ` Stefano Babic 0 siblings, 1 reply; 9+ messages in thread From: zhengruoqin @ 2020-06-18 7:08 UTC (permalink / raw) To: Stefano Babic, yocto; +Cc: swupdate Hi, Stefano I'm sorry I'm so late for your reply. > I do not think the issue exists. I bitbake swupdate with poky and meta-oe without any modifications, depend error happens, isn't it an issue? > In fact, CONFIG_MONGOOSESSL is protected by > CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set > CONFIG_MONGOOSESSL, SSL is already set. I don't find the relationship of CONFIG_MONGOOSESSL and CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS in menuconfig step. --------------------------- $ grep MONGOOSESSL tmp/work/core2-64-poky-linux/swupdate/2020.04-r0/git/ -r | grep Kconfig $ --------------------------- By default CONFIG_SSL_IMPL_OPENSSL will be enable during configure step. But openssl is not in the dependences of swupdate. That' s why error happens by default. Of course, YP users can fix this build error by manuconfig or modified the defconfig. But that's not friendly. In fact, after configuring the CONFIG_SSL_IMPL_OPENSSL will be added into .config file. But swupdate.inc judges which dependence will be added by defconfig file before configure step. So, I thought out two methods to fix this issue to make swupdate can be passed by default. 1. Added openssl into dependences by DEPENDS parameter, such as the following: ---------------------------- --- a/recipes-support/swupdate/swupdate.inc +++ b/recipes-support/swupdate/swupdate.inc @@ -72,7 +72,7 @@ S = "${WORKDIR}/git/" ...... -DEPENDS += "kern-tools-native" +DEPENDS += "kern-tools-native openssl" ---------------------------- 2. Added CONFIG_SSL_IMPL_OPENSSL into defconfig file. ---------------------------- --- a/recipes-support/swupdate/swupdate/defconfig +++ b/recipes-support/swupdate/swupdate/defconfig @@ -96,3 +96,4 @@ CONFIG_SHELLSCRIPTHANDLER=y # CONFIG_ARCHIVE is not set # CONFIG_REMOTE_HANDLER is not set # CONFIG_BOOTLOADERHANDLER is not set +CONFIG_SSL_IMPL_OPENSSL=y ---------------------------- Users who want to use mbedtls instead if openssl can added bbappend file to modify the DEPENDS parameter or defconfig file. How about my suggestion? Best regards Zheng ________________________________________ 发件人: Stefano Babic <sbabic@denx.de> 发送时间: 2020年6月8日 16:55 收件人: Zheng, Ruoqin; Stefano Babic; yocto@yoctoproject.org 抄送: swupdate 主题: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. Hi Zheng, On 08.06.20 10:49, Zheng, Ruoqin wrote: > Hi Stefano > >> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >> posting here: > Thank you, I got it. > >> openSSL is not the only supported SSL library, the Webserver runs also with >> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >> used as signing and crypto library. >> >>> if 'CONFIG_JSON=y\n' in features: >>> depends += ' json-c' > OK, I'll update my patch and send V2 version. > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set CONFIG_MONGOOSESSL, SSL is already set. The only way to go into the issue is to modify directly your defconfig file instead of running "bitbake -c menuconfig" and then apply the diffconfig or the new defconfig. But this is of course a wrong way, exactly as in kernel. Try to run "mnuconfig" and then you can apply the fragment to your diffconfig (fragments are supported, too). Best regards, Stefano > -------------------------------------------------- > Zheng Ruoqin > Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) > ADDR.: No.6 Wenzhu Road, Software Avenue, > Nanjing, 210012, China > MAIL : zhengrq.fnst@cn.fujistu.com > > > >> -----Original Message----- >> From: Stefano Babic <sbabic@denx.de> >> Sent: Thursday, June 4, 2020 3:22 PM >> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com>; >> yocto@yoctoproject.org >> Cc: swupdate <swupdate@googlegroups.com> >> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. >> >> Hi Zheng, >> >> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >> posting here: >> >> On 04.06.20 14:26, zhengruoqin wrote: >>> -c -o mongoose/mongoose.o mongoose/mongoose.c >>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such >>> | file >>> or directory >>> >>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> >>> --- >>> recipes-support/swupdate/swupdate.inc | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/recipes-support/swupdate/swupdate.inc >>> b/recipes-support/swupdate/swupdate.inc >>> index 9ea0a8a..3d9a3fa 100644 >>> --- a/recipes-support/swupdate/swupdate.inc >>> +++ b/recipes-support/swupdate/swupdate.inc >>> @@ -123,6 +123,9 @@ python () { >>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: >>> depends += ' mbedtls' >>> >>> + if 'CONFIG_MONGOOSESSL=y\n' in features: >>> + depends += ' openssl' >>> + >> >> openSSL is not the only supported SSL library, the Webserver runs also with >> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >> used as signing and crypto library. >> >>> if 'CONFIG_JSON=y\n' in features: >>> depends += ' json-c' >>> >>> >>> >>> >>> >> >> Best regards, >> Stefano Babic >> >> >> >> >> > > > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-18 7:08 ` zhengruoqin @ 2020-06-18 7:42 ` Stefano Babic [not found] ` <CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw=mFwL-T66oOe+w@mail.gmail.com> 0 siblings, 1 reply; 9+ messages in thread From: Stefano Babic @ 2020-06-18 7:42 UTC (permalink / raw) To: Zheng, Ruoqin, Stefano Babic, yocto; +Cc: swupdate Hi Zheng, On 18.06.20 09:08, Zheng, Ruoqin wrote: > Hi, Stefano > > > I'm sorry I'm so late for your reply. > > >> I do not think the issue exists. > > > I bitbake swupdate with poky and meta-oe without any modifications, depend error happens, isn't it an issue? > It depends what you are meaning. > >> In fact, CONFIG_MONGOOSESSL is protected by > >> CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set > >> CONFIG_MONGOOSESSL, SSL is already set. > > > I don't find the relationship of CONFIG_MONGOOSESSL and CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS in menuconfig step. In mongoose/Config.in: config MONGOOSESSL bool "SSL support" depends on MONGOOSE depends on SSL_IMPL_OPENSSL || SSL_IMPL_MBEDTLS CONFIG_MONGOOSESSL cannot be set if one of the SSL options is not set, and they can be chosen via SSL menu. > > --------------------------- > > $ grep MONGOOSESSL tmp/work/core2-64-poky-linux/swupdate/2020.04-r0/git/ -r | grep Kconfig > Search for Config.in and Kconfig in root. > $ > > --------------------------- > > By default CONFIG_SSL_IMPL_OPENSSL will be enable during configure step. But openssl is not in the dependences of swupdate. That' s why error happens by default. Because the tree of dependency is SSL, then MBEDTLS or OPENSSL > > Of course, YP users can fix this build error by manuconfig or modified the defconfig. But that's not friendly. > > > > In fact, after configuring the CONFIG_SSL_IMPL_OPENSSL will be added into .config file. But swupdate.inc judges which dependence will be added by defconfig file before configure step. Ok - if you are using fragments, you have to add the missing dependencies on your own, that is true. > > So, I thought out two methods to fix this issue to make swupdate can be passed by default. > > 1. Added openssl into dependences by DEPENDS parameter, such as the following: Ok, so you are saying that the defconfig shipped with meta-swupdate is inconsistent. This can be because it was not updated with last SWUpdate's version, and it does not know about the new CONFIG options. Anyway, think about that SWUpdate should match the MACHINE, that is most users need to configure it and add a swupdate_%.bbappend with own configuration. In any case, if this is broken, I agree it should be fixed. > > ---------------------------- > > --- a/recipes-support/swupdate/swupdate.inc > > +++ b/recipes-support/swupdate/swupdate.inc > > @@ -72,7 +72,7 @@ S = "${WORKDIR}/git/" > > ...... > > -DEPENDS += "kern-tools-native" > > +DEPENDS += "kern-tools-native openssl" No, this is wrong, as users can select MBEDTLS. > > ---------------------------- > > > 2. Added CONFIG_SSL_IMPL_OPENSSL into defconfig file. > > ---------------------------- > > --- a/recipes-support/swupdate/swupdate/defconfig > > +++ b/recipes-support/swupdate/swupdate/defconfig > > @@ -96,3 +96,4 @@ CONFIG_SHELLSCRIPTHANDLER=y > > # CONFIG_ARCHIVE is not set > > # CONFIG_REMOTE_HANDLER is not set > > # CONFIG_BOOTLOADERHANDLER is not set > > +CONFIG_SSL_IMPL_OPENSSL=y > > ---------------------------- > > > Users who want to use mbedtls instead if openssl can added bbappend file to modify the DEPENDS parameter or defconfig file. But they need to remove openssl from DEPENDS and they need to know wich dependencies are already set - it does not work. > > > How about my suggestion? Default defconfig does not match current SWUpdate and it is what should be fixed. Best regards, Stefano > > > Best regards > > Zheng > > ________________________________________ > 发件人: Stefano Babic <sbabic@denx.de> > 发送时间: 2020年6月8日 16:55 > 收件人: Zheng, Ruoqin; Stefano Babic; yocto@yoctoproject.org > 抄送: swupdate > 主题: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. > > Hi Zheng, > > On 08.06.20 10:49, Zheng, Ruoqin wrote: >> Hi Stefano >> >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >>> posting here: >> Thank you, I got it. >> >>> openSSL is not the only supported SSL library, the Webserver runs also with >>> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >>> used as signing and crypto library. >>> >>>> if 'CONFIG_JSON=y\n' in features: >>>> depends += ' json-c' >> OK, I'll update my patch and send V2 version. >> > > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is > protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you > set CONFIG_MONGOOSESSL, SSL is already set. > > The only way to go into the issue is to modify directly your defconfig > file instead of running "bitbake -c menuconfig" and then apply the > diffconfig or the new defconfig. But this is of course a wrong way, > exactly as in kernel. > > Try to run "mnuconfig" and then you can apply the fragment to your > diffconfig (fragments are supported, too). > > Best regards, > Stefano > >> -------------------------------------------------- >> Zheng Ruoqin >> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) >> ADDR.: No.6 Wenzhu Road, Software Avenue, >> Nanjing, 210012, China >> MAIL : zhengrq.fnst@cn.fujistu.com >> >> >> >>> -----Original Message----- >>> From: Stefano Babic <sbabic@denx.de> >>> Sent: Thursday, June 4, 2020 3:22 PM >>> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com>; >>> yocto@yoctoproject.org >>> Cc: swupdate <swupdate@googlegroups.com> >>> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. >>> >>> Hi Zheng, >>> >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are cross- >>> posting here: >>> >>> On 04.06.20 14:26, zhengruoqin wrote: >>>> -c -o mongoose/mongoose.o mongoose/mongoose.c >>>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such >>>> | file >>>> or directory >>>> >>>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com> >>>> --- >>>> recipes-support/swupdate/swupdate.inc | 3 +++ >>>> 1 file changed, 3 insertions(+) >>>> >>>> diff --git a/recipes-support/swupdate/swupdate.inc >>>> b/recipes-support/swupdate/swupdate.inc >>>> index 9ea0a8a..3d9a3fa 100644 >>>> --- a/recipes-support/swupdate/swupdate.inc >>>> +++ b/recipes-support/swupdate/swupdate.inc >>>> @@ -123,6 +123,9 @@ python () { >>>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: >>>> depends += ' mbedtls' >>>> >>>> + if 'CONFIG_MONGOOSESSL=y\n' in features: >>>> + depends += ' openssl' >>>> + >>> >>> openSSL is not the only supported SSL library, the Webserver runs also with >>> mbedTLS. This patch forces to use openSSL and conflicts in case mbedTLS is >>> used as signing and crypto library. >>> >>>> if 'CONFIG_JSON=y\n' in features: >>>> depends += ' json-c' >>>> >>>> >>>> >>>> >>>> >>> >>> Best regards, >>> Stefano Babic >>> >>> >>> >>> >>> >> >> >> > > > -- > ===================================================================== > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de > ===================================================================== > > > > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
[parent not found: <CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw=mFwL-T66oOe+w@mail.gmail.com>]
* Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. [not found] ` <CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw=mFwL-T66oOe+w@mail.gmail.com> @ 2020-06-18 8:22 ` Stefano Babic 2020-06-19 6:20 ` zhengruoqin 0 siblings, 1 reply; 9+ messages in thread From: Stefano Babic @ 2020-06-18 8:22 UTC (permalink / raw) To: Alexander Vickberg, Stefano Babic; +Cc: Zheng, Ruoqin, yocto, swupdate Hi Alexander, On 18.06.20 10:18, Alexander Vickberg wrote: > Hi, > > This issue surfaced as a side effect of my patch in commit 293f273, > sorry about that. The defconfig was not updated and does not include any > default SSL implementation choice but it will be added > by cml1_do_configure (yes "" | make oldconfig). That is after the > DEPENDS variable has been built. I propose the defconfig is updated to > include the default SSL implementation option. Agree on this. Regards, Stefano > > Best Regards > Alexander Vickberg > > Den tors 18 juni 2020 kl 09:43 skrev Stefano Babic <sbabic@denx.de > <mailto:sbabic@denx.de>>: > > Hi Zheng, > > On 18.06.20 09:08, Zheng, Ruoqin wrote: > > Hi, Stefano > > > > > > I'm sorry I'm so late for your reply. > > > > > >> I do not think the issue exists. > > > > > > I bitbake swupdate with poky and meta-oe without any > modifications, depend error happens, isn't it an issue? > > > > It depends what you are meaning. > > > > >> In fact, CONFIG_MONGOOSESSL is protected by > > > >> CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set > > > >> CONFIG_MONGOOSESSL, SSL is already set. > > > > > > I don't find the relationship of CONFIG_MONGOOSESSL and > CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS in menuconfig step. > > In mongoose/Config.in: > > config MONGOOSESSL > bool "SSL support" > depends on MONGOOSE > depends on SSL_IMPL_OPENSSL || SSL_IMPL_MBEDTLS > > CONFIG_MONGOOSESSL cannot be set if one of the SSL options is not set, > and they can be chosen via SSL menu. > > > > > --------------------------- > > > > $ grep MONGOOSESSL > tmp/work/core2-64-poky-linux/swupdate/2020.04-r0/git/ -r | grep Kconfig > > > > Search for Config.in and Kconfig in root. > > > $ > > > > --------------------------- > > > > By default CONFIG_SSL_IMPL_OPENSSL will be enable during > configure step. But openssl is not in the dependences of swupdate. > That' s why error happens by default. > > Because the tree of dependency is SSL, then MBEDTLS or OPENSSL > > > > > Of course, YP users can fix this build error by manuconfig or > modified the defconfig. But that's not friendly. > > > > > > > > In fact, after configuring the CONFIG_SSL_IMPL_OPENSSL will be > added into .config file. But swupdate.inc judges which dependence > will be added by defconfig file before configure step. > > Ok - if you are using fragments, you have to add the missing > dependencies on your own, that is true. > > > > > So, I thought out two methods to fix this issue to make swupdate > can be passed by default. > > > > 1. Added openssl into dependences by DEPENDS parameter, such as > the following: > > Ok, so you are saying that the defconfig shipped with meta-swupdate is > inconsistent. This can be because it was not updated with last > SWUpdate's version, and it does not know about the new CONFIG options. > > Anyway, think about that SWUpdate should match the MACHINE, that is > most > users need to configure it and add a swupdate_%.bbappend with own > configuration. In any case, if this is broken, I agree it should be > fixed. > > > > > ---------------------------- > > > > --- a/recipes-support/swupdate/swupdate.inc > > > > +++ b/recipes-support/swupdate/swupdate.inc > > > > @@ -72,7 +72,7 @@ S = "${WORKDIR}/git/" > > > > ...... > > > > -DEPENDS += "kern-tools-native" > > > > +DEPENDS += "kern-tools-native openssl" > > No, this is wrong, as users can select MBEDTLS. > > > > > ---------------------------- > > > > > > 2. Added CONFIG_SSL_IMPL_OPENSSL into defconfig file. > > > > ---------------------------- > > > > --- a/recipes-support/swupdate/swupdate/defconfig > > > > +++ b/recipes-support/swupdate/swupdate/defconfig > > > > @@ -96,3 +96,4 @@ CONFIG_SHELLSCRIPTHANDLER=y > > > > # CONFIG_ARCHIVE is not set > > > > # CONFIG_REMOTE_HANDLER is not set > > > > # CONFIG_BOOTLOADERHANDLER is not set > > > > +CONFIG_SSL_IMPL_OPENSSL=y > > > > ---------------------------- > > > > > > Users who want to use mbedtls instead if openssl can added > bbappend file to modify the DEPENDS parameter or defconfig file. > > But they need to remove openssl from DEPENDS and they need to know wich > dependencies are already set - it does not work. > > > > > > > How about my suggestion? > > Default defconfig does not match current SWUpdate and it is what should > be fixed. > > Best regards, > Stefano > > > > > > > Best regards > > > > Zheng > > > > ________________________________________ > > 发件人: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> > > 发送时间: 2020年6月8日 16:55 > > 收件人: Zheng, Ruoqin; Stefano Babic; yocto@yoctoproject.org > <mailto:yocto@yoctoproject.org> > > 抄送: swupdate > > 主题: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix > build error of dependence. > > > > Hi Zheng, > > > > On 08.06.20 10:49, Zheng, Ruoqin wrote: > >> Hi Stefano > >> > >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are > cross- > >>> posting here: > >> Thank you, I got it. > >> > >>> openSSL is not the only supported SSL library, the Webserver > runs also with > >>> mbedTLS. This patch forces to use openSSL and conflicts in case > mbedTLS is > >>> used as signing and crypto library. > >>> > >>>> if 'CONFIG_JSON=y\n' in features: > >>>> depends += ' json-c' > >> OK, I'll update my patch and send V2 version. > >> > > > > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is > > protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. > If you > > set CONFIG_MONGOOSESSL, SSL is already set. > > > > The only way to go into the issue is to modify directly your > defconfig > > file instead of running "bitbake -c menuconfig" and then apply the > > diffconfig or the new defconfig. But this is of course a wrong way, > > exactly as in kernel. > > > > Try to run "mnuconfig" and then you can apply the fragment to your > > diffconfig (fragments are supported, too). > > > > Best regards, > > Stefano > > > >> -------------------------------------------------- > >> Zheng Ruoqin > >> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) > >> ADDR.: No.6 Wenzhu Road, Software Avenue, > >> Nanjing, 210012, China > >> MAIL : zhengrq.fnst@cn.fujistu.com > <mailto:zhengrq.fnst@cn.fujistu.com> > >> > >> > >> > >>> -----Original Message----- > >>> From: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> > >>> Sent: Thursday, June 4, 2020 3:22 PM > >>> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com > <mailto:zhengrq.fnst@cn.fujitsu.com>>; > >>> yocto@yoctoproject.org <mailto:yocto@yoctoproject.org> > >>> Cc: swupdate <swupdate@googlegroups.com > <mailto:swupdate@googlegroups.com>> > >>> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of > dependence. > >>> > >>> Hi Zheng, > >>> > >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are > cross- > >>> posting here: > >>> > >>> On 04.06.20 14:26, zhengruoqin wrote: > >>>> -c -o mongoose/mongoose.o mongoose/mongoose.c > >>>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such > >>>> | file > >>>> or directory > >>>> > >>>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com > <mailto:zhengrq.fnst@cn.fujitsu.com>> > >>>> --- > >>>> recipes-support/swupdate/swupdate.inc | 3 +++ > >>>> 1 file changed, 3 insertions(+) > >>>> > >>>> diff --git a/recipes-support/swupdate/swupdate.inc > >>>> b/recipes-support/swupdate/swupdate.inc > >>>> index 9ea0a8a..3d9a3fa 100644 > >>>> --- a/recipes-support/swupdate/swupdate.inc > >>>> +++ b/recipes-support/swupdate/swupdate.inc > >>>> @@ -123,6 +123,9 @@ python () { > >>>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: > >>>> depends += ' mbedtls' > >>>> > >>>> + if 'CONFIG_MONGOOSESSL=y\n' in features: > >>>> + depends += ' openssl' > >>>> + > >>> > >>> openSSL is not the only supported SSL library, the Webserver > runs also with > >>> mbedTLS. This patch forces to use openSSL and conflicts in case > mbedTLS is > >>> used as signing and crypto library. > >>> > >>>> if 'CONFIG_JSON=y\n' in features: > >>>> depends += ' json-c' > >>>> > >>>> > >>>> > >>>> > >>>> > >>> > >>> Best regards, > >>> Stefano Babic > >>> > >>> > >>> > >>> > >>> > >> > >> > >> > > > > > > -- > > ===================================================================== > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: > sbabic@denx.de <mailto:sbabic@denx.de> > > ===================================================================== > > > > > > > > > > -- > ===================================================================== > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: > sbabic@denx.de <mailto:sbabic@denx.de> > ===================================================================== > > -- > You received this message because you are subscribed to the Google > Groups "swupdate" group. > To unsubscribe from this group and stop receiving emails from it, > send an email to swupdate+unsubscribe@googlegroups.com > <mailto:swupdate%2Bunsubscribe@googlegroups.com>. > To view this discussion on the web visit > https://groups.google.com/d/msgid/swupdate/647f2241-e847-1476-a294-7d4b4882e236%40denx.de. > > -- > You received this message because you are subscribed to the Google > Groups "swupdate" group. > To unsubscribe from this group and stop receiving emails from it, send > an email to swupdate+unsubscribe@googlegroups.com > <mailto:swupdate+unsubscribe@googlegroups.com>. > To view this discussion on the web visit > https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com > <https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com?utm_medium=email&utm_source=footer>. -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-18 8:22 ` Stefano Babic @ 2020-06-19 6:20 ` zhengruoqin 2020-06-19 8:53 ` Stefano Babic 0 siblings, 1 reply; 9+ messages in thread From: zhengruoqin @ 2020-06-19 6:20 UTC (permalink / raw) To: Stefano Babic, Alexander Vickberg; +Cc: yocto, swupdate Hi, So, the conclusion is adding a default SSL implementation option(CONFIG_SSL_IMPL_OPENSSL) into defconfig, is it? Is there anybody plans to do it? Or let me submit a patch? Best regards Zheng Ruoqin ________________________________________ From: Stefano Babic <sbabic@denx.de> Sent: Thursday, June 18, 2020 16:22 To: Alexander Vickberg; Stefano Babic Cc: Zheng, Ruoqin; yocto@yoctoproject.org; swupdate Subject: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. Hi Alexander, On 18.06.20 10:18, Alexander Vickberg wrote: > Hi, > > This issue surfaced as a side effect of my patch in commit 293f273, > sorry about that. The defconfig was not updated and does not include any > default SSL implementation choice but it will be added > by cml1_do_configure (yes "" | make oldconfig). That is after the > DEPENDS variable has been built. I propose the defconfig is updated to > include the default SSL implementation option. Agree on this. Regards, Stefano > > Best Regards > Alexander Vickberg > > Den tors 18 juni 2020 kl 09:43 skrev Stefano Babic <sbabic@denx.de > <mailto:sbabic@denx.de>>: > > Hi Zheng, > > On 18.06.20 09:08, Zheng, Ruoqin wrote: > > Hi, Stefano > > > > > > I'm sorry I'm so late for your reply. > > > > > >> I do not think the issue exists. > > > > > > I bitbake swupdate with poky and meta-oe without any > modifications, depend error happens, isn't it an issue? > > > > It depends what you are meaning. > > > > >> In fact, CONFIG_MONGOOSESSL is protected by > > > >> CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set > > > >> CONFIG_MONGOOSESSL, SSL is already set. > > > > > > I don't find the relationship of CONFIG_MONGOOSESSL and > CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS in menuconfig step. > > In mongoose/Config.in: > > config MONGOOSESSL > bool "SSL support" > depends on MONGOOSE > depends on SSL_IMPL_OPENSSL || SSL_IMPL_MBEDTLS > > CONFIG_MONGOOSESSL cannot be set if one of the SSL options is not set, > and they can be chosen via SSL menu. > > > > > --------------------------- > > > > $ grep MONGOOSESSL > tmp/work/core2-64-poky-linux/swupdate/2020.04-r0/git/ -r | grep Kconfig > > > > Search for Config.in and Kconfig in root. > > > $ > > > > --------------------------- > > > > By default CONFIG_SSL_IMPL_OPENSSL will be enable during > configure step. But openssl is not in the dependences of swupdate. > That' s why error happens by default. > > Because the tree of dependency is SSL, then MBEDTLS or OPENSSL > > > > > Of course, YP users can fix this build error by manuconfig or > modified the defconfig. But that's not friendly. > > > > > > > > In fact, after configuring the CONFIG_SSL_IMPL_OPENSSL will be > added into .config file. But swupdate.inc judges which dependence > will be added by defconfig file before configure step. > > Ok - if you are using fragments, you have to add the missing > dependencies on your own, that is true. > > > > > So, I thought out two methods to fix this issue to make swupdate > can be passed by default. > > > > 1. Added openssl into dependences by DEPENDS parameter, such as > the following: > > Ok, so you are saying that the defconfig shipped with meta-swupdate is > inconsistent. This can be because it was not updated with last > SWUpdate's version, and it does not know about the new CONFIG options. > > Anyway, think about that SWUpdate should match the MACHINE, that is > most > users need to configure it and add a swupdate_%.bbappend with own > configuration. In any case, if this is broken, I agree it should be > fixed. > > > > > ---------------------------- > > > > --- a/recipes-support/swupdate/swupdate.inc > > > > +++ b/recipes-support/swupdate/swupdate.inc > > > > @@ -72,7 +72,7 @@ S = "${WORKDIR}/git/" > > > > ...... > > > > -DEPENDS += "kern-tools-native" > > > > +DEPENDS += "kern-tools-native openssl" > > No, this is wrong, as users can select MBEDTLS. > > > > > ---------------------------- > > > > > > 2. Added CONFIG_SSL_IMPL_OPENSSL into defconfig file. > > > > ---------------------------- > > > > --- a/recipes-support/swupdate/swupdate/defconfig > > > > +++ b/recipes-support/swupdate/swupdate/defconfig > > > > @@ -96,3 +96,4 @@ CONFIG_SHELLSCRIPTHANDLER=y > > > > # CONFIG_ARCHIVE is not set > > > > # CONFIG_REMOTE_HANDLER is not set > > > > # CONFIG_BOOTLOADERHANDLER is not set > > > > +CONFIG_SSL_IMPL_OPENSSL=y > > > > ---------------------------- > > > > > > Users who want to use mbedtls instead if openssl can added > bbappend file to modify the DEPENDS parameter or defconfig file. > > But they need to remove openssl from DEPENDS and they need to know wich > dependencies are already set - it does not work. > > > > > > > How about my suggestion? > > Default defconfig does not match current SWUpdate and it is what should > be fixed. > > Best regards, > Stefano > > > > > > > Best regards > > > > Zheng > > > > ________________________________________ > > 发件人: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> > > 发送时间: 2020年6月8日 16:55 > > 收件人: Zheng, Ruoqin; Stefano Babic; yocto@yoctoproject.org > <mailto:yocto@yoctoproject.org> > > 抄送: swupdate > > 主题: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix > build error of dependence. > > > > Hi Zheng, > > > > On 08.06.20 10:49, Zheng, Ruoqin wrote: > >> Hi Stefano > >> > >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are > cross- > >>> posting here: > >> Thank you, I got it. > >> > >>> openSSL is not the only supported SSL library, the Webserver > runs also with > >>> mbedTLS. This patch forces to use openSSL and conflicts in case > mbedTLS is > >>> used as signing and crypto library. > >>> > >>>> if 'CONFIG_JSON=y\n' in features: > >>>> depends += ' json-c' > >> OK, I'll update my patch and send V2 version. > >> > > > > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is > > protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. > If you > > set CONFIG_MONGOOSESSL, SSL is already set. > > > > The only way to go into the issue is to modify directly your > defconfig > > file instead of running "bitbake -c menuconfig" and then apply the > > diffconfig or the new defconfig. But this is of course a wrong way, > > exactly as in kernel. > > > > Try to run "mnuconfig" and then you can apply the fragment to your > > diffconfig (fragments are supported, too). > > > > Best regards, > > Stefano > > > >> -------------------------------------------------- > >> Zheng Ruoqin > >> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) > >> ADDR.: No.6 Wenzhu Road, Software Avenue, > >> Nanjing, 210012, China > >> MAIL : zhengrq.fnst@cn.fujistu.com > <mailto:zhengrq.fnst@cn.fujistu.com> > >> > >> > >> > >>> -----Original Message----- > >>> From: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> > >>> Sent: Thursday, June 4, 2020 3:22 PM > >>> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com > <mailto:zhengrq.fnst@cn.fujitsu.com>>; > >>> yocto@yoctoproject.org <mailto:yocto@yoctoproject.org> > >>> Cc: swupdate <swupdate@googlegroups.com > <mailto:swupdate@googlegroups.com>> > >>> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of > dependence. > >>> > >>> Hi Zheng, > >>> > >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are > cross- > >>> posting here: > >>> > >>> On 04.06.20 14:26, zhengruoqin wrote: > >>>> -c -o mongoose/mongoose.o mongoose/mongoose.c > >>>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such > >>>> | file > >>>> or directory > >>>> > >>>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com > <mailto:zhengrq.fnst@cn.fujitsu.com>> > >>>> --- > >>>> recipes-support/swupdate/swupdate.inc | 3 +++ > >>>> 1 file changed, 3 insertions(+) > >>>> > >>>> diff --git a/recipes-support/swupdate/swupdate.inc > >>>> b/recipes-support/swupdate/swupdate.inc > >>>> index 9ea0a8a..3d9a3fa 100644 > >>>> --- a/recipes-support/swupdate/swupdate.inc > >>>> +++ b/recipes-support/swupdate/swupdate.inc > >>>> @@ -123,6 +123,9 @@ python () { > >>>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: > >>>> depends += ' mbedtls' > >>>> > >>>> + if 'CONFIG_MONGOOSESSL=y\n' in features: > >>>> + depends += ' openssl' > >>>> + > >>> > >>> openSSL is not the only supported SSL library, the Webserver > runs also with > >>> mbedTLS. This patch forces to use openSSL and conflicts in case > mbedTLS is > >>> used as signing and crypto library. > >>> > >>>> if 'CONFIG_JSON=y\n' in features: > >>>> depends += ' json-c' > >>>> > >>>> > >>>> > >>>> > >>>> > >>> > >>> Best regards, > >>> Stefano Babic > >>> > >>> > >>> > >>> > >>> > >> > >> > >> > > > > > > -- > > ===================================================================== > > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: > sbabic@denx.de <mailto:sbabic@denx.de> > > ===================================================================== > > > > > > > > > > -- > ===================================================================== > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: > sbabic@denx.de <mailto:sbabic@denx.de> > ===================================================================== > > -- > You received this message because you are subscribed to the Google > Groups "swupdate" group. > To unsubscribe from this group and stop receiving emails from it, > send an email to swupdate+unsubscribe@googlegroups.com > <mailto:swupdate%2Bunsubscribe@googlegroups.com>. > To view this discussion on the web visit > https://groups.google.com/d/msgid/swupdate/647f2241-e847-1476-a294-7d4b4882e236%40denx.de. > > -- > You received this message because you are subscribed to the Google > Groups "swupdate" group. > To unsubscribe from this group and stop receiving emails from it, send > an email to swupdate+unsubscribe@googlegroups.com > <mailto:swupdate+unsubscribe@googlegroups.com>. > To view this discussion on the web visit > https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com > <https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com?utm_medium=email&utm_source=footer>. -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. 2020-06-19 6:20 ` zhengruoqin @ 2020-06-19 8:53 ` Stefano Babic 0 siblings, 0 replies; 9+ messages in thread From: Stefano Babic @ 2020-06-19 8:53 UTC (permalink / raw) To: Zheng, Ruoqin, Stefano Babic, Alexander Vickberg; +Cc: yocto, swupdate Hi Zheng, On 19.06.20 08:20, Zheng, Ruoqin wrote: > Hi, > > So, the conclusion is adding a default SSL implementation option(CONFIG_SSL_IMPL_OPENSSL) into defconfig, is it? Yes > Is there anybody plans to do it? Or let me submit a patch? Just send the patch to SWUpdate's ML, I will integrate then. Regards, Stefano > > Best regards > Zheng Ruoqin > > ________________________________________ > From: Stefano Babic <sbabic@denx.de> > Sent: Thursday, June 18, 2020 16:22 > To: Alexander Vickberg; Stefano Babic > Cc: Zheng, Ruoqin; yocto@yoctoproject.org; swupdate > Subject: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix build error of dependence. > > Hi Alexander, > > On 18.06.20 10:18, Alexander Vickberg wrote: >> Hi, >> >> This issue surfaced as a side effect of my patch in commit 293f273, >> sorry about that. The defconfig was not updated and does not include any >> default SSL implementation choice but it will be added >> by cml1_do_configure (yes "" | make oldconfig). That is after the >> DEPENDS variable has been built. I propose the defconfig is updated to >> include the default SSL implementation option. > > Agree on this. > > Regards, > Stefano > >> >> Best Regards >> Alexander Vickberg >> >> Den tors 18 juni 2020 kl 09:43 skrev Stefano Babic <sbabic@denx.de >> <mailto:sbabic@denx.de>>: >> >> Hi Zheng, >> >> On 18.06.20 09:08, Zheng, Ruoqin wrote: >> > Hi, Stefano >> > >> > >> > I'm sorry I'm so late for your reply. >> > >> > >> >> I do not think the issue exists. >> > >> > >> > I bitbake swupdate with poky and meta-oe without any >> modifications, depend error happens, isn't it an issue? >> > >> >> It depends what you are meaning. >> >> > >> >> In fact, CONFIG_MONGOOSESSL is protected by >> > >> >> CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. If you set >> > >> >> CONFIG_MONGOOSESSL, SSL is already set. >> > >> > >> > I don't find the relationship of CONFIG_MONGOOSESSL and >> CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS in menuconfig step. >> >> In mongoose/Config.in: >> >> config MONGOOSESSL >> bool "SSL support" >> depends on MONGOOSE >> depends on SSL_IMPL_OPENSSL || SSL_IMPL_MBEDTLS >> >> CONFIG_MONGOOSESSL cannot be set if one of the SSL options is not set, >> and they can be chosen via SSL menu. >> >> > >> > --------------------------- >> > >> > $ grep MONGOOSESSL >> tmp/work/core2-64-poky-linux/swupdate/2020.04-r0/git/ -r | grep Kconfig >> > >> >> Search for Config.in and Kconfig in root. >> >> > $ >> > >> > --------------------------- >> > >> > By default CONFIG_SSL_IMPL_OPENSSL will be enable during >> configure step. But openssl is not in the dependences of swupdate. >> That' s why error happens by default. >> >> Because the tree of dependency is SSL, then MBEDTLS or OPENSSL >> >> > >> > Of course, YP users can fix this build error by manuconfig or >> modified the defconfig. But that's not friendly. >> > >> > >> > >> > In fact, after configuring the CONFIG_SSL_IMPL_OPENSSL will be >> added into .config file. But swupdate.inc judges which dependence >> will be added by defconfig file before configure step. >> >> Ok - if you are using fragments, you have to add the missing >> dependencies on your own, that is true. >> >> > >> > So, I thought out two methods to fix this issue to make swupdate >> can be passed by default. >> > >> > 1. Added openssl into dependences by DEPENDS parameter, such as >> the following: >> >> Ok, so you are saying that the defconfig shipped with meta-swupdate is >> inconsistent. This can be because it was not updated with last >> SWUpdate's version, and it does not know about the new CONFIG options. >> >> Anyway, think about that SWUpdate should match the MACHINE, that is >> most >> users need to configure it and add a swupdate_%.bbappend with own >> configuration. In any case, if this is broken, I agree it should be >> fixed. >> >> > >> > ---------------------------- >> > >> > --- a/recipes-support/swupdate/swupdate.inc >> > >> > +++ b/recipes-support/swupdate/swupdate.inc >> > >> > @@ -72,7 +72,7 @@ S = "${WORKDIR}/git/" >> > >> > ...... >> > >> > -DEPENDS += "kern-tools-native" >> > >> > +DEPENDS += "kern-tools-native openssl" >> >> No, this is wrong, as users can select MBEDTLS. >> >> > >> > ---------------------------- >> > >> > >> > 2. Added CONFIG_SSL_IMPL_OPENSSL into defconfig file. >> > >> > ---------------------------- >> > >> > --- a/recipes-support/swupdate/swupdate/defconfig >> > >> > +++ b/recipes-support/swupdate/swupdate/defconfig >> > >> > @@ -96,3 +96,4 @@ CONFIG_SHELLSCRIPTHANDLER=y >> > >> > # CONFIG_ARCHIVE is not set >> > >> > # CONFIG_REMOTE_HANDLER is not set >> > >> > # CONFIG_BOOTLOADERHANDLER is not set >> > >> > +CONFIG_SSL_IMPL_OPENSSL=y >> > >> > ---------------------------- >> > >> > >> > Users who want to use mbedtls instead if openssl can added >> bbappend file to modify the DEPENDS parameter or defconfig file. >> >> But they need to remove openssl from DEPENDS and they need to know wich >> dependencies are already set - it does not work. >> >> > >> > >> > How about my suggestion? >> >> Default defconfig does not match current SWUpdate and it is what should >> be fixed. >> >> Best regards, >> Stefano >> >> > >> > >> > Best regards >> > >> > Zheng >> > >> > ________________________________________ >> > 发件人: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> >> > 发送时间: 2020年6月8日 16:55 >> > 收件人: Zheng, Ruoqin; Stefano Babic; yocto@yoctoproject.org >> <mailto:yocto@yoctoproject.org> >> > 抄送: swupdate >> > 主题: Re: [swupdate] RE: [yocto] [PATCH] [meta-swupdate] Fix >> build error of dependence. >> > >> > Hi Zheng, >> > >> > On 08.06.20 10:49, Zheng, Ruoqin wrote: >> >> Hi Stefano >> >> >> >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are >> cross- >> >>> posting here: >> >> Thank you, I got it. >> >> >> >>> openSSL is not the only supported SSL library, the Webserver >> runs also with >> >>> mbedTLS. This patch forces to use openSSL and conflicts in case >> mbedTLS is >> >>> used as signing and crypto library. >> >>> >> >>>> if 'CONFIG_JSON=y\n' in features: >> >>>> depends += ' json-c' >> >> OK, I'll update my patch and send V2 version. >> >> >> > >> > I do not think the issue exists. In fact, CONFIG_MONGOOSESSL is >> > protected by CONFIG_SSL_IMPL_OPENSSL or CONFIG_SSL_IMPL_MBEDTLS. >> If you >> > set CONFIG_MONGOOSESSL, SSL is already set. >> > >> > The only way to go into the issue is to modify directly your >> defconfig >> > file instead of running "bitbake -c menuconfig" and then apply the >> > diffconfig or the new defconfig. But this is of course a wrong way, >> > exactly as in kernel. >> > >> > Try to run "mnuconfig" and then you can apply the fragment to your >> > diffconfig (fragments are supported, too). >> > >> > Best regards, >> > Stefano >> > >> >> -------------------------------------------------- >> >> Zheng Ruoqin >> >> Nanjing Fujitsu Nanda Software Tech. Co., Ltd.(FNST) >> >> ADDR.: No.6 Wenzhu Road, Software Avenue, >> >> Nanjing, 210012, China >> >> MAIL : zhengrq.fnst@cn.fujistu.com >> <mailto:zhengrq.fnst@cn.fujistu.com> >> >> >> >> >> >> >> >>> -----Original Message----- >> >>> From: Stefano Babic <sbabic@denx.de <mailto:sbabic@denx.de>> >> >>> Sent: Thursday, June 4, 2020 3:22 PM >> >>> To: Zheng, Ruoqin/郑 若钦 <zhengrq.fnst@cn.fujitsu.com >> <mailto:zhengrq.fnst@cn.fujitsu.com>>; >> >>> yocto@yoctoproject.org <mailto:yocto@yoctoproject.org> >> >>> Cc: swupdate <swupdate@googlegroups.com >> <mailto:swupdate@googlegroups.com>> >> >>> Subject: Re: [yocto] [PATCH] [meta-swupdate] Fix build error of >> dependence. >> >>> >> >>> Hi Zheng, >> >>> >> >>> there is a ML for SWUpdate and meta-swupdate (see CC), we are >> cross- >> >>> posting here: >> >>> >> >>> On 04.06.20 14:26, zhengruoqin wrote: >> >>>> -c -o mongoose/mongoose.o mongoose/mongoose.c >> >>>> | mongoose/mongoose.c:4496:10: fatal error: openssl/ssl.h: No such >> >>>> | file >> >>>> or directory >> >>>> >> >>>> Signed-off-by: Zheng Ruoqing <zhengrq.fnst@cn.fujitsu.com >> <mailto:zhengrq.fnst@cn.fujitsu.com>> >> >>>> --- >> >>>> recipes-support/swupdate/swupdate.inc | 3 +++ >> >>>> 1 file changed, 3 insertions(+) >> >>>> >> >>>> diff --git a/recipes-support/swupdate/swupdate.inc >> >>>> b/recipes-support/swupdate/swupdate.inc >> >>>> index 9ea0a8a..3d9a3fa 100644 >> >>>> --- a/recipes-support/swupdate/swupdate.inc >> >>>> +++ b/recipes-support/swupdate/swupdate.inc >> >>>> @@ -123,6 +123,9 @@ python () { >> >>>> elif 'CONFIG_SSL_IMPL_MBEDTLS=y\n' in features: >> >>>> depends += ' mbedtls' >> >>>> >> >>>> + if 'CONFIG_MONGOOSESSL=y\n' in features: >> >>>> + depends += ' openssl' >> >>>> + >> >>> >> >>> openSSL is not the only supported SSL library, the Webserver >> runs also with >> >>> mbedTLS. This patch forces to use openSSL and conflicts in case >> mbedTLS is >> >>> used as signing and crypto library. >> >>> >> >>>> if 'CONFIG_JSON=y\n' in features: >> >>>> depends += ' json-c' >> >>>> >> >>>> >> >>>> >> >>>> >> >>>> >> >>> >> >>> Best regards, >> >>> Stefano Babic >> >>> >> >>> >> >>> >> >>> >> >>> >> >> >> >> >> >> >> > >> > >> > -- >> > ===================================================================== >> > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk >> > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany >> > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: >> sbabic@denx.de <mailto:sbabic@denx.de> >> > ===================================================================== >> > >> > >> > >> > >> >> -- >> ===================================================================== >> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk >> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany >> Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: >> sbabic@denx.de <mailto:sbabic@denx.de> >> ===================================================================== >> >> -- >> You received this message because you are subscribed to the Google >> Groups "swupdate" group. >> To unsubscribe from this group and stop receiving emails from it, >> send an email to swupdate+unsubscribe@googlegroups.com >> <mailto:swupdate%2Bunsubscribe@googlegroups.com>. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/swupdate/647f2241-e847-1476-a294-7d4b4882e236%40denx.de. >> >> -- >> You received this message because you are subscribed to the Google >> Groups "swupdate" group. >> To unsubscribe from this group and stop receiving emails from it, send >> an email to swupdate+unsubscribe@googlegroups.com >> <mailto:swupdate+unsubscribe@googlegroups.com>. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com >> <https://groups.google.com/d/msgid/swupdate/CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw%3DmFwL-T66oOe%2Bw%40mail.gmail.com?utm_medium=email&utm_source=footer>. > > -- > ===================================================================== > DENX Software Engineering GmbH, Managing Director: Wolfgang Denk > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany > Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de > ===================================================================== > > > > -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic@denx.de ===================================================================== ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-06-19 8:53 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-04 12:26 [PATCH] [meta-swupdate] Fix build error of dependence zhengruoqin
2020-06-04 7:22 ` [yocto] " Stefano Babic
2020-06-08 8:49 ` zhengruoqin
2020-06-08 8:55 ` [swupdate] " Stefano Babic
2020-06-18 7:08 ` zhengruoqin
2020-06-18 7:42 ` Stefano Babic
[not found] ` <CAAxSGfY9uDN708Cy5LFd3cgeRMgfBT85o8Pw=mFwL-T66oOe+w@mail.gmail.com>
2020-06-18 8:22 ` Stefano Babic
2020-06-19 6:20 ` zhengruoqin
2020-06-19 8:53 ` Stefano Babic
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.