Merge ~liushuyu-011/ubuntu/+source/ogre-1.12:ubuntu/devel into ubuntu/+source/ogre-1.12:ubuntu/devel

Proposed by Zixing Liu
Status: Needs review
Proposed branch: ~liushuyu-011/ubuntu/+source/ogre-1.12:ubuntu/devel
Merge into: ubuntu/+source/ogre-1.12:ubuntu/devel
Diff against target: 63 lines (+29/-1)
4 files modified
debian/changelog (+7/-0)
debian/control (+2/-1)
debian/patches/fix-imgui-freetype-api-deprecation.patch (+19/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Andreas Hasenack Needs Information
Review via email: mp+462537@code.launchpad.net

Description of the change

This MP fixes an FTBFS by adding a patch to adapt ogre-1.12 font loading to the new ImGUI API.

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Hi,

which bug are you trying to fix? You say it's an FTBFS, but it's not referenced in your MP, https://bugs.launchpad.net/ubuntu/+source/ogre-1.12 is empty, and the current package[1] in noble-proposed built just fine.

1. https://launchpad.net/ubuntu/+source/ogre-1.12/1.12.10+dfsg2-3.1~exp1ubuntu2

review: Needs Information

Unmerged commits

677bb17... by Zixing Liu

d/p/fix-imgui-freetype-api-deprecation.patch: add a patch to fix build with ...

... newer ImGUI

6b0d547... by Matthias Klose

1.12.10+dfsg2-3build3 (patches unapplied)

Imported using git-ubuntu import.

73e76d5... by Graham Inggs

1.12.10+dfsg2-3build2 (patches unapplied)

Imported using git-ubuntu import.

ec2b283... by Graham Inggs

1.12.10+dfsg2-3build1 (patches unapplied)

Imported using git-ubuntu import.

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 bf17248..597d30c 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+ogre-1.12 (1.12.10+dfsg2-3ubuntu1) noble; urgency=medium
7+
8+ * debian/patches/fix-imgui-freetype-api-deprecation.patch: Adapt to
9+ new ImGUI FreeType API.
10+
11+ -- Zixing Liu <zixing.liu@canonical.com> Fri, 15 Mar 2024 15:54:21 -0600
12+
13 ogre-1.12 (1.12.10+dfsg2-3build3) noble; urgency=medium
14
15 * No-change rebuild for python3.12 t64.
16diff --git a/debian/control b/debian/control
17index 1ce5260..43c6083 100644
18--- a/debian/control
19+++ b/debian/control
20@@ -2,7 +2,8 @@ Source: ogre-1.12
21 Priority: optional
22 Section: libs
23 Standards-Version: 4.5.1
24-Maintainer: Debian Games Team <pkg-games-devel@lists.alioth.debian.org>
25+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
26+XSBC-Original-Maintainer: Debian Games Team <pkg-games-devel@lists.alioth.debian.org>
27 Uploaders: Manuel A. Fernandez Montecelo <mafm@debian.org>,
28 Simon Schmeisser <s.schmeisser@gmx.net>
29 Homepage: https://ogre3d.org/
30diff --git a/debian/patches/fix-imgui-freetype-api-deprecation.patch b/debian/patches/fix-imgui-freetype-api-deprecation.patch
31new file mode 100644
32index 0000000..8b0bb6c
33--- /dev/null
34+++ b/debian/patches/fix-imgui-freetype-api-deprecation.patch
35@@ -0,0 +1,19 @@
36+Description: Adapt to new ImGUI FreeType API
37+ The new API removed BuildFontAtlas function but split the previous function
38+ into two smaller ones.
39+Author: Zixing Liu <zixing.liu@canonical.com>
40+Forwarded: not-needed
41+Last-Update: 2024-03-15
42+
43+--- ogre-1.12-1.12.10+dfsg2.orig/Components/Overlay/src/OgreImGuiOverlay.cpp
44++++ ogre-1.12-1.12.10+dfsg2/Components/Overlay/src/OgreImGuiOverlay.cpp
45+@@ -115,7 +115,8 @@ void ImGuiOverlay::ImGUIRenderable::crea
46+ ImGuiIO& io = ImGui::GetIO();
47+ if (io.Fonts->Fonts.empty())
48+ io.Fonts->AddFontDefault();
49+- ImGuiFreeType::BuildFontAtlas(io.Fonts, 0);
50++ io.Fonts->FontBuilderIO = ImGuiFreeType::GetBuilderForFreeType();
51++ io.Fonts->FontBuilderFlags = 0;
52+
53+ unsigned char* pixels;
54+ int width, height;
55diff --git a/debian/patches/series b/debian/patches/series
56index ef24b22..1bb9d90 100644
57--- a/debian/patches/series
58+++ b/debian/patches/series
59@@ -8,3 +8,4 @@ OgreGLSupport_2.patch
60 OGREConfig_fix_plugin_dir.patch
61 0009-Overlay-update-imgui-to-1.83.patch
62 0010-CMake-rename-FeatureSummary.cmake-to-avoid-name-clas.patch
63+fix-imgui-freetype-api-deprecation.patch

Subscribers

People subscribed via source and target branches

to all changes: