OGRECave / ogre-next

aka ogre v2 - scene-oriented, flexible 3D C++ engine
https://ogrecave.github.io/ogre-next/api/latest
Other
1.03k stars 227 forks source link

rename OGER to OGER-Next in pc files. #420

Closed talregev closed 10 months ago

talregev commented 11 months ago

System Information

Detailled description

I compile ogre-next 2.3.2 on vcpkg with the flag: -DOGRE_USE_NEW_PROJECT_NAME=ON to have the new name of ogre-next. It compile successful with the new name on the libs. but the pc file didn't not update to the new name:

prefix=${pcfiledir}/../..
exec_prefix=${prefix}
libdir=${prefix}/lib
includedir=${prefix}/../include
plugindir=${libdir}/OGRE

Name: OGRE (static lib)
Description: Object-Oriented Graphics Rendering Engine
Version: 2.3.2
URL: http://www.ogre3d.org

Libs: "-L${libdir}" "-L${plugindir}" -lOgreMainStatic_d -lpthread -lboost-thread-mt -lfreeimage
Requires: freetype2, zziplib, x11, xt, xaw7, gl
Cflags: "-I${includedir}" "-I${includedir}/OGRE" -pthread

it should change the name the include folder and the linkage flags:

plugindir=${libdir}/OGRE -> plugindir=${libdir}/OGRE-Next
Name: OGRE (static lib) -> Name: OGRE-Next (static lib)
 -lOgreMainStatic_d ->  -lOgreNextMainStatic_d
-I${includedir}/OGRE -> -I${includedir}/OGRE-Next

Ogre.log

Callstack

Not relevant. not in run time.

talregev commented 11 months ago

It coming from here: https://github.com/OGRECave/ogre-next/blob/master/CMake/Templates/OGREStatic.pc.in Is it not update by purpose? I can make a patch on vcpkg side.

talregev commented 11 months ago

I think also here: https://github.com/OGRECave/ogre-next/blob/491d37df15c048cd4757cd57d3091cec7c255fc5/RenderSystems/Direct3D11/CMakeLists.txt#L64

install(FILES ${HEADER_FILES} DESTINATION include/OGRE/RenderSystems/Direct3D11) ->
install(FILES ${HEADER_FILES} DESTINATION include/${OGRE_NEXT_PREFIX}/RenderSystems/Direct3D11)
talregev commented 10 months ago

Thank you!

darksylinc commented 10 months ago

Thank you for reporting it!

I'm planning on making a new 2.3 release to address this issue.