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 fails to compile in Fedora 28 / gcc version 8.0.1 #3716

Closed
itamarjp opened this issue Apr 13, 2018 · 6 comments
Closed

rocksdb fails to compile in Fedora 28 / gcc version 8.0.1 #3716

itamarjp opened this issue Apr 13, 2018 · 6 comments
Assignees

Comments

@itamarjp
Copy link

[root@f28 rocksdb]# make
Makefile:129: Warning: Compiling in debug mode. Don't use the resulting binary in production
GEN util/build_version.cc
CC db/c.o
db/c.cc: In function 'void rocksdb_writebatch_wi_iterate(rocksdb_writebatch_wi_t*, void*, void ()(void, const char*, size_t, const char*, size_t), void ()(void, const char*, size_t))':
db/c.cc:1683:5: error: conflicting C language linkage declaration 'rocksdb_writebatch_wi_iterate(rocksdb_writebatch_wi_t*, void*, void ()(void, const char*, size_t, const char*, size_t), void ()(void, const char*, size_t))::H handler' [-Werror]
H handler;
^~~~~~~
db/c.cc:1427:5: note: previous declaration 'rocksdb_writebatch_iterate(rocksdb_writebatch_t*, void*, void ()(void, const char*, size_t, const char*, size_t), void ()(void, const char*, size_t))::H handler'
H handler;
^~~~~~~
db/c.cc: In function 'rocksdb_slicetransform_t* rocksdb_slicetransform_create_fixed_prefix(size_t)':
db/c.cc:3315:12: error: conflicting C language linkage declaration 'rocksdb_slicetransform_create_fixed_prefix(size_t)::Wrapper* wrapper' [-Werror]
Wrapper* wrapper = new Wrapper;
^~~~~~~
db/c.cc:2869:12: note: previous declaration 'rocksdb_filterpolicy_create_bloom_format(int, bool)::Wrapper* wrapper'
Wrapper* wrapper = new Wrapper;
^~~~~~~
db/c.cc: In function 'rocksdb_slicetransform_t* rocksdb_slicetransform_create_noop()':
db/c.cc:3336:12: error: conflicting C language linkage declaration 'rocksdb_slicetransform_create_noop()::Wrapper* wrapper' [-Werror]
Wrapper* wrapper = new Wrapper;
^~~~~~~
db/c.cc:2869:12: note: previous declaration 'rocksdb_filterpolicy_create_bloom_format(int, bool)::Wrapper* wrapper'
Wrapper* wrapper = new Wrapper;
^~~~~~~
cc1plus: all warnings being treated as errors
make: *** [Makefile:1868: db/c.o] Error 1
[root@f28 rocksdb]#

@itamarjp
Copy link
Author

full build log with the failure, here ->

https://koji.fedoraproject.org/koji/taskinfo?taskID=26351599

@itamarjp
Copy link
Author

I think probably should be a good idea do use github webhooks to launch automatic builds on copr / fedora, they allways use the lasted gcc-c++ please take a look -> copr.fedorainfracloud.org

@inikep
Copy link
Contributor

inikep commented Apr 18, 2018

I confirm the same issues with gcc-8 under Ubuntu 14.04 and 16.04.

You can test gcc-8 with Travis and Ubuntu 14.04 by adding:

sudo -E apt-add-repository -y "ppa:jonathonf/binutils";
sudo -E apt-add-repository -y "ppa:jonathonf/gcc-8.0";
sudo -E apt-get -yq --no-install-suggests --no-install-recommends install g++-8;

inikep added a commit to inikep/rocksdb that referenced this issue Apr 18, 2018
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]
@yiwu-arbug
Copy link
Contributor

@inikep thanks for sending the fix and I think it is a good idea to test with gcc8 on travis. I'll check with @gfosco to see if travis capacity is a concern.

@yiwu-arbug yiwu-arbug self-assigned this Apr 19, 2018
facebook-github-bot pushed a commit that referenced this issue Apr 20, 2018
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
@tamird
Copy link
Contributor

tamird commented Apr 23, 2018

This was fixed in #3736.

@itamarjp
Copy link
Author

thank you.

Hoverbear pushed a commit to tikv/rocksdb that referenced this issue May 31, 2018
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves facebook#3716
Closes facebook#3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
Hoverbear pushed a commit to tikv/rocksdb that referenced this issue Jun 2, 2018
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves facebook#3716
Closes facebook#3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
benesch pushed a commit to cockroachdb/rocksdb that referenced this issue Jul 24, 2018
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves facebook#3716
Closes facebook#3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
ltamasi pushed a commit that referenced this issue Oct 30, 2019
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves #3716
Closes #3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
jimharris pushed a commit to spdk/rocksdb that referenced this issue Feb 5, 2020
Summary:
Fix the following gcc-8 warnings:
- conflicting C language linkage declaration [-Werror]
- writing to an object with no trivial copy-assignment [-Werror=class-memaccess]
- array subscript -1 is below array bounds [-Werror=array-bounds]

Solves facebook#3716
Closes facebook#3736

Differential Revision: D7684161

Pulled By: yiwu-arbug

Change-Id: Iee07a868dd704b4dffa63997c62182b498645b3e
fbshipit-source-id: 47c0423d26b74add251f1d3595211eee1e41e54a
Signed-off-by: Seth Howell <[email protected]>
Reviewed-on: https://review.gerrithub.io/424729
Reviewed-by: Jim Harris <[email protected]>
Reviewed-by: Shuhei Matsumoto <[email protected]>
Tested-by: Jim Harris <[email protected]>
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

No branches or pull requests

4 participants