search

freedomjs / freedom

Embracing a distributed web
http://freedomjs.org
Apache License 2.0
512 stars 53 forks source link

fix build, update dependencies #286

Closed agallant closed 9 years ago

agallant commented 9 years ago

Fixes #285, #278, and more definitively resolves the issue first discussed in #276.

This takes a rather brute but hopefully robust-to-latent-dependency-changes approach to fixing our build issue. concat doesn't play nicely with sourcemaps-as-files that have unicode, but we don't need any unicode (our codebase and generated files are entirely ascii) so we can just remove the charset:utf-8 clause and things work as before. Obviously ideally we'll go back and remove this replacement approach once dependencies are more cooperative - I'll open a tracking issue so we don't lose track, but think that at the moment more investigation would be more cost than benefit.

The other changes in this request are just to bump dependencies and smooth over the few spots where that means changing things (of note, junit reporter now takes an outputDir rather than file).

agallant commented 9 years ago

@ryscheng @willscott @dborkan PTAL - not urgent in and of itself, but getting this (or something similar) merged will be blocking for getting other changes out (e.g. cutting a release for Dan's changes). Thanks!