`--asar` should be enabled by default

[sindresorhus]

I don't see why you wouldn't want those and I think most do. We could instead have --no-asar and --no-prune flags. I much prefer us to have sane defaults, which means less config mess for users.

[junosuarez]

I support both of these as defaults

[max-mapper]

i had seen a lot of open issues involving asar on the electron repo since it's relatively new. but if we think its bug-free now then i'm good with it

[sindresorhus]

I don't think it's bug free, but it's pretty stable now, and Atom is using it so any issue will be fixed fast.

[junosuarez]

aside: as someone who just had to leave their computer on overnight copying over a large ~/dev directory full of hundreds of thousands of files from node_modules directory trees on a faulty SSD, I kinda wish a certain software platform would move to archive-based dependencies, too :)

[ccorcos]

Only bad part of this is you'll have to re-install when you prune. Here's what I'm doing:

# make a regex of node_modules to ignore
ignores=`npm ls --dev --parseable`
# get the package name
ignores=$(echo $ignores | sed -e 's@.*/@@')
# skip the first line
ignores=`echo $ignores | awk '{if(NR > 1) print $0}'`
# join with a pipe
ignores=${ignores//$'\n'/'|'}

# build mac distibutable app
./node_modules/.bin/electron-packager . "Dev Tool" \
  --platform=darwin --arch=x64 --version=0.35.6 \
  --ignore=\"$ignores\" \
  --app-bundle-id "com.dev-too.appl" --app-version 0.1.0 \
  --out ./built

[ccorcos]

zsh: command too long :(

[max-mapper]

if you pass the --prune flag to electron-packager it prunes for you in its tmpdir copy, which doesnt affect your source folder

[ccorcos]

well then there we go!

[malept]

I have no problem with making prune default to true.

The main problem I have with making asar default to true is that as soon as that gets released, there is going to be an influx of issues filed on electron-packager with "module not found" errors when people use native extensions. Or really, any exception listed in the Electron docs.