Re: [Buildroot] [PATCH v2] package/python-pygments: support host package

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Takumi,

On Sun,  1 Oct 2023 12:42:46 +0900
Takumi Takahashi <takumiiinn@gmail.com> wrote:

> The reason this patch is needed is because gtk-doc requires pygments.
> The final goal is to submit a patch for lightdm, but since gtk-doc is
> required to build lightdm, pygments will also be required.
> 
> Signed-off-by: Takumi Takahashi <takumiiinn@gmail.com>
> ---
>  package/python-pygments/python-pygments.mk | 1 +
>  1 file changed, 1 insertion(+)

Thanks for your patch, and sorry for the long delay with which we are
getting back to you. However, this patch, as well as the patching
adding itstool need to go together, in a patch series that also adds
the lightdm package. Indeed we want to see the big picture, and not
have those two "orphan" host packages without the actual package using
them.

That being said, I think the approach should rather to make sure that
lightdm doesn't require gtk-doc. In the context of Buildroot, we are
not interested in building the documentation. Are you sure that lightdm
absolutely requires gtk-doc? I see it's using the GTK_DOC_CHECK m4
macro, so it supports --disable-gtk-doc, and that should do the trick.
If not, it should be fixed to do the trick :)

If you're still interested into this, could you send the full series of
patches including lightdm?

Thanks a lot!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot