search

lasso-js / lasso

Advanced JavaScript module bundler, asset pipeline and optimizer
581 stars 75 forks source link

Fixes #180 - Defining bundles with "intersection" does not work for "require" dependencies #181

Closed philidem closed 7 years ago

philidem commented 7 years ago

Defining bundles with "intersection" does not work for "require" dependencies.

dependency-require uses data in lasso-context.phaseData['dependency-require'] to avoid walking the same dependency multiple times. It seems like a hack, but I simply reset the lassoContext.phaseData object before walking the dependencies and then I restore the old phase data when done.

See https://github.com/lasso-js/lasso-require/blob/master/src/util/Deduper.js#L20