Re: [ptxdist] [APPLIED] polkit: add pkexec install option

[Michael Olbrich <m.olbrich@pengutronix.de>]

Thanks, applied as 0bbd0652f102aa42186ca1eb27fce5a0d4dc9d52.

Michael

[sent from post-receive hook]

On Mon, 20 Jun 2022 08:19:25 +0200, Bruno Thomsen <bruno.thomsen@gmail.com> wrote:
> Disable default install of pkexec SUID binary as it's an old way of
> executing a command as another user or with elevated privileges.
> 
> Fedora will also avoid installing pkexec[1], and put it in an optional
> compat RPM package.
> 
> [1] https://fedoraproject.org/wiki/Changes/polkit_recommends_pkla_pkexec
> 
> Signed-off-by: Bruno Thomsen <bruno.thomsen@gmail.com>
> Message-Id: <20220430113724.6133-1-bruno.thomsen@gmail.com>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
> 
> diff --git a/rules/polkit.in b/rules/polkit.in
> index 6085fbf783d6..8227cafbcaf8 100644
> --- a/rules/polkit.in
> +++ b/rules/polkit.in
> @@ -29,4 +29,14 @@ config POLKIT_SYSTEMD
>  	help
>  	  Use systemd for session tracking, else ConsoleKit is used
>  
> +comment "install options   ---"
> +
> +config POLKIT_PKEXEC
> +	bool
> +	prompt "Install pkexec"
> +	help
> +	  pkexec - Execute a command as another user or with elevated privileges.
> +
> +	  Old and obsolete tool. Use 'sudo' for this kind of task.
> +
>  endif
> diff --git a/rules/polkit.make b/rules/polkit.make
> index 4fa8d9c0edb1..368fbe3e4d3c 100644
> --- a/rules/polkit.make
> +++ b/rules/polkit.make
> @@ -96,7 +96,9 @@ endif
>  	@$(call install_copy, polkit, 0, 0, 0755, -, /usr/libexec/polkitd)
>  
>  # binaries with suid
> +ifdef PTXCONF_POLKIT_PKEXEC
>  	@$(call install_copy, polkit, 0, 0, 4755, -, /usr/bin/pkexec)
> +endif
>  	@$(call install_copy, polkit, 0, 0, 4755, -, \
>  		/usr/libexec/polkit-agent-helper-1)
>