mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Bastian Krause <bst@pengutronix.de>
Subject: Re: [ptxdist] [APPLIED] cog: add option to control cog via D-Bus system bus
Date: Fri, 23 Apr 2021 15:47:34 +0200	[thread overview]
Message-ID: <20210423134734.2557871-1-m.olbrich@pengutronix.de> (raw)
In-Reply-To: <20210415102923.21134-1-bst@pengutronix.de>

Thanks, applied as 7a23508e15b14bd9a977d8b9308932cfb03ef8ca.

Michael

[sent from post-receive hook]

On Fri, 23 Apr 2021 15:47:34 +0200, Bastian Krause <bst@pengutronix.de> wrote:
> This allows remote control of cog via D-Bus, e.g. open an URL, reload,
> navigate in page view history.
> 
> Signed-off-by: Bastian Krause <bst@pengutronix.de>
> Message-Id: <20210415102923.21134-1-bst@pengutronix.de>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/patches/cog-0.8.1/0001-CMake-install-D-Bus-policy-configuration-to-CMAKE_IN.patch b/patches/cog-0.8.1/0001-CMake-install-D-Bus-policy-configuration-to-CMAKE_IN.patch
> new file mode 100644
> index 000000000000..9f2a1900981f
> --- /dev/null
> +++ b/patches/cog-0.8.1/0001-CMake-install-D-Bus-policy-configuration-to-CMAKE_IN.patch
> @@ -0,0 +1,31 @@
> +From: Bastian Krause <bst@pengutronix.de>
> +Date: Wed, 14 Apr 2021 17:36:24 +0200
> +Subject: [PATCH] CMake: install D-Bus policy configuration to
> + CMAKE_INSTALL_DATADIR
> +
> +The D-BUS system bus policy config should reside in
> +CMAKE_INSTALL_DATADIR rather than in CMAKE_INSTALL_SYSCONFDIR.
> +
> +See:
> +
> +  https://gitlab.freedesktop.org/dbus/dbus/-/blob/ef55a3db0d8f17848f8a579092fb05900cc076f5/bus/CMakeLists.txt#L117
> +
> +Forwarded: https://github.com/Igalia/cog/pull/296
> +Signed-off-by: Bastian Krause <bst@pengutronix.de>
> +---
> + CMakeLists.txt | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/CMakeLists.txt b/CMakeLists.txt
> +index cb51087359fc..2fcd63845212 100644
> +--- a/CMakeLists.txt
> ++++ b/CMakeLists.txt
> +@@ -157,7 +157,7 @@ if (COG_DBUS_SYSTEM_BUS)
> +     configure_file(dbus/policy.conf.in ${COG_DEFAULT_APPID}.conf @ONLY)
> +     install(
> +         FILES ${CMAKE_CURRENT_BINARY_DIR}/${COG_DEFAULT_APPID}.conf
> +-        DESTINATION ${CMAKE_INSTALL_SYSCONFDIR}/dbus-1/system.d
> ++        DESTINATION ${CMAKE_INSTALL_DATADIR}/dbus-1/system.d
> +         COMPONENT "runtime"
> +     )
> + 
> diff --git a/patches/cog-0.8.1/series b/patches/cog-0.8.1/series
> new file mode 100644
> index 000000000000..14639312eb47
> --- /dev/null
> +++ b/patches/cog-0.8.1/series
> @@ -0,0 +1,4 @@
> +# generated by git-ptx-patches
> +#tag:base --start-number 1
> +0001-CMake-install-D-Bus-policy-configuration-to-CMAKE_IN.patch
> +# a7cfa1be71dd856d687d8a3863e5a56f  - git-ptx-patches magic
> diff --git a/rules/cog.in b/rules/cog.in
> index 23b9de3cd931..ccc7362950c2 100644
> --- a/rules/cog.in
> +++ b/rules/cog.in
> @@ -1,12 +1,21 @@
>  ## SECTION=applications
>  
> -config COG
> +menuconfig COG
>  	tristate
> -	prompt "cog"
> +	prompt "cog                           "
>  	select HOST_CMAKE
>  	select HOST_NINJA
>  	select WAYLAND_PROTOCOLS
>  	select WPEWEBKIT
>  	select WPEBACKEND_FDO
> +	select DBUS			if COG_REMOTE_DBUS_SYSTEM_BUS
>  	help
>  	  Cog launcher and webapp container.
> +
> +if COG
> +
> +config COG_REMOTE_DBUS_SYSTEM_BUS
> +	bool
> +	prompt "Expose remote control interface on system bus"
> +
> +endif
> diff --git a/rules/cog.make b/rules/cog.make
> index 758b4f7f4a84..445c76617246 100644
> --- a/rules/cog.make
> +++ b/rules/cog.make
> @@ -38,7 +38,7 @@ COG_CONF_OPT	:= \
>  	-DCOG_APPID= \
>  	-DCOG_BUILD_PROGRAMS=ON \
>  	-DCOG_DBUS_OWN_USER= \
> -	-DCOG_DBUS_SYSTEM_BUS=OFF \
> +	-DCOG_DBUS_SYSTEM_BUS=$(call ptx/onoff,PTXCONF_COG_REMOTE_DBUS_SYSTEM_BUS) \
>  	-DCOG_HOME_URI=https://ptxdist.org/ \
>  	-DCOG_PLATFORM_DRM=OFF \
>  	-DCOG_PLATFORM_FDO=ON \
> @@ -64,6 +64,11 @@ $(STATEDIR)/cog.targetinstall:
>  	@$(call install_lib, cog, 0, 0, 0644, libcogplatform-fdo)
>  	@$(call install_lib, cog, 0, 0, 0644, libcogcore)
>  
> +ifdef PTXCONF_COG_REMOTE_DBUS_SYSTEM_BUS
> +	@$(call install_copy, cog, 0, 0, 0644, -, \
> +		/usr/share/dbus-1/system.d/com.igalia.Cog.conf)
> +endif
> +
>  	@$(call install_finish, cog)
>  
>  	@$(call touch)

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de


      parent reply	other threads:[~2021-04-23 13:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-15 10:29 [ptxdist] [PATCH 1/2] " Bastian Krause
2021-04-15 10:29 ` [ptxdist] [PATCH 2/2] cog: add option to install cogctl Bastian Krause
2021-04-23 13:47   ` [ptxdist] [APPLIED] " Michael Olbrich
2021-04-23 13:47 ` Michael Olbrich [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20210423134734.2557871-1-m.olbrich@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=bst@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox