Enable the .pak files on mac builds also.

Enable the .pak files on mac builds also.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=10446

[tony, 2009-02-25 20:43:23]

http://codereview.chromium.org/27153/diff/1/2
File tools/grit/grit/tool/build.py (right):

http://codereview.chromium.org/27153/diff/1/2#newcode164
Line 164: if output.GetType() == 'data_package' and not_windows:
I think this is backwards.  If the platform is windows, then go down the if path
(write an empty file).

I also think that this code could be completely removed and windows will work
fine now, but I can try that myself later.

[TVL, 2009-02-25 20:50:37]

http://codereview.chromium.org/27153/diff/1/2
File tools/grit/grit/tool/build.py (right):

http://codereview.chromium.org/27153/diff/1/2#newcode164
Line 164: if output.GetType() == 'data_package' and not_windows:
On 2009/02/25 20:43:23, tony wrote:
> I think this is backwards.  If the platform is windows, then go down the if
path
> (write an empty file).
> 
> I also think that this code could be completely removed and windows will work
> fine now, but I can try that myself later.

oops, right.  new version going up.

[TVL, 2009-02-25 20:55:38]

On 2009/02/25 20:50:37, TVL wrote:
> http://codereview.chromium.org/27153/diff/1/2
> File tools/grit/grit/tool/build.py (right):
> 
> http://codereview.chromium.org/27153/diff/1/2#newcode164
> Line 164: if output.GetType() == 'data_package' and not_windows:
> On 2009/02/25 20:43:23, tony wrote:
> > I think this is backwards.  If the platform is windows, then go down the if
> path
> > (write an empty file).
> > 
> > I also think that this code could be completely removed and windows will
work
> > fine now, but I can try that myself later.
> 
> oops, right.  new version going up.

i'll post when i have this updated.  having issues uploading at the moment, and
gotta run to a meeting.

[TVL, 2009-02-25 22:58:50]

sorry about that, this should have the right logic now.

[tony, 2009-02-25 23:05:03]

LGTM!