* [Buildroot] [PATCH v3] package/ogre: bump to version 1.12.12
@ 2021-04-28 22:48 Romain Naour
2021-12-22 20:59 ` Arnout Vandecappelle
0 siblings, 1 reply; 2+ messages in thread
From: Romain Naour @ 2021-04-28 22:48 UTC (permalink / raw)
To: buildroot
Remove upstream patch [1].
tinyxml has been replaced by pugixml [2].
Switch to git download method since github release without
the complete source code in the tarball [3].
Update indentation of hash file (two spaces).
Previous ogre version unconditionally include <sys/sysctl.h>
header that has been removed from glibc since 2.32.
This has been fixed since Ogre 1.12.7 [4].
[1] https://github.com/OGRECave/ogre/commit/de4d5c920e23e1e2b21dc5c8192ef74ba6210cca
[2] https://github.com/OGRECave/ogre/commit/338103324096b78ae4718f97ac2150c4bb98c2d1
[3] https://github.com/OGRECave/ogre/issues/1332#issuecomment-563189058
[4] https://github.com/OGRECave/ogre/commit/8ec086e9bc2e24fab373b514c572483b69071d69
Signed-off-by: Romain Naour <romain.naour@gmail.com>
---
v3: improve commit log.
bump ogre 1.12.12
---
...ompiledHeader.cmake-Add-c-argument-t.patch | 57 -------------------
package/ogre/Config.in | 1 +
package/ogre/ogre.hash | 4 +-
package/ogre/ogre.mk | 14 +++--
4 files changed, 13 insertions(+), 63 deletions(-)
delete mode 100644 package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
diff --git a/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch b/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
deleted file mode 100644
index 97f4e24890..0000000000
--- a/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
+++ /dev/null
@@ -1,57 +0,0 @@
-From f480ac538eb69086d4b7db855c2a457d5d6420d4 Mon Sep 17 00:00:00 2001
-From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-Date: Mon, 10 Feb 2020 14:05:12 +0100
-Subject: [PATCH] CMake/Utils/PrecompiledHeader.cmake: Add -c argument to build
- precompiled headers
-
-Add "-c" argument when building precompiled headers to fix build with
-RELRO.
-
-More information on a similar issue with domoticz can be found here:
-https://patchwork.ozlabs.org/patch/1187328:
-
-"The problem AFAICS is that if no -c or similar option is given, GCC
-decides what needs to be done based on the rest of the arguments. If the
-rest of the arguments include a -Wl,... option, it decides that linking
-needs to be done. If the rest of the arguments are just header files, it
-decides to create a precompiled header."
-
-Fixes:
- - http://autobuild.buildroot.org/results/8fabf8d270b9257c3a9db6a2f17f1c08ec9428d3
-
-Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
-[Upstream: https://github.com/OGRECave/ogre/commit/de4d5c920e23e1e2b21dc5c8192ef74ba6210cca]
----
- CMake/Utils/PrecompiledHeader.cmake | 6 +++---
- 1 file changed, 3 insertions(+), 3 deletions(-)
-
-diff --git a/CMake/Utils/PrecompiledHeader.cmake b/CMake/Utils/PrecompiledHeader.cmake
-index a02d99acd..bfb0059db 100644
---- a/CMake/Utils/PrecompiledHeader.cmake
-+++ b/CMake/Utils/PrecompiledHeader.cmake
-@@ -133,11 +133,11 @@ MACRO(_PCH_GET_COMPILE_COMMAND out_command _input _output)
- STRING(REGEX REPLACE "^ +" "" pchsupport_compiler_cxx_arg1 ${CMAKE_CXX_COMPILER_ARG1})
-
- SET(${out_command}
-- ${CMAKE_CXX_COMPILER} ${pchsupport_compiler_cxx_arg1} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}
-+ ${CMAKE_CXX_COMPILER} ${pchsupport_compiler_cxx_arg1} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}
- )
- ELSE(CMAKE_CXX_COMPILER_ARG1)
- SET(${out_command}
-- ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}
-+ ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}
- )
- ENDIF(CMAKE_CXX_COMPILER_ARG1)
- ELSE(CMAKE_COMPILER_IS_GNUCXX)
-@@ -291,7 +291,7 @@ MACRO(ADD_PRECOMPILED_HEADER _targetName _input)
- set_target_properties(${_targetName}_pch_dephelp PROPERTIES INCLUDE_DIRECTORIES "${DIRINC}")
-
- #MESSAGE("_compile_FLAGS: ${_compile_FLAGS}")
-- #message("COMMAND ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}")
-+ #message("COMMAND ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}")
-
- ADD_CUSTOM_COMMAND(
- OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${_name}"
---
-2.24.1
-
diff --git a/package/ogre/Config.in b/package/ogre/Config.in
index 1d7784714d..7d57d3f82a 100644
--- a/package/ogre/Config.in
+++ b/package/ogre/Config.in
@@ -10,6 +10,7 @@ config BR2_PACKAGE_OGRE
depends on BR2_USE_WCHAR # use wchar_t
select BR2_PACKAGE_FREETYPE
select BR2_PACKAGE_LIBFREEIMAGE
+ select BR2_PACKAGE_PUGIXML
select BR2_PACKAGE_SDL2
select BR2_PACKAGE_SDL2_OPENGL
select BR2_PACKAGE_SDL2_X11 # use wmInfo.info.x11
diff --git a/package/ogre/ogre.hash b/package/ogre/ogre.hash
index 8bb4ae96a1..0028a4b97c 100644
--- a/package/ogre/ogre.hash
+++ b/package/ogre/ogre.hash
@@ -1,3 +1,3 @@
# sha256 locally computed
-sha256 163e7700e319532d4389ecba91e3ab88551f78610886fa36f8f262f9a5080988 ogre-1.12.0.tar.gz
-sha256 82758e8d1d72139904b9b7472ef0f6544413d2871b58540307fdcc20e473e5f8 LICENSE
+sha256 805b97aacd2070f19394fe92442307a7ba6036ad95c511f551817d694c8dea37 ogre-v1.12.12-br1.tar.gz
+sha256 82758e8d1d72139904b9b7472ef0f6544413d2871b58540307fdcc20e473e5f8 LICENSE
diff --git a/package/ogre/ogre.mk b/package/ogre/ogre.mk
index 2df685a23a..72c96cb33a 100644
--- a/package/ogre/ogre.mk
+++ b/package/ogre/ogre.mk
@@ -4,17 +4,21 @@
#
################################################################################
-OGRE_VERSION = 1.12.0
-OGRE_SITE = $(call github,OGRECave,ogre,v$(OGRE_VERSION))
-OGRE_LICENSE = MIT (main library, DeferredShadingMedia samples), Public Domain (samples and plugins), Zlib (tinyxml)
+OGRE_VERSION = v1.12.12
+OGRE_SITE = https://github.com/OGRECave/ogre
+OGRE_SITE_METHOD = git
+OGRE_LICENSE = MIT (main library, DeferredShadingMedia samples), Public Domain (samples and plugins)
OGRE_LICENSE_FILES = LICENSE
OGRE_INSTALL_STAGING = YES
-# Ogre use a bundled version of tinyxml
+# Download with imgui submodule (https://github.com/ocornut/imgui
+OGRE_GIT_SUBMODULES = YES
+
OGRE_DEPENDENCIES = host-pkgconf \
freetype \
libfreeimage \
libgl \
+ pugixml \
sdl2 \
xlib_libX11 \
xlib_libXaw \
@@ -27,9 +31,11 @@ OGRE_CXXFLAGS = $(TARGET_CXXFLAGS) -DGLEW_NO_GLU
# Unbundle freetype and zziplib.
# Disable java and nvidia cg support.
+# Disable imgui overlay to avoid extra download from CMake.
OGRE_CONF_OPTS = -DOGRE_BUILD_DEPENDENCIES=OFF \
-DOGRE_BUILD_COMPONENT_JAVA=OFF \
-DOGRE_BUILD_PLUGIN_CG=OFF \
+ -DOGRE_BUILD_COMPONENT_OVERLAY_IMGUI=OFF \
-DOGRE_INSTALL_DOCS=OFF \
-DCMAKE_C_FLAGS="$(OGRE_CFLAGS)" \
-DCMAKE_CXX_FLAGS="$(OGRE_CXXFLAGS)"
--
2.30.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Buildroot] [PATCH v3] package/ogre: bump to version 1.12.12
2021-04-28 22:48 [Buildroot] [PATCH v3] package/ogre: bump to version 1.12.12 Romain Naour
@ 2021-12-22 20:59 ` Arnout Vandecappelle
0 siblings, 0 replies; 2+ messages in thread
From: Arnout Vandecappelle @ 2021-12-22 20:59 UTC (permalink / raw)
To: Romain Naour, buildroot
On 29/04/2021 00:48, Romain Naour wrote:
> Remove upstream patch [1].
>
> tinyxml has been replaced by pugixml [2].
>
> Switch to git download method since github release without
> the complete source code in the tarball [3].
>
> Update indentation of hash file (two spaces).
>
> Previous ogre version unconditionally include <sys/sysctl.h>
> header that has been removed from glibc since 2.32.
> This has been fixed since Ogre 1.12.7 [4].
>
> [1] https://github.com/OGRECave/ogre/commit/de4d5c920e23e1e2b21dc5c8192ef74ba6210cca
> [2] https://github.com/OGRECave/ogre/commit/338103324096b78ae4718f97ac2150c4bb98c2d1
> [3] https://github.com/OGRECave/ogre/issues/1332#issuecomment-563189058
> [4] https://github.com/OGRECave/ogre/commit/8ec086e9bc2e24fab373b514c572483b69071d69
>
> Signed-off-by: Romain Naour <romain.naour@gmail.com>
Applied to master, thanks.
Congratulations for winning the 20 years of Buildroot quiz!
Regards,
Arnout
> ---
> v3: improve commit log.
> bump ogre 1.12.12
> ---
> ...ompiledHeader.cmake-Add-c-argument-t.patch | 57 -------------------
> package/ogre/Config.in | 1 +
> package/ogre/ogre.hash | 4 +-
> package/ogre/ogre.mk | 14 +++--
> 4 files changed, 13 insertions(+), 63 deletions(-)
> delete mode 100644 package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
>
> diff --git a/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch b/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
> deleted file mode 100644
> index 97f4e24890..0000000000
> --- a/package/ogre/0001-CMake-Utils-PrecompiledHeader.cmake-Add-c-argument-t.patch
> +++ /dev/null
> @@ -1,57 +0,0 @@
> -From f480ac538eb69086d4b7db855c2a457d5d6420d4 Mon Sep 17 00:00:00 2001
> -From: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -Date: Mon, 10 Feb 2020 14:05:12 +0100
> -Subject: [PATCH] CMake/Utils/PrecompiledHeader.cmake: Add -c argument to build
> - precompiled headers
> -
> -Add "-c" argument when building precompiled headers to fix build with
> -RELRO.
> -
> -More information on a similar issue with domoticz can be found here:
> -https://patchwork.ozlabs.org/patch/1187328:
> -
> -"The problem AFAICS is that if no -c or similar option is given, GCC
> -decides what needs to be done based on the rest of the arguments. If the
> -rest of the arguments include a -Wl,... option, it decides that linking
> -needs to be done. If the rest of the arguments are just header files, it
> -decides to create a precompiled header."
> -
> -Fixes:
> - - http://autobuild.buildroot.org/results/8fabf8d270b9257c3a9db6a2f17f1c08ec9428d3
> -
> -Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> -[Upstream: https://github.com/OGRECave/ogre/commit/de4d5c920e23e1e2b21dc5c8192ef74ba6210cca]
> ----
> - CMake/Utils/PrecompiledHeader.cmake | 6 +++---
> - 1 file changed, 3 insertions(+), 3 deletions(-)
> -
> -diff --git a/CMake/Utils/PrecompiledHeader.cmake b/CMake/Utils/PrecompiledHeader.cmake
> -index a02d99acd..bfb0059db 100644
> ---- a/CMake/Utils/PrecompiledHeader.cmake
> -+++ b/CMake/Utils/PrecompiledHeader.cmake
> -@@ -133,11 +133,11 @@ MACRO(_PCH_GET_COMPILE_COMMAND out_command _input _output)
> - STRING(REGEX REPLACE "^ +" "" pchsupport_compiler_cxx_arg1 ${CMAKE_CXX_COMPILER_ARG1})
> -
> - SET(${out_command}
> -- ${CMAKE_CXX_COMPILER} ${pchsupport_compiler_cxx_arg1} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}
> -+ ${CMAKE_CXX_COMPILER} ${pchsupport_compiler_cxx_arg1} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}
> - )
> - ELSE(CMAKE_CXX_COMPILER_ARG1)
> - SET(${out_command}
> -- ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}
> -+ ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}
> - )
> - ENDIF(CMAKE_CXX_COMPILER_ARG1)
> - ELSE(CMAKE_COMPILER_IS_GNUCXX)
> -@@ -291,7 +291,7 @@ MACRO(ADD_PRECOMPILED_HEADER _targetName _input)
> - set_target_properties(${_targetName}_pch_dephelp PROPERTIES INCLUDE_DIRECTORIES "${DIRINC}")
> -
> - #MESSAGE("_compile_FLAGS: ${_compile_FLAGS}")
> -- #message("COMMAND ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -o ${_output} ${_input}")
> -+ #message("COMMAND ${CMAKE_CXX_COMPILER} ${_compile_FLAGS} -x c++-header -c -o ${_output} ${_input}")
> -
> - ADD_CUSTOM_COMMAND(
> - OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${_name}"
> ---
> -2.24.1
> -
> diff --git a/package/ogre/Config.in b/package/ogre/Config.in
> index 1d7784714d..7d57d3f82a 100644
> --- a/package/ogre/Config.in
> +++ b/package/ogre/Config.in
> @@ -10,6 +10,7 @@ config BR2_PACKAGE_OGRE
> depends on BR2_USE_WCHAR # use wchar_t
> select BR2_PACKAGE_FREETYPE
> select BR2_PACKAGE_LIBFREEIMAGE
> + select BR2_PACKAGE_PUGIXML
> select BR2_PACKAGE_SDL2
> select BR2_PACKAGE_SDL2_OPENGL
> select BR2_PACKAGE_SDL2_X11 # use wmInfo.info.x11
> diff --git a/package/ogre/ogre.hash b/package/ogre/ogre.hash
> index 8bb4ae96a1..0028a4b97c 100644
> --- a/package/ogre/ogre.hash
> +++ b/package/ogre/ogre.hash
> @@ -1,3 +1,3 @@
> # sha256 locally computed
> -sha256 163e7700e319532d4389ecba91e3ab88551f78610886fa36f8f262f9a5080988 ogre-1.12.0.tar.gz
> -sha256 82758e8d1d72139904b9b7472ef0f6544413d2871b58540307fdcc20e473e5f8 LICENSE
> +sha256 805b97aacd2070f19394fe92442307a7ba6036ad95c511f551817d694c8dea37 ogre-v1.12.12-br1.tar.gz
> +sha256 82758e8d1d72139904b9b7472ef0f6544413d2871b58540307fdcc20e473e5f8 LICENSE
> diff --git a/package/ogre/ogre.mk b/package/ogre/ogre.mk
> index 2df685a23a..72c96cb33a 100644
> --- a/package/ogre/ogre.mk
> +++ b/package/ogre/ogre.mk
> @@ -4,17 +4,21 @@
> #
> ################################################################################
>
> -OGRE_VERSION = 1.12.0
> -OGRE_SITE = $(call github,OGRECave,ogre,v$(OGRE_VERSION))
> -OGRE_LICENSE = MIT (main library, DeferredShadingMedia samples), Public Domain (samples and plugins), Zlib (tinyxml)
> +OGRE_VERSION = v1.12.12
> +OGRE_SITE = https://github.com/OGRECave/ogre
> +OGRE_SITE_METHOD = git
> +OGRE_LICENSE = MIT (main library, DeferredShadingMedia samples), Public Domain (samples and plugins)
> OGRE_LICENSE_FILES = LICENSE
> OGRE_INSTALL_STAGING = YES
>
> -# Ogre use a bundled version of tinyxml
> +# Download with imgui submodule (https://github.com/ocornut/imgui
> +OGRE_GIT_SUBMODULES = YES
> +
> OGRE_DEPENDENCIES = host-pkgconf \
> freetype \
> libfreeimage \
> libgl \
> + pugixml \
> sdl2 \
> xlib_libX11 \
> xlib_libXaw \
> @@ -27,9 +31,11 @@ OGRE_CXXFLAGS = $(TARGET_CXXFLAGS) -DGLEW_NO_GLU
>
> # Unbundle freetype and zziplib.
> # Disable java and nvidia cg support.
> +# Disable imgui overlay to avoid extra download from CMake.
> OGRE_CONF_OPTS = -DOGRE_BUILD_DEPENDENCIES=OFF \
> -DOGRE_BUILD_COMPONENT_JAVA=OFF \
> -DOGRE_BUILD_PLUGIN_CG=OFF \
> + -DOGRE_BUILD_COMPONENT_OVERLAY_IMGUI=OFF \
> -DOGRE_INSTALL_DOCS=OFF \
> -DCMAKE_C_FLAGS="$(OGRE_CFLAGS)" \
> -DCMAKE_CXX_FLAGS="$(OGRE_CXXFLAGS)"
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-12-22 20:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-28 22:48 [Buildroot] [PATCH v3] package/ogre: bump to version 1.12.12 Romain Naour
2021-12-22 20:59 ` Arnout Vandecappelle
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.