[Bug 510613] Re: look(1) can't open bigfiles

2011-05-28 Thread Mark Nieweglowski
Sorry about the noise. I attached the wrong patch last time. ** Patch added: "Enable look to handle large files" https://bugs.launchpad.net/ubuntu/+source/bsdmainutils/+bug/510613/+attachment/2146461/+files/look2.diff -- You received this bug notification because you are a member of Ubuntu

[Bug 510613] Re: look(1) can't open bigfiles

2011-05-28 Thread Mark Nieweglowski
This patch defines _LARGEFILE64_SOURCE and _FILE_OFFSET_BITS so fstat and mmap become fstat64 and mmap64. It replaces the previous patch and is meant to replace the package's debian/patches/look.diff ** Patch added: "look.diff" https://bugs.launchpad.net/ubuntu/+source/bsdmainutils/+bug/5106

[Bug 510613] Re: look(1) can't open bigfiles

2011-05-28 Thread Mark Nieweglowski
** Changed in: bsdmainutils (Ubuntu) Status: New => Confirmed -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/510613 Title: look(1) can't open bigfiles -- ubuntu-bugs mailing list ubuntu-bugs

[Bug 510613] Re: look(1) can't open bigfiles

2011-05-27 Thread Mark Nieweglowski
BTW: The original bug mentioned EOVERFLOW and in hindsight I see that my patch doesn't address it, but EOVERFLOW is not the bug I'm seeing. I'm seeing the max size check fail. --- look.c:145 if (sb.st_size > SIZE_T_MAX) errx(2, "%s: %s", file, strerror(EF

[Bug 510613] Re: look(1) can't open bigfiles

2011-05-27 Thread Mark Nieweglowski
Attached is a patch so "look" will accept files larger than INT_MAX on 64 bit architectures. The original debian/patches/look.diff allows look.c to compile on Debian by defining SIZE_T_MAX as INT_MAX. But it should have defined SIZE_T_MAX as SIZE_MAX. SIZE_MAX is defined in stdint.h which wasn't