apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <jor...@redhat.com>
Subject Re: [PATCH] apr_dso_load2() with flags support
Date Wed, 21 Sep 2005 20:19:56 GMT
On Wed, Sep 21, 2005 at 11:25:33AM -0500, William Rowe wrote:
> Joe Orton wrote:
> >
> >If you want to propose an alternative interface please do it in full in 
> >"diff -u" format, otherwise please give a technical justification for 
> >your veto on what I proposed which makes more sense than "It must not 
> >expose special platform knowledge".
> 
> That is completely valid justification since this project's inception.
> If you aren't looking for portability, use dlopen().  If we are offering
> platform independence, we need to offer options (yes, I agree we need
> such options) that cleanly map to a certain behavior that can be 
> supported on most platforms.

So by "it must not expose special platform knowledge" you really mean 
"it must not allow callers to take advantage of features not available 
on all supported platforms"?  I'd hoped to have seen the back of that 
tired old argument.

1) APR already has many such features, from apr_env_delete to
APR_POLLSET_THREADSAFE
2) the flags other than DEEPBIND can be supported on all POSIX platforms 
and also the legacy HP-UX loader and the Darwin loader as already 
discussed.  that is "most platforms" enough for me anyway.
3) the flags can be exposed as hints/optional features without harming 
application portability

If APR is constrained to being no better than the lowest common 
denominator then APR is doomed.  Please rescind the veto and let those 
of us who care about moving the code forward do so.

joe

Mime
View raw message