vm_tools: vm_cicerone: send sftp_port to ash.

Ash needs to know the port number so it can tell cros-disks
to mount sshfs using that port.


Change-Id: I50f1bb575302d0417d06ef036a9bd841a13939ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/3708623
Reviewed-by: David Munro <davidmunro@google.com>
Tested-by: Emil Mikulic <easy@google.com>
Commit-Queue: Emil Mikulic <easy@google.com>
GitOrigin-RevId: 68cb45097035d9c5b9e08a99515b333f3f452a76
1 file changed
tree: 76b19263257badb0e04c6f0ff71b7d96a97dd346
  1. constants/
  2. dbus/
  3. mojo/
  4. src/
  5. switches/
  6. .gitignore
  7. BUILD.gn
  8. build.rs
  9. Cargo.toml
  11. OWNERS
  12. README.md
  13. system_api.pc

This directory (platform2/system_api) contains constants and definitions like D-Bus service names that are shared between Chromium and Chromium OS.

This directory is only for things like headers and .proto files. No implementation should be added.

When writting a .proto file make sure to use:

option optimize_for = LITE_RUNTIME;

This will force usage of a lite protobuf instead of a full/heavy weight protobuf. The browser only links against the light version, so you will get cryptic link errors about missing parts of Message if you define a protobuf here and then try to use it in Chrome. Currently CrOS links against the full protobuffer library, but that might change in the future.

When declaring a protobuf, avoid use of required unless it is exactly what you mean. “Required is Forever” and very rarely should actually be used. Consult Protocol Buffer Basics: C++ for a detailed of this issue.