Merge ~j-rivero/ubuntu/+source/ogre-next:ubuntu/devel into ubuntu/+source/ogre-next:ubuntu/devel

Proposed by Jose Luis Rivero
Status: Needs review
Proposed branch: ~j-rivero/ubuntu/+source/ogre-next:ubuntu/devel
Merge into: ubuntu/+source/ogre-next:ubuntu/devel
Diff against target: 80 lines (+13/-11)
3 files modified
debian/changelog (+8/-0)
debian/extras/OGRE-NextConfig.cmake (+0/-10)
debian/rules (+5/-1)
Reviewer Review Type Date Requested Status
Dave Jones (community) Approve
Review via email: mp+464598@code.launchpad.net

Commit message

Include the OGRE-Next CMake Module to facilitate build.
Fix the autopkgtest execution.

Description of the change

From the work of bumping the version to 2.3.3 there was a large cleanup of custom code in d/rules that ended up with the lack of the installation of the CMake Module in d/extras. The OGRE-Next project does not provide any CMake module ready to use we are inheriting this from previous version 2.2.5 already in Noble.

The merge-request should fix the problems with autopkgtest and facilitate the transition from noble-proposed currently blocked.

@Dave Jones: adding you as a reviewer since you were the uploader for ogre-2.3.3. Not sure if this is the right procedure for this kind of follow-ups.

Code builds fine here: https://code.launchpad.net/~j-rivero/+archive/ubuntu/ogre-next-recipe/+build/28119759

To post a comment you must log in.
Revision history for this message
Pushkar Kulkarni (pushkarnk) wrote :

Thanks for the MP! I filed a bug report yesterday, before noticing this. It might help to add the LP# in the changelog.

https://bugs.launchpad.net/ubuntu/+source/ogre-next/+bug/2062378

3440a9f... by Jose Luis Rivero

Include launchpad bug 2062378

Signed-off-by: Jose Luis Rivero <email address hidden>

Revision history for this message
Jose Luis Rivero (j-rivero) wrote :

Thanks Pushkar for the bug. I've added it to the d/changelog entry.

Revision history for this message
Dave Jones (waveform) wrote :

This looks like it builds fine and passes autopkgtests locally on an arm64 box. I'll sponsor shortly.

review: Approve

Unmerged commits

3440a9f... by Jose Luis Rivero

Include launchpad bug 2062378

Signed-off-by: Jose Luis Rivero <email address hidden>

1157754... by Jose Luis Rivero

Update d/changelog

Signed-off-by: Jose Luis Rivero <email address hidden>

ceac5df... by Jose Luis Rivero

Cleanup debug messages from OGRE-Next Module

Signed-off-by: Jose Luis Rivero <email address hidden>

53cd5c1... by Jose Luis Rivero

Include the OGRE-Next CMake Module

Signed-off-by: Jose Luis Rivero <email address hidden>

b0bb82f... by Jose Luis Rivero

Merge remote-tracking branch 'ubuntu/ubuntu/devel' into ubuntu/devel

f4f60bc... by Jose Luis Rivero

Update d/changelog

Signed-off-by: Jose Luis Rivero <email address hidden>

a590b64... by Jose Luis Rivero

[PATCH] Fix armhf vulkan build

Signed-off-by: Jose Luis Rivero <email address hidden>

7d84f42... by Jose Luis Rivero

Update d/changelog

Signed-off-by: Jose Luis Rivero <email address hidden>

bba8030... by Jose Luis Rivero

[PATCH] Avoid shaderrc vulkan dependency

Signed-off-by: Jose Luis Rivero <email address hidden>

1b581a8... by Jose Luis Rivero

Fix vulkan dep on noble

Signed-off-by: Jose Luis Rivero <email address hidden>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 4e524d4..36f8444 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,11 @@
6+ogre-next (2.3.3+dfsg-0ubuntu2) noble; urgency=medium
7+
8+ * Include the OGRE-Next Module in the installation.
9+ (LP: #2062378)
10+ * Cleanup debug message from Ogre-Next module
11+
12+ -- Jose Luis Rivero <jrivero@osrfoundation.org> Fri, 19 Apr 2024 14:18:04 +0200
13+
14 ogre-next (2.3.3+dfsg-0ubuntu1) noble; urgency=medium
15
16 * New upstream version 2.3.3+dfsg
17diff --git a/debian/extras/OGRE-NextConfig.cmake b/debian/extras/OGRE-NextConfig.cmake
18index 921f583..67b656b 100644
19--- a/debian/extras/OGRE-NextConfig.cmake
20+++ b/debian/extras/OGRE-NextConfig.cmake
21@@ -190,9 +190,7 @@ endif()
22 # locate Ogre include files
23
24 find_path(OGRE-Next_CONFIG_INCLUDE_DIR NAMES OgreBuildSettings.h HINTS ${OGRE-Next_INC_SEARCH_PATH} ${OGRE-Next_FRAMEWORK_INCLUDES} ${OGRE-Next_PKGC_INCLUDE_DIRS} PATH_SUFFIXES "OGRE-Next")
25-message(STATUS "${OGRE-Next_CONFIG_INCLUDE_DIR}")
26 find_path(OGRE-Next_INCLUDE_DIR NAMES OgreRoot.h HINTS ${OGRE-Next_CONFIG_INCLUDE_DIR} ${OGRE-Next_INC_SEARCH_PATH} ${OGRE-Next_FRAMEWORK_INCLUDES} ${OGRE-Next_PKGC_INCLUDE_DIRS} PATH_SUFFIXES "OGRE-Next")
27-message(STATUS "${OGRE-Next_INCLUDE_DIR}")
28 set(OGRE-Next_INCOMPATIBLE FALSE)
29
30 if (OGRE-Next_INCLUDE_DIR)
31@@ -238,13 +236,9 @@ else ()
32 set(OGRE-Next_INCOMPATIBLE FALSE)
33 endif ()
34
35-message(STATUS "1. waypoint ${OGRE-Next_INCOMPATIBLE}")
36-
37 find_library(OGRE-Next_LIBRARY_REL NAMES ${OGRE-Next_LIBRARY_NAMES} HINTS ${OGRE-Next_LIB_SEARCH_PATH} ${OGRE-Next_PKGC_LIBRARY_DIRS} ${OGRE-Next_FRAMEWORK_SEARCH_PATH} PATH_SUFFIXES "" "Release" "RelWithDebInfo" "MinSizeRel")
38 find_library(OGRE-Next_LIBRARY_DBG NAMES ${OGRE-Next_LIBRARY_NAMES_DBG} HINTS ${OGRE-Next_LIB_SEARCH_PATH} ${OGRE-Next_PKGC_LIBRARY_DIRS} ${OGRE-Next_FRAMEWORK_SEARCH_PATH} PATH_SUFFIXES "" "Debug")
39
40-message(STATUS "REL ${OGRE-Next_LIBRARY_REL}")
41-message(STATUS "DBG ${OGRE-Next_LIBRARY_DBG}")
42 make_library_set(OGRE-Next_LIBRARY)
43
44 if (OGRE-Next_INCOMPATIBLE)
45@@ -265,14 +259,10 @@ endif()
46
47 mark_as_advanced(OGRE-Next_CONFIG_INCLUDE_DIR OGRE-Next_MEDIA_DIR OGRE-Next_PLUGIN_DIR_REL OGRE-Next_PLUGIN_DIR_DBG)
48
49-message(STATUS "2. waypoint ${OGRE-Next_FOUND}")
50-
51 if (NOT OGRE-Next_FOUND)
52 return()
53 endif ()
54
55-message(STATUS "3. waypoint ${OGRE-Next_FOUND}")
56-
57 # look for required Ogre dependencies in case of static build and/or threading
58 if (OGRE-Next_STATIC)
59 set(OGRE-Next_DEPS_FOUND TRUE)
60diff --git a/debian/rules b/debian/rules
61index c13aea2..0bd67ca 100755
62--- a/debian/rules
63+++ b/debian/rules
64@@ -29,12 +29,16 @@ override_dh_auto_configure:
65 -DOGRE_GLSUPPORT_USE_EGL_HEADLESS:BOOL=TRUE \
66 -DOGRE_GLSUPPORT_USE_GLX:BOOL=TRUE $(EXTRA_CMAKE_FLAGS)
67
68-# TODO: REVERT
69 override_dh_auto_build-indep:
70 dh_auto_build -i
71 # generate doxygen API doc
72 dh_auto_build -i -- OgreDoc
73
74+execute_before_dh_install:
75+ # Inject the custom cmake configure file from extras
76+ install -m644 debian/extras/OGRE-NextConfig.cmake \
77+ debian/tmp/usr/lib/$(DEB_HOST_MULTIARCH)/OGRE-Next/cmake/
78+
79 override_dh_installchangelogs:
80 dh_installchangelogs -- Docs/ChangeLog.html
81

Subscribers

People subscribed via source and target branches