Page MenuHomePhabricator

[tunnelbroker] Remove lazy_static dependency
ClosedPublic

Authored by bartek on Aug 2 2023, 12:57 AM.
Tags
None
Referenced Files
F3242599: D8703.diff
Wed, Nov 13, 11:45 PM
Unknown Object (File)
Fri, Oct 18, 8:15 AM
Unknown Object (File)
Fri, Oct 18, 8:15 AM
Unknown Object (File)
Fri, Oct 18, 8:15 AM
Unknown Object (File)
Fri, Oct 18, 8:14 AM
Unknown Object (File)
Sep 26 2024, 4:17 PM
Unknown Object (File)
Sep 26 2024, 4:39 AM
Unknown Object (File)
Sep 20 2024, 1:39 PM
Subscribers

Details

Summary

Follow up ENG-2900.
This was a dead dependency - it's unused in tunnelbroker.

Test Plan

cd services/tunnelbroker && cargo build

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek requested review of this revision.Aug 2 2023, 1:19 AM

i think you can actually remove more than this :)

varun@varuns-MacBook-Pro tunnelbroker % cargo machete
Analyzing dependencies of crates in this directory...
cargo-machete found the following unused dependencies in /Users/varun/Code/comm/services/tunnelbroker:
tunnelbroker -- /Users/varun/Code/comm/services/tunnelbroker/Cargo.toml:
        a2
        aws-types
        base64
        env_logger
        fcm
        futures
        lazy_static
        log
        openssl
        prost
        prost-types
        tokio-stream
Done!

I think prost is actually required but everything else can probably be removed

This revision is now accepted and ready to land.Aug 3 2023, 11:16 AM