Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to send SIGTERM or SIGINT to Processes started by dart #1756

Closed
efortuna opened this issue Feb 18, 2012 · 7 comments
Closed

Ability to send SIGTERM or SIGINT to Processes started by dart #1756

efortuna opened this issue Feb 18, 2012 · 7 comments
Assignees
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io type-enhancement A request for a change that isn't a bug
Milestone

Comments

@efortuna
Copy link
Contributor

Currently, if you create a Process, your only have two ways to end the proces: close() and kill(). Kill sends a SIGKILL signal, and close seems to just close the input/output streams without sending any signal. We'd like the functionality to also send signals like SIGINT to Processes we create in Dart.

@sethladd
Copy link
Contributor

Set owner to @madsager.
Added Triaged label.

@madsager
Copy link
Contributor

We can definitely add support for sending signals to processes. Could you give me more information about the use case for this? Sending SIGINT is Linux/Mac specific so I would like to figure out if we can provide something else that will cover your needs in a platform independent way. I do see the use case for sending signals on Mac and Linux but would like to understand if there is something else we can do for this particular case that would work on all platforms.


cc @sgjesse.
Added Accepted label.

@jmesserly
Copy link

The use case is to kill a process in the default Linux/Mac way (SIGTERM). This is preferred because the process can implement cleanup logic.

e.g. http://nodejs.org/docs/latest/api/child_processes.html#child.kill

@madsager
Copy link
Contributor

madsager commented Jun 7, 2012

Added this to the M1 milestone.

@andersjohnsen
Copy link

Set owner to @Skabet.

@andersjohnsen
Copy link

Fixed in http://code.google.com/p/dart/source/detail?r=8541

Default kill is now SIGTERM, while it's possible to provide another. This argument is ignored on windows.


Added Fixed label.

@kevmoo
Copy link
Member

kevmoo commented May 14, 2014

Removed Area-IO label.
Added Area-Library, Library-IO labels.

@efortuna efortuna added Type-Enhancement area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io labels May 14, 2014
@efortuna efortuna added this to the M1 milestone May 14, 2014
@kevmoo kevmoo added type-enhancement A request for a change that isn't a bug and removed type-enhancement labels Mar 1, 2016
copybara-service bot pushed a commit that referenced this issue Oct 11, 2022
…ptor, webdev

collection (https://github.com/dart-lang/collection/compare/414ffa1..ca45fc4):
  ca45fc4  2022-10-07  Devon Carew  update CI config; prep for publishing (#251)

glob (https://github.com/dart-lang/glob/compare/1d51fcc..ee81279):
  ee81279  2022-10-10  Moritz  Merge pull request #62 from dart-lang/bump_deps
  b6747a1  2022-10-10  moritz  Add changelog entry
  d069e13  2022-10-10  moritz  Bump depencies

intl (https://github.com/dart-lang/intl/compare/7639a15..dda8ade):
  dda8ade  2022-10-10  Moritz  GitHub Sync (#501)

markdown (https://github.com/dart-lang/markdown/compare/f387340..d72ae07):
  d72ae07  2022-10-07  Sam Rawlins  Use fewer empty lists; non-growable (#463)

stack_trace (https://github.com/dart-lang/stack_trace/compare/17f09c2..2194227):
  2194227  2022-10-10  Kevin Moore  A bunch of cleanup, prepare for v1.11.0 (#118)

test_descriptor (https://github.com/dart-lang/test_descriptor/compare/f392f85..66f14ce):
  66f14ce  2022-10-07  Devon Carew  update the CI configuration; prep for publishing (#44)

webdev (https://github.com/dart-lang/webdev/compare/7416956..69aac60):
  69aac60  2022-10-07  Parker Lougheed  Update min SDK restraint mentioned to match pubspec (#1755)
  b175072  2022-10-07  Elliott Brooks (she/her)  Start migration of WebDev to null-safety (#1756)

Change-Id: I452b89a44cfcb7b0d9f36eeb1539ea362b29b6b8
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/263521
Commit-Queue: Nate Bosch <nbosch@google.com>
Auto-Submit: Devon Carew <devoncarew@google.com>
Reviewed-by: Nate Bosch <nbosch@google.com>
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

6 participants