Grive / grive

an open source Linux client for Google Drive
http://www.lbreda.com/grive/start
GNU General Public License v2.0
1.59k stars 378 forks source link

Error compile in Fedora 18 #72

Open Vascom opened 12 years ago

Vascom commented 12 years ago

Build fails: [ 94%] Building CXX object libgrive/CMakeFiles/grive.dir/src/bfd/SymbolInfo.cc.o cd /builddir/build/BUILD/Grive-grive-f4b3e48/libgrive && /usr/bin/c++ -DHAVE_BFD -DVERSION=\"0.2.0\" -DTEST_DATA=\"/builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/test/data/\" -DSRC_DIR=\"/builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/src\" -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -I/builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/src -I/builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/test -o CMakeFiles/grive.dir/src/bfd/SymbolInfo.cc.o -c /builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/src/bfd/SymbolInfo.cc In file included from /builddir/build/BUILD/Grive-grive-f4b3e48/libgrive/src/bfd/SymbolInfo.cc:25:0: /usr/include/bfd.h:37:2: error: #error config.h must be included before this header

http://koji.russianfedora.ru/koji/getfile?taskID=38730&name=build.log

nestal commented 12 years ago

This is because of a new check in the new bfd library: http://sourceware.org/bugzilla/show_bug.cgi?id=14243

Can you change the CMakeLists not to use the bfd library?

Vascom commented 12 years ago

Can you change libgrive/src/bfd/SymbolInfo.cc instead?

nestal commented 12 years ago

I don't have a config.h. I dont know how to fix this problem.

Vascom commented 12 years ago

OK. I change CMakeLists. But we need to solve this problem in the future.

nestal commented 12 years ago

I agree.

junghans commented 12 years ago

Same here: https://bugs.gentoo.org/show_bug.cgi?id=428636

ian-abbott commented 12 years ago

Is it sufficient to define a dummy macro called PACKAGE in libgrive/src/bfd/SymbolInfo.cc before the #include <bfd.h>? It seems that's enough to stop <bfd.h> complaining.

alycolas commented 11 years ago

how to change CMakelists

Vascom commented 11 years ago

sed -i -e '/find_package(BFD)/d' libgrive/CMakeLists.txt

ernstki commented 10 years ago

@ian-abbott It is sufficient. That's the solution that the Gentoo folks came up with in the bug report for the same issue in a different package.

linwiz commented 9 years ago

Fixed in linwiz/grive#3