[ffx] Fix bug that was leaving orphaned `ffx log` processes

When working in tree, `.jiri_root/bin/ffx` launches a `fx ffx`
subprocess. We were leaving orphaned process as `kill()` wasn't doing
what we expected. This change fixes that.

Before (note how each time the logs view is closed and reopened a new
process appears):
https://screencast.googleplex.com/cast/NjIwMzkwMTcyMTE4MjIwOHw1MGQ4Mzg1NC0xMA

After (only a single process when the view is active, zero processes
when it's closed):
https://screencast.googleplex.com/cast/NTYzNzk0MjUyMjQxMzA1Nnw0N2E0OTI5ZS02MA

An interesting side effect of this worth exploring more is that I
don't have to run `ffx doctor --restart-daemon` as often anymore.

Change-Id: I78fdd7866d4d115378a93570114cd35b42d2b42e
Reviewed-on: https://fuchsia-review.googlesource.com/c/vscode-plugins/+/732203
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: Solly Ross <sollyross@google.com>
2 files changed
tree: 3fdfb69a6424a3459d40e3544a66bd51f3745016
  1. .ci/
  2. .kokoro/
  3. .vscode/
  4. docs/
  5. media/
  6. resources/
  7. src/
  8. syntax/
  9. testdata/
  10. webviews/
  11. .eslintrc.json
  12. .gitignore
  13. .vscodeignore
  14. AUTHORS
  15. build-helpers.mjs
  16. build.mjs
  17. BUILDING.md
  18. CHANGELOG.md
  19. CONTRIBUTING.md
  20. LICENSE
  21. OWNERS
  22. package-lock.json
  23. package.json
  24. PATENTS
  25. README.md
  26. THIRD_PARTY_NOTICES.txt
  27. TROUBLESHOOTING_ZXDB.md
  28. tsconfig.json
README.md

Fuchsia extension for VS Code

This VS Code extension implements the commonly needed functionality for Fuchsia development.

This is a Work In Progress Currently supporting:

  • Interacting with the fuchsia target device.
    • Setting the default target.
    • Sending commands to the target.
  • Debugging C/C++ components.
  • Filtering and listening to target logs.
  • FIDL syntax highlighting

FFX path settings

The FFX tool path can be set under Settings > Extensions > Fuchsia SDK. If this path is not set, the extension will automatically search for it at:

  • ./tools/ffx
  • ./.jiri_root/bin/ffx

Recommended extensions

Contributing

Issues/feedback

  • Please report issues/feedback here.

Legal Notice

Use of the Fuchsia core developer tools is subject to the Google Terms of Service. The Google Privacy Policy describes how data is handled in your use of this service.