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

[onetbb] Reproduce error from #9164 #9231

Closed
wants to merge 4 commits into from

Conversation

uilianries
Copy link
Member

/cc @SSE4


  • 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.

SpaceIm and others added 3 commits January 30, 2022 15:03
tbb recupe is deprecated, all versions which were supported in tbb recipe are backported to onetbb recipe
@SpaceIm
Copy link
Contributor

SpaceIm commented Feb 2, 2022

To save CI resources, you may want to trigger the build of 2021.3.0 only.

@conan-center-bot

This comment has been minimized.

@ericLemanissier
Copy link
Contributor

for the linter error => #9380

@conan-center-bot

This comment has been minimized.

@uilianries
Copy link
Member Author

All good now!

@uilianries uilianries closed this Feb 14, 2022
@SpaceIm
Copy link
Contributor

SpaceIm commented Feb 14, 2022

Side question: why #9208 didn't produce binaries for apple-clang 13? We can see this error in onetbb/2021.3 build for apple-clang 13.

Signed-off-by: Uilian Ries <uilianries@gmail.com>
@uilianries uilianries reopened this Feb 17, 2022
@uilianries
Copy link
Member Author

Side question: why #9208 didn't produce binaries for apple-clang 13? We can see this error in onetbb/2021.3 build for apple-clang 13.

Maybe because epochs.

@SSE4
Copy link
Contributor

SSE4 commented Feb 17, 2022

I think OpenSSL didn't enter a new epoch due to some requirements (maybe perl? not sure)
then TBB incorrectly entered new epoch because it didn't consider requirements from test_package (known bug, which we're working on)

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@SSE4 SSE4 marked this pull request as ready for review February 23, 2022 04:03
@conan-center-bot

This comment has been minimized.

@SSE4 SSE4 changed the title Reproduce error from #9164 [onetbb] Reproduce error from #9164 Mar 11, 2022
@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

All green in build 3 (7a1f8967250d1ba80c533374e8e579564a582d4b):

  • onetbb/2021.3.0@:
    All packages built successfully! (All logs)

@SpaceIm
Copy link
Contributor

SpaceIm commented Mar 13, 2022

I think this PR can be closed since #9164 has been merged.

@SSE4
Copy link
Contributor

SSE4 commented Mar 14, 2022

@uilianries WDYT?

@uilianries
Copy link
Member Author

Yes, it can be closed, thank you for reminding

@uilianries uilianries closed this Mar 14, 2022
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.

None yet

5 participants