Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/libxkbcommon: provide xkbregistry when libxml2 is enabled
@ 2026-05-27 19:53 Raphaël Gallais-Pou
  2026-06-24 15:51 ` Romain Naour via buildroot
  0 siblings, 1 reply; 3+ messages in thread
From: Raphaël Gallais-Pou @ 2026-05-27 19:53 UTC (permalink / raw)
  To: buildroot; +Cc: Thomas Petazzoni

xkbregistry is disabled by default.  Yet, it only needs libxml2 to be
built, which is already supported.

Add the compile flags to build xkbregistry when possible.

Signed-off-by: Raphaël Gallais-Pou <rgallaispou@gmail.com>
---
 package/libxkbcommon/libxkbcommon.mk | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/package/libxkbcommon/libxkbcommon.mk b/package/libxkbcommon/libxkbcommon.mk
index 21a2b937ef02..98154369b419 100644
--- a/package/libxkbcommon/libxkbcommon.mk
+++ b/package/libxkbcommon/libxkbcommon.mk
@@ -12,8 +12,7 @@ LIBXKBCOMMON_CPE_ID_VENDOR = xkbcommon
 LIBXKBCOMMON_INSTALL_STAGING = YES
 LIBXKBCOMMON_DEPENDENCIES = host-bison host-flex
 LIBXKBCOMMON_CONF_OPTS = \
-	-Denable-docs=false \
-	-Denable-xkbregistry=false
+	-Denable-docs=false
 
 ifeq ($(BR2_PACKAGE_XORG7),y)
 LIBXKBCOMMON_CONF_OPTS += -Denable-x11=true
@@ -22,6 +21,13 @@ else
 LIBXKBCOMMON_CONF_OPTS += -Denable-x11=false
 endif
 
+ifeq ($(BR2_PACKAGE_LIBXML2),y)
+LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=true
+LIBXKBCOMMON_DEPENDENCIES += libxml2
+else
+LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=false
+endif
+
 ifeq ($(BR2_PACKAGE_LIBXKBCOMMON_TOOLS),y)
 LIBXKBCOMMON_CONF_OPTS += -Denable-tools=true
 else

---
base-commit: 08f3757b0f2133b0ed50e1ef3d7a5a1d9c80de1c
change-id: 20260527-master-2e5299aaa8ca

Best regards,
--  
Raphaël Gallais-Pou <rgallaispou@gmail.com>

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH] package/libxkbcommon: provide xkbregistry when libxml2 is enabled
  2026-05-27 19:53 [Buildroot] [PATCH] package/libxkbcommon: provide xkbregistry when libxml2 is enabled Raphaël Gallais-Pou
@ 2026-06-24 15:51 ` Romain Naour via buildroot
  2026-07-06 15:34   ` Raphaël Gallais-Pou
  0 siblings, 1 reply; 3+ messages in thread
From: Romain Naour via buildroot @ 2026-06-24 15:51 UTC (permalink / raw)
  To: Raphaël Gallais-Pou, buildroot; +Cc: Thomas Petazzoni

Hello Raphaël,

Le 27/05/2026 à 21:53, Raphaël Gallais-Pou a écrit :
> xkbregistry is disabled by default.  Yet, it only needs libxml2 to be
> built, which is already supported.
> 
> Add the compile flags to build xkbregistry when possible.
> 
> Signed-off-by: Raphaël Gallais-Pou <rgallaispou@gmail.com>
> ---
>  package/libxkbcommon/libxkbcommon.mk | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/package/libxkbcommon/libxkbcommon.mk b/package/libxkbcommon/libxkbcommon.mk
> index 21a2b937ef02..98154369b419 100644
> --- a/package/libxkbcommon/libxkbcommon.mk
> +++ b/package/libxkbcommon/libxkbcommon.mk
> @@ -12,8 +12,7 @@ LIBXKBCOMMON_CPE_ID_VENDOR = xkbcommon
>  LIBXKBCOMMON_INSTALL_STAGING = YES
>  LIBXKBCOMMON_DEPENDENCIES = host-bison host-flex
>  LIBXKBCOMMON_CONF_OPTS = \
> -	-Denable-docs=false \
> -	-Denable-xkbregistry=false

xkbregistry has been disabled on purpose when libxkbcommon has been updated to
1.0.1 [1].

It seems to requires xkeyboard-config at runtime [2], can you verify?

[1]
https://gitlab.com/buildroot.org/buildroot/-/commit/1791bc30a5b19bb52366375d03e66348242ff25a

[2]
https://github.com/xkbcommon/libxkbcommon/blob/2c295b10aa87e9bd8c11954adcf7e92b031b92f9/PACKAGING#L61

Best regards,
Romain

> +	-Denable-docs=false
>  
>  ifeq ($(BR2_PACKAGE_XORG7),y)
>  LIBXKBCOMMON_CONF_OPTS += -Denable-x11=true
> @@ -22,6 +21,13 @@ else
>  LIBXKBCOMMON_CONF_OPTS += -Denable-x11=false
>  endif
>  
> +ifeq ($(BR2_PACKAGE_LIBXML2),y)
> +LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=true
> +LIBXKBCOMMON_DEPENDENCIES += libxml2
> +else
> +LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=false
> +endif
> +
>  ifeq ($(BR2_PACKAGE_LIBXKBCOMMON_TOOLS),y)
>  LIBXKBCOMMON_CONF_OPTS += -Denable-tools=true
>  else
> 
> ---
> base-commit: 08f3757b0f2133b0ed50e1ef3d7a5a1d9c80de1c
> change-id: 20260527-master-2e5299aaa8ca
> 
> Best regards,
> --  
> Raphaël Gallais-Pou <rgallaispou@gmail.com>
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH] package/libxkbcommon: provide xkbregistry when libxml2 is enabled
  2026-06-24 15:51 ` Romain Naour via buildroot
@ 2026-07-06 15:34   ` Raphaël Gallais-Pou
  0 siblings, 0 replies; 3+ messages in thread
From: Raphaël Gallais-Pou @ 2026-07-06 15:34 UTC (permalink / raw)
  To: Romain Naour, buildroot; +Cc: Thomas Petazzoni



On 6/24/26 17:51, Romain Naour wrote:
> Hello Raphaël,

Hi,

> 
> Le 27/05/2026 à 21:53, Raphaël Gallais-Pou a écrit :
>> xkbregistry is disabled by default.  Yet, it only needs libxml2 to be
>> built, which is already supported.
>>
>> Add the compile flags to build xkbregistry when possible.
>>
>> Signed-off-by: Raphaël Gallais-Pou <rgallaispou@gmail.com>
>> ---
>>   package/libxkbcommon/libxkbcommon.mk | 10 ++++++++--
>>   1 file changed, 8 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/libxkbcommon/libxkbcommon.mk b/package/libxkbcommon/libxkbcommon.mk
>> index 21a2b937ef02..98154369b419 100644
>> --- a/package/libxkbcommon/libxkbcommon.mk
>> +++ b/package/libxkbcommon/libxkbcommon.mk
>> @@ -12,8 +12,7 @@ LIBXKBCOMMON_CPE_ID_VENDOR = xkbcommon
>>   LIBXKBCOMMON_INSTALL_STAGING = YES
>>   LIBXKBCOMMON_DEPENDENCIES = host-bison host-flex
>>   LIBXKBCOMMON_CONF_OPTS = \
>> -	-Denable-docs=false \
>> -	-Denable-xkbregistry=false
> 
> xkbregistry has been disabled on purpose when libxkbcommon has been updated to
> 1.0.1 [1].

Indeed I haven't see this.

> 
> It seems to requires xkeyboard-config at runtime [2], can you verify?

Yes: libxkbcommon does not provide dataset, which is what 
xkeyboard-config is.
 From [2]:
It is [preferred] that libxkbregistry be split into a separate packages 
as most
clients that require libxkbcommon do not require libxkbregistry and clients
requiring libxkbregistry may not need libxkbcommon.

Does that mean that a libxkbregistry package needs to be created since 
it is independant from libxkbcommon ?

Best regards,
Raphaël

> 
> [1]
> https://gitlab.com/buildroot.org/buildroot/-/commit/1791bc30a5b19bb52366375d03e66348242ff25a
> 
> [2]
> https://github.com/xkbcommon/libxkbcommon/blob/2c295b10aa87e9bd8c11954adcf7e92b031b92f9/PACKAGING#L61
> 
> Best regards,
> Romain
> 
>> +	-Denable-docs=false
>>   
>>   ifeq ($(BR2_PACKAGE_XORG7),y)
>>   LIBXKBCOMMON_CONF_OPTS += -Denable-x11=true
>> @@ -22,6 +21,13 @@ else
>>   LIBXKBCOMMON_CONF_OPTS += -Denable-x11=false
>>   endif
>>   
>> +ifeq ($(BR2_PACKAGE_LIBXML2),y)
>> +LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=true
>> +LIBXKBCOMMON_DEPENDENCIES += libxml2
>> +else
>> +LIBXKBCOMMON_CONF_OPTS += -Denable-xkbregistry=false
>> +endif
>> +
>>   ifeq ($(BR2_PACKAGE_LIBXKBCOMMON_TOOLS),y)
>>   LIBXKBCOMMON_CONF_OPTS += -Denable-tools=true
>>   else
>>
>> ---
>> base-commit: 08f3757b0f2133b0ed50e1ef3d7a5a1d9c80de1c
>> change-id: 20260527-master-2e5299aaa8ca
>>
>> Best regards,
>> --
>> Raphaël Gallais-Pou <rgallaispou@gmail.com>
>>
>> _______________________________________________
>> buildroot mailing list
>> buildroot@buildroot.org
>> https://lists.buildroot.org/mailman/listinfo/buildroot
> 

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-07-06 15:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-27 19:53 [Buildroot] [PATCH] package/libxkbcommon: provide xkbregistry when libxml2 is enabled Raphaël Gallais-Pou
2026-06-24 15:51 ` Romain Naour via buildroot
2026-07-06 15:34   ` Raphaël Gallais-Pou

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox