Only run gulp build:dist if we're in production

[jeffposnick]

R: @petele

This should fix the issue @petele was seeing when running npm install from within the app-shell-demo/ directory in a freshly cloned copy of the code.

[echeese]

This is causing npm postinstall to fail on Windows.

[jeffposnick]

Sorry about that, @echeese. I've reverted and will come up with an alternative approach.

Just to confirm, this didn't impact usage of the core sw-precache library, right? It's just when you tried to test out app-shell-demo locally?

[echeese]

It's okay, it seems like it works fine, the postinstall just fails. However, everything else seems to work fine.

[jeffposnick]

Right, but just to clarify, you were specifically running an npm install for a local copy of the app-shell-demo/ project when you encountered this, right?

I want to confirm that this isn't impacting all Windows users who just do a general npm install sw-precache (and don't have a Windows machine handy to test at the moment...).

[echeese]

sw-precache installed fine, the error only happened when I explicitly changed to the app-shell-demo subdirectory and ran npm install.

[jeffposnick]

Thanks for confirming!