Page MenuHomePhabricator

[blob] Use `FetchContent_Declare` to pull in `gRPC`
AbandonedPublicDraft

Authored by atul on Jul 25 2022, 11:42 AM.
Tags
None
Referenced Files
F2180828: D4625.id14884.diff
Wed, Jul 3, 3:39 PM
Unknown Object (File)
Tue, Jul 2, 8:25 PM
Unknown Object (File)
Tue, Jul 2, 4:15 AM
Unknown Object (File)
Tue, Jul 2, 12:34 AM
Unknown Object (File)
Sat, Jun 29, 8:42 AM
Unknown Object (File)
Wed, Jun 26, 6:12 AM
Unknown Object (File)
Wed, Jun 26, 6:12 AM
Unknown Object (File)
Wed, Jun 26, 6:12 AM
Subscribers

Details

Summary

Context from D4624 (replace glog with gRPC):
Right now we handle dependencies for services in a very weird way that depends on various shell scripts and assumes a Docker environment built atop the base-image.

This diff uses the CMake FetchContent_Declare functionality to pull in the correct version of glog (as described in services/base-image/docker/install_glog.sh) without depending on a shell script/base image.

In my opinion it makes way more sense for each service to handle required dependencies rather than assuming there's some "base image" that magically takes care of things.

The coupling of services builds with assumption of Docker environment leads to CMake projects that aren't portable... making it impossible to get things to work directly on dev machines. IMO the right level of abstraction for handling dependencies is the build system (in this case CMake).


Depends on D4624

Test Plan

Blob CI continues to work.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2022, 11:45 AM
Harbormaster failed remote builds in B10821: Diff 14884!
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2022, 11:54 AM
Harbormaster failed remote builds in B10822: Diff 14885!
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2022, 12:00 PM
Harbormaster failed remote builds in B10823: Diff 14886!
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2022, 12:07 PM
Harbormaster failed remote builds in B10824: Diff 14887!
Harbormaster returned this revision to the author for changes because remote builds failed.Jul 25 2022, 12:10 PM
Harbormaster failed remote builds in B10825: Diff 14888!