Re: [PATCH] Makefile: update the default build options for AIX
- From
Mike Ralphson <mike.ralphson@gmail.com>
- Date
- May 7, 2008, 15:15 UTC
- Message-ID
- <e2b179460805070815u6cc627feo6137084fe7c5a635@mail.gmail.com>
- In-Reply-To
- <4821BECA.8020509@nrlssc.navy.mil>
2008/5/7 Brandon Casey <casey@nrlssc.navy.mil>:
Show 26 quoted lines
> Johannes Sixt wrote:
> > The #define fopen in git-compat-util.h essentially defeats the effect of
> > _LARGE_FILES as far as fopen() calls are concerned: If
> > FREAD_READS_DIRECTORIES is not defined, fopen() would be redirected to
> > fopen64(), but when it is defined, it is redirected to git_fopen(), which
> > in turn uses fopen() instead of fopen64() (due to the #undef in
> > compat/fopen.c).
> >
>
> How about something like this?
>
> diff --git a/compat/fopen.c b/compat/fopen.c
> index ccb9e89..70b0d4d 100644
> --- a/compat/fopen.c
> +++ b/compat/fopen.c
> @@ -1,5 +1,5 @@
> +#undef FREAD_READS_DIRECTORIES
> #include "../git-compat-util.h"
> -#undef fopen
> FILE *git_fopen(const char *path, const char *mode)
> {
> FILE *fp;
>
>
> -brandon
>Ta. I still get all the warnings with that, was that what you were trying to solve? The 64 bit specific tests in t5302 do still pass.
Mike