https://bugzilla.redhat.com/show_bug.cgi?id=1510565



--- Comment #9 from Miroslav Lichvar <[email protected]> ---
(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 -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to