search

noseglid / atom-build

:hammer: Build your project directly from the Atom editor
https://atom.io/packages/build
MIT License
248 stars 97 forks source link

Fix #484, #447 #502

Closed sphaerophoria closed 7 years ago

sphaerophoria commented 7 years ago

In TargetManager, getTargets, and by extension fillTargets, has an implicit requirement that pathTarget.targets is not empty. If it is we re-call refreshTargets which would end up with infinite recursion. We don't actually need to call fillTargets as we already know which targets are available, we can just set them ourselves

sphaerophoria commented 7 years ago

I seem to have missed some of the side-effects of this change closing for the time being.