search

rollup / rollup-watch

Fast incremental rebuilds with Rollup CLI
MIT License
91 stars 23 forks source link

Regression: Rebuilds only once in v4.0.0. Works in v3.2.2 #51

Open btakita opened 7 years ago

btakita commented 7 years ago

Upon upgrading to v4.0.0, my watch scripts only fire on the first change.

Downgrading to v3.2.2 fixes the issue. I don't know the cause of the issue & I will further investigate if nobody else has already done so.

useChokidar is true in this case.

btakita commented 7 years ago

Adding the rollup option {watch: {useChokidar: false}} fixes the issue

Rich-Harris commented 7 years ago

I suspect this is because chokidar didn't get used at all prior to version 4, due to a silly bug. Are you able to create a repo reproducing the issue, so we can investigate further? Thanks

btakita commented 7 years ago

This issue is related to: https://github.com/paulmillr/chokidar/issues/556

btakita commented 7 years ago

I'm trying to pass in the config:

watch: { chokidar: {usePolling: true} }

However, inputOptions from function mergeOptions in bin/rollup does not include the watch member.

Rich-Harris commented 7 years ago

This repo is deprecated — rollup.watch is now part of core. If this is still an issue with the latest Rollup version, could you open a ticket over there please? thanks