Bug 228082 - Wrappers need to support legacy bundled builds
Wrappers need to support legacy bundled builds
Status: CLOSED CURRENTRELEASE
Product: 389
Classification: Community
Component: Directory Server (Show other bugs)
1.0.4
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nathan Kinder
Viktor Ashirov
:
Depends On:
Blocks: 152373 240316 FDS1.1.0
  Show dependency treegraph
 
Reported: 2007-02-09 16:47 EST by Nathan Kinder
Modified: 2015-12-07 11:40 EST (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-12-07 11:40:35 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Proposed Fix (2.88 KB, patch)
2007-02-09 16:51 EST, Nathan Kinder
no flags Details | Diff
Revised Diffs (3.02 KB, patch)
2007-02-09 18:19 EST, Nathan Kinder
no flags Details | Diff
CVS Commit (1.46 KB, text/plain)
2007-02-09 18:31 EST, Nathan Kinder
no flags Details

  None (edit)
Description Nathan Kinder 2007-02-09 16:47:02 EST
We still need to support legacy-style bundled builds where we bundle all of our
run-time dependencies with the directory server itself.

With the current build-system, we use the library paths that are used at
link-time to set the library search path in our wrapper scripts.  This will not
work for the old style bundled builds.  We should only be using libpath
(/opt/fedora-ds/lib by default) in the wrappers.
Comment 1 Nathan Kinder 2007-02-09 16:51:05 EST
Created attachment 147818 [details]
Proposed Fix

The fix adds a new "--enable-bundle" configure option that will only be used
for legacy-style packaging.  Using this option will make the build use a
different sed command for wrapper creation.
Comment 2 Rich Megginson 2007-02-09 17:02:19 EST
OK, but you might need to add svrcore_libdir, perldap_libdir, and icu_libdir.
Comment 3 Noriko Hosoi 2007-02-09 17:04:45 EST
Looks good to me.
(So, there's no platform restriction for --enable-bundle.  If we wish, we can do
it on RHEL and Solaris, as well.  It's nice to have the way!)
Comment 4 Nathan Kinder 2007-02-09 17:16:50 EST
(In reply to comment #2)
> OK, but you might need to add svrcore_libdir, perldap_libdir, and icu_libdir.

I intend to have all of those libraries placed in libdir by the packaging script
(to-be-written).
Comment 5 Rich Megginson 2007-02-09 17:21:15 EST
(In reply to comment #4)
> (In reply to comment #2)
> > OK, but you might need to add svrcore_libdir, perldap_libdir, and icu_libdir.
> 
> I intend to have all of those libraries placed in libdir by the packaging script
> (to-be-written).

OK.
Comment 6 Nathan Kinder 2007-02-09 18:19:02 EST
Created attachment 147822 [details]
Revised Diffs

This new set of diffs merges in some changes that Rich checked in since the
previous attachment.  The only change is the addition of a few more
substitutions in the sed command.
Comment 7 Rich Megginson 2007-02-09 18:24:42 EST
(In reply to comment #6)
> Created an attachment (id=147822) [edit]
> Revised Diffs
> 
> This new set of diffs merges in some changes that Rich checked in since the
> previous attachment.  The only change is the addition of a few more
> substitutions in the sed command.

Ok.
Comment 8 Nathan Kinder 2007-02-09 18:31:58 EST
Created attachment 147824 [details]
CVS Commit

Checked into ldapserver (HEAD).  Thanks to Rich and Noriko for their reviews!

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