Page MenuHomePhabricator

[services] Delete shared C++ lib
AbandonedPublic

Authored by bartek on Jun 19 2023, 4:01 AM.
Tags
None
Referenced Files
F5108740: D8244.diff
Tue, Apr 1, 3:00 PM
Unknown Object (File)
Fri, Mar 14, 5:06 AM
Unknown Object (File)
Mon, Mar 10, 8:09 PM
Unknown Object (File)
Mon, Mar 10, 8:09 PM
Unknown Object (File)
Mon, Mar 10, 7:56 PM
Unknown Object (File)
Mon, Mar 10, 7:49 PM
Unknown Object (File)
Mon, Mar 3, 7:59 PM
Unknown Object (File)
Feb 23 2025, 7:52 PM
Subscribers

Details

Summary

Part of ENG-4098.
Deletes the services/lib directory which contained the shared C++ code.

Depends on D8230
old Tunnelbroker Dockerfile still required these files

Test Plan

CI, Tunnelbroker build should pass with D8230

Diff Detail

Repository
rCOMM Comm
Branch
barthap/remove-cpp-lib
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek held this revision as a draft.
bartek published this revision for review.Jun 28 2023, 12:21 PM

Ah, I didn't see this diff earlier and put out D8336

atul added a subscriber: atul.

Nice, no more reactors!

This revision is now accepted and ready to land.Jun 28 2023, 12:25 PM
In D8244#246350, @varun wrote:

Ah, I didn't see this diff earlier and put out D8336

Your diff actually removes more stuff than mine

atul added a reviewer: ashoat. atul added 1 blocking reviewer(s): jon.Jun 28 2023, 12:33 PM

Should've added @ashoat and @jon as blocking to take another look

This revision now requires review to proceed.Jun 28 2023, 12:33 PM

I'm not super familiar with the services codebase

This revision is now accepted and ready to land.Jun 29 2023, 7:15 AM