<div dir="ltr">All,<div>Is there anything preventing this being merged? Can I help with any issues?<div>Thanks, Adrian</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 4 Feb 2021 at 20:33, Adrian Perez de Castro <<a href="mailto:aperez@igalia.com">aperez@igalia.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Adrian,<br>
<br>
Thanks for the patch, I think adding the option is fine given that there<br>
are people around who would use it—which seems to be the case.<br>
<br>
Unless somebody else has an opinion against, I think we could merge this<br>
change =]<br>
<br>
On Tue, 02 Feb 2021 11:57:21 +0000 Adrian Sweet <<a href="mailto:asweet@thegoodpenguin.co.uk" target="_blank">asweet@thegoodpenguin.co.uk</a>> wrote:<br>
> Added config option to expose browser remote control on the D-Bus system bus<br>
> <br>
> Signed-off-by: Adrian Sweet <<a href="mailto:asweet@thegoodpenguin.co.uk" target="_blank">asweet@thegoodpenguin.co.uk</a>><br>
<br>
Reviewed-by: Adrian Perez de Castro <<a href="mailto:aperez@igalia.com" target="_blank">aperez@igalia.com</a>><br>
<br>
> ---<br>
>  package/cog/Config.in | 6 ++++++<br>
>  package/cog/<a href="http://cog.mk" rel="noreferrer" target="_blank">cog.mk</a>    | 6 ++++++<br>
>  2 files changed, 12 insertions(+)<br>
> <br>
> diff --git a/package/cog/Config.in b/package/cog/Config.in<br>
> index a9ca7473a9..c19f1c200e 100644<br>
> --- a/package/cog/Config.in<br>
> +++ b/package/cog/Config.in<br>
> @@ -47,6 +47,12 @@ config BR2_PACKAGE_COG_PLATFORM_DRM<br>
>         with video drivers that support kernel mode-setting (KMS)<br>
>         via the DRM user-space API.<br>
>  <br>
> +config BR2_PACKAGE_COG_USE_SYSTEM_DBUS<br>
> +     bool "Use D-Bus system bus"<br>
> +     default n<br>
> +     help<br>
> +       Expose remote control interface on system bus<br>
> +<br>
>  comment "DRM platform needs mesa3d w/ EGL driver and GBM"<br>
>       depends on !BR2_PACKAGE_MESA3D_OPENGL_EGL<br>
>  <br>
> diff --git a/package/cog/<a href="http://cog.mk" rel="noreferrer" target="_blank">cog.mk</a> b/package/cog/<a href="http://cog.mk" rel="noreferrer" target="_blank">cog.mk</a><br>
> index 47a5573dda..c5a1cc0df2 100644<br>
> --- a/package/cog/<a href="http://cog.mk" rel="noreferrer" target="_blank">cog.mk</a><br>
> +++ b/package/cog/<a href="http://cog.mk" rel="noreferrer" target="_blank">cog.mk</a><br>
> @@ -31,4 +31,10 @@ else<br>
>  COG_CONF_OPTS += -DCOG_PLATFORM_DRM=OFF<br>
>  endif<br>
>  <br>
> +ifeq ($(BR2_PACKAGE_COG_USE_SYSTEM_DBUS),y)<br>
> +COG_CONF_OPTS += -DCOG_DBUS_SYSTEM_BUS=ON<br>
> +else<br>
> +COG_CONF_OPTS += -DCOG_DBUS_SYSTEM_BUS=OFF<br>
> +endif<br>
> +<br>
>  $(eval $(cmake-package))<br>
> -- <br>
> 2.25.1<br>
> <br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div style="color:rgb(80,0,80)">Adrian Sweet, Software Engineer<br><a href="https://www.thegoodpenguin.co.uk/" style="color:rgb(17,85,204)" target="_blank">https://www.thegoodpenguin.co.uk</a></div><div style="color:rgb(80,0,80)"><br></div><div style="color:rgb(80,0,80)"><font color="#999999">The Good Penguin Ltd is a company registered in England and Wales with company number 12374667 and VAT number 341687879. </font><span style="color:rgb(153,153,153)">Registered office: The Good Penguin Ltd, Westcott, Glasllwch Lane, Newport, NP20 3PS.</span></div></div></div>