apr: version bump plus configure changes

[wsiegrist@…]

Attached is a patch for apr to bump to v1.3.3 as well as apply some improvements for darwin_9:

• allow use of sendfile now that APR can handle darwin's sendfile
• use flocks for locking
• dont use kqueue or poll

[wsiegrist@…]

Patch for apr Portfile

[danielluke (Daniel J. Luke)]

I should get a chance to look at this and do some testing in the next couple of days.

[danielluke (Daniel J. Luke)]

Can you explain why we want to override the configure selection (for in favor of flock and no kqueue)?

Thanks.

[blair (Blair Zajac)]

Yes, it definitely would be good to know from an Apple employee why to have APR use those defaults.

One reason alone not to use flock() is that flock() doesn't work over NFS, while fcntl() based locks do.

Subversion needs locking, so this could prevent Subversion fsfs repositories from working correctly on NFS.

[blair (Blair Zajac)]

Adding myself to the cc list.

[danielluke (Daniel J. Luke)]

1.3.3 committed without the darwin 9 platform changes, pending a response for why we might want things set that way.

[wsiegrist@…]

Flock is a little faster, but maybe it should be a variant since it doesnt work on NFS. The use of select instead of kqueue and poll is due to kqueue and poll not working as well on Mac OS for some file types.

[danielluke (Daniel J. Luke)]

Replying to wsiegrist@…:

The use of select instead of kqueue and poll is due to kqueue and poll not working as well on Mac OS for some file types.

Can you be more specific and/or point to where it's been reported upstream? ;-)

[danielluke (Daniel J. Luke)]

closing because of lack of response, feel free to re-open if you have more information.