Re: _stati64 patch (Was: [PATCH] Fix include path separator)

[Mark Wielaard <mark@klomp.org>]

[-- Attachment #1: Type: text/plain, Size: 1451 bytes --]

Hi Joshua,

On Tue, 2019-07-09 at 10:00 -0500, Joshua Watt wrote:
> On 7/5/19 3:38 AM, Mark Wielaard wrote:
> > There is another Windows specific patch which we haven't applied yet
> > to the 1.0.x branch because we don't have Windows builders/testers.
> > 
> > It is the attached patch from Phil Ross (CCed) to use _stati64 instead
> > of _stat to support large >4GB files on Windows.
> > 
> > If you could test the above works on your setups that would be
> > appreciated.  Do we need any checks to see whether _stati64 is
> > available? Or can it be used unconditionally as in this patch?
> 
> The patch compiles without issues using msvc 18.0 (Visual Studio 2013) 
> and 64-bit MinGW gcc 7.3.0. 'make test' passes also.

Thanks for the extra testing. Lets apply this patch then.
It would be good to be able to handle large files on Windows too.

> The "patch in a patch" format was a little confusing at first :)

Oops, sorry, I wanted to make sure I cherry-picked the original patch
from Phil, but in doing so, I got the patch-in-patch variant. I applied
the straight diff one (as attached) so that the source matches what was
applied on the 1.1.x branch.

BTW. I don't know much about windows, and I assume it doesn't ship with
bash. But maybe you could take a peek at the bzip2-tests repository and
see if you could somehow make that work on Windows?
https://sourceware.org/git/bzip2-tests.git

Cheers,

Mark

[-- Attachment #2: 0001-Fix-a-not-a-normal-file-error-when-compressing-large.patch --]
[-- Type: text/x-patch, Size: 981 bytes --]

From 13d8bce0393ca21cbca1e8ba7692466a64fd46dd Mon Sep 17 00:00:00 2001
From: Phil Ross <phil.ross@gmail.com>
Date: Tue, 21 May 2019 20:46:14 +0100
Subject: [PATCH] Fix a 'not a normal file' error when compressing large files.

The bzip2 command line would report 'not a normal file' for files of
size larger than 2^32 - 1 bytes.

Patch bzip2.c to use _stati64 instead of _stat so that a successful
result is returned for large files.

Resolves https://github.com/philr/bzip2-windows/issues/3.
---
 bzip2.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/bzip2.c b/bzip2.c
index be3b3be..76c6c94 100644
--- a/bzip2.c
+++ b/bzip2.c
@@ -132,8 +132,8 @@
 
 #   define NORETURN       /**/
 #   define PATH_SEP       '\\'
-#   define MY_LSTAT       _stat
-#   define MY_STAT        _stat
+#   define MY_LSTAT       _stati64
+#   define MY_STAT        _stati64
 #   define MY_S_ISREG(x)  ((x) & _S_IFREG)
 #   define MY_S_ISDIR(x)  ((x) & _S_IFDIR)
 
-- 
1.8.3.1