dsheiko / nw-autoupdater

⛔️ [DEPRECATED] Library provides low-level API to control NW.js app auto-updates
MIT License
75 stars 36 forks source link

Bugfix for Mac OS + new feature. #7

Closed cry0m1 closed 7 years ago

cry0m1 commented 7 years ago

Hi Dmitry,

I prepared new PR.

  1. Added key -l to rsync SwapScript strategy, why? As you know nwjs starting from 0.22.0 uses relative symlinks inside package. So we need copy them (if applicable) properly:
    $ man rsync
    -l, --links                 copy symlinks as symlinks
  2. Better to create different backups for every update to distinguish them.
  3. Added cleanUp() method to solve Mac OS bug, which hangs forever as installing....

Now update process works like a charm on Mac OS. I see another float bug w/ Mac OS, but not critical, it is regarding consequent two updates, which prevents run for final build, though you can do it manually without any issues or any backup build as well (symlinks are correct).