[lkml]   [1998]   [Jun]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: OFFTOPIC: e2fsprogs and +2Gb partitions
"Theodore Y. Ts'o" <tytso@MIT.EDU> writes:

> But leaving llseek() in the library but removing it from the prototype,
> such that programs that use autoconf to determine whether or not llseek
> exists was just irresponsible.

What has this to do with irresponsibility? It certainly was a mistake
to add llseek but when I realized this the damage was done (i.e., the
ABI was used). But since I never added the prototype this keeps the
attention away from this function which never ever should be used.

And a test for the prototype isn't necessary. Simply add a test for
lseek64 and if this function is not available test for llseek with
code like this:

[AC_CACHE_CHECK([for usable llseek], e2_cv_usable_llseek,
[AC_TRY_LINK([#include <stdio.h>],[#if __GLIBC__ == 2 && __GLIBC_MINOR__ == 0
#error "not here"
llseek (0, 0, 0);

---------------. drepper at ,-. 1325 Chesapeake Terrace
Ulrich Drepper \ ,-------------------' \ Sunnyvale, CA 94089 USA
Cygnus Solutions `--' drepper at `------------------------
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to

 \ /
  Last update: 2005-03-22 13:43    [from the cache]
©2003-2014 Jasper Spaans. hosted at Digital Ocean