>> what _does_ need to happen, though, is for Apache::compat to redefine >> filename() so that it behaves like mp1 did. I just don't have the >> tuits at >> the moment. > > > Right, but there are issues with doing that: > > 1) that should probably be overridable and not loaded by default: > http://perl.apache.org/docs/2.0/api/Apache/compat.html#Compatibility_Functions_Colliding_with_mod_perl_2_0_API > > since if not it'll replace the real mp2 API and may be someone doesn't > want the stat struct to be updated.
yup. > > 2) filename() should behave differently depending on which phase was it > called from. What if someone just sets filename() to skip the stat > calls? Should it still call the finfo update? I'd think so - this is a compat layer, and mod_perl updated finfo blindly on each $r->filename call. it also updated _ with the new stat information, so compat.pm should do that as well. --Geoff -- Report problems: http://perl.apache.org/bugs/ Mail list info: http://perl.apache.org/maillist/modperl.html List etiquette: http://perl.apache.org/maillist/email-etiquette.html