[PATCH] xmlcatalog: add exit 0

[PATCH] xmlcatalog: add exit 0

[Jeremy Puhlman]

In most sysroot creation steps, the sysroots components are installed in
depenancy order. However, in the case of build-sysroots, since it is
not using standard dependacies, post installs that use xmlcatalog fail
because xmlcatalog is not installed. Since the last execution in the
script is xmlcatalog, the script returns 127.

Signed-off-by: Jeremy A. Puhlman <jpuhlman@mvista.com>
---
 meta/classes/xmlcatalog.bbclass | 1 +
 1 file changed, 1 insertion(+)

diff --git a/meta/classes/xmlcatalog.bbclass b/meta/classes/xmlcatalog.bbclass
index ae4811fdeb..e8d73abcf5 100644
--- a/meta/classes/xmlcatalog.bbclass
+++ b/meta/classes/xmlcatalog.bbclass
@@ -15,6 +15,7 @@ xmlcatalog_complete() {
 	for CATALOG in ${XMLCATALOGS}; do
 		xmlcatalog --noout --add nextCatalog unused file://$CATALOG $ROOTCATALOG
 	done
+	exit 0
 }
 
 xmlcatalog_sstate_postinst() {
-- 
2.20.1

Re: [OE-core] [PATCH] xmlcatalog: add exit 0

[Alexander Kanavin]

[-- Attachment #1: Type: text/plain, Size: 1250 bytes --]

There is already a patch for fixing this properly:
http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=fdee2d49fc9139ff7e0b5a8b1b6476224540ca57

Does it work for you?

Alex

On Tue, 23 Jun 2020 at 02:02, Jeremy Puhlman <jpuhlman@mvista.com> wrote:

> In most sysroot creation steps, the sysroots components are installed in
> depenancy order. However, in the case of build-sysroots, since it is
> not using standard dependacies, post installs that use xmlcatalog fail
> because xmlcatalog is not installed. Since the last execution in the
> script is xmlcatalog, the script returns 127.
>
> Signed-off-by: Jeremy A. Puhlman <jpuhlman@mvista.com>
> ---
>  meta/classes/xmlcatalog.bbclass | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/meta/classes/xmlcatalog.bbclass
> b/meta/classes/xmlcatalog.bbclass
> index ae4811fdeb..e8d73abcf5 100644
> --- a/meta/classes/xmlcatalog.bbclass
> +++ b/meta/classes/xmlcatalog.bbclass
> @@ -15,6 +15,7 @@ xmlcatalog_complete() {
>         for CATALOG in ${XMLCATALOGS}; do
>                 xmlcatalog --noout --add nextCatalog unused
> file://$CATALOG $ROOTCATALOG
>         done
> +       exit 0
>  }
>
>  xmlcatalog_sstate_postinst() {
> --
> 2.20.1
>
> 
>

[-- Attachment #2: Type: text/html, Size: 1843 bytes --]

Re: [OE-core] [PATCH] xmlcatalog: add exit 0

[Jeremy Puhlman]

[-- Attachment #1: Type: text/plain, Size: 1912 bytes --]



On 6/23/2020 12:16 AM, Alexander Kanavin wrote:
> There is already a patch for fixing this properly:
> http://git.yoctoproject.org/cgit/cgit.cgi/poky/commit/?id=fdee2d49fc9139ff7e0b5a8b1b6476224540ca57
>
> Does it work for you?

Yup. That works.

>
> Alex
>
> On Tue, 23 Jun 2020 at 02:02, Jeremy Puhlman <jpuhlman@mvista.com 
> <mailto:jpuhlman@mvista.com>> wrote:
>
>     In most sysroot creation steps, the sysroots components are
>     installed in
>     depenancy order. However, in the case of build-sysroots, since it is
>     not using standard dependacies, post installs that use xmlcatalog fail
>     because xmlcatalog is not installed. Since the last execution in the
>     script is xmlcatalog, the script returns 127.
>
>     Signed-off-by: Jeremy A. Puhlman <jpuhlman@mvista.com
>     <mailto:jpuhlman@mvista.com>>
>     ---
>      meta/classes/xmlcatalog.bbclass | 1 +
>      1 file changed, 1 insertion(+)
>
>     diff --git a/meta/classes/xmlcatalog.bbclass
>     b/meta/classes/xmlcatalog.bbclass
>     index ae4811fdeb..e8d73abcf5 100644
>     --- a/meta/classes/xmlcatalog.bbclass
>     +++ b/meta/classes/xmlcatalog.bbclass
>     @@ -15,6 +15,7 @@ xmlcatalog_complete() {
>             for CATALOG in ${XMLCATALOGS}; do
>                     xmlcatalog --noout --add nextCatalog unused
>     file://$CATALOG $ROOTCATALOG
>             done
>     +       exit 0
>      }
>
>      xmlcatalog_sstate_postinst() {
>     -- 
>     2.20.1
>
>     
>
>
> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> 
> 	Virus-free. www.avg.com 
> <http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient> 
>
>
> <#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

-- 
Jeremy A. Puhlman
jpuhlman@mvista.com


[-- Attachment #2: Type: text/html, Size: 4673 bytes --]