1
0
mirror of https://github.com/facebookincubator/mvfst.git synced 2025-04-18 17:24:03 +03:00

32 Commits

Author SHA1 Message Date
Konstantin Tsoy
adf16f9a07 Remove libccp from mvfst
Summary: We don't use it, and the OSS lib hasn't been updated in a while.

Reviewed By: mjoras

Differential Revision: D46707559

fbshipit-source-id: ec102a52183a736cfb1c0241600816a837062108
2023-06-15 18:17:53 -07:00
Luca Niccolini
ce0873c14d make build script read gtest/fmt/zstd version from manifest
Reviewed By: jbeshay, kvtsoy

Differential Revision: D38965330

fbshipit-source-id: 240171d9d8e7ee6039d4dc301903287963559ffd
2022-08-24 09:41:01 -07:00
Luca Niccolini
ab9cb63da2 fix zstd OSS build
Summary:
* pull the same version used by getdeps
 * actually build zstd (we were not entering the directory)

Reviewed By: jbeshay

Differential Revision: D38952608

fbshipit-source-id: 68763cf558e9cfd11873f51e50b93ff6f4d0cad3
2022-08-23 12:41:46 -07:00
Luca Niccolini
7127a107fb fix OSS build
Reviewed By: jbeshay

Differential Revision: D34739371

fbshipit-source-id: 9e60ee1c0b664785751b2da1f3b9d9d67386c436
2022-03-08 22:17:19 -08:00
Gilson Takaasi Gil
7f2f302f96 Revert D34351084: Migrate from googletest 1.8 to googletest 1.10
Differential Revision:
D34351084 (715dec85be)

Original commit changeset: 939b3985ab63

Original Phabricator Diff: D34351084 (715dec85be)

fbshipit-source-id: 2fd17e0ccd9d1f1d643f4a372d84cb95f5add1f8
2022-03-03 04:41:46 -08:00
Dimitri Bouche
715dec85be Migrate from googletest 1.8 to googletest 1.10 (#67)
Summary:
X-link: https://github.com/facebookincubator/hsthrift/pull/67

Updating `googletest` from `1.8.0` to `1.10.0`

Reviewed By: mzlee, igorsugak, luciang, meyering, r-barnes

Differential Revision: D34351084

fbshipit-source-id: 939b3985ab63a06b6d511ec8711c2d5863bdfea8
2022-03-03 03:46:03 -08:00
Hani Damlaj
00e67c1bf9 mvfst License Header Update
Reviewed By: lnicco

Differential Revision: D33587012

fbshipit-source-id: 972eb440f0156c9c04aa6e8787561b18295c1a97
2022-01-18 13:56:12 -08:00
Hani Damlaj
2660a288b3 Update Company Name
Summary: - as title

Reviewed By: lnicco

Differential Revision: D33513410

fbshipit-source-id: 282b6f512cf83b9abb7990402661135b658f7bd1
2022-01-13 12:07:48 -08:00
Andrii Vasylevskyi
e4359fc12f Setup zstd in build_helper
Summary:
Right now ubuntu build fails on folly dependency on newer zstd lib than is provided by distro.
Adding setup_zstd to clone and build zstd, similar to other dependencies.

Reviewed By: lnicco

Differential Revision: D26692582

fbshipit-source-id: bd7514cbf3e40271b25658fc543e6dc3d3c8b2ee
2021-03-03 07:26:27 -08:00
Luca Niccolini
d73e3db2ad CLI flag to skip fetching dependencies
Summary: to make local changes to dependencies after the first build

Reviewed By: mjoras

Differential Revision: D25377891

fbshipit-source-id: fe4c8621c3845debe36432229ac4658d9cfbddfe
2020-12-07 13:07:56 -08:00
Luca Niccolini
f400d748cd no need to build tests for dependencies
Summary: faster build

Reviewed By: mjoras

Differential Revision: D23700094

fbshipit-source-id: d15bf29bfaacf49b92fc3002d5c9ed6627544d04
2020-09-15 09:30:01 -07:00
Luca Niccolini
ac33c9e7d3 googletest 1.8.0
Summary: required. not installed with CMake ExternalProject anymore

Reviewed By: xttjsn

Differential Revision: D23700095

fbshipit-source-id: 807105a1e3d4bd837a35e39c6dced53744edd8de
2020-09-14 23:58:35 -07:00
Luca Niccolini
692597298e fix build script array expansion
Summary:
this is the most portable way according to:
https://gist.github.com/dimo414/2fb052d230654cc0c25e9e41a9651ebe

fixed #143

Reviewed By: avasylev

Differential Revision: D23013599

fbshipit-source-id: 5d766e9f4ab5cc6cdc3311a5dbd745f23a5854b8
2020-08-10 06:55:52 -07:00
Frank Cangialosi
1819cc152b start ccp as thread within mvfst
Summary:
This diff implements running ccp as a thread within mvfst.

1. It adds `libstartccp`, which is a tiny rust library that provides an interface between multiple ccp algorithms (written in rust) and the calling code in mvfst (written in c++). It exposes a single function `ccp_run_forever`, which takes an algorithm and optional parameters, and then starts ccp using the correct algorithm. This function never returns, so it is intended to be called from a dedicated thread.
2. When ccp is enabled and a `ccpConfig` is provided, `QuicServer` simply calls `ccp_run_forever` its own thread, passing the `ccpConfig`, which specifies both the algorithm name and any optional parameters.

To add a new ccp algorithm:
1. Place the algorithm source code in its own folder here: `quic/congestion_control/third_party/ccp/[ALGORITHM]`
2. Add the algorithm to `quic/congestion_control/third_party/ccp/src.rs`:
(a) Add `extern crate [ALGORITHM];` at the top.
(b) Add `register_alg!([ALGORITHM]);` at the end of the `_start` function.

Reviewed By: udippant

Differential Revision: D21854346

fbshipit-source-id: 5b0718a90d560e9cbe6e616daea5e910e00e6aeb
2020-07-27 14:09:26 -07:00
Frank Cangialosi
267bd4d3d2 OSS build support for quic+CCP
Summary: Adds dependencies and build rules for building quic with CCP_ENABLED using CMake.

Reviewed By: udippant

Differential Revision: D22736839

fbshipit-source-id: a59ac2df89312ca264ee78b217c6ccd0fcb64f29
2020-07-27 14:09:26 -07:00
Francis Cangialosi
5312c13bbf use latest release from fmtlib instead of master
Summary:
mvfst and proxygen both depend on folly, which depends on fmt, which must be built from source. the current build scripts get fmt from master, which could include a breaking change. this diff instead checks out the current latest version (6.2.1) which is known to build successfully.

related discussion in comments: https://fb.workplace.com/groups/248214662741731/permalink/601467044083156/

Reviewed By: lnicco

Differential Revision: D22213809

fbshipit-source-id: cf5efd2bf2bc694a3c246db9666088962465d4e9
2020-07-13 21:59:07 -07:00
Luca Niccolini
9e9fb71ce0 install m4 as a dependency for oss build
Summary: as in title.

Reviewed By: avasylev

Differential Revision: D22325090

fbshipit-source-id: d35477c4c3fc29c0138f4f9cd7c29c0f845110ea
2020-06-30 21:28:53 -07:00
Amaury Séchet
bea6efc248 Add ccache support to the build_helper.sh (#107)
Summary:
This adds a -c option to the build helper that will use ccache to build everything.
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/107

Reviewed By: yangchi

Differential Revision: D20457572

Pulled By: mjoras

fbshipit-source-id: b8cf13c49a2040acf211b1f443539c440d99e6fc
2020-03-15 18:32:48 -07:00
Olivier Delalleau
d968de4625 Minor improvements to folly & fizz synch (#94)
Summary:
Workaround for https://github.com/facebookincubator/mvfst/issues/86
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/94

Reviewed By: lnicco

Differential Revision: D19744560

Pulled By: udippant

fbshipit-source-id: 77554feef74356ee76250f527319f8787060e7f8
2020-02-11 15:12:12 -08:00
Luca Niccolini
fce811976f use Subproject hash for folly and fizz
Summary:
this is how we do dependencies

fixes #86

Reviewed By: mjoras

Differential Revision: D19742747

fbshipit-source-id: bf90e1e327216e0342cd87ccfd1736d9078c8c3e
2020-02-05 07:20:38 -08:00
Luca Niccolini
ea573aa66b Properly build fmt as a dependency for proxygen and mvfst
Summary:
^

(Note: this ignores all push blocking failures!)

Reviewed By: mjoras

Differential Revision: D19742355

fbshipit-source-id: c5a6b7b553d3429e505616fad52facd2f9845a39
2020-02-05 07:18:24 -08:00
Yang Chi
4b014858f2 add fmt install instruction in Quic README
Summary:
Quic OSS Readme file missed one dep. The build script is likely wrong
too. On both Ubuntu and WSL, libfmt-dev installed from apt-get is too old to
have fmt/core.h file. So i removed the apt-get install libfmt-dev thing.

Reviewed By: mjoras

Differential Revision: D18763548

fbshipit-source-id: 6a83644dc3d825d0e7fd7ebed1af0df1ff0551dd
2019-12-02 10:22:34 -08:00
Luca Niccolini
aa3975852e install fmt for osx build
Summary: ^

Reviewed By: mjoras

Differential Revision: D18575839

fbshipit-source-id: e50f44c17a2535d1822cf934386b82a7a9ffa166
2019-11-18 11:25:43 -08:00
Amaury Séchet
9493126ea4 Add libfmt-dev as a dependency (#68)
Summary:
Looks like it is required.
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/68

Reviewed By: lnicco

Differential Revision: D18573868

Pulled By: mjoras

fbshipit-source-id: 60a2e0c8d5139a08a38f6877241acc699a309d90
2019-11-18 10:33:35 -08:00
bottler
2bf5cbc8e0 Option to not install dependencies (#45)
Summary:
This adds an argument to skip installing the system dependencies. This is useful to a project I am working on which wraps `mvfst` and installs dependent system packages in its own way.
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/45

Reviewed By: yangchi

Differential Revision: D17553209

Pulled By: udippant

fbshipit-source-id: 427a1df562ba7b6f6962aa42fb6f4ea5636e8101
2019-10-11 11:29:17 -07:00
Udip Pant
d8d153c13b Stop consuming fizz via ExternalProject
Summary:
After changing fizz to export test headers and lib, we do not need to pull in
fizz as an ExternalProject anymore. This simplifies and fixes a lot of build
issues

Reviewed By: yangchi

Differential Revision: D15930220

fbshipit-source-id: dcb01c46b8ade545047c20af2fc5245223afff99
2019-06-20 21:26:41 -07:00
Udip Pant
2be8bd37e2 fix oss build failures
Summary: fix for the unbound variable and include fizz header

Reviewed By: mjoras

Differential Revision: D15565195

fbshipit-source-id: 1b02955a2546d57da71cab40ad0ceccdf4cee577
2019-05-30 15:49:59 -07:00
Akshay Narayan
b336524398 build_helper: Optionally allow building without jemalloc (#14)
Summary:
As a workaround for https://github.com/facebook/folly/issues/976, modify build_helper.sh to allow disabling jemalloc use by passing the appropriate argument to the cmake invocation on folly.
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/14

Reviewed By: mjoras

Differential Revision: D15521328

Pulled By: udippant

fbshipit-source-id: 75a5f5abd99fd50fa939d32da99153ede9d8804e
2019-05-29 15:11:38 -07:00
Jeff Squyres
ad3d7d79b9 Various fixes for MacOS Mojave 10.14 (#3)
Summary:
I cloned this repo for the first time today and ran into a few problems building on MacOS Mojave 10.14.

Here's a small number of standalone fixes to the `build_helper.sh` script.
Pull Request resolved: https://github.com/facebookincubator/mvfst/pull/3

Reviewed By: lnicco

Differential Revision: D15212179

Pulled By: udippant

fbshipit-source-id: 86b5619954851654cc756d0b238c89896d7b42d1
2019-05-08 14:28:54 -07:00
Udip Pant
d92acd1225 consume fizz as an external project in cmake
Summary: Favor standalone cmake config than relying on custom script

Reviewed By: siyengar

Differential Revision: D15227020

fbshipit-source-id: 949885b06daeef5e496bf5f1b67a2a3a89750ebf
2019-05-06 16:56:29 -07:00
Udip Pant
6e361a20a7 pull in appropriate googletest via cmake ExternalProject
Summary: Do not rely on downloading it via wget in `build_helper.sh`

Reviewed By: siyengar

Differential Revision: D15215226

fbshipit-source-id: aeef31b30964674d80346aa6a1480ce03fdf478d
2019-05-05 13:04:43 -07:00
udippant
e0d0e8c6c4 Add script to build mvfst and update README 2019-04-24 16:27:03 -07:00