Bug 672313
Summary: | Spec file should use Requires(post) instead of Requires(posttrans) | |||
---|---|---|---|---|
Product: | [Retired] 389 | Reporter: | Rich Megginson <rmeggins> | |
Component: | Install/Uninstall | Assignee: | Nathan Kinder <nkinder> | |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Viktor Ashirov <vashirov> | |
Severity: | high | Docs Contact: | ||
Priority: | high | |||
Version: | 1.2.8 | CC: | jgalipea, pmatilai | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: | Story Points: | --- | ||
Clone Of: | ||||
: | 672315 (view as bug list) | Environment: | ||
Last Closed: | 2015-12-07 16:59:44 UTC | Type: | --- | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 656390, 672315 |
Description
Rich Megginson
2011-01-24 19:39:44 UTC
I have changed the specfiles to use "Requires(post)" for F13, F14, and rawhide. This will be fixed when we build 389-ds-base-1.2.8.a1. I also updated the specfile for the el5 EPEL branch. A small correction: posttrans as a Requires qualifier is not deprecated. It simply has never been supported, but a spec parser flaw has let it slip through as a regular require (which actually happens to be a fairly close approximation of its real needs) Is this enough to verify this bug? # cat 389-ds-base-1.2.8.1-1.el6.src.rpm.spec | grep posttrans %posttrans # echo posttrans - upgrading - looking for instances in $instbase - %posttrans now runs update to update the server instances cat 389-ds-base-1.2.8.1-1.el6.src.rpm.spec | grep Requires\(post\) Requires(post): /sbin/chkconfig (In reply to comment #5) > cat 389-ds-base-1.2.8.1-1.el6.src.rpm.spec | grep Requires\(post\) > Requires(post): /sbin/chkconfig ack closing verified, per comment #6 |