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

Fix Folly to build with current VS version #4048

Merged
merged 10 commits into from
Jan 7, 2021
Merged

Fix Folly to build with current VS version #4048

merged 10 commits into from
Jan 7, 2021

Conversation

dmn-star
Copy link
Contributor

@dmn-star dmn-star commented Dec 31, 2020

Specify library name and version: folly/2019.10.21.00

Fixes #3983

Relates to facebook/folly#1412

Fix Folly to build on Linux

Relates to facebook/folly#1414

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the
    conan-center hook activated.

dmn-star and others added 3 commits December 28, 2020 15:48
Fix to compile libbsd on Xcode 12
Fix Folly to build with current VS version
@ghost
Copy link

ghost commented Dec 31, 2020

I detected other pull requests that are modifying folly/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot
Copy link
Collaborator

An unexpected error happened and has been reported. Help is on its way! 🏇

@dmn-star dmn-star closed this Jan 3, 2021
@dmn-star dmn-star reopened this Jan 3, 2021
@conan-center-bot
Copy link
Collaborator

All green in build 2 (f4a7bc4ee2eca3bc1c7076a2dfb74a04aa732af5)! 😊

@SSE4 SSE4 requested a review from uilianries January 4, 2021 04:43
@conan-center-bot
Copy link
Collaborator

All green in build 3 (f4a7bc4ee2eca3bc1c7076a2dfb74a04aa732af5)! 😊

@conan-center-bot
Copy link
Collaborator

An unexpected error happened and has been reported. Help is on its way! 🏇

prince-chrismc
prince-chrismc previously approved these changes Jan 4, 2021
@dmn-star dmn-star closed this Jan 4, 2021
@dmn-star dmn-star reopened this Jan 4, 2021
@conan-center-bot
Copy link
Collaborator

Some configurations of 'folly/2019.10.21.00' failed in build 5 (220de6538abf1c6d6e6063bcb7935c5c74b48840):

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 4, 2021

Now I am not sure which is better: "unexpected error" or failed. 😂
Do you have updated the CI to use the new CMake version?

https://github.com/facebook/folly/blob/master/CMake/GenPkgConfig.cmake

  # Since CMake 3.18 FindThreads may include a generator expression requiring
  # a target, which gets propagated to us through INTERFACE_COMPILE_OPTIONS.
  # Before CMake 3.19 there's no way to solve this in a general way, so we
  # work around the specific case. See #1414 and CMake bug #21074.
  if(CMAKE_VERSION VERSION_LESS 3.19)
    string(REPLACE
      "<COMPILE_LANG_AND_ID:CUDA,NVIDIA>" "<COMPILE_LANGUAGE:CUDA>"
      cflags "${cflags}"
    )
  endif()

Error log

CMake Error at source_subfolder/CMakeLists.txt:417 (file):
Error evaluating generator expression:

$<COMPILE_LANG_AND_ID:CUDA,NVIDIA>

$<COMPILE_LANG_AND_ID:lang,id> may only be used with binary targets to
specify include directories, compile definitions, and compile options. It
may not be used with the add_custom_command, add_custom_target, or
file(GENERATE) commands.

CMake Error at source_subfolder/CMakeLists.txt:417 (file):
Error evaluating generator expression:

$<COMPILE_LANG_AND_ID:CUDA,NVIDIA>

$<COMPILE_LANG_AND_ID:lang,id> may only be used with binary targets to
specify include directories, compile definitions, and compile options. It
may not be used with the add_custom_command, add_custom_target, or
file(GENERATE) commands.

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 5, 2021

Failed again but why? 🤨

@prince-chrismc
Copy link
Contributor

Probably not enough build agents

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 6, 2021

Ok. I give Facebook one last chance. 😂

@dmn-star dmn-star closed this Jan 6, 2021
@dmn-star dmn-star reopened this Jan 6, 2021
@conan-center-bot
Copy link
Collaborator

All green in build 10 (e679ee7e56fac049730cd5c18898326b53dea158)! 😊

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 6, 2021

Now we have to be quick before any new build tools update break the old folly again. Please review.

@conan-center-bot
Copy link
Collaborator

All green in build 11 (e679ee7e56fac049730cd5c18898326b53dea158)! 😊

prince-chrismc
prince-chrismc previously approved these changes Jan 6, 2021
@prince-chrismc
Copy link
Contributor

We need to merge this so I can fix my PR too

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 6, 2021

as often we need more reviewers. :-)

@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 6, 2021

@Croydon @uilianries @danimtb

Any idea why the build failed again without any response message?

@uilianries
Copy link
Member

@dmn-star It occurred an internal timeout while building Folly. As you know, it requires more time than other projects to be built. Let's try again.

@uilianries uilianries closed this Jan 6, 2021
@uilianries uilianries reopened this Jan 6, 2021
@conan-center-bot
Copy link
Collaborator

All green in build 13 (c483c50887c231cabd2061f0218ad5f3e59414e4)! 😊

@dmn-star dmn-star requested a review from uilianries January 6, 2021 19:28
@dmn-star
Copy link
Contributor Author

dmn-star commented Jan 7, 2021

We need to merge this so I can fix my PR too

May be today 😂

#2857

@conan-center-bot conan-center-bot merged commit 25833cd into conan-io:master Jan 7, 2021
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

Successfully merging this pull request may close these issues.

[package] folly/2019.10.21.00: Error building with Visual Studio 16 (release/debug)
6 participants