[EMAIL PROTECTED] wrote:
> dougm 2002/05/30 19:19:11
>
> Modified: t/response/TestAPR perlio.pm
> xs/APR/PerlIO apr_perlio.c
> Log:
> we cannot support passing Off_t from the perl runtime compiled with
> largfile flags into the apr runtime compiled without.
> only APR::PerlIO::seek has this issue, problem can be seen failing
> apr/perlio tests 3-5 on hpux and solaris, and likely elsewhere.
> ok when offset is 0, but nothing else since cast turns it into 0.
But isn't this ifdef test one sided? i.e. you test only if perl was
compiled with USE_LARGE_FILES, but not APR. Shouldn't it be:
#ifdef USE_LARGE_FILES && !APR_HAS_LARGE_FILES
and the adjusted croak
> +#ifdef USE_LARGE_FILES
> + if (offset != 0) {
> + Perl_croak(aTHX_ "PerlIO::APR::seek with non-zero offset"
> + " not supported with -Duselargefiles");
> + }
> +#else
> + seek_offset = offset;
> +#endif
is there some workaround for this problem? or the only solution is to
make sure that users compile both with largefiles support? If not, does
it mean that it renders APR::PerlIO::seek useless, since the code is
becoming not portable?
__________________________________________________________________
Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org http://ticketmaster.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
