Overview
Request 834570 superseded
- Initial v0.6.1 greetd package
* includes greetd service
* includes agreety text-based greeter
- Created by dkondratenko
- In state superseded
- Supersedes 833797
- Superseded by 834573
-
Open review for
licensedigger
- Open review for openSUSE:Factory:Staging:adi:121
-
Open review for
opensuse-review-team
found conflict of greetd-0.6.1-5.1.x86_64 with openSUSEway-0.11-19.1.noarch /etc/greetd [mode mismatch: d755 greeter:greeter, d755 root:root]
how to fix this and where is it seen? Does it mean that /etc/greetd should be owned by root but everything under by greetd ?
It's not clear what the correct fix is - all I see is two packages create the same directory with different permissions - which would result in a random result on the installed system, depending on the order the packages are being installed.
Considering the mode is 755 - there is very likely no reason to have the directory owned by greetd: the daemon is not supposed to write in /etc anyway, so owning it root:root should be perfectly valid (there should not even be a reason to have the files differently owned, especialy not if they are mode 644)
i.e.: drop the %attr() statement in the files section for the stuff in sysconfdr
92+%config %{_sysconfdir}/pam.d/greetd
Please see: https://en.opensuse.org/openSUSE:Packaging_UsrEtc#pam.2Fpam-config
pam config should to to /usr/etc by now
Request History
dkondratenko created request
- Initial v0.6.1 greetd package
* includes greetd service
* includes agreety text-based greeter
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:121"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:121"
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
92+%config %{_distconfdir}/pam.d/greetd => /usr/etc is not %config files - they are not supposed to be modified by the admin.
an admin that wants to override those defaults will copy the same file to /etc/pam.d and modify it there, not being affected by rpm file replacements