cloudflare / json-schema-tools

Packages for working with JSON Schema and JSON Hyper-Schema
BSD 3-Clause "New" or "Revised" License
321 stars 30 forks source link

Windows support #5

Open handrews opened 6 years ago

handrews commented 6 years ago

The new @cloudflare/doca probably doesn't work any better on Windows than the old.

You must set an environment variable by set SOMEVAR=value. Further more this is a separete command so you'll have to add an additional && before the call to webpack like for instance

"cleanwin": "rmdir /S /Q build",
"buildwin": "npm run cleanwin && set NODE_ENV=production && webpack --config webpack/config.prod.withjs.babel.js && del build\\static-*", 
"buildwin:nojs": "npm run cleanwin set && NODE_ENV=production && webpack --config webpack/config.prod.nojs.babel.js && del build\\static-* && del build\\app-*.js"

From @aphilaja in cloudflare/doca#26:

I don't think Windows build has to be a special case:

use rimraf module instead of del and rm -rf. use webpack -p instead of NODE_ENV=production Then it should works equally in win & *nix.

Edit: Hmm.. not sure if -p sets NODE_ENV to production. If that's not the case and the env var is needed, I think there is Webpack plugin for it.

zhangyongjing commented 5 years ago

'cross-env' can solve the NODE_ENV issue, - by installing 'cross-env' and add 'cross-env' before 'NODE_ENV'. But one still needs to manually fix require paths with forward slashes in 'schema.js'.

handrews commented 5 years ago

Thanks, @zhangyongjing !