Google Git
Sign in
android / platform / system / tools / aidl / fc2ddbfcf7c8eddb03b4820095a494f572d1dbbd
commitfc2ddbfcf7c8eddb03b4820095a494f572d1dbbd[log] [tgz]
authorSteven Moreland <smoreland@google.com>Thu Jul 29 16:52:34 2021 -0700
committerSteven Moreland <smoreland@google.com>Fri Jul 30 16:36:08 2021 +0000
treee9adb0853b4a5bab9e694ec447503572ec3804ab
parent2cb8ecdf6a135360a8141f17f0f0db772e6f81a0 [diff]
.hash files can have multiple hashes

I have wanted this since the beginning :) HIDL interfaces have benefited
from supporting this repeatedly. Although we encounter the problem a lot
less in AIDL, since we already allow backwards compatible new versions,
and we don't have comments in aidl_api files, there are two cases where
we have hit this recently:

- a late backwards compatible change to the biometrics face AIDL HAL
(changing a type to be @nullable). While this doesn't affect the wire
protocol or stability over the wire, it does require adding a new hash.
So, VTS will fail on old devices that don't have the patch. Late in
release, this has caused problems for certain HIDL changes.

- we want to add '@hide' to V1 of FMQ, so we need two hashes for it
(this is the most straightforward resolution of b/194739778 to decouple
the issue from -java-shallow, which is slightly contentious/complex)

Of course, THIS SHOULD BE USED AS A SOLUTION OF LAST RESORT!

Fixes: 194509098
Test: libaidlmetadata_test
Change-Id: Id5080d026e9352dac17e02a1895ae9abfbac030c
4 files changed
tree: e9adb0853b4a5bab9e694ec447503572ec3804ab
  1. aidl_api/
  2. build/
  3. metadata/
  4. tests/
  5. trace/
  6. .clang-format
  7. aidl-format.sh
  8. aidl.cpp
  9. aidl.h
  10. aidl_checkapi.cpp
  11. aidl_checkapi.h
  12. aidl_const_expressions.cpp
  13. aidl_dumpapi.cpp
  14. aidl_dumpapi.h
  15. aidl_language.cpp
  16. aidl_language.h
  17. aidl_language_l.ll
  18. aidl_language_y.yy
  19. aidl_to_cpp.cpp
  20. aidl_to_cpp.h
  21. aidl_to_cpp_common.cpp
  22. aidl_to_cpp_common.h
  23. aidl_to_java.cpp
  24. aidl_to_java.h
  25. aidl_to_ndk.cpp
  26. aidl_to_ndk.h
  27. aidl_to_rust.cpp
  28. aidl_to_rust.h
  29. aidl_typenames.cpp
  30. aidl_typenames.h
  31. aidl_unittest.cpp
  32. Android.bp
  33. ast_cpp.cpp
  34. ast_cpp.h
  35. ast_cpp_unittest.cpp
  36. ast_java.cpp
  37. ast_java.h
  38. ast_java_unittest.cpp
  39. code_writer.cpp
  40. code_writer.h
  41. code_writer_unittest.cpp
  42. comments.cpp
  43. comments.h
  44. diagnostics.cpp
  45. diagnostics.h
  46. diagnostics.inc
  47. diagnostics_unittest.cpp
  48. generate_aidl_mappings.cpp
  49. generate_aidl_mappings.h
  50. generate_cpp.cpp
  51. generate_cpp.h
  52. generate_cpp_unittest.cpp
  53. generate_java.cpp
  54. generate_java.h
  55. generate_java_binder.cpp
  56. generate_ndk.cpp
  57. generate_ndk.h
  58. generate_rust.cpp
  59. generate_rust.h
  60. hiddenapi-greylist
  61. import_resolver.cpp
  62. import_resolver.h
  63. io_delegate.cpp
  64. io_delegate.h
  65. io_delegate_unittest.cpp
  66. location.cpp
  67. location.h
  68. logging.cpp
  69. logging.h
  70. main.cpp
  71. NOTICE
  72. options.cpp
  73. options.h
  74. options_unittest.cpp
  75. os.h
  76. OWNERS
  77. parser.cpp
  78. parser.h
  79. preprocess.cpp
  80. preprocess.h
  81. PREUPLOAD.cfg
  82. README.md
  83. run_integration_tests.sh
  84. TEST_MAPPING
README.md

Documentation for this project is currently maintained here:

https://source.android.com/devices/architecture/aidl/overview