Bug 455279 - Define _initddir, not _initrddir
Define _initddir, not _initrddir
Product: Fedora
Classification: Fedora
Component: redhat-rpm-config (Show other bugs)
All Linux
low Severity low
: ---
: ---
Assigned To: Panu Matilainen
Fedora Extras Quality Assurance
: FutureFeature, Patch
Depends On:
  Show dependency treegraph
Reported: 2008-07-14 12:42 EDT by Ville Skyttä
Modified: 2010-10-14 07:43 EDT (History)
4 users (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2010-02-02 04:59:02 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Define _initddir, not _initrddir (431 bytes, patch)
2008-07-14 12:42 EDT, Ville Skyttä
no flags Details | Diff

  None (edit)
Description Ville Skyttä 2008-07-14 12:42:34 EDT
rpm 4.5.90+ fixes the ages-old _initrddir misspelling, the canonical name of the
macro is now _initddir.  Even though the old form still works, only the newer
should be defined so that both the old and the new stay in sync.

Patch attached; perhaps guarding it with "Requires: rpm >= 4.5.90" in
redhat-rpm-config wouldn't be a bad idea.

On the other hand, I'm not sure if redefining this _initddir (or _initrddir) is
necessary in redhat-rpm-config at all as inheriting it from rpm's defaults seems
to work just fine, perhaps it could be just removed (no versioned rpm dependency
would be needed in that case)?  Ditto bunch of other *dir macros in
Comment 1 Ville Skyttä 2008-07-14 12:42:34 EDT
Created attachment 311730 [details]
Define _initddir, not _initrddir
Comment 2 Bug Zapper 2008-11-25 21:33:26 EST
This bug appears to have been reported against 'rawhide' during the Fedora 10 development cycle.
Changing version to '10'.

More information and reason for this action is here:
Comment 3 Panu Matilainen 2010-02-02 04:59:02 EST
Removed in current rawhide, rpm defaults to sane %_initddir (and %_initrddir for compatibility) for all versions that we care about, no reason to override it here.
Comment 4 Michael Shigorin 2010-10-14 07:43:28 EDT
Folks, did anyone consider plain %_initdir for clarity instead of all those typos?

Note You need to log in before you can comment on or make changes to this bug.