proxygen icon indicating copy to clipboard operation
proxygen copied to clipboard

Git Submodules

Open Maktm opened this issue 7 years ago • 4 comments

Since proxygen uses folly and wangle as dependencies, wouldn't it be easier to include them as submodules into the project? A simple command such as:

git clone --recursive https://github.com/facebook/proxygen.git

would pull all the required dependencies?

I noticed the files folly-rev.txt and wange-rev.txt under build/deps/github_hashes/facebook but I was under the impression that everything under build is for contributors since it's related to continous integration.

Maktm avatar Jul 05 '17 21:07 Maktm

Sounds like a good idea and something I'll look into. For now those *-rev.txt files linked are auto updated and so things are kept in sync automatically and is a system I believe other open sourced Fb projects buy into.

dddmello avatar Jul 06 '17 17:07 dddmello

on the same topic, I wanted to clone proxygen so I would have a stable release which I've checked and would not have different code running between different machines. The problem is that the deps script pulls the latest of folly, wangle and other libraries which defeats the whole purpose of my clone. Any ideas how to solve this?

Adam-DM avatar Aug 01 '17 00:08 Adam-DM

@AdamDailyMotion : you could locally modify the deps script to pin it to the folly/wangle head revisions. I realize it's not ideal.

afrind avatar Aug 07 '17 16:08 afrind

#226 should offer a simple workaround

arminfelder avatar May 13 '18 15:05 arminfelder

Closing old issue. We're committed to the current system for now, but if this is still a pain point please reopen.

afrind avatar Sep 18 '24 20:09 afrind