On Thu, 02.03.06 01:16, Joe Orton ([EMAIL PROTECTED]) wrote:
Hi!
> If you compile everything, including neon, with _FILE_OFFSET_BITS=64,
> then everything works fine (at least from neon's perspective). If you
> mix and match apps and libraries built with _F_O_B=64, then yes, you'll
> have lots of strange and horrible problems. So... don't do that!
>
> Having neon export _F_O_B=64 by default would cause similar problems for
> apps which *don't* build using that flag. Really, just avoiding _F_O_B
> definitions completely is the only safe bet.
I believe that *all* apps and libs should define _F_O_B=64
(if supported by the libc).
The thing is that it is not my decision whether to compile libneon
with 64bit support or without. If Debian (Laszlo Boszormenyi) chose to
compile the lib with 64bit off_t enabled it would become
ABI-incompatible with all other distributions that do not enable
it. Therefore I am asking you, as the maintainer of NEON, to make that
switch upstream. In my opinion all new libraries/apps should use 64bit
off_t, support for 32bit off_t should be retained only for legacy
sources.
> It is simple enough to add an LFS-style ne_get_range64 to neon to allow
> taking off64_t offsets; e.g. as below. If you want to mix'n'match
> apps+libs using differently sized off_t, then you can do that safely
> with neon so long as you stick to using the LFS interfaces.
I don't want to mix'n'match apps+libs with differntly sized
off_t. It's just that neon in Debian is compiled with 32bit off_t and
all other libs are compiled with 64bit off_t. And the reason is mostly
that changing to 64bit off_t would make neon ABI incompatible with
upstream.
However, such an API addition (ne_get_range64()) would be certainly
better than nothing. Though I'd definitely prefer to use _F_O_B=64
everyhwere and drop those special 64 versions of those functions.
When do you plan to release the next version of neon?
Thanks,
Lennart
--
Lennart Poettering; lennart [at] poettering [dot] net
ICQ# 11060553; GPG 0x1A015CC4; http://0pointer.net/lennart/
_______________________________________________
neon mailing list
[email protected]
http://mailman.webdav.org/mailman/listinfo/neon
