Media splitting: just copy the files for build instead of uglifying them

[iseulde]

• Don't uglify, just copy like the other watch tasks.
• Only copy the changed files, currently it uglifies all of them.
• Exclude media files from the all task.
• Clean up the callback a bit.

[iseulde]

Makes it pretty fast: Completed in 0.012s at Mon Apr 06 2015 16:46:39 GMT-0400 (EDT)

[wonderboymusic]

In 33959:

Pre-req for Media builds: make sure the watcher actually works for browserify.

See #31911.

[wonderboymusic]

In 33960:

Speed up browserify builds, don't uglify the media builds on watch.

Props iseulde.
Fixes #31911.

[jdgrimes]

I'm not sure why the *.manifest files aren't being watched. Shouldn't the browserify task be run on the media files when the manifests are edited as well?

[wonderboymusic]

Replying to jdgrimes:

I'm not sure why the *.manifest files aren't being watched. Shouldn't the browserify task be run on the media files when the manifests are edited as well?

They should be - the watcher was totally broken, so the fixes above were a start. Not watching them is an oversight, wanna patch it for some easy props?

[jdgrimes]

Don't ignore manifest files

[jdgrimes]

Replying to wonderboymusic:

Replying to jdgrimes:

I'm not sure why the *.manifest files aren't being watched. Shouldn't the browserify task be run on the media files when the manifests are edited as well?

They should be - the watcher was totally broken, so the fixes above were a start. Not watching them is an oversight, wanna patch it for some easy props?

31911.diff​

[wonderboymusic]

one step further, we need the watcher to:

• watch the proper files
• build them into media-*.js files
• only copy the built files over, not the files in media/*

I'm not sure how or when we borked this so completely

[netweb]

Replying to wonderboymusic:

I'm not sure how or when we borked this so completely

wonderboymusic is fired r31629

[wonderboymusic]

Replying to netweb:

Replying to wonderboymusic:

I'm not sure how or when we borked this so completely

wonderboymusic is fired r31629

I would focus on the Props iseulde... portion, mostly.

[wonderboymusic]

hey @iseulde, welcome back! funny story: you, me, we, at some point, broke this. Still isn't in an ideal state - see comment:11

[iseulde]

@wonderboymusic No need to watch the modules, and the built media files are copied through all. I can't get watchify to work with grunt-browserify v 3.8, updating to 4.0 fixes the problem for me.

[iseulde]

Please review, let me know if anything unexpected happens.

[wonderboymusic]

In 34888:

Build Tools: upgrade grunt-browserify to ~4.0.0 and remove the watch task for browserify, which has its own internal watcher. Run npm install to update.

Props iseulde.
Fixes #31911.