FS#69394 - [brltty] /usr/share/polkit-1/rules.d/ owned by root group instead of polkitd in brltty 6.2-2

Attached to Project: Arch Linux
Opened by gamezelda (gamezelda) - Thursday, 21 January 2021, 15:11 GMT
Last edited by David Runge (dvzrv) - Thursday, 21 January 2021, 21:58 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To David Runge (dvzrv)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

brltty 6.2-2 now packages a polkit rule in /usr/share/polkit-1/rules.d/org.a11y.brlapi.rules so it also packages the parent folder /usr/share/polkit-1/rules.d/.

In the package however, this folder is owned by the group 'root' instead of 'polkitd' like in other packages such as gvfs, polkit or systemd. (As far as I know, this is harmless, but it causes a warning to appear when checking permissions with 'pacman -Qkk'.)

Looking at the package definition, I see gvfs sets the folder GID to 102=polkitd when creating it, while brltty appears to leave it at the default which is root:

https://github.com/archlinux/svntogit-packages/blob/3fd837a3c5442b4951b01ccaa7d6d9b7f42eadc2/trunk/brltty-6.2-polkit_path_permission.patch#L9

https://github.com/archlinux/svntogit-packages/blob/79b2eb949dc733baa5d87dafebba97c6bdce46bd/trunk/PKGBUILD#L68

Additional info:
* package version(s): brltty 6.2-2
* config and/or log files etc.: None (see below for console)
* link to upstream bug report, if any: Unknown (suspect packaging issue)

Steps to reproduce:

$ curl -s https://mirror.f4st.host/archlinux/core/os/x86_64/systemd-247.2-1-x86_64.pkg.tar.zst | tar tvf - --zstd | grep 'usr/share/polkit-1/rules.d/$'
drwxr-x--- root/102 0 2020-12-16 15:38 usr/share/polkit-1/rules.d/
$ curl -s https://mirror.f4st.host/archlinux/extra/os/x86_64/polkit-0.118-1-x86_64.pkg.tar.zst | tar tvf - --zstd | grep 'usr/share/polkit-1/rules.d/$'
drwxr-x--- root/102 0 2020-09-25 13:13 usr/share/polkit-1/rules.d/
$ curl -s https://mirror.f4st.host/archlinux/extra/os/x86_64/gvfs-1.46.2-1-x86_64.pkg.tar.zst | tar tvf - --zstd | grep 'usr/share/polkit-1/rules.d/$'
drwxr-x--- root/polkitd 0 2021-01-08 14:00 usr/share/polkit-1/rules.d/
$ curl -s https://mirror.f4st.host/archlinux/extra/os/x86_64/brltty-6.1-5-x86_64.pkg.tar.zst | tar tvf - --zstd | grep 'usr/share/polkit-1/rules.d/$'
$ curl -s https://mirror.f4st.host/archlinux/testing/os/x86_64/brltty-6.2-2-x86_64.pkg.tar.zst | tar tvf - --zstd | grep 'usr/share/polkit-1/rules.d/$'
drwxr-x--- root/root 0 2021-01-21 00:05 usr/share/polkit-1/rules.d/
$ LANG=C sudo pacman -Qkk brltty
warning: brltty: /usr/share/polkit-1/rules.d (GID mismatch)
brltty: 795 total files, 1 altered file
This task depends upon

Closed by  David Runge (dvzrv)
Thursday, 21 January 2021, 21:58 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed with brltty 6.2-3
Comment by David Runge (dvzrv) - Thursday, 21 January 2021, 15:19 GMT
@gamezelda: Thanks for the report!

This is indeed a packaging issue I introduced with my latest patches/additions (to fix a directory permission problem).
I look into this and release a pkgrel bump with a fix.

Loading...