On Wed, May 11, 2011 at 11:53:53AM -0700, Edward Pilatowicz wrote: > On Wed, May 11, 2011 at 11:03:06AM -0700, [email protected] wrote: > > On Wed, May 11, 2011 at 09:03:23AM -0700, Danek Duvall wrote: > > > [email protected] wrote: > > > > > > > http://cr.opensolaris.org/~johansen/webrev-18277/ > > > > > > This isn't part of your change, but if someone could explain why we're > > > setting LD_LIBRARY_PATH in the sysrepo method, I'd greatly appreciate it. > > > > I don't know the reason for this either. > > > > i think this is currently required for apache. > > if you start apache via it's normal scripts, it sources: > /usr/apache2/2.2/bin/envvars > > which includes a LD_LIBRARY_PATH directive. we don't want to depend on > any of the site apache configuration files (ie, any apache files in > /etc, which the above script also sources) so we have to set > LD_LIBRARY_PATH ourselves. > > there should probably be a comment explaining this in the code. >
i just filed: 18313 sysrepo svc method needs a comment explaining LD_LIBRARY_PATH ed _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
