Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependeny on fmt not mentioned in installation instructions #1262

Closed
SashaNullptr opened this issue Nov 19, 2019 · 4 comments
Closed

Dependeny on fmt not mentioned in installation instructions #1262

SashaNullptr opened this issue Nov 19, 2019 · 4 comments

Comments

@SashaNullptr
Copy link
Contributor

fmt not mentioned as dependency

Folly depends on the fmt library in order to build, as seen in folly-deps.cmake as of da41ae5 . This isn't mentioned in the installation instructions. For example if one is naively following the Ubuntu 16.04 LTS install instructions CMake will fail to find fmt and error out.

On Ubuntu 18.04 LTS installing libfmt-dev doesn't fix things, at least not with CMake running without any flags (i.e. cmake ..). It seems that fmt needs to be installed from source as described in the fmt usage guide.

@SashaNullptr
Copy link
Contributor Author

In particular, if libfmt-dev from Ubuntu 18,04 is installed ( version 4.0.0+ds-2 ) the following error occurs:

In file included from /folly/folly/logging/LogStreamProcessor.cpp:17:0:
/folly/folly/logging/LogStreamProcessor.h:19:10: fatal error: fmt/core.h: No such file or directory

@yfeldblum
Copy link
Contributor

@vitaut

@vitaut
Copy link
Contributor

vitaut commented Nov 20, 2019

I'll update the instructions.

@vitaut
Copy link
Contributor

vitaut commented Nov 20, 2019

Oh, I see it's already done in #1263. Thanks, @SashaNullptr.

facebook-github-bot pushed a commit that referenced this issue Nov 20, 2019
Summary:
Dovetailing off of #1262 ; Folly's dependency on `fmt` isn't explicitly mentioned nor are from-source install instructions provided. This PR adds a short section to the README describing how to build and install fmt from source.
Pull Request resolved: #1263

Reviewed By: vitaut

Differential Revision: D18622693

Pulled By: yfeldblum

fbshipit-source-id: e43ab8c5524df9decc79cdd0cb2aea3d88826617
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants