tags 221618 patch thanks Quoting Steve Langasek (vor...@debian.org): > On Wed, Jun 08, 2011 at 06:39:06AM +0200, Christian PERRIER wrote: > > Quoting Steve Langasek (vor...@debian.org): > > > On Tue, Jun 07, 2011 at 07:32:17PM +0200, Christian PERRIER wrote: > > > > Do you guys ahave an advice about this bug report? > > > > Should be fixed by setting '#define _LARGEFILE64_SOURCE' and '#define > > > _FILE_OFFSET_BITS 64' in the header. > > > <mode bubulle_is_very_dumb> > > > You mean just dropping these two lines at top of libsmbclient.h? > > Yep. > > > Should this be done upstream? > > It should be discussed upstream, but it's potentially an ABI change upstream > for those platforms which were *not* previously building with LFS support so > more caution is required there.
So, the attached patch shold be the fix. Let's record that.
Index: samba/source3/include/libsmbclient.h =================================================================== --- samba.orig/source3/include/libsmbclient.h +++ samba/source3/include/libsmbclient.h @@ -79,6 +79,10 @@ #include <fcntl.h> #include <utime.h> + /* Debian bug #221618 */ +#define _LARGEFILE64_SOURCE +#define _FILE_OFFSET_BITS 64 + #define SMBC_BASE_FD 10000 /* smallest file descriptor returned */ #define SMBC_WORKGROUP 1
signature.asc
Description: Digital signature