Skip to content
This repository was archived by the owner on Feb 20, 2019. It is now read-only.

Activate stdin handler processing#57

Open
OvermindDL1 wants to merge 1 commit intorollup:masterfrom
OvermindDL1:patch-1
Open

Activate stdin handler processing#57
OvermindDL1 wants to merge 1 commit intorollup:masterfrom
OvermindDL1:patch-1

Conversation

@OvermindDL1
Copy link

The close handler when stdin is ended is not being called. Calling resume() on process.stdin will enable that handler so it will be called properly. It should 'probably' be called earlier in the program, perhaps at the very start, but if it spools up fast enough it should not matter, and this is the same place similar code in other projects places it as well.

Properly fixes #30

The `close` handler when `stdin` is `end`ed is not being called.  Calling `resume()` on `process.stdin` will enable that handler so it will be called properly.  It should 'probably' be called earlier in the program, perhaps at the very start, but if it spools up fast enough it should not matter, and this is the same place similar code in other projects places it as well.
@LostKobrakai
Copy link

@OvermindDL1 it's now merged in the rollup repo

@OvermindDL1
Copy link
Author

Ah, wouldn't have thought it goes there, weird. ^.^;

Does it work for the watcher from the main rollup repo @LostKobrakai ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

<stdin> closing and parent process closing do not close rollup --watch

2 participants