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 version 6.0.2 #5900

Closed
wants to merge 4 commits into from

Conversation

fennibay
Copy link
Contributor

Specify library name and version: rocksdb/6.0.2

For backwards compatibility on one of my projects, I need exactly the version 6.0.2. And instead of solving this for myself, I thought contributing to this repo would help others, too.

Current latest version of RocksDB is 6.20.3, I could also add that as a separate PR if desired.


  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes. // no change needed in the py files.
  • I've used the latest Conan client version. // v1.37.2
  • I've tried at least one configuration locally with the
    conan-center hook activated. // tested with conan create and conan test

fennibay added 2 commits June 15, 2021 10:14
This is needed for backwards compatibility on a project.
@CLAassistant
Copy link

CLAassistant commented Jun 15, 2021

CLA assistant check
All committers have signed the CLA.

@conan-center-bot

This comment has been minimized.

@ghost
Copy link

ghost commented Jun 17, 2021

I detected other pull requests that are modifying rocksdb/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.

@prince-chrismc
Copy link
Contributor

Thank you for stating you need this exact version

Current latest version of RocksDB is 6.20.3, I could also add that as a separate PR if desired.

You can add it in this one... if it takes more changes (ie recipe modifications) save it for another PR

prince-chrismc
prince-chrismc previously approved these changes Jun 22, 2021
@conan-center-bot

This comment has been minimized.

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
@fennibay fennibay marked this pull request as draft June 22, 2021 15:07
@fennibay
Copy link
Contributor Author

Setting back to draft until I figure out the build problems

@conan-center-bot
Copy link
Collaborator

Failure in build 4 (dfd98147508802b1dac998d61732d238a5f12800):

  • rocksdb/6.0.2@:
    CI failed to create some packages (All logs)

    Logs for packageID 41f9523a0b8902f880bc9b145935e445607146b0:
    [settings]
    arch=x86_64
    arch_build=x86_64
    build_type=Debug
    compiler=clang
    compiler.libcxx=libstdc++
    compiler.version=10
    os=Linux
    os_build=Linux
    [options]
    rocksdb:shared=False
    
    [...]
        NUMBER_KEYS_WRITTEN,
        ^
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/include/rocksdb/statistics.h:121:3: note: previous declaration is here
      NUMBER_KEYS_WRITTEN,
      ^
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.cc:8:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_db_impl.h:22:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_iter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_impl.h:23:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/compaction_job.h:24:
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/internal_stats.h:116:5: warning: declaration shadows a variable in namespace 'rocksdb' [-Wshadow]
        WRITE_DONE_BY_OTHER,
        ^
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/include/rocksdb/statistics.h:198:3: note: previous declaration is here
      WRITE_DONE_BY_OTHER,  // Equivalent to writes done for others
      ^
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.cc:8:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_db_impl.h:22:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_iter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_impl.h:23:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/compaction_job.h:24:
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/internal_stats.h:117:5: warning: declaration shadows a variable in namespace 'rocksdb' [-Wshadow]
        WRITE_DONE_BY_SELF,
        ^
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/include/rocksdb/statistics.h:197:3: note: previous declaration is here
      WRITE_DONE_BY_SELF,
      ^
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.cc:8:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_compaction_filter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/utilities/blob_db/blob_db_impl.h:22:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_iter.h:13:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/db_impl.h:23:
    In file included from /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/compaction_job.h:24:
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/db/internal_stats.h:118:5: warning: declaration shadows a variable in namespace 'rocksdb' [-Wshadow]
        WRITE_WITH_WAL,
        ^
    /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/source_subfolder/include/rocksdb/statistics.h:200:3: note: previous declaration is here
      WRITE_WITH_WAL,       // Number of Write calls that request WAL
      ^
    8 warnings generated.
    make[1]: *** [source_subfolder/CMakeFiles/rocksdb-shared.dir/all] Error 2
    make: *** [all] Error 2
    rocksdb/6.0.2: WARN: Build folder is dirty, removing it: /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0
    rocksdb/6.0.2: ERROR: Package '41f9523a0b8902f880bc9b145935e445607146b0' build failed
    rocksdb/6.0.2: WARN: Build folder /home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0
    ERROR: rocksdb/6.0.2: Error in build() method, line 164
    	cmake.build()
    	ConanException: Error 2 while executing cmake --build '/home/conan/w/BuildSingleReference/.conan/data/rocksdb/6.0.2/_/_/build/41f9523a0b8902f880bc9b145935e445607146b0/build_subfolder' '--' '-j3'
    
  • rocksdb/6.8.1@:
    All packages built successfully! (All logs)

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


Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@fennibay
Copy link
Contributor Author

Superseded by #6018. Closing

@fennibay fennibay closed this Jun 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants