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

rocksdb: add versions 6.0.2 and 6.20.3 #6018

Merged
merged 9 commits into from
Jun 26, 2021

Conversation

fennibay
Copy link
Contributor

Specify library name and version: rocksdb/6.0.2 rocksdb/6.20.3

I actually need exactly v6.0.2 for backwards compatibility on one of my projects. I created PR #5900 for that.

If things go well, we could actually add also the latest version v6.20.3, so I'm creating this PR for this. If this succeeds, we can ignore #5900.


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

fennibay and others added 5 commits June 15, 2021 10:14
This is needed for backwards compatibility on a project.
RocksDB 6.0.2 had an invalid use of const, which was later fixed in facebook/rocksdb#6048 Applying the fix as a patch to 6.0.2 could result in a regression, so just don't compile with this compiler instead
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@fennibay
Copy link
Contributor Author

fennibay commented Jun 23, 2021

I investigated the error for version 6.10.2 and package ID e9395492247c902e986bb43e0038bb2a0c2e606e. It seems to be a temporary problem on the runner machine while executing tests. And the same version and package ID for roughly the same recipe succeeded on #5900. So I now re-triggered a build by fetching master. I think that shall work.

@fennibay fennibay changed the title Rocksdb/add v6.0.2 v6.20.3 rocksdb: add versions 6.0.2 and 6.20.3 Jun 23, 2021
@fennibay fennibay marked this pull request as ready for review June 23, 2021 14:09
@fennibay fennibay marked this pull request as draft June 23, 2021 14:17
@fennibay
Copy link
Contributor Author

@conan-center-bot how can I trigger the build? Why is it aborted?

@jgsogo
Copy link
Contributor

jgsogo commented Jun 24, 2021

Hi! I'm tracking this PR and (besides fighting for resources with some Qt pull-request) it is taking too long to build, longer that I would expect in the Linux machines. I've restarted it just in case. I hope it succeeds or I'll try to investigate the failure. 🕵️

@conan-center-bot
Copy link
Collaborator

All green in build 8 (921a2498814943090f6d4e1aceea747571b76978):

  • rocksdb/6.0.2@:
    All packages built successfully! (All logs)

  • rocksdb/6.8.1@:
    All packages built successfully! (All logs)

  • rocksdb/6.10.2@:
    All packages built successfully! (All logs)

  • rocksdb/6.20.3@:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit b601896 into conan-io:master Jun 26, 2021
@fennibay fennibay deleted the rocksdb/add_v6.0.2_v6.20.3 branch June 26, 2021 16:11
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.

5 participants