mozilla-jetpack / jpm

Jetpack Manager for Node.js
https://www.npmjs.org/package/jpm
Mozilla Public License 2.0
164 stars 75 forks source link

`jpm xpi` should work with `--prefs` #555

Open gregglind opened 8 years ago

gregglind commented 8 years ago

I don't see anything in lib/xpi.js that does this.

gregglind commented 8 years ago

https://github.com/mozilla-jetpack/jpm/blob/a7e064217cc976326955e76afeba61b521931fc4/lib/task.js#L38

is where it happens in run and test.

CLAIM: it should be during INSTALL ONLY, and happen during all the other startup stuff, IFF reason = install.

I have no idea if what should happen during uninstall or reset. Maybe that needs thinking?