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

Ubuntu CI not finding Ogre 1.9 #400

Closed
chapulina opened this issue Sep 9, 2021 · 2 comments
Closed

Ubuntu CI not finding Ogre 1.9 #400

chapulina opened this issue Sep 9, 2021 · 2 comments
Assignees
Labels
bug Something isn't working ogre1.x tests Broken or missing tests / testing infra

Comments

@chapulina
Copy link
Contributor

Our CI is not finding Ogre 1.9, it may be related to #376

Environment

  • OS Version: Ubuntu Bionic or Focal
  • Source or binary build? Source, on CI

Description

  • Expected behavior: CI runs for both Ogre 1 and 2
  • Actual behavior: CI is only finding Ogre 2

Steps to reproduce

Run CI on Jenkins or GitHub actions. Jenkins is yellow because of the warning, but Actions doesn't fail on warnings, so it shows up green.

Output

Ogre 1.9 is installed:

2021-09-09T00:23:39.9181608Z Selecting previously unselected package libogre-1.9.0v5:amd64.
2021-09-09T00:23:39.9217794Z Preparing to unpack .../270-libogre-1.9.0v5_1.9.0+dfsg1-10_amd64.deb ...
2021-09-09T00:23:39.9329386Z Unpacking libogre-1.9.0v5:amd64 (1.9.0+dfsg1-10) ...
2021-09-09T00:23:40.1933838Z Selecting previously unselected package libogre-1.9-dev.
2021-09-09T00:23:40.1977322Z Preparing to unpack .../271-libogre-1.9-dev_1.9.0+dfsg1-10_amd64.deb ...
2021-09-09T00:23:40.1989527Z Unpacking libogre-1.9-dev (1.9.0+dfsg1-10) ...

But not found:

2021-09-09T00:23:47.9754351Z    CONFIGURATION WARNINGS:
2021-09-09T00:23:47.9755220Z    -- Missing dependency [IgnOGRE] (Components: RTShaderSystem, Terrain, Overlay)
2021-09-09T00:23:47.9756184Z    -- Skipping component [optix]: Missing dependency [OptiX].
2021-09-09T00:23:47.9756851Z       ^~~~~ Set SKIP_optix=true in cmake to suppress this warning.

Example builds:

@chapulina chapulina added bug Something isn't working ogre1.x tests Broken or missing tests / testing infra labels Sep 9, 2021
@chapulina chapulina mentioned this issue Sep 9, 2021
@Blast545
Copy link
Contributor

I'm not a Cmake expert, but I think I know what's happening.

https://github.com/ignitionrobotics/ign-cmake/blob/eac0b654a19dc67c580e8ceab494915efc6ab796/cmake/IgnUtils.cmake#L196-L209

This portion of code tries to find a IgnOgre, but when it fails to find it, it still tries to find the selected components here:
https://github.com/ignitionrobotics/ign-rendering/blob/49e7a06024a4adb7f2ec68cac1ebd33318cb9b8b/CMakeLists.txt#L64

Then, the components are marked as IgnOgre dependency is not available, even when IgnOgre is found below. This is confirmed by the Jenkins output:

-- Looking for IgnOGRE - not found

-- Checking for module 'OGRE-RTShaderSystem >= 1.9'
--   Found OGRE-RTShaderSystem , version 1.9.0
-- Checking for module 'OGRE-Terrain >= 1.9'
--   Found OGRE-Terrain , version 1.9.0
-- Checking for module 'OGRE-Overlay >= 1.9'
--   Found OGRE-Overlay , version 1.9.0
-- Looking for IgnOGRE - found

...

CMake Warning at /usr/share/cmake/ignition-cmake2/cmake2/IgnConfigureBuild.cmake:53 (message):
   CONFIGURATION WARNINGS:
   -- Missing dependency [IgnOGRE] (Components: RTShaderSystem, Terrain, Overlay)
Call Stack (most recent call first):
  CMakeLists.txt:156 (ign_configure_build)

I think this should be fixed by changing the statement so the components require IgnOgre, otherwise do not import them. @j-rivero

@chapulina
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ogre1.x tests Broken or missing tests / testing infra
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants