Gitiles
Code Review Sign In
gerrit.omnirom.org / android_system_update_engine / 135197526b9c0f5a9acac57a05129339f311bbca
commit135197526b9c0f5a9acac57a05129339f311bbca[log] [tgz]
authorSen Jiang <senj@google.com>Tue Aug 02 16:10:52 2016 -0700
committerSen Jiang <senj@google.com>Fri Aug 05 11:19:04 2016 -0700
tree986e0ca3438de3f96235207597d82f9ba1965e63
parent44348e01c0c66e6baf1376f0b712ecae26fc5595 [diff]
brillo_update_payload: Show correct error when delta_generator not found.

If delta_generator is not in PATH, $(which delta_generator) will return 1
and then the script just go ahead to cleanup because trap is set on ERR,
but the error message never got a chance to be showed.

Bug: None
Test: run brillo_update_payload without delta_generator in PATH.

Change-Id: Ie7a6943f174f69e11ca795351d72c20516792f6f
1 file changed
tree: 986e0ca3438de3f96235207597d82f9ba1965e63
  1. binder_bindings/
  2. client_library/
  3. common/
  4. dbus_bindings/
  5. include/
  6. init/
  7. payload_consumer/
  8. payload_generator/
  9. sample_images/
  10. scripts/
  11. update_manager/
  12. update_payload_key/
  13. weaved/
  14. .clang-format ⇨ ../../build/tools/brillo-clang-format
  15. .gitignore
  16. Android.mk
  17. binder_service_android.cc
  18. binder_service_android.h
  19. binder_service_brillo.cc
  20. binder_service_brillo.h
  21. boot_control_android.cc
  22. boot_control_android.h
  23. boot_control_chromeos.cc
  24. boot_control_chromeos.h
  25. boot_control_chromeos_unittest.cc
  26. boot_control_recovery_stub.cc
  27. certificate_checker.cc
  28. certificate_checker.h
  29. certificate_checker_unittest.cc
  30. chrome_browser_proxy_resolver.cc
  31. chrome_browser_proxy_resolver.h
  32. chrome_browser_proxy_resolver_unittest.cc
  33. common_service.cc
  34. common_service.h
  35. common_service_unittest.cc
  36. connection_manager.cc
  37. connection_manager.h
  38. connection_manager_android.cc
  39. connection_manager_android.h
  40. connection_manager_interface.h
  41. connection_manager_unittest.cc
  42. connection_utils.cc
  43. connection_utils.h
  44. daemon.cc
  45. daemon.h
  46. daemon_state_android.cc
  47. daemon_state_android.h
  48. daemon_state_interface.h
  49. dbus_connection.cc
  50. dbus_connection.h
  51. dbus_service.cc
  52. dbus_service.h
  53. dbus_test_utils.h
  54. fake_file_writer.h
  55. fake_p2p_manager.h
  56. fake_p2p_manager_configuration.h
  57. fake_shill_proxy.cc
  58. fake_shill_proxy.h
  59. fake_system_state.cc
  60. fake_system_state.h
  61. generate_pc_file.sh
  62. hardware_android.cc
  63. hardware_android.h
  64. hardware_chromeos.cc
  65. hardware_chromeos.h
  66. hardware_chromeos_unittest.cc
  67. image_properties.h
  68. image_properties_android.cc
  69. image_properties_chromeos.cc
  70. image_properties_chromeos_unittest.cc
  71. libcros_proxy.cc
  72. libcros_proxy.h
  73. libcurl_http_fetcher.cc
  74. libcurl_http_fetcher.h
  75. libupdate_engine-client-test.pc.in
  76. libupdate_engine-client.pc.in
  77. local_coverage_rate
  78. main.cc
  79. metrics.cc
  80. metrics.h
  81. metrics_utils.cc
  82. metrics_utils.h
  83. metrics_utils_unittest.cc
  84. mock_certificate_checker.h
  85. mock_connection_manager.h
  86. mock_file_writer.h
  87. mock_omaha_request_params.h
  88. mock_p2p_manager.h
  89. mock_payload_state.h
  90. mock_power_manager.h
  91. mock_proxy_resolver.h
  92. mock_update_attempter.h
  93. MODULE_LICENSE_APACHE2
  94. network_selector.h
  95. network_selector_android.cc
  96. network_selector_android.h
  97. network_selector_interface.h
  98. network_selector_stub.cc
  99. network_selector_stub.h
  100. NOTICE
  101. omaha_request_action.cc
  102. omaha_request_action.h
  103. omaha_request_action_unittest.cc
  104. omaha_request_params.cc
  105. omaha_request_params.h
  106. omaha_request_params_unittest.cc
  107. omaha_response.h
  108. omaha_response_handler_action.cc
  109. omaha_response_handler_action.h
  110. omaha_response_handler_action_unittest.cc
  111. omaha_utils.cc
  112. omaha_utils.h
  113. omaha_utils_unittest.cc
  114. OWNERS
  115. p2p_manager.cc
  116. p2p_manager.h
  117. p2p_manager_unittest.cc
  118. parcelable_update_engine_status.cc
  119. parcelable_update_engine_status.h
  120. payload_state.cc
  121. payload_state.h
  122. payload_state_interface.h
  123. payload_state_unittest.cc
  124. power_manager_android.cc
  125. power_manager_android.h
  126. power_manager_chromeos.cc
  127. power_manager_chromeos.h
  128. power_manager_interface.h
  129. PRESUBMIT.cfg
  130. proxy_resolver.cc
  131. proxy_resolver.h
  132. real_system_state.cc
  133. real_system_state.h
  134. run_unittests
  135. sample_omaha_v3_response.xml
  136. service_delegate_android_interface.h
  137. service_observer_interface.h
  138. shill_proxy.cc
  139. shill_proxy.h
  140. shill_proxy_interface.h
  141. sideload_main.cc
  142. system_state.h
  143. tar_bunzip2.gypi
  144. test_http_server.cc
  145. test_subprocess.cc
  146. testrunner.cc
  147. unittest_key.pem
  148. unittest_key2.pem
  149. update_attempter.cc
  150. update_attempter.h
  151. update_attempter_android.cc
  152. update_attempter_android.h
  153. update_attempter_unittest.cc
  154. update_engine-client.gyp
  155. update_engine.conf
  156. update_engine.gyp
  157. update_engine.rc
  158. update_engine_client.cc
  159. update_engine_client_android.cc
  160. update_metadata.proto
  161. update_status_utils.cc
  162. update_status_utils.h
  163. UpdateEngine.conf
  164. utils_android.cc
  165. utils_android.h
  166. WATCHLISTS
  167. weave_service.cc
  168. weave_service.h
  169. weave_service_factory.cc
  170. weave_service_factory.h
  171. weave_service_interface.h