You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by tr...@apache.org on 2009/12/15 13:55:10 UTC

svn commit: r890781 - /apr/apr-util/branches/1.4.x/STATUS

Author: trawick
Date: Tue Dec 15 12:55:09 2009
New Revision: 890781

URL: http://svn.apache.org/viewvc?rev=890781&view=rev
Log:
why are hooks in APR?

Modified:
    apr/apr-util/branches/1.4.x/STATUS

Modified: apr/apr-util/branches/1.4.x/STATUS
URL: http://svn.apache.org/viewvc/apr/apr-util/branches/1.4.x/STATUS?rev=890781&r1=890780&r2=890781&view=diff
==============================================================================
--- apr/apr-util/branches/1.4.x/STATUS [utf-8] (original)
+++ apr/apr-util/branches/1.4.x/STATUS [utf-8] Tue Dec 15 12:55:09 2009
@@ -57,6 +57,11 @@
 
 RELEASE SHOWSTOPPERS:
 
+    * Resolve hook probe support; current APR-Util 1.4.0-dev has only the
+      --enable-dtrace configure option which can't be used in a general
+     purpose build of APR-Util.  See this thread:
+       http://www.mail-archive.com/dev@apr.apache.org/msg22534.html
+
 RELEASE NON-SHOWSTOPPERS BUT WOULD BE REAL NICE TO WRAP THESE UP:
 
     * Solaris's Sun Freeware (sfw) package has a busted gcc/ld setup.



Re: svn commit: r890781 - /apr/apr-util/branches/1.4.x/STATUS

Posted by Jeff Trawick <tr...@gmail.com>.
On Tue, Dec 15, 2009 at 5:43 PM, Graham Leggett <mi...@sharp.fm> wrote:
> Jeff Trawick wrote:
>
>> *my "why are hooks in apr?" lament in the commit log was referring to
>> APR[-Util] as a whole; what besides httpd actually uses these?
>
> A system developed for one of my current clients uses APR provided hooks
> extensively.

thanks for the info!

Re: svn commit: r890781 - /apr/apr-util/branches/1.4.x/STATUS

Posted by Graham Leggett <mi...@sharp.fm>.
Jeff Trawick wrote:

> *my "why are hooks in apr?" lament in the commit log was referring to
> APR[-Util] as a whole; what besides httpd actually uses these?

A system developed for one of my current clients uses APR provided hooks
extensively.

Regards,
Graham
--

Re: svn commit: r890781 - /apr/apr-util/branches/1.4.x/STATUS

Posted by Jeff Trawick <tr...@gmail.com>.
On Tue, Dec 15, 2009 at 4:55 PM, William A. Rowe Jr.
<wr...@rowe-clan.net> wrote:
>>
>>  RELEASE SHOWSTOPPERS:
>>
>> +    * Resolve hook probe support; current APR-Util 1.4.0-dev has only the
>> +      --enable-dtrace configure option which can't be used in a general
>> +     purpose build of APR-Util.  See this thread:
>> +       http://www.mail-archive.com/dev@apr.apache.org/msg22534.html
>> +
>
> To clarify, is the suggestion to move hooks to apr?  I'm not 100% clear.

(oops, my lackadaisical STATUS commit log led you astray*)

no; I'm just warning that I intend to replace the curent DTrace-only
hook probe support in the 1.4.x branch with something more generally
useful, at the same time axing the configure-time enable-ment of
probes which meant that any app using hooks that built against that
APR-Util include directory had to also implement the probes

I'll first commit my current patch with some commentary fixes to trunk
in the a.m. unless someone has some concerns that need to be
addressed.

*my "why are hooks in apr?" lament in the commit log was referring to
APR[-Util] as a whole; what besides httpd actually uses these?

Re: svn commit: r890781 - /apr/apr-util/branches/1.4.x/STATUS

Posted by "William A. Rowe Jr." <wr...@rowe-clan.net>.
>  
>  RELEASE SHOWSTOPPERS:
>  
> +    * Resolve hook probe support; current APR-Util 1.4.0-dev has only the
> +      --enable-dtrace configure option which can't be used in a general
> +     purpose build of APR-Util.  See this thread:
> +       http://www.mail-archive.com/dev@apr.apache.org/msg22534.html
> +

To clarify, is the suggestion to move hooks to apr?  I'm not 100% clear.

If so, I'm all for scuttling 1.4.1 since I also want to see us introduce
incomplete type macros in 1.4 to solve problems similar to those encountered
in the apr_crypto_t types.  Although it appears this may not help the current
crypto_t problems since it the context omitted the crypto_device_t *.

Let us know.