leaflet-geopackage build problems

[jsuwala]

Hi!

Thanks for your work on leaflet-geopackage; it's great to see this integration between leaflet and geopackage-js working.

I thought I should report two problems with your current package.json that I found when trying to compile leaflet-geopackage:

1. the webworkify package is missing (I got the error "Error: Cannot find module 'webworkify' from '/home/user/geopackage/leaflet/node_modules/geopackage/lib/tiles/creator'")
2. the default uglify-js installed does not support es6, so gives the error "uglifyjs failed: SyntaxError: Unexpected token: operator (>)".

A fix to the missing webworkify package is straight-forward.

And I found a fix for the uglify-js problem here: ionic-team/ionic-framework#8817 , which says you need to reference the "harmony" branch of uglify-js.

The revised package.json file that includes these two fixes is attached.

• Jason
  package.json.txt

[danielbarela]

I added webworkify but I cannot make the syntax error occur. What did you run to make that happen?
Thanks

[jsuwala]

I was just running npm commands:
1: npm install

"Error: Cannot find module 'webworkify' from '/home/user/geopackage/leaflet/node_modules/geopackage/lib/tiles/creator'"

2: npm install webworkify --save
3: npm run build-min

Parse error at -:20762,24 module.exports = input => { ^ SyntaxError: Unexpected token: operator (>)

I'm running Ubuntu 14.04.5 LTS, with npm version 4.0.5

[danielbarela]

AH, npm 4.0.5 was the key. Added in your uglify fix and merged to master.
Thanks