jsvpx icon indicating copy to clipboard operation
jsvpx copied to clipboard

stray smart quote in Makefile

Open bvibber opened this issue 8 years ago • 5 comments

$ make
rm -rf build 
mkdir build  
VERSION=`node -pe "require('./package.json').version"` && \
	browserify src/main.js -t [ envify purge --MODE global ] | \
	tee  build/flare-vpx-“$VERSION".js 
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make: *** [build-release] Error 2

bvibber avatar Feb 17 '17 16:02 bvibber

It occurs to me the Makefile looks like it's been replaced by the webpack build; should it be removed?

bvibber avatar Feb 17 '17 17:02 bvibber

Yes, originally started with a makefile, but then started using npm scripts cause easier. But now if the scripts start getting longer, it might make more sense to just use a separate make or build script again. Maybe remove for now?

brianxautumn avatar Feb 17 '17 17:02 brianxautumn

Actually it does make more sense to use makefile. Going move the build scripts to there. Just added a new one to build the ogv example

brianxautumn avatar Feb 17 '17 19:02 brianxautumn

My one warning on using a Makefile is that it's harder to make a local build that works on both Unix and Windows.

For ogv.js I'm planning to split out the emscripten-built modules, which require a Unix build infrastructure for all the autoconf libraries anyway, and avoid Makefiles on the front-end build in favor of Grunt.

bvibber avatar Feb 17 '17 21:02 bvibber

That is a very good point. Npm for now it is!

brianxautumn avatar Feb 17 '17 23:02 brianxautumn