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

PS-4961 : Fix MyRocks compilation errors for Ubuntu 18.10 gcc 8.2.0 #2622

Merged
merged 1 commit into from
Oct 30, 2018

Conversation

george-lorch
Copy link
Contributor

  • Fixed error storage/rocksdb/ha_rocksdb.cc:561:70: error: type qualifiers
    ignored on cast result type [-Werror=ignored-qualifiers]

  • Fixed error storage/rocksdb/rdb_datadic.cc:1108:78: error: type qualifiers
    ignored on cast result type [-Werror=ignored-qualifiers]

  • Fixed error storage/rocksdb/rdb_threads.cc:29:71: error: type qualifiers
    ignored on cast result type [-Werror=ignored-qualifiers]

  • Fixed error storage/rocksdb/rdb_datadic.cc:124:44: error:
    ‘void* memcpy(void*, const void*, size_t)’ writing to an object of type
    ‘class myrocks::Rdb_field_packing’ with no trivial copy-assignment
    [-Werror=class-memaccess]

- Fixed error storage/rocksdb/ha_rocksdb.cc:561:70: error: type qualifiers
  ignored on cast result type [-Werror=ignored-qualifiers]

- Fixed error storage/rocksdb/rdb_datadic.cc:1108:78: error: type qualifiers
  ignored on cast result type [-Werror=ignored-qualifiers]

- Fixed error storage/rocksdb/rdb_threads.cc:29:71: error: type qualifiers
  ignored on cast result type [-Werror=ignored-qualifiers]

- Fixed error storage/rocksdb/rdb_datadic.cc:124:44: error:
  ‘void* memcpy(void*, const void*, size_t)’ writing to an object of type
  ‘class myrocks::Rdb_field_packing’ with no trivial copy-assignment
  [-Werror=class-memaccess]
@george-lorch
Copy link
Contributor Author

@george-lorch george-lorch merged commit 86bdf96 into percona:5.7 Oct 30, 2018
@george-lorch george-lorch deleted the ps-5.7-4961 branch October 30, 2018 13:53
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.

2 participants