Re: [PATCH] enable LFS support by default

2009-04-04 Thread Mike Frysinger
On Tuesday 24 March 2009 21:44:25 Roland McGrath wrote: > Of course I understand LFS. I don't understand what you think is wrong nor > what you think you are fixing. Those are what need to precede any patch. > > -D_FILE_OFFSET_BITS=64 might be intractable for strace. It needs to decode > the 32-

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Dmitry V. Levin
On Tue, Mar 24, 2009 at 07:07:33PM -0700, Roland McGrath wrote: > Does that bug really still exist? In my build, fopen_for_output is fopen64. According to changelog entry, it is fixed long time ago: 2007-07-05 Roland McGrath * strace.c (strace_fopen): [_LFS64_LARGEFILE]: Use fope

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Roland McGrath
Does that bug really still exist? In my build, fopen_for_output is fopen64. Thanks, Roland -- Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are powering Web 2.0 with engaging, cross-platform capabi

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Dmitry V. Levin
On Tue, Mar 24, 2009 at 06:44:25PM -0700, Roland McGrath wrote: > Of course I understand LFS. I don't understand what you think is wrong nor > what you think you are fixing. Those are what need to precede any patch. > > -D_FILE_OFFSET_BITS=64 might be intractable for strace. It needs to decode

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Roland McGrath
Of course I understand LFS. I don't understand what you think is wrong nor what you think you are fixing. Those are what need to precede any patch. -D_FILE_OFFSET_BITS=64 might be intractable for strace. It needs to decode the 32-bit struct stat et al formats in the syscalls that use them. In

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Mike Frysinger
On Tuesday 24 March 2009 20:23:36 Dmitry V. Levin wrote: > On Thu, Mar 05, 2009 at 10:02:53PM -0500, Mike Frysinger wrote: > > configure.ac |1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff --git a/configure.ac b/configure.ac > > index ffbfeb4..e0b341a 100644 > > --- a/co

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Dmitry V. Levin
On Thu, Mar 05, 2009 at 10:02:53PM -0500, Mike Frysinger wrote: > configure.ac |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/configure.ac b/configure.ac > index ffbfeb4..e0b341a 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -173,6 +173,7 @@ AC_PROG_CPP > A

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Mike Frysinger
On Tuesday 24 March 2009 03:38:31 Roland McGrath wrote: > Include ChangeLog entries before a patch, please. > We do not use Signed-off-by lines. > > If you want a patch considered, say what it is for. considering the developers involved with strace maintenance, i didnt think i needed to explain L

Re: [PATCH] enable LFS support by default

2009-03-24 Thread Roland McGrath
Include ChangeLog entries before a patch, please. We do not use Signed-off-by lines. If you want a patch considered, say what it is for. Thanks, Roland -- Apps built with the Adobe(R) Flex(R) framework and Flex Builde

[PATCH] enable LFS support by default

2009-03-05 Thread Mike Frysinger
Signed-off-by: Mike Frysinger --- configure.ac |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/configure.ac b/configure.ac index ffbfeb4..e0b341a 100644 --- a/configure.ac +++ b/configure.ac @@ -173,6 +173,7 @@ AC_PROG_CPP AC_PROG_GCC_TRADITIONAL AC_PROG_INSTALL AC_C_C