For the needs of the Rust service development, I moved the C++ implementation to the old dir.
- Modified Dockerfile accordingly
- Modified CMakeLists accordingly
- Left blob_client untouched as it is convenient for me to keep it where it is.
Differential D5841 Authored by bartek on Dec 8 2022, 12:03 PM.
Details Summary For the needs of the Rust service development, I moved the C++ implementation to the old dir.
Test Plan
Diff Detail
Event Timelinebartek held this revision as a draft. Herald added subscribers: atul, ashoat. · View Herald TranscriptDec 8 2022, 12:03 PM2022-12-08 12:03:36 (UTC+0) Harbormaster completed remote builds in B14059: Diff 19246.Dec 8 2022, 12:15 PM2022-12-08 12:15:42 (UTC+0) • jon added inline comments.
This revision now requires changes to proceed.Dec 8 2022, 8:21 PM2022-12-08 20:21:45 (UTC+0)
Comment Actions Moved the blob_client to the old directory, reverted the CMake logic. I decided to do this as it turned out I won't be re-using this code anyway. Harbormaster completed remote builds in B15076: Diff 20567.Jan 4 2023, 1:15 PM2023-01-04 13:15:58 (UTC+0) This revision is now accepted and ready to land.Jan 4 2023, 5:15 PM2023-01-04 17:15:29 (UTC+0) Comment Actions
Yes, the blob_client Rust library is going to be deleted along with the C++ codebase in ENG-2403. Closed by commit rCOMM97f2b69903cf: [services][backup] Move C++ impl to 'old' directory. · Explain WhyJan 11 2023, 9:30 AM2023-01-11 09:30:09 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 20567 | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||