> On 16.10.2021 23:02, Arseny Maslennikov wrote: > > On Sat, Oct 16, 2021 at 10:46:19PM +0700, Илья Курдюков wrote: > > > Заметил при сборке python3-module-PySide2, что макрос %cmake устанавливает > > > CMAKE_CXX_FLAGS, в который включается -O%_optlevel. Но потом этот -On > > > затирается флагом -O2, который содержится в CMAKE_CXX_FLAGS_RELEASE ("-O2 > > > -DNDEBUG" по умолчанию). > > > > > > Не надо ли тогда в %cmake добавить -DCMAKE_CXX_FLAGS_RELEASE="-DNDEBUG" ? > > Не все собирают с -DCMAKE_BUILD_TYPE=Release. > > Если, например, явно собирать с -DCMAKE_BUILD_TYPE=RelWithDebInfo, > > флаги в CMAKE_CXX_FLAGS_RELEASE будут неактуальны. On Sat, Oct 16, 2021 at 11:12:28PM +0700, Илья Курдюков wrote: > Там в CMakeLists везде проставлено: > > set(CMAKE_BUILD_TYPE Release CACHE STRING "Build Type") Там Release — это значение по умолчанию, судя по вашим словам, у конкретного апстрима; заданное в командной строке другое значение его перебьёт. > > Это какая-то редкость? Апстримы пишут, как хотят, в меру своего разумения. ;] > |CMAKE_CXX_FLAGS_RELWITHDEBINFO тоже должен ставить опцию -O2. Полагаю что > если |CMAKE_BUILD_TYPE никто не трогал, то > используется|CMAKE_CXX_FLAGS_DEBUG установленный в -g? ||| Из https://cmake.org/cmake/help/latest/variable/CMAKE_LANG_FLAGS.html: >> The flags in this variable will be passed to the compiler before those >> in the per-configuration CMAKE__FLAGS_ variant, and before >> flags added by the add_compile_options() or target_compile_options() >> commands. Вот в meson для этого придумали --buildtype=plain, который означает "никаких оптфлагов не дописывать". У нас в макросах он и передаётся.