https://bugzilla.redhat.com/show_bug.cgi?id=1510565 --- Comment #9 from Miroslav Lichvar <mlichvar@xxxxxxxxxx> --- (In reply to Pavel Zhukov from comment #8) > (In reply to Miroslav Lichvar from comment #7) > > Wouldn't it better to just explicitly list the supported services in the > > ntpstat spec using a rich dependency, e.g. "Requires: (ntp or chrony)" ? > looks good for me. It's maintainer decision at the end :) > Are you going to update spec or it'll be done after formal review? I've updated the spec and srpm: https://mlichvar.fedorapeople.org/tmp/ntpstat.spec https://mlichvar.fedorapeople.org/tmp/ntpstat-0.4-2.fc28.src.rpm > > I tried adding "Obsoletes: ntp < 4.2.8p10-4" to ntpstat.spec, expecting ntp > > would be dropped without replacement as chrony is installed, but the result > > was the same as before. > weird. maybe bug in libsolv? I'm not sure. Maybe the obsoletes should be considered only if ntpstat is already installed? I guess the other option is to simply add "Requires: ntpstat" to ntp. > Anyway I'll post formal review soon... Thanks. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
