Initialize struct flock for AIX with O_LARGEFILE used by Python.

Initialize struct flock for AIX with O_LARGEFILE used by Python.

gyp issue 440

Python on AIX is compiled with LARGEFILE support.  gyp flock_tool must
use the appropriate stucture member definitions to properly initialize
the structure.

BUG=

r2008

[David Edelsohn, 2014-11-21 15:56:33]

dje.gcc@gmail.com changed reviewers:
+ thakis@chromium.org

[David Edelsohn, 2014-11-21 15:56:34]

Hi, Nico

This is the new issue created for the AIX flock LARGEFILE initialization bug.

Thanks, David

[Nico, 2014-11-21 17:32:14]

lgtm with a comment:

https://codereview.chromium.org/738943004/diff/1/pylib/gyp/flock_tool.py
File pylib/gyp/flock_tool.py (right):

https://codereview.chromium.org/738943004/diff/1/pylib/gyp/flock_tool.py#newc...
pylib/gyp/flock_tool.py:43: if sys.platform.startswith('aix'):
Please add a comment saying

  # Python on AIX is compiled with LARGEFILE support, which changes the struct
size.

above the new struct.pack line.

[David Edelsohn, 2014-11-21 23:45:29]

On 2014/11/21 17:32:14, Nico wrote:
> lgtm with a comment:
> 
> https://codereview.chromium.org/738943004/diff/1/pylib/gyp/flock_tool.py
> File pylib/gyp/flock_tool.py (right):
> 
>
https://codereview.chromium.org/738943004/diff/1/pylib/gyp/flock_tool.py#newc...
> pylib/gyp/flock_tool.py:43: if sys.platform.startswith('aix'):
> Please add a comment saying
> 
>   # Python on AIX is compiled with LARGEFILE support, which changes the struct
> size.
> 
> above the new struct.pack line.

I uploaded the revised patch.  I don't believe that I have an authorized account
on Chromium to push the change.

[Nico, 2014-11-22 00:06:55]

lgtm, I'll land

Thanks!

[Nico, 2014-11-22 00:08:08]

gyp r2008