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

sophus: fails in do_compile on x86-64 architecture #497

Closed
bulwahn opened this issue Apr 29, 2017 · 3 comments
Closed

sophus: fails in do_compile on x86-64 architecture #497

bulwahn opened this issue Apr 29, 2017 · 3 comments

Comments

@bulwahn
Copy link
Contributor

bulwahn commented Apr 29, 2017

bitbake sophus fails during compiling with:

[...]
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/recipe-sysroot/usr/include/eigen3/Eigen/src/Core/Matrix.h:311:7:   required from 'Eigen::Matrix<_Scalar, _Rows, _Cols, _Options, _MaxRows, _MaxCols>::Matrix(const Eigen::ReturnByValue<OtherDerived>&) [with OtherDerived = Eigen::MatrixExponentialReturnValue<Eigen::Matrix<float, 4, 4> >; _Scalar = float; int _Rows = 4; int _Cols = 4; int _Options = 0; int _MaxRows = 4; int _MaxCols = 4]'
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/tests.hpp:96:64:   required from 'bool Sophus::Tests<LieGroup>::expMapTest() [with LieGroup = Sophus::Sim3Group<float>]'
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/tests.hpp:211:24:   required from 'void Sophus::Tests<LieGroup>::runAllTests() [with LieGroup = Sophus::Sim3Group<float>]'
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/test_sim3.cpp:97:3:   required from 'void tests() [with Scalar = float]'
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/test_sim3.cpp:107:16:   required from here
| /home/lukas/work/repositories/openembedded.org/openembedded-core/build/tmp-glibc/work/core2-64-oe-linux/sophus/0.9.1-r0/recipe-sysroot/usr/include/eigen3/Eigen/src/Core/DenseCoeffsBase.h:56:30: error: ignoring attributes on template argument 'Eigen::internal::packet_traits<float>::type {aka __vector(4) float}' [-Werror=ignored-attributes]
| ERROR: oe_runmake failed

The first build configuration I noticed the issue was:

Build Configuration:
BB_VERSION        = "1.33.4"
BUILD_SYS         = "x86_64-linux"
NATIVELSBSTRING   = "ubuntu-14.04"
TARGET_SYS        = "x86_64-oe-linux"
MACHINE           = "qemux86-64"
DISTRO            = "nodistro"
DISTRO_VERSION    = "nodistro.0"
TUNE_FEATURES     = "m64 core2"
TARGET_FPU        = ""
meta              = "master:e584be78f92ee6f08f570c239698d56ac78d05f9"
meta-oe           
meta-multimedia   
meta-python       = "master:2c21aa8633e4e3538099bcc56d76ab0a594411b4"
meta-ros          = "master:f162d103a3b0fa97133fd15dd56eca34fcdf264e"

The issue however occurs since sophus' initial commit in meta-ros at 7b08f25.

@graugans
Copy link

graugans commented May 17, 2017

I have an similar error when building for x86-64:

/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/x86_64-linux/usr/bin/x86_64-fslc-linux/x86_64-fslc-linux-g++    -I/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86/usr/include/eigen3  -m64 -march=corei7 -mtune=corei7 -mfpmath=sse -msse4.2  --sysroot=/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86  -O2 -pipe -g -feliminate-unused-debug-types -fdebug-prefix-map=/home/chris/src/udoo-meta-freescale/build/tmp/work/corei7-64-fslc-linux/sophus/0.9.1-r0=/usr/src/debug/sophus/0.9.1-r0 -fdebug-prefix-map=/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/x86_64-linux= -fdebug-prefix-map=/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86=  -fvisibility-inlines-hidden  -m64 -march=corei7 -mtune=corei7 -mfpmath=sse -msse4.2  --sysroot=/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86 -Wno-deprecated-declarations -Wno-misleading-indentation -O3 -DNDEBUG    -Wall -Werror -Wno-unused-variable                    -Wno-unused-but-set-variable -Wno-unknown-pragmas  -o CMakeFiles/test_rxso3.dir/sophus/test_rxso3.cpp.o -c /home/chris/src/udoo-meta-freescale/build/tmp/work/corei7-64-fslc-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/test_rxso3.cpp
In file included from /home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86/usr/include/eigen3/Eigen/Core:263:0,
                 from /home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86/usr/include/eigen3/unsupported/Eigen/MatrixFunctions:19,
                 from /home/chris/src/udoo-meta-freescale/build/tmp/work/corei7-64-fslc-linux/sophus/0.9.1-r0/sophus-0.9.1/sophus/test_se2.cpp:26:
/home/chris/src/udoo-meta-freescale/build/tmp/sysroots/udoox86/usr/include/eigen3/Eigen/src/Core/arch/SSE/PacketMath.h:29:39: error: ignoring attributes on template argument '__m128 {aka __vector(4) float}' [-Werror=ignored-attributes]
 template<> struct is_arithmetic<__m128>  { enum { value = true }; };
                                       ^                    ^

Looks like this is an warning what is turned to an error due to -Wall -Werror It seems there is a fix in the latest libeigen beta

@graugans
Copy link

I've fixed the issue with the following patch

diff --git a/recipes-extended/sophus/sophus_0.9.1.bb b/recipes-extended/sophus/sophus_0.9.1.bb
index 64cbc8b..406647b 100644
--- a/recipes-extended/sophus/sophus_0.9.1.bb
+++ b/recipes-extended/sophus/sophus_0.9.1.bb
@@ -14,4 +14,4 @@ S = "${WORKDIR}/sophus-${PV}"
 inherit cmake

 # CXXFLAGS are needed to compile eigen 3.3.1 headers properly
-CXXFLAGS += "-Wno-deprecated-declarations -Wno-misleading-indentation"
+CXXFLAGS += "-Wno-deprecated-declarations -Wno-misleading-indentation -Wno-ignored-attributes"

@bulwahn
Copy link
Contributor Author

bulwahn commented May 19, 2017

👍 Great. Just open a pull request, and I will quickly test it and then merge it. By the way, if you add (resolves #497) in the commit message, the commit is immediately linked to this issue and the issue is closed once it has been merged.

@rojkov rojkov closed this as completed in 9d6db38 Jul 6, 2017
rojkov added a commit that referenced this issue Jul 6, 2017
sophus: compile on x86-64 arch (resolves #497)
allenh1 pushed a commit to allenh1/meta-ros that referenced this issue Jul 17, 2017
This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>
allenh1 added a commit to allenh1/meta-ros that referenced this issue Dec 1, 2017
* Signing the bbclasses so that we can modify stuff.

* Added a temporary lunar directory

* First ros-lunar recipes. (#1)

* regenerate ros-lunar, Thu Jul  6 11:15:15 2017

* regenerate ros-lunar, Mon Jul 17 12:29:45 2017 (#4)

* sophus: compile on x86-64 arch (resolves ros#497)

This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>

* regenerate ros-lunar, Mon Jul 17 13:43:54 2017 (#5)

* regenerate ros-lunar, Thu Jul 20 10:46:00 2017 (#6)

* regenerate ros-lunar, Thu Jul 20 12:28:02 2017 (#7)

* Old version of geometry 2.

* Old geometry build.

* regenerate ros-lunar, Thu Jul 20 15:14:27 2017 (#8)

* Added defusedxml bb

* regenerate ros-lunar, Fri Jul 21 14:36:57 2017 (#9)

* regenerate ros-lunar, Fri Jul 21 15:20:35 2017 (#10)

* regenerate ros-lunar, Mon Jul 24 10:57:45 2017 (#11)

* Bumped catkin version, as well as added a non-binary option for building catkin.

* Necessary changes to get catkin working.

* Remove extraneous recipes.

* regenerate ros-lunar, Mon Jul 24 16:24:15 2017 (#12)

* regenerate ros-lunar, Mon Jul 24 18:24:27 2017
allenh1 added a commit to allenh1/meta-ros that referenced this issue Dec 23, 2017
* Signing the bbclasses so that we can modify stuff.

* Added a temporary lunar directory

* First ros-lunar recipes. (#1)

* regenerate ros-lunar, Thu Jul  6 11:15:15 2017

* regenerate ros-lunar, Mon Jul 17 12:29:45 2017 (#4)

* sophus: compile on x86-64 arch (resolves ros#497)

This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>

* regenerate ros-lunar, Mon Jul 17 13:43:54 2017 (#5)

* regenerate ros-lunar, Thu Jul 20 10:46:00 2017 (#6)

* regenerate ros-lunar, Thu Jul 20 12:28:02 2017 (#7)

* Old version of geometry 2.

* Old geometry build.

* regenerate ros-lunar, Thu Jul 20 15:14:27 2017 (#8)

* Added defusedxml bb

* regenerate ros-lunar, Fri Jul 21 14:36:57 2017 (#9)

* regenerate ros-lunar, Fri Jul 21 15:20:35 2017 (#10)

* regenerate ros-lunar, Mon Jul 24 10:57:45 2017 (#11)

* Bumped catkin version, as well as added a non-binary option for building catkin.

* Necessary changes to get catkin working.

* Remove extraneous recipes.

* regenerate ros-lunar, Mon Jul 24 16:24:15 2017 (#12)

* regenerate ros-lunar, Mon Jul 24 18:24:27 2017
allenh1 added a commit to allenh1/meta-ros that referenced this issue Dec 23, 2017
* rosdistro sync, Mon Jul 24 18:24:27 2017 (#13)

* Signing the bbclasses so that we can modify stuff.

* Added a temporary lunar directory

* First ros-lunar recipes. (#1)

* regenerate ros-lunar, Thu Jul  6 11:15:15 2017

* regenerate ros-lunar, Mon Jul 17 12:29:45 2017 (#4)

* sophus: compile on x86-64 arch (resolves ros#497)

This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>

* regenerate ros-lunar, Mon Jul 17 13:43:54 2017 (#5)

* regenerate ros-lunar, Thu Jul 20 10:46:00 2017 (#6)

* regenerate ros-lunar, Thu Jul 20 12:28:02 2017 (#7)

* Old version of geometry 2.

* Old geometry build.

* regenerate ros-lunar, Thu Jul 20 15:14:27 2017 (#8)

* Added defusedxml bb

* regenerate ros-lunar, Fri Jul 21 14:36:57 2017 (#9)

* regenerate ros-lunar, Fri Jul 21 15:20:35 2017 (#10)

* regenerate ros-lunar, Mon Jul 24 10:57:45 2017 (#11)

* Bumped catkin version, as well as added a non-binary option for building catkin.

* Necessary changes to get catkin working.

* Remove extraneous recipes.

* regenerate ros-lunar, Mon Jul 24 16:24:15 2017 (#12)

* regenerate ros-lunar, Mon Jul 24 18:24:27 2017

* regenerate ros-lunar, Thu Nov 30 20:21:29 2017 (#16)

* Add updated version of defusedxml

* Remove extra lines form bbclass files.

* update rosdistro.

* Update rosdep.

* Update catkin_pkg

* Update rospkg.

* Fix checksums, move checksums out of include files, and remove a patch from rosdep.

* regenerate ros-lunar, Fri Dec  1 11:34:40 2017 (#18)

* Use the catkin.inc file.

* Add more patches for catkin.

* Change patch set.

* Still working on catkin stuff

* Ok, catkin is working now. Fixes #17.

* regenerate ros-lunar, Mon Dec 18 16:50:31 2017 (#20)

* regenerate ros-lunar, Fri Dec 22 16:50:36 2017 (#23)

* regenerate ros-kinetic, Sat Dec 23 13:24:12 2017
allenh1 added a commit to allenh1/meta-ros that referenced this issue Jan 9, 2018
* Signing the bbclasses so that we can modify stuff.

* Added a temporary lunar directory

* First ros-lunar recipes. (#1)

* regenerate ros-lunar, Thu Jul  6 11:15:15 2017

* regenerate ros-lunar, Mon Jul 17 12:29:45 2017 (#4)

* sophus: compile on x86-64 arch (resolves ros#497)

This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>

* regenerate ros-lunar, Mon Jul 17 13:43:54 2017 (#5)

* regenerate ros-lunar, Thu Jul 20 10:46:00 2017 (#6)

* regenerate ros-lunar, Thu Jul 20 12:28:02 2017 (#7)

* Old version of geometry 2.

* Old geometry build.

* regenerate ros-lunar, Thu Jul 20 15:14:27 2017 (#8)

* Added defusedxml bb

* regenerate ros-lunar, Fri Jul 21 14:36:57 2017 (#9)

* regenerate ros-lunar, Fri Jul 21 15:20:35 2017 (#10)

* regenerate ros-lunar, Mon Jul 24 10:57:45 2017 (#11)

* Bumped catkin version, as well as added a non-binary option for building catkin.

* Necessary changes to get catkin working.

* Remove extraneous recipes.

* regenerate ros-lunar, Mon Jul 24 16:24:15 2017 (#12)

* regenerate ros-lunar, Mon Jul 24 18:24:27 2017
allenh1 added a commit to allenh1/meta-ros that referenced this issue Jan 16, 2018
* Signing the bbclasses so that we can modify stuff.

* Added a temporary lunar directory

* First ros-lunar recipes. (#1)

* regenerate ros-lunar, Thu Jul  6 11:15:15 2017

* regenerate ros-lunar, Mon Jul 17 12:29:45 2017 (#4)

* sophus: compile on x86-64 arch (resolves ros#497)

This commit implements Christian Ege's suggestion in the meta-ros
issue tracker to make gcc only warn on ignored template attributes.

Signed-off-by: Lukas Bulwahn <[email protected]>

* regenerate ros-lunar, Mon Jul 17 13:43:54 2017 (#5)

* regenerate ros-lunar, Thu Jul 20 10:46:00 2017 (#6)

* regenerate ros-lunar, Thu Jul 20 12:28:02 2017 (#7)

* Old version of geometry 2.

* Old geometry build.

* regenerate ros-lunar, Thu Jul 20 15:14:27 2017 (#8)

* Added defusedxml bb

* regenerate ros-lunar, Fri Jul 21 14:36:57 2017 (#9)

* regenerate ros-lunar, Fri Jul 21 15:20:35 2017 (#10)

* regenerate ros-lunar, Mon Jul 24 10:57:45 2017 (#11)

* Bumped catkin version, as well as added a non-binary option for building catkin.

* Necessary changes to get catkin working.

* Remove extraneous recipes.

* regenerate ros-lunar, Mon Jul 24 16:24:15 2017 (#12)

* regenerate ros-lunar, Mon Jul 24 18:24:27 2017
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

2 participants