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.
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.
OK, but you might need to add svrcore_libdir, perldap_libdir, and icu_libdir.
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!)
(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).
(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.
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.
(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.
Created attachment 147824 [details] CVS Commit Checked into ldapserver (HEAD). Thanks to Rich and Noriko for their reviews!