Overview
Request 871212 accepted
- bsc#1181788: Make the code for the service name change robust to
the simplified systemd-sysv-convert script.
- Fix systemd dependencies.
- Package statsdir /var/log/ntpstats/
- Let system-user-ntp handle the user/group generation
- Introduce subpackage dcf77-tools
* testdcf, a simple DCF77 raw impulse test program
* dcfd, a simple DCF77 raw impulse receiver
- Add patch testdcf-gude.diff
* Improves testdcf's compatibility with GUDE DCF77 receivers
- Silence an OpenSSL version warning (bsc#992038,bsc#1125401
- removal of ntp.firewall, since SuSEfirewall2 has been replaced by
- Drop the omc config ntp.xml fate#301838:
- Update to 4.2.8p12 (bsc#1111853):
15+%systemd_requires => better %systemd_ordering
(not an explicit hard dep on systems, but if systemd, ensure to systemd being installed first
Request History
rmax created request
- bsc#1181788: Make the code for the service name change robust to
the simplified systemd-sysv-convert script.
- Fix systemd dependencies.
- Package statsdir /var/log/ntpstats/
- Let system-user-ntp handle the user/group generation
- Introduce subpackage dcf77-tools
* testdcf, a simple DCF77 raw impulse test program
* dcfd, a simple DCF77 raw impulse receiver
- Add patch testdcf-gude.diff
* Improves testdcf's compatibility with GUDE DCF77 receivers
- Silence an OpenSSL version warning (bsc#992038,bsc#1125401
- removal of ntp.firewall, since SuSEfirewall2 has been replaced by
- Drop the omc config ntp.xml fate#301838:
- Update to 4.2.8p12 (bsc#1111853):
factory-auto added opensuse-review-team as a reviewer
Please review sources
factory-auto accepted review
Check script succeeded
licensedigger accepted review
ok
dimstar_suse added as a reviewer
Being evaluated by staging project "openSUSE:Factory:Staging:adi:143"
dimstar_suse accepted review
Picked "openSUSE:Factory:Staging:adi:143"
dimstar accepted review
dimstar_suse accepted review
Staging Project openSUSE:Factory:Staging:adi:143 got accepted.
dimstar_suse approved review
Staging Project openSUSE:Factory:Staging:adi:143 got accepted.
dimstar_suse accepted request
Staging Project openSUSE:Factory:Staging:adi:143 got accepted.
ntp and system-user-ntp disagree on permissions/users to be assigned for the directory. as ntp has Requires(pre) user(ntp), it is guaranteed for system-user-ntp to be present (and thus also the directory in question) before ntp is being installed.
Seems like ntp should simply give up on caring for this directory and let system-user-ntp handle it
The dir still also being owned by the ntp package seems to be an oversight by the contributor who implemented this and I agree that it should be owned by the user package only.
But I don't think the ownership and permissions of the dir should be as they are now in the user package. The ntp user should never log in, so it does not need a writable home, especially as it also serves as the root of ntp's chroot environment (if configured so), which should never be writable by the daemon it hosts.
The only place that ntpd needs to be able to write to is
/var/lib/ntp/drift/for the drift file.The other thing is the permissions. I don't think the dir has to be locked down to 750 and would hence keep it at 755.