diff mbox series

[meta-gnome] appstream-glib: fix manpage generation

Message ID 20220920083157.660480-1-f_l_k@t-online.de
State New
Headers show
Series [meta-gnome] appstream-glib: fix manpage generation | expand

Commit Message

Markus Volk Sept. 20, 2022, 8:31 a.m. UTC
Creating manpages is set by default but fails with:

| I/O error : Attempt to load network entity http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
| warning: failed to load external entity "http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl"
| cannot parse http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl

Disable manpage generation by default but fix the dependencies, so it wouldn't fail

Signed-off-by: Markus Volk <f_l_k@t-online.de>
---
 .../recipes-support/appstream-glib/appstream-glib_0.7.18.bb    | 3 +++
 1 file changed, 3 insertions(+)

Comments

Mittal, Anuj Sept. 20, 2022, 1:43 p.m. UTC | #1
On Tue, 2022-09-20 at 10:31 +0200, Markus Volk wrote:
> Creating manpages is set by default but fails with:
> 
> > I/O error : Attempt to load network entity
> > http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
> > warning: failed to load external entity
> > "http://docbook.sourceforge.net/release/xsl/current/manpages/docboo
> > k.xsl"
> > cannot parse
> > http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl
> 
> Disable manpage generation by default but fix the dependencies, so it
> wouldn't fail
> 
> Signed-off-by: Markus Volk <f_l_k@t-online.de>
> ---
>  .../recipes-support/appstream-glib/appstream-glib_0.7.18.bb    | 3
> +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/meta-gnome/recipes-support/appstream-glib/appstream-
> glib_0.7.18.bb b/meta-gnome/recipes-support/appstream-glib/appstream-
> glib_0.7.18.bb
> index 45a28ed96..b5d60b13a 100644
> --- a/meta-gnome/recipes-support/appstream-glib/appstream-
> glib_0.7.18.bb
> +++ b/meta-gnome/recipes-support/appstream-glib/appstream-
> glib_0.7.18.bb
> @@ -26,6 +26,9 @@ EXTRA_OEMESON = " \
>      -Drpm=false \
>  "
>  
> +PACKAGECONFIG ?= ""
> +PACKAGECONFIG[man] = "-Dman=true,-Dman=false,docbook-xml-dtd4-native
> docbook-xsl-stylesheets-native libxslt-native"

I think this should also inherit manpages bbclass for it to be enabled
correctly. The PACKAGECONFIG will also have to be changed to manpages
then.

Thanks,

Anuj

> +
>  FILES:${PN} += "${libdir}/asb-plugins-5"
>  
>  FILES:${PN}-dev += " \
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#98900):
> https://lists.openembedded.org/g/openembedded-devel/message/98900
> Mute This Topic: https://lists.openembedded.org/mt/93799217/3616702
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe:
> https://lists.openembedded.org/g/openembedded-devel/unsub [
> anuj.mittal@intel.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Markus Volk Sept. 20, 2022, 1:52 p.m. UTC | #2
Thanks for the suggestions. I've sent a v2

Am Di, 20. Sep 2022 um 13:43:34 +0000 schrieb Anuj Mittal 
<anuj.mittal@intel.com>:
> On Tue, 2022-09-20 at 10:31 +0200, Markus Volk wrote:
>>  Creating manpages is set by default but fails with:
>> 
>>  > I/O error : Attempt to load network entity
>>  > 
>> <http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl>
>>  > warning: failed to load external entity
>>  > 
>> "<http://docbook.sourceforge.net/release/xsl/current/manpages/docboo>
>>  > k.xsl"
>>  > cannot parse
>>  > 
>> <http://docbook.sourceforge.net/release/xsl/current/manpages/docbook.xsl>
>> 
>>  Disable manpage generation by default but fix the dependencies, so 
>> it
>>  wouldn't fail
>> 
>>  Signed-off-by: Markus Volk <f_l_k@t-online.de 
>> <mailto:f_l_k@t-online.de>>
>>  ---
>>   .../recipes-support/appstream-glib/appstream-glib_0.7.18.bb    | 3
>>  +++
>>   1 file changed, 3 insertions(+)
>> 
>>  diff --git a/meta-gnome/recipes-support/appstream-glib/appstream-
>>  glib_0.7.18.bb 
>> b/meta-gnome/recipes-support/appstream-glib/appstream-
>>  glib_0.7.18.bb
>>  index 45a28ed96..b5d60b13a 100644
>>  --- a/meta-gnome/recipes-support/appstream-glib/appstream-
>>  glib_0.7.18.bb
>>  +++ b/meta-gnome/recipes-support/appstream-glib/appstream-
>>  glib_0.7.18.bb
>>  @@ -26,6 +26,9 @@ EXTRA_OEMESON = " \
>>       -Drpm=false \
>>   "
>> 
>>  +PACKAGECONFIG ?= ""
>>  +PACKAGECONFIG[man] = 
>> "-Dman=true,-Dman=false,docbook-xml-dtd4-native
>>  docbook-xsl-stylesheets-native libxslt-native"
> 
> I think this should also inherit manpages bbclass for it to be enabled
> correctly. The PACKAGECONFIG will also have to be changed to manpages
> then.
> 
> Thanks,
> 
> Anuj
> 
>>  +
>>   FILES:${PN} += "${libdir}/asb-plugins-5"
>> 
>>   FILES:${PN}-dev += " \
>> 
>> 
>> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#98907): 
> <https://lists.openembedded.org/g/openembedded-devel/message/98907>
> Mute This Topic: <https://lists.openembedded.org/mt/93799217/3618223>
> Group Owner: openembedded-devel+owner@lists.openembedded.org 
> <mailto:openembedded-devel+owner@lists.openembedded.org>
> Unsubscribe: 
> <https://lists.openembedded.org/g/openembedded-devel/unsub> 
> [f_l_k@t-online.de <mailto:f_l_k@t-online.de>]
> -=-=-=-=-=-=-=-=-=-=-=-
>
diff mbox series

Patch

diff --git a/meta-gnome/recipes-support/appstream-glib/appstream-glib_0.7.18.bb b/meta-gnome/recipes-support/appstream-glib/appstream-glib_0.7.18.bb
index 45a28ed96..b5d60b13a 100644
--- a/meta-gnome/recipes-support/appstream-glib/appstream-glib_0.7.18.bb
+++ b/meta-gnome/recipes-support/appstream-glib/appstream-glib_0.7.18.bb
@@ -26,6 +26,9 @@  EXTRA_OEMESON = " \
     -Drpm=false \
 "
 
+PACKAGECONFIG ?= ""
+PACKAGECONFIG[man] = "-Dman=true,-Dman=false,docbook-xml-dtd4-native docbook-xsl-stylesheets-native libxslt-native"
+
 FILES:${PN} += "${libdir}/asb-plugins-5"
 
 FILES:${PN}-dev += " \